This project is archived and is in readonly mode.
index does not work on fields_for select
fields_for :post, @post, :index => 1 do |f|
f.select(:category, %w( abe <mus> hest))
end
currently does not add the index to the id and name.
This patch fixes and tests for it (including the other select methods)
It also adds tests to prove that f.select works with auto_index
-
index-on-fields_for.select.patch
7.27 KB
-
RSLMerged Kevin's patch and my old patch for what I feel like is a better patch. Any love?
-
RSL
Updated to right format. My bad :(
-
-
Repository- State changed from new to resolved
(from [95812d5eafc3b63ce5eeb0748a5d0132f5108b64]) Ensure :index works with fields_for select methods. [#518 state:resolved]
Signed-off-by: Pratik Naik
-
Tom StuartCan we have the same treatment for date_helper.rb, please?
-
Dewey Bushaw-
patch.diff
3.69 KB
-
-
Dewey Bushaw
Is this the same issue as stated here (http://dev.rubyonrails.org/ticke...?
Seems to me that date_helper.rb needs to be updated as Tom Stuart pointed out.
Patch on Rails Trac works (I did not run the test) but basically it changes the following:
elsif @auto_index to elsif defined?(@auto_index)
Attached patch.diff which includes test updates but this was for rails v.2.0.1. This must have gotten lost during the switch to Lighthouse.
-
Tom Stuart
Resolved by this commit.