From 21837821a804f6c26a0672f9ce5697e198624968 Mon Sep 17 00:00:00 2001 From: Michael Koziarski Date: Sun, 2 Sep 2007 23:53:31 +0000 Subject: Add array support to remote_form_for for polymorphic urls. Closes #8654 [jade] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@7400 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- .../lib/action_view/helpers/prototype_helper.rb | 50 ++++++++++++++++--- actionpack/test/template/prototype_helper_test.rb | 56 +++++++++++++++++----- 2 files changed, 87 insertions(+), 19 deletions(-) (limited to 'actionpack') diff --git a/actionpack/lib/action_view/helpers/prototype_helper.rb b/actionpack/lib/action_view/helpers/prototype_helper.rb index 0da5cc5579..4ece6f8709 100644 --- a/actionpack/lib/action_view/helpers/prototype_helper.rb +++ b/actionpack/lib/action_view/helpers/prototype_helper.rb @@ -180,16 +180,54 @@ module ActionView form_tag(options[:html].delete(:action) || url_for(options[:url]), options[:html], &block) end - # Works like form_remote_tag, but uses form_for semantics. - def remote_form_for(record_or_name, *args, &proc) + # Creates a form that will submit using XMLHttpRequest in the background + # instead of the regular reloading POST arrangement and a scope around a + # specific resource that is used as a base for questioning about + # values for the fields. + # + # === Resource + # + # Example: + # <% remote_form_for(@post) do |f| %> + # ... + # <% end %> + # + # This will expand to be the same as: + # + # <% remote_form_for :post, @post, :url => post_path(@post), :html => { :method => :put, :class => "edit_post", :id => "edit_post_45" } do |f| %> + # ... + # <% end %> + # + # === Nested Resource + # + # Example: + # <% remote_form_for([@post, @comment]) do |f| %> + # ... + # <% end %> + # + # This will expand to be the same as: + # + # <% remote_form_for :comment, @comment, :url => post_comment_path(@post, @comment), :html => { :method => :put, :class => "edit_comment", :id => "edit_comment_45" } do |f| %> + # ... + # <% end %> + # + # If you don't need to attach a form to a resource, then check out form_remote_tag. + # + # See FormHelper#form_for for additional semantics. + def remote_form_for(record_or_name_or_array, *args, &proc) options = args.extract_options! - case record_or_name + case record_or_name_or_array when String, Symbol - object_name = record_or_name + object_name = record_or_name_or_array + when Array + object = record_or_name_or_array.last + object_name = ActionController::RecordIdentifier.singular_class_name(object) + apply_form_for_options!(record_or_name_or_array, options) + args.unshift object else - object = record_or_name - object_name = ActionController::RecordIdentifier.singular_class_name(record_or_name) + object = record_or_name_or_array + object_name = ActionController::RecordIdentifier.singular_class_name(record_or_name_or_array) apply_form_for_options!(object, options) args.unshift object end diff --git a/actionpack/test/template/prototype_helper_test.rb b/actionpack/test/template/prototype_helper_test.rb index a26ca680a8..a0217d84d1 100644 --- a/actionpack/test/template/prototype_helper_test.rb +++ b/actionpack/test/template/prototype_helper_test.rb @@ -11,6 +11,16 @@ class Author end end +class Article + attr_reader :id + attr_reader :author_id + def save; @id = 1; @author_id = 1 end + def new_record?; @id.nil? end + def name + @id.nil? ? 'new article' : "article ##{@id}" + end +end + class Author::Nested < Author; end @@ -30,6 +40,7 @@ module BaseTest include ActionView::Helpers::FormHelper include ActionView::Helpers::CaptureHelper include ActionView::Helpers::RecordIdentificationHelper + include ActionController::PolymorphicRoutes def setup @template = nil @@ -59,7 +70,8 @@ class PrototypeHelperTest < Test::Unit::TestCase include BaseTest def setup - @record = Author.new + @record = @author = Author.new + @article = Article.new super end @@ -114,7 +126,7 @@ class PrototypeHelperTest < Test::Unit::TestCase _erbout = '' remote_form_for(@record, {:html => { :id => 'create-author' }}) {} - expected = %(
) + expected = %(
) assert_dom_equal expected, _erbout end @@ -122,7 +134,7 @@ class PrototypeHelperTest < Test::Unit::TestCase _erbout = '' remote_form_for(@record) {} - expected = %(
) + expected = %(
) assert_dom_equal expected, _erbout end @@ -131,7 +143,25 @@ class PrototypeHelperTest < Test::Unit::TestCase _erbout = '' remote_form_for(@record) {} - expected = %(
) + expected = %(
) + assert_dom_equal expected, _erbout + end + + def test_remote_form_for_with_new_object_in_list + _erbout = '' + remote_form_for([@author, @article]) {} + + expected = %(
) + assert_dom_equal expected, _erbout + end + + def test_remote_form_for_with_existing_object_in_list + @author.save + @article.save + _erbout = '' + remote_form_for([@author, @article]) {} + + expected = %(
) assert_dom_equal expected, _erbout end @@ -233,20 +263,20 @@ class PrototypeHelperTest < Test::Unit::TestCase protected - def author_url(record) + def author_path(record) "/authors/#{record.id}" end - def authors_url + def authors_path "/authors" end - - def polymorphic_path(record) - if record.new_record? - "/authors" - else - "/authors/#{record.id}" - end + + def author_articles_path(author) + "/authors/#{author.id}/articles" + end + + def author_article_path(author, article) + "/authors/#{author.id}/articles/#{article.id}" end end -- cgit v1.2.3