You said at the top that there is no model to get the values from, so how can there be a saved value in the db?
Colin
Assuming that you have form_for @some_object do |f| where @some_object is the object being edited and that @some_object.year_made contains the previous selection then the dropdown should default to the current selection, I think.
Colin
Well with this code,
<%= f.select(:year_made, ((1980..Time.now.year).collect {|p| [ p, p ]}).reverse, {:prompt=>"Select a Year"}) %>
it does not seem to work and that makes me wonder what is wrong with this code?
What does the form_for line look like? What does generated html of the select look like? (View, Page Source or similar in your browser) Have you checked that year_made contains the previous value. Possibly put <%= @object.year_made %> in the form to check.
Colin
Odd, it all looks ok to me. Is year_made an integer or a string? I wonder whether it is expecting an integer as all my uses have always been with an id value.
Any other ideas anyone?
Colin
Well it turns out that it was a string vs integer issue. I have my fields in db as string and the code was generating a integer so that caused issues at edit time.
I made the following change and now it seems to work.
<%= f.select(:year_made, ((1980..Time.now.year).collect {|p| [ p.to_s, p.to_s ]}).reverse, {:prompt=>"Select a Year"}) %>
Does anyone else have a better idea on how to do this?
No. This is the right way to do it. Data type matches are necessary for functionality, you can't ignore those.
Well it turns out that it was a string vs integer issue. I have my fields in db as string and the code was generating a integer so that caused issues at edit time.
I made the following change and now it seems to work.
<%= f.select(:year_made, ((1980..Time.now.year).collect {|p| [ p.to_s, p.to_s ]}).reverse, {:prompt=>"Select a Year"}) %>
Does anyone else have a better idea on how to do this?
You probably only need the to_s on one of them not both.
Presumably it would also work if, instead, the type of year_made were changed to integer. Arguably this might be a more aesthetically pleasing solution.
Colin
I agree with you on that. And will think about converting to integer, I do have two more drop downs with similar values but they do have one or more string values so they do require to be strings but the year_made does not.
Hi,
What essentially you are doing here is creating an array. So you can prepend “Local” in the array like this
((1980…Time.now.year).collect {|p| [p.to_s, p.to_s ]}).reverse.unshift “Local”
I will also suggest you move out this array creation, and put it as a helper method.
