How do I set a time in a time_select view helper?

Question

I have a time_select in which I am trying to set a time value as follows;

<%= f.time_select :start_time, :value => (@invoice.start_time ? @invoice.start_time : Time.now) %>

This always produces a time selector with the current time rather than the value for @invoice.start_time.

@invoice.start_time is in fact a datetime object but this is passed to the time selector just fine if I use

<%= f.time_select :start_time %>

I guess what I'm really asking is how to use the :value option with the time_select helper. Attempts like the following don't seem to produce the desired result;

<%= f.time_select :start_time, :value => (Time.now + 2.hours) %>
<%= f.time_select :start_time, :value => "14:30" %>

Answer 1

Has @invoice.start_time have a value assigned to it? I guess not. @invoice.start_time will return nil if you use that code.. and hence the :value will always default to Time.now. The problem here is the conditional statement that you are using. I am assuming this is happening when you try to create new data. When you filling up the form, @invoice.start_time is not filled with any value. Hence its nil throughout till you save.

I would suggest that you change the code to:

<%= f.time_select :start_time, :value => @invoice.start_time, :default => Time.now %>

Actually if you could be more clear in your question as to what you want your time_select helper to do then it would make things easier.