Thanks everyone for the help. I didn't end up figuring out what was causing the problem but changing the code to current_user.Objective... fixed the problem and makes a lot more sense.
I hope not too many puppies were hurt during this exchange :). On Oct 26, 10:56 am, Marnen Laibow-Koser <rails-mailing-l...@andreas- s.net> wrote: > 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-Koserhttp://www.marnen.org > [email protected] > -- > Posted viahttp://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 -~----------~----~----~----~------~----~------~--~---
