Rename section to dom_section, raise errors on invalid routes.

Author: ryanstout

lib/volt/models/url.rb

@@ -130,7 +130,13 @@ def assign_query_hash_to_params
       query_hash = self.query_hash
 
       # Get the params that are in the route
-      query_hash.merge!(@router.url_to_params(@path))
+      new_params = @router.url_to_params(@path)
+
+      if new_params == false
+        raise "no routes match path: #{@path}"
+      end
+
+      query_hash.merge!(new_params)
 
       # Loop through the .params we already have assigned.
       assign_from_old(@params, query_hash)

lib/volt/page/bindings/base_binding.rb

@@ -20,21 +20,21 @@ def initialize(page, target, context, binding_name)
     @@binding_number ||= 10000
   end
 
-  def section
-    @section ||= target.section(@binding_name)
+  def dom_section
+    @dom_section ||= target.dom_section(@binding_name)
   end
 
   def remove
-    section.remove if @section
+    @dom_section.remove if @dom_section
 
     # Clear any references
     @target = nil
     @context = nil
-    @section = nil
+    @dom_section = nil
   end
 
   def remove_anchors
-    section.remove_anchors
+    @dom_section.remove_anchors if @dom_section
   end
 
   def queue_update

lib/volt/page/bindings/content_binding.rb

@@ -23,7 +23,7 @@ def update
 
     # Update the html in this section
     # TODO: Move the formatter into another class.
-    section.html = value.gsub("\n", "<br />\n")
+    dom_section.html = value.gsub("\n", "<br />\n")
   end
 
   def remove

lib/volt/page/bindings/each_binding.rb

@@ -56,10 +56,10 @@ def item_added(position)
 
     if position >= @templates.size
       # Setup new bindings in the spot we want to insert the item
-      section.insert_anchor_before_end(binding_name)
+      dom_section.insert_anchor_before_end(binding_name)
     else
       # Insert the item before an existing item
-      section.insert_anchor_before(binding_name, @templates[position].binding_name)
+      dom_section.insert_anchor_before(binding_name, @templates[position].binding_name)
     end
 
     index = ReactiveValue.new(position)

lib/volt/page/bindings/template_binding.rb

@@ -174,7 +174,7 @@ def call_ready
       # Set the current section on the controller if it wants so it can manipulate
       # the dom if needed
       if @controller.respond_to?(:section=)
-        @controller.section = @current_template.section
+        @controller.dom_section = @current_template.dom_ection
       end
 
       if @controller.respond_to?(:dom_ready)

lib/volt/page/targets/attribute_target.rb

@@ -8,7 +8,7 @@
 
 class AttributeTarget < ComponentNode
 
-  def section(*args)
+  def dom_section(*args)
     return AttributeSection.new(self, *args)
   end
 end

lib/volt/page/targets/dom_target.rb

@@ -5,7 +5,7 @@
 # the dom.  Currently only one "dom" is supported, but multiple
 # may be allowed in the future (iframes?)
 class DomTarget < BaseSection
-  def section(*args)
+  def dom_section(*args)
     return DomSection.new(*args)
   end
 end

lib/volt/page/template_renderer.rb

@@ -7,7 +7,7 @@ def initialize(page, target, context, binding_name, template_name)
 
     @sub_bindings = []
 
-    bindings = self.section.set_content_to_template(page, template_name)
+    bindings = self.dom_section.set_content_to_template(page, template_name)
 
     bindings.each_pair do |id,bindings_for_id|
       bindings_for_id.each do |binding|