acts_as_multipart_form 0.0.5 → 0.0.6

data/CHANGELOG

@@ -1,3 +1,13 @@
+== unreleased changes
+
+== 0.0.6
+
+* Added indices to the migration
+* Updated the controller mixin to do a better job handling namespaced models
+* Clarified how the route needs to be setup in the readme
+* Changed the ordering of the default form partial in the controller mixin
+* Updated index links partial to not use the locals hash directly
+
 == 0.0.5
 
 * Updated gem requirements

data/README.rdoc

@@ -20,7 +20,9 @@ The parts correspond to methods in the controller and partials in the controller
 
 Each form has a polymorphic relationship with a single record.  This relationship is specified by the model key's value.  This is how the form is loaded on the index page and the only association it has with the rest of the system.
 
-Two routes can be specified for the form system. The form route corresponds to the route to the form itself.  It's default value is model_name_form_name.  In the example, the specified value would be the default value.  The show route, specified with the tag :show_route is redirected to after the last part of the multipart form is submitted and passes validations.  It defauls to model_name to correspond with the model's view page.
+Two routes can be specified for the form system. The form route corresponds to the route to the form itself.  It's default value is model_name_form_name.  In the example, the specified value would be the default value.  The show route, specified with the tag :show_route is redirected to after the last part of the multipart form is submitted and passes validations.  It defaults to model_name to correspond with the model's view page.
+
+Note that when you add the route to your route file, it must be able to match both get and post requests.
 
 == Add the mixin to your model
 

data/VERSION

@@ -1 +1 @@
-0.0.5
+0.0.6

data/acts_as_multipart_form.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "acts_as_multipart_form"
-  s.version = "0.0.5"
+  s.version = "0.0.6"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jeremiah Hemphill", "Ethan Pemble"]

data/app/views/multipart_form/_index_links.html.erb

@@ -1,16 +1,17 @@
 <div class='multipart_form_index_links'>
   <div class='multipart_form_index_links_links'>
-    <% @multipart_form_index_parts[locals[:form_subject].id][:parts].each do |part| %>
+    <% debugger %>
+    <% @multipart_form_index_parts[form_subject.id][:parts].each do |part| %>
       <% if ActsAsMultipartForm.config.use_numbered_parts_on_index %>
-        <%= link_to part[:number].to_s, send(@multipart_form_path, :id => locals[:form_subject].id, :multipart_form_part => part[:name]) %>
+        <%= link_to part[:number].to_s, send(@multipart_form_path, :id => form_subject.id, :multipart_form_part => part[:name]) %>
       <% else %>
-        <%= link_to part[:name].to_s.titleize, send(@multipart_form_path, :id => locals[:form_subject].id, :multipart_form_part => part[:name]) %>
+        <%= link_to part[:name].to_s.titleize, send(@multipart_form_path, :id => form_subject.id, :multipart_form_part => part[:name]) %>
       <% end %>
     <% end %>
   </div>
   <% if ActsAsMultipartForm.config.show_completed %>
     <div class='multipart_form_index_links_complete'>
-      <% if @multipart_form_index_parts[locals[:form_subject].id][:completed] %>
+      <% if @multipart_form_index_parts[form_subject.id][:completed] %>
         Complete
       <% else %>
         Incomplete

data/lib/acts_as_multipart_form/multipart_form_in_controller.rb

@@ -67,8 +67,8 @@ module ActsAsMultipartForm
           end
           # sets default model if it is not set
           arg[:model] = self.to_s.gsub("Controller", "").singularize unless arg.has_key?(:model)
-          arg[:form_route] = (arg[:model].downcase + "_" + arg[:name].to_s) unless arg.has_key?(:form_route)
-          arg[:show_route] = (arg[:model].downcase) unless arg.has_key?(:show_route)
+          arg[:form_route] = (arg[:name].to_s + "_" + arg[:model].gsub("::", "").underscore) unless arg.has_key?(:form_route)
+          arg[:show_route] = (arg[:model].gsub("::", "").underscore) unless arg.has_key?(:show_route)
           # copy args to fields
           self.multipart_forms[arg[:name]] = arg
           forms << arg[:name]

data/lib/generators/acts_as_multipart_form/templates/migrations/install_migration.rb.erb

@@ -9,6 +9,8 @@ class <%= migration_class_name %> < ActiveRecord::Migration
 
       t.timestamps
     end
+    add_index :multipart_form_in_progress_forms, [:form_subject_type, :form_subject_id], :name => "multipart_form_ipf_form_subject"
+    add_index :multipart_form_in_progress_forms, :form_name
   end
   
   def self.down

data/spec/dummy/app/views/people/index.html.erb

@@ -9,7 +9,7 @@
       <td><%= person.id %></td>
       <td><%= person.name %></td>
       <td>
-        <%= render "multipart_form/index_links", :locals => {:form_subject => person } %>
+        <%= render :partial => "multipart_form/index_links", :locals => {:form_subject => person } %>
       </td>
     </tr>
   <% end %>

metadata

@@ -2,7 +2,7 @@
 name: acts_as_multipart_form
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors: 
 - Jeremiah Hemphill
@@ -257,7 +257,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -873948551073501972
+      hash: 3542899139585014364
       segments: 
       - 0
       version: "0"