drewB wrote: > On Oct 26, 9:51�am, Marnen Laibow-Koser <rails-mailing-l...@andreas- > s.net> wrote: >> >> No! �Create shouldn't be a controller method. > > Why not?
Sorry, that was a flash of abject stupidity on my part. :) Please feel free to ignore it. > >> >> >> > And as others have said, don't overwrite >> >> over*ride* >> >> > AR class methods like >> > 'create', otherwise you will enter a world of pain. > > Thanks for the good practice tip. > >> > Also, you are deleting a param called 'freq_options', if the model >> > does not need it then it shouldn't be in the hash. If the controller >> > needs it, then pass it as params[:freq_options] rather than params >> > [:objective][:freq_options]. If it's not used there, then why pass it >> > at all? >> >> Good catch! >> > > It will eventually be used, just not yet. Then you don't need it. Put it in when you do. Best, -- Marnen Laibow-Koser http://www.marnen.org [email protected] -- Posted via http://www.ruby-forum.com/. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
