On Sunday, February 2, 2014 2:10:10 PM UTC, Colin Law wrote: > On 2 February 2014 13:57, Frederick Cheung <[email protected]> wrote: > > > ... > > > You could update params in place , ie params.merge!(...). > > > > Would that not use the value from defaults if it was already present in > params? >
I was assuming this was after checking that params didn't have the values in question, although if not you could use reverse_merge! Fred > > > > > > > Personally I wouldn't do this. I'd be more likely to have a controller > > method called something like get_date_range that would get the dates from > > params that would do things like parse the strings into actual dates and/or > > replace missing params with defaults. > > > > +1 > > > > Colin -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/581a9dc1-482c-4adb-8291-a8b1ff6a650c%40googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.

