diff options
author | David Heinemeier Hansson <david@loudthinking.com> | 2006-02-12 00:22:24 +0000 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2006-02-12 00:22:24 +0000 |
commit | a04a849d155997d0c3a6fcae4688a45c421e7a00 (patch) | |
tree | c48f0ba561eb97d1ed9c49059d70fb68f6348f86 /actionpack/lib | |
parent | 344ae8326a51beabbc5b3f4f851c3d65ec789e2d (diff) | |
download | rails-a04a849d155997d0c3a6fcae4688a45c421e7a00.tar.gz rails-a04a849d155997d0c3a6fcae4688a45c421e7a00.tar.bz2 rails-a04a849d155997d0c3a6fcae4688a45c421e7a00.zip |
Clean up style around form_for and friends a bit
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@3573 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'actionpack/lib')
-rw-r--r-- | actionpack/lib/action_view/helpers/form_helper.rb | 11 | ||||
-rw-r--r-- | actionpack/lib/action_view/helpers/prototype_helper.rb | 4 |
2 files changed, 7 insertions, 8 deletions
diff --git a/actionpack/lib/action_view/helpers/form_helper.rb b/actionpack/lib/action_view/helpers/form_helper.rb index 81f6fec827..ab387b7bf8 100644 --- a/actionpack/lib/action_view/helpers/form_helper.rb +++ b/actionpack/lib/action_view/helpers/form_helper.rb @@ -121,9 +121,10 @@ module ActionView # end # def form_for(object_name, object, options = {}, &proc) - concat form_tag(options, options.delete(:html) || {}), proc.binding + raise ArgumentError, "Missing block" unless block_given? + concat(form_tag(options, options.delete(:html) || {}), proc.binding) fields_for(object_name, object, options, &proc) - concat '</form>', proc.binding + concat('</form>', proc.binding) end # Creates a scope around a specific model object like form_for, but doesn't create the form tags themselves. This makes @@ -141,10 +142,8 @@ module ActionView # Note: This also works for the methods in FormOptionHelper and DateHelper that are designed to work with an object as base. # Like collection_select and datetime_select. def fields_for(object_name, object, options = {}, &proc) - raise ArgumentError, "fields_for requires a block!" unless block_given? - builder_klass = options[:builder] || FormBuilder - form_builder = builder_klass.new(object_name, object, self, options, proc) - yield form_builder + raise ArgumentError, "Missing block" unless block_given? + yield (options[:builder] || FormBuilder).new(object_name, object, self, options, proc) end # Returns an input tag of the "text" type tailored for accessing a specified attribute (identified by +method+) on an object diff --git a/actionpack/lib/action_view/helpers/prototype_helper.rb b/actionpack/lib/action_view/helpers/prototype_helper.rb index 96f065cc11..8aabe04aac 100644 --- a/actionpack/lib/action_view/helpers/prototype_helper.rb +++ b/actionpack/lib/action_view/helpers/prototype_helper.rb @@ -172,9 +172,9 @@ module ActionView # Works like form_remote_tag, but uses form_for semantics. def remote_form_for(object_name, object, options = {}, &proc) - concat form_remote_tag(options), proc.binding + concat(form_remote_tag(options), proc.binding) fields_for(object_name, object, options, &proc) - concat '</form>', proc.binding + concat('</form>', proc.binding) end alias_method :form_remote_for, :remote_form_for |