@@ -6,17 +6,13 @@ module ApplicationHelper def less_form_for name, *args, &block - options = args.last.is_a?(Hash) ? args.pop : {} - options = options.merge(:builder=>LessFormBuilder) - args = (args << options) - form_for name, *args, &block + options = args.extract_options! + form_for name, *(args << options.merge(:builder=>LessFormBuilder)), &block end def less_remote_form_for name, *args, &block - options = args.last.is_a?(Hash) ? args.pop : {} - options = options.merge(:builder=>LessFormBuilder) - args = (args << options) - remote_form_for name, *args, &block + options = args.extract_options! + form_for name, *(args << options.merge(:builder=>LessFormBuilder)), &block end app/helpers/application_helper.rb @@ -29,7 +29,7 @@ class LessFormBuilder < ActionView::Helpers::FormBuilder "<label for='#{options.delete(:for)}'>#{text}</label>" else #need to use InstanceTag to build the correct ID for :for - ActionView::Helpers::InstanceTag.new(@object_name, method, self, nil, @object).to_label_tag(text, options) + ActionView::Helpers::InstanceTag.new(@object_name, method, self, @object).to_label_tag(text, options) end end lib/less_form_builder.rb 688d30a367657752bd33e5d11a32457350d83a18 Jonathon Brenner jon@ladidadi.net http://github.com/stevenbristol/lovd-by-less/commit/601cad8cafbaea79cbc16b77d1de0430d7bcdaae 601cad8cafbaea79cbc16b77d1de0430d7bcdaae 2008-11-29T18:24:08-08:00 2008-11-29T18:24:08-08:00 Fixed the call to the _undocumented_ InstanceTag. Bad Steven\! 52aa908f32ed8679787f029ed7da927d8063ff89 Jonathon Brenner jon@ladidadi.net