hobo 1.0.3 → 1.1.0.pre0

data/lib/hobo/user_controller.rb

@@ -70,6 +70,7 @@ module Hobo
         user = model.authenticate(params[:login], params[:password])
         if user.nil?
           flash[:error] = options[:failure_notice]
+          hobo_ajax_response if request.xhr? && !performed?
         else
           old_user = current_user
           self.current_user = user

data/lib/hobo/view_hints.rb

@@ -3,11 +3,7 @@ module Hobo
   class ViewHints
     
     def self.enable
-      if ActiveSupport::Dependencies.respond_to?(:autoload_paths)
-        ActiveSupport::Dependencies.autoload_paths |= [ "#{RAILS_ROOT}/app/viewhints" ]
-      else
-        ActiveSupport::Dependencies.load_paths |= [ "#{RAILS_ROOT}/app/viewhints" ]
-      end
+      ActiveSupport::Dependencies.load_paths |= ["#{RAILS_ROOT}/app/viewhints"]
     end
   
     def self.setter(name, default=nil, &block)

data/rails_generators/hobo/templates/initializer.rb

@@ -1,2 +1,2 @@
 Hobo::ModelRouter.reload_routes_on_every_request = true
-# Hobo::Dryml.precompile_taglibs if File.basename($0) != "rake" && Rails.env.production? 
+# Dryml.precompile_taglibs if File.basename($0) != "rake" && Rails.env.production? 

data/rails_generators/hobo_front_controller/templates/summary.dryml

@@ -67,11 +67,12 @@
 
     <h2>Models</h2>
     <table class="app-summary">
-      <tr><th>Class</th><th>Table</th></tr>
+      <tr><th>Class</th><th>Table</th><th></th></tr>
       <with-models>
         <tr>
           <td><model-name/></td>
           <td><model-table-name/></td>
+          <td><model-table-comment/></td>
         </tr>
       </with-models>
     </table>        
@@ -80,10 +81,11 @@
       <h3 if="&this.try.table_name"><model-name /></h3>
       <table class="app-summary">
         <with-model-columns>
-          <tr if="&first_item?"><th>Column</th><th>Type</th></tr>
+          <tr if="&first_item?"><th>Column</th><th>Type</th><th></th></tr>
           <tr>
             <td><model-column-name/></td>
             <td><model-column-type/></td>
+            <td><model-column-comment/></td>
           </tr>
         </with-model-columns>
       </table>

data/rails_generators/hobo_model/hobo_model_generator.rb

@@ -27,5 +27,17 @@ class HoboModelGenerator < Rails::Generator::NamedBase
     def max_attribute_length
       attributes.*.name.*.length.max
     end
+    
+    def field_attributes
+      attributes.reject { |a| a.name == "bt" || a.name == "hm" }
+    end
+    
+    def hms
+      attributes.select { |a| a.name == "hm" }.*.type
+    end
+
+    def bts
+      attributes.select { |a| a.name == "bt" }.*.type
+    end
 
 end

data/rails_generators/hobo_model/templates/model.rb

@@ -3,13 +3,20 @@ class <%= class_name %> < ActiveRecord::Base
   hobo_model # Don't put anything above this
 
   fields do
-<% for attribute in attributes -%>
+<% for attribute in field_attributes -%>
     <%= "%-#{max_attribute_length}s" % attribute.name %> :<%= attribute.type %>
 <% end -%>
     timestamps
   end
 
-
+<% for bt in bts -%>
+  belongs_to :<%= bt %>
+<% end -%>
+<%= "\n" unless bts.empty? -%>
+<% for hm in hms -%>
+  has_many :<%= hm %>, :dependent => :destroy
+<% end -%>
+<%= "\n" unless hms.empty? -%>
   # --- Permissions --- #
 
   def create_permitted?

data/rails_generators/hobo_rapid/templates/hobo-rapid.js

@@ -408,8 +408,8 @@ var Hobo = {
 
 
     updateElement: function(id, content) {
-        // TODO: Do we need this method?
         Element.update(id, content)
+        Element.fire($(id), "rapid:partupdated")
     },
 
     getStyle: function(el, styleProp) {
@@ -560,7 +560,7 @@ HoboBehavior = Class.create({
 })
 
 
-new HoboBehavior("ul.input-many", {
+HoboInputMany = {
   
   events: {
       "> li > div.buttons": {
@@ -569,9 +569,9 @@ new HoboBehavior("ul.input-many", {
       }
   },
 
-  initialize: function(ul) {
+  initialize: function(ev) {
       /* the second clause should be sufficient, but it isn't in IE7.  See bug 603  */
-      $$(".input-many-template input:hidden, .input-many-template select:hidden, .input-many-template textarea:hidden, .input-many-template button:hidden").each(function(input) {
+      Element.select(ev.target, ".input-many-template input:hidden, .input-many-template select:hidden, .input-many-template textarea:hidden, .input-many-template button:hidden").each(function(input) {
           if(!input.disabled) {
               input.disabled = true;
               input.addClassName("input_many_template_input");
@@ -579,10 +579,33 @@ new HoboBehavior("ul.input-many", {
       });
 
       // disable all elements inside our template, and mark them so we can find them later.
-      $$(".input-many-template input:enabled, .input-many-template select:enabled, .input-many-template textarea:enabled, .input-many-template button:enabled").each(function(input) {
+      Element.select(ev.target, ".input-many-template input:enabled, .input-many-template select:enabled, .input-many-template textarea:enabled, .input-many-template button:enabled").each(function(input) {
           input.disabled = true;
           input.addClassName("input_many_template_input");
       });
+
+      Element.select(ev.target, ".sortable-input-many").each(function(el) {
+          HoboInputMany.createSortable.call(el);
+      });
+
+      /* need to reinitialize after every change */
+      Event.addBehavior({".sortable-input-many:rapid:change": function(ev) {
+          HoboInputMany.createSortable.call(this);
+      }});
+
+      document.observe("rapid:partupdated", HoboInputMany.initialize);
+  },
+
+  createSortable: function() {
+      Sortable.create(this.id, {
+          constraint: 'vertical', 
+          handle: 'ordering-handle', 
+          overlap: 'vertical', 
+          scroll: 'window',
+          onUpdate: function(list) {
+              HoboInputMany.fixIndices.call(list);                
+          }
+      });
   },
 
     // given this==the input-many, returns a lambda that updates the name & id for an element
@@ -619,23 +642,56 @@ new HoboBehavior("ul.input-many", {
 
   // given this==an input-many item, get the submit index
   getIndex: function() {
-      return Number(this.id.match(/\[([-0-9]+)\]$/)[1]);
+      return Number(this.id.match(/_([-0-9]+)$/)[1]);
   },
 
-  /* For some reason, select() and down() and all those useful functions aren't working for us.  Roll our own replacement. */
-  recurse_elements_with_class: function(el, klass, f) {
-      var that=this;
-      if(klass==null || el.hasClassName(klass)) {
-          f(el);
+    /* For some reason, select() and down() and all those useful functions aren't working for us.  Roll our own replacement.   
+
+      this: element to recurse on.   
+      klass: class to filter on
+      f: function to invoke
+    */
+  recurse_elements_with_class: function(klass,f ) {
+      if(klass==null || this.hasClassName(klass)) {
+          f(this);
       }
-      el.childElements().each(function(el2) {that.recurse_elements_with_class.call(that, el2, klass, f);});
+      this.childElements().each(function(el2) {HoboInputMany.recurse_elements_with_class.call(el2, klass, f);});
   },
 
+/* fixes the indices on an input-many so they're in order. */
+  fixIndices: function() {
+    var lis = this.immediateDescendants();
+    var minimum = parseInt(Hobo.getClassData(this, 'minimum'));
+    /* first two lis are hidden/disabled on an input-many */
+    for(var i=0; i<lis.length-2; i++) {
+        var il=i+2;
+        if(i!=HoboInputMany.getIndex.call(lis[il])) {
+            var updater = HoboInputMany.getNameUpdater.call(this, i);
+            HoboInputMany.recurse_elements_with_class.call(lis[il], null, function(el) {
+                updater.call(el);
+            });
+            var position=lis[il].childWithClass("sortable-position");
+            if(position) position.value=i+1;
+            if(i==minimum-1 && il==lis.length-1) {
+                lis[il].childWithClass("buttons").childWithClass("remove-item").addClassName("hidden");
+            } else {
+                lis[il].childWithClass("buttons").childWithClass("remove-item").removeClassName("hidden");
+            }
+            if(il==lis.length-1) {
+                lis[il].childWithClass("buttons").childWithClass("add-item").removeClassName("hidden");
+            } else {
+                lis[il].childWithClass("buttons").childWithClass("add-item").addClassName("hidden");
+            }
+        }
+    }
+  },
+
+
   addOne: function(ev, el) {
       Event.stop(ev);
       var ul = el.up('ul.input-many'), li = el.up('li.input-many-li');
 
-      if(li.id.search(/\[-1\]/)>=0 && ul.immediateDescendants().length>2) {
+      if(li.id.search(/_-1$/ && ul.immediateDescendants().length>2)>=0) {
           /* if(console) console.log("IE7 messed up again (bug 605)"); */
           return;
       }
@@ -658,7 +714,7 @@ new HoboBehavior("ul.input-many", {
       reenable_inputs(clone);
 
       // update id & name
-      this.recurse_elements_with_class.call(this, clone, null, function(el) {
+      HoboInputMany.recurse_elements_with_class.call(clone, null, function(el) {
           name_updater.call(el);
       });
 
@@ -691,7 +747,7 @@ new HoboBehavior("ul.input-many", {
       var ul = el.up('ul.input-many'), li = el.up('li.input-many-li')
       var minimum = parseInt(Hobo.getClassData(ul, 'minimum'));
 
-      if(li.id.search(/\[-1\]/)>=0) {
+      if(li.id.search(/_-1$/)>=0) {
           /* if(console) console.log("IE7 messed up again (bug 605)"); */
           return;
       }
@@ -703,7 +759,7 @@ new HoboBehavior("ul.input-many", {
       var n=li.next();
       for(; n; i+=1, n=n.next()) {          
           var name_updater = this.getNameUpdater.call(ul, i);
-          this.recurse_elements_with_class.call(this, n, null, function(el) {name_updater.call(el);});
+          HoboInputMany.recurse_elements_with_class.call(n, null, function(el) {name_updater.call(el);});
       } 
 
       // adjust +/- buttons on the button element as appropriate
@@ -714,7 +770,7 @@ new HoboBehavior("ul.input-many", {
 
       if(last.hasClassName("empty")) {
           last.removeClassName("hidden");
-          this.recurse_elements_with_class.call(this, last, "empty-input", function(el) {el.disabled=false;});
+          HoboInputMany.recurse_elements_with_class.call(last, "empty-input", function(el) {el.disabled=false;});
       } else {
           // if we've reached the minimum, we don't want to add the '-' button
           if(ul.childElements().length-3 <= minimum||0) {
@@ -734,7 +790,9 @@ new HoboBehavior("ul.input-many", {
 
 
 
-})
+}
+  
+new HoboBehavior("ul.input-many", HoboInputMany);
 
 
 SelectManyInput = Behavior.create({
@@ -811,9 +869,10 @@ NameManyInput = Object.extend(SelectManyInput, {
     }
 })
 
-                              
+             
 AutocompleteBehavior = Behavior.create({
     initialize : function() {
+        this.minChars  = parseInt(Hobo.getClassData(this.element, "min-chars")); 
         var match     = this.element.className.match(/complete-on::([\S]+)/)
         var target    = match[1].split('::')
         var typedId   = target[0]
@@ -822,11 +881,27 @@ AutocompleteBehavior = Behavior.create({
         var spec = Hobo.parseModelSpec(typedId)
         var url = urlBase + "/" + Hobo.pluralise(spec.name) +  "/complete_" + completer
         var parameters = spec.id ? "id=" + spec.id : ""
-        new Ajax.Autocompleter(this.element, 
-                               this.element.next('.completions-popup'), 
-                               url, 
-                               {paramName:'query', method:'get', parameters: parameters});
+        this.autocompleter = new Ajax.Autocompleter(this.element, 
+            this.element.next('.completions-popup'), 
+            url, 
+            {paramName:'query', method:'get', parameters: parameters, minChars: this.minChars,
+            afterUpdateElement: this.afterUpdateElement});
+    },
+
+    onfocus: function() {
+        if(this.element.hasClassName("nil-value")) {
+            this.element.value = '';
+            this.element.removeClassName("nil-value");
+        }
+        if(this.minChars==0) { 
+            this.autocompleter.activate();
+        }
+    },
+
+    afterUpdateElement: function(input, li) {
+        input.fire("rapid:autocomplete-assigned");
     }
+        
 })
 
 

data/rails_generators/hobo_rapid/templates/themes/clean/public/stylesheets/clean.css

@@ -57,7 +57,7 @@ pre, code {
     font-family:  "Courier New", Courier, monospace;
 }
 
-input.text, input.string, input.email-address, input.password, input.search, input.integer, input.float, textarea {
+input.text, input.string, input.email-address, input.password, input.search, input.integer, input.float, input.autocompleter, input.decimal, textarea {
 	border-top:1px solid #7c7c7c;
 	border-left:1px solid #c3c3c3;
 	border-right:1px solid #c3c3c3;

data/rails_generators/hobo_rapid/templates/themes/clean/public/stylesheets/rapid-ui.css

@@ -97,4 +97,6 @@ select.dev-user-changer:hover { opacity: 1; }
 optgroup.disabled-option {
     color: #ccc;
     height: 1em;
-}
+}
+
+input.nil-value { color:grey; }

data/{dryml_generators → rapid_generators}/rapid/pages.dryml.erb

@@ -165,7 +165,7 @@ end
 <% end -%>
           </header>
 
-          <section param="content-body">
+          <section param="content-body"<%= ' with-flash-messages' if aside_collections %>>
 <% if main_content -%>
             <view:<%= main_content %> param="description"/>
 <% end -%>

data/taglibs/rapid.dryml

@@ -6,7 +6,9 @@ The Rapid tag library makes web development go fast. The Rapid tag library is yo
 
 -->
 
+<include module="Hobo::HoboHelper"/>
 <include module="Hobo::RapidHelper"/>
+<include module="Hobo::Translations"/>
 
 <include src="rapid_core"/>
 <include src="rapid_support"/>

data/taglibs/rapid_core.dryml

@@ -1,3 +1,4 @@
+<!-- Core Rapid tags and tags that don't belong anywhere else. -->
 
 <!-- Renders a table with one row per field, where each row contains a `<th>` with the field name, and a `<td>` with (by default)
      a `<view>` of the field.
@@ -35,11 +36,11 @@
        input_attrs = {:no_edit => no_edit} if tag == "input"
     -%>
       <labelled-item unless="&tag == 'input' && no_edit == 'skip' && !can_edit?">
-        <item-label param="#{scope.field_name.to_s.sub('?', '').sub('.', '-')}-label" unless="&field_name.blank?">
+        <item-label param="#{scope.field_name.to_s.sub('?', '').gsub('.', '-')}-label" unless="&field_name.blank?">
           <do param="label"><%= field_name %></do>
         </item-label>
-        <item-value param="#{scope.field_name.to_s.sub('?', '').sub('.', '-')}-view" colspan="&2 if field_name.blank?">
-          <do param="view"><call-tag tag="&tag" param="#{scope.field_name.to_s.sub('?', '').sub('.', '-')}-tag" merge-attrs="&input_attrs"/></do>
+        <item-value param="#{scope.field_name.to_s.sub('?', '').gsub('.', '-')}-view" colspan="&2 if field_name.blank?">
+          <do param="view"><call-tag tag="&tag" param="#{scope.field_name.to_s.sub('?', '').gsub('.', '-')}-tag" merge-attrs="&input_attrs"/></do>
           <div param="input-help" if="&tag.to_sym == :input && !this_field_help.blank?"><%= this_field_help %></div>
         </item-value>
       </labelled-item>
@@ -107,7 +108,7 @@ This will use `<input/>` as the tag in each table cell instead of `<view/>`
       <thead if="&all_parameters[:thead] || fields" param>
         <tr param="field-heading-row">
           <with-field-names merge-attrs="&all_attributes & attrs_for(:with_fields)">
-            <th param="#{scope.field_name}-heading"><%= this.member_class.try.view_hints.try.field_name(scope.field_name) if scope %></th>
+            <th param="#{scope.field_name}-heading"><%= (this.member_class.try.view_hints.try.field_name(scope.field_name) || scope.field_name.titleize) if scope %></th>
           </with-field-names>
           <th if="&all_parameters[:controls]" class="controls"/>
         </tr>
@@ -335,7 +336,7 @@ Or a new page if the context is a class:
     target = to || this
   
     if target.nil?
-      Hobo::Dryml.last_if = false
+      Dryml.last_if = false
       nil_view
     elsif action == "new"
       # Link to a new object form
@@ -352,10 +353,10 @@ Or a new page if the context is a class:
 
         add_classes!(attributes, "new-#{new_class_name.underscore}-link")
         content = "New #{new_class_name.titleize}" if content.blank?
-        Hobo::Dryml.last_if = true
+        Dryml.last_if = true
         element(:a, attributes.update(:href => href), content)
       else
-        Hobo::Dryml.last_if = false
+        Dryml.last_if = false
         ""
       end
     else
@@ -456,7 +457,7 @@ Assuming the context is a blog post...
             end
           end
         end
-   Hobo::Dryml.last_if = !res.blank?
+   Dryml.last_if = !res.blank?
    res
 %></def>
 
@@ -535,7 +536,7 @@ The label can be customised using the `label` attribute, e.g.
 
   label = label.downcase if lowercase
    
-  Hobo::Dryml.last_if = c > 0 if if_any
+  Dryml.last_if = c > 0 if if_any
   if if_any && c == 0
     ""
   else

data/taglibs/rapid_forms.dryml

@@ -51,8 +51,7 @@ executed at various points in the ajax request cycle:
             hiddens = case fields
                       when '*', nil
                         # TODO: Need a better (i.e. extensible) way to eleminate certain fields
-                        # marking a field as attr_protected is one way to eliminate a field
-                        this.class.column_names - [this.class.inheritance_column, 'created_at', 'updated_at']
+                        this.class.column_names - ['type', 'created_at', 'updated_at']
                       else
                         comma_split(fields)
                       end
@@ -103,6 +102,8 @@ AJAX based submission can be enabled by simply adding an `update` attribute. e.g
         <div part="comments"><collection:comments/></div>
         <form with="&Comment.new" update="comments"/>
 
+`<form>` support all of the standard ajax attributes.
+
 ### Additional Notes
 
 - Hobo automatically inserts an `auth_token` hidden field if forgery protection is enabled
@@ -111,19 +112,6 @@ AJAX based submission can be enabled by simply adding an `update` attribute. e.g
   validation error occurs.
 
 - `<form>` supports all of the standrd ajax attributes - (see the main taglib docs for Rapid Forms)
-
-- `<form>` resets `last_if` if it does not have permission to display the form.   The `<else>` clause may be used to display alternate content.   For example:
-
-    <form>...</form>
-    <else>You do not have permission to edit this form</else>
-
-or on a standard generated page using a default form:
-
-    <some-page>
-      <after-form:>
-        <else>You do not have permission to edit this form</else>
-      </after-form:>
-    </some-page>
   
 ### Attributes
 
@@ -186,7 +174,7 @@ The standard form tag does not have any parameters, nor does it have any default
   if action.nil? && (html_attrs[:action].nil? ||
                      (lifecycle.nil? && new_record && !this.creatable_by?(current_user)) ||
                      (lifecycle.nil? && !new_record && !can_edit?))
-    Hobo::Dryml.last_if = false
+    Dryml.last_if = false
     ""
   else
     if method == "put"
@@ -223,7 +211,7 @@ The standard form tag does not have any parameters, nor does it have any default
       page_path = if (request.post? || request.put?) && params[:page_path]
                     params[:page_path]
                   else
-                    view_name.sub(Hobo::Dryml::EMPTY_PAGE, params[:action] || '')
+                    view_name.sub(Dryml::EMPTY_PAGE, params[:action] || '')
                   end
       page_path_hidden = hidden_field_tag("page_path", page_path) 
     end
@@ -240,7 +228,7 @@ The standard form tag does not have any parameters, nor does it have any default
       end
     end
 
-    Hobo::Dryml.last_if = true
+    Dryml.last_if = true
     element("form", html_attrs, body)
   end
 %></def>
@@ -350,12 +338,11 @@ edit collections a `Category` model in your application:
 <def tag="collection-input" for="ActiveRecord::Base"><select-many merge/></def>
 
 
-<!-- A `<textarea>` input.  Attributes are passed through to [text_area_tag](http://api.rubyonrails.org/classes/ActionView/Helpers/FormTagHelper.html#method-i-text_area_tag). -->
+<!-- A `<textarea>` input -->
 <def tag="input" for="text" attrs="name">
-  <%= text_area_tag(name, attributes["escape"]==false ? this : html_escape(this), attributes) %>
+  <%= text_area_tag(name, this, attributes) %>
 </def>
 
-
 <!-- A checkbox plus a hidden-field. The hidden field trick comes from Rails - it means that when the checkbox is not checked, the parameter name is still submitted, with a '0' value (the value is '1' when the checkbox is checked) -->
 <def tag="input" for="boolean" attrs="name">
   <%= unless attributes[:disabled]
@@ -570,7 +557,7 @@ All the standard ajax attributes *except the callbacks* are supported (see the m
 <def tag="delete-button" attrs="label, update, in-place, image, confirm, fade, subsite"><%=
   in_place = false if in_place.nil? && this == @this && request.method == :get
   url = object_url(this, :method => :delete, :subsite => subsite)
-  if (Hobo::Dryml.last_if = url && can_delete?)
+  if (Dryml.last_if = url && can_delete?)
     attributes = attributes.merge(if image
                                     { :type => "image", :src => "#{base_url}/images/#{image}" }
                                   else
@@ -730,14 +717,26 @@ We're using an object as the complete-target rather than a class.   This allows
 
 There's another example of `<name-one>` use in the [recipes](http://cookbook.hobocentral.net/recipes/36-using-a-name-one-one-a).
 
+### Attributes:
+
+- `complete-target`, `completer`: see above
+- `min-chars`: The minimum number of characters that must be entered in the input field before an Ajax request is made.
+- `nil-value`: If there is no current value, this text will appear greyed out inside the control, and will disappear on focus.
+
+### Note:
+
+If you wish to set `min-chars` to 0, you will require this [patch to controls.js](http://github.com/bryanlarsen/scriptaculous/commit/3915b7b).   'controls.js' was added to your project via the rails generator, not via Hobo.
+
   -->
-<def tag="name-one" attrs="complete-target, completer"><%
+<def tag="name-one" attrs="complete-target, completer, min-chars, nil-value"><%
   complete_target ||= this_field_reflection.klass
   completer ||= (complete_target.is_a?(Class) ? complete_target : complete_target.class).name_attribute
+  min_chars ||= 1
+  value = name(:no_wrapper => true, :if_present => true)
   -%>
   <input type="text" name="#{param_name_for_this}" 
-         class="autocompleter #{type_and_field.dasherize} #{css_data :complete_on, typed_id(complete_target), completer}"
-         value="&name :no_wrapper => true, :if_present => true"
+         class="autocompleter #{type_and_field._?.dasherize} #{css_data :complete_on, typed_id(complete_target), completer} #{css_data :min_chars, min_chars} #{'nil-value' if value==''}"
+         value="#{value=='' ? nil_value : value}"
          merge-attrs/>
   <div class="completions-popup" style="display:none"></div>
 </def>
@@ -976,8 +975,11 @@ Example javascript:
         }
       });
 
+Note: if your javascript does not work, please ensure that you have
+the Hobo version of lowpro.js.
+
   -->
-<def tag="input-many" attrs="minimum, fields, skip, template" polymorphic >
+<def tag="input-many" attrs="minimum, fields, skip, more-skip, template" polymorphic >
 <%
 # helper function to create id's on buttons to facilitate testing
 def underize(s)
@@ -987,10 +989,11 @@ end
   <set empty="&this.empty?"/>
   <% template ||= this.try.new_candidate || this.member_class.new %>
   <% minimum ||= 0 ; minimum = minimum.to_i %>
-  <% skip ||= this.proxy_reflection.klass.reflect_on_all_associations.detect {|p| p.primary_key_name==this.proxy_reflection.primary_key_name}.try.name.to_s if this.respond_to? :proxy_reflection %>           
-  <ul class="input-many #{this_field.dasherize} #{css_data :input_many_prefix, param_name_for_this} #{css_data(:minimum, minimum)}">
+  <% skip ||= this.proxy_reflection.klass.reflect_on_all_associations.detect {|p| p.primary_key_name==this.proxy_reflection.primary_key_name}.try.name.to_s if this.respond_to? :proxy_reflection %>
+  <% skip += ",#{more_skip}" if more_skip -%>
+  <ul class="input-many #{this_field.dasherize} #{css_data :input_many_prefix, param_name_for_this} #{css_data(:minimum, minimum)}" merge-attrs>
     <fake-field-context fake-field="-1" context="&template">
-      <li class="input-many-li input-many-template" id="#{param_name_for_this}">
+      <li class="input-many-li input-many-template" id="#{underize param_name_for_this}">
         <div class="input-many-item" param="default">
           <field-list param merge-attrs="fields" skip="&skip" />
         </div>
@@ -1000,13 +1003,13 @@ end
         </div>
       </li>
     </fake-field-context>
-    <li class="input-many-li empty #{'hidden' unless this.empty? and minimum==0}" id="#{param_name_for_this}[-1]_empty">
+    <li class="input-many-li empty #{'hidden' unless this.empty? and minimum==0}" id="#{underize param_name_for_this}_-1_empty">
       <!-- HACK way to signal an empty collection to the controller -->
-      <input type="hidden" class="empty-input" id="#{param_name_for_this}" name="#{param_name_for_this}" value="" disabled="&(!this.empty? || minimum>0)" />
+      <input type="hidden" class="empty-input" id="#{underize param_name_for_this}" name="#{param_name_for_this}" value="" disabled="&(!this.empty? || minimum>0)" />
       <fake-field-context fake-field="-1" context="&template">
         <div param="empty-message">
-          <ht key="#{this.class.name.tableize}.collection.empty_message">          
-            No <%= this.class.name.titleize.downcase.pluralize %>.
+          <ht key="#{this.class.class_name.tableize}.collection.empty_message">          
+            No <%= this.class.class_name.titleize.downcase.pluralize %>.
           </ht>
         </div>
         <div class="buttons">
@@ -1016,7 +1019,7 @@ end
       </fake-field-context>      
     </li>
     <fake-field-context fake-field="0" context="&template">
-      <li class="input-many-li" if="&(this_parent.empty? && minimum>0)" id="#{param_name_for_this}">
+      <li class="input-many-li" if="&(this_parent.empty? && minimum>0)" id="#{underize param_name_for_this}">
         <div class="input-many-item" param="default">
           <field-list param merge-attrs="fields" skip="&skip" />
         </div>
@@ -1026,7 +1029,7 @@ end
         </div>
       </li>
     </fake-field-context>       
-    <li repeat class="input-many-li #{'record-with-errors' unless this.errors.empty?}" id="#{param_name_for_this}">
+    <li repeat class="input-many-li #{'record-with-errors' unless this.errors.empty?}" id="#{underize param_name_for_this}">
       <error-messages without-heading class="sub-record"/>
       <hidden-id-field/>
       <div class="input-many-item" param="default">
@@ -1055,6 +1058,38 @@ end
   </ul>
 </def>
 
+<!-- An enhanced version of [`<input-many>`](/api_tag_defs/input-many)  that supports drag-and-drop re-ordering.
+
+Each item in the collection has a `<div class="ordering-handle" param="handle">` added, which can be used to drag the item up and down.
+
+If the items in the collection contain an [`acts_as_list`](http://ar.rubyonrails.org/classes/ActiveRecord/Acts/List/ClassMethods.html) declaration, it is updated.
+
+The specified sort order may be maintained even without `acts_as_list`.  The items will be passed to the controller in the correct order, so the order may be persisted there.
+
+### Attributes
+
+ - `id`:  Due to a limitation in script.aculo.us, an id is required.  If you do not supply one, one will be generated.
+ - `position-column`:  The position column may be specified via `acts_as_list`, via a `position_column` method on your model or via this attribute.
+ - others:  all other attributes are passed through to `<input-many>`
+
+-->
+
+<def tag="sortable-input-many" attrs="id, position-column, template">
+  <% this_id = this_parent.id || rand(100000) -%>
+  <% id ||= "sortable-input-many-#{this_parent.class.name.underscore.dasherize}-#{this_id}-#{this_field_reflection.name}" -%>
+  <% template ||= this.try.new_candidate || this.member_class.new %>
+  <% position_column ||= template.try.position_column -%>
+  <input-many merge id="&id" class="sortable-input-many" template="&template" more-skip="&position_column">
+    <default: replace>
+        <div class="ordering-handle" param="handle" if="&can_edit?">&uarr;<br/>&darr;</div>
+        <if test="&position_column">
+          <input class="sortable-position" type="hidden" value="&this.send(position_column)" name="#{param_name_for_this}[#{position_column}]" />
+        </if>
+        <default restore/>
+    </default:>
+  </input-many>
+</def>
+
 <!-- Renders the common "or (Cancel)" for a form. Attributes are merged into the link (`<a>Cancel</a>`), making it easy to customise the destination of the cancel link. By default it will link to `this` or `this.class`.
   -->
 <def tag="or-cancel">