Could you explain why the extra method in the controller bothers you? Thanks,
-Amiel On Wed, Aug 27, 2014 at 2:55 AM, Elad Meidar <[email protected]> wrote: > Hi All, > > Since strong parameters came out I was struggling with the thought that I > need to keep an extra method around in the controller for parameter > whitelisting. It was absolutely right to move the responsibility to > whitelist end user submissions from the ActiveModel to ActionController - > but the extra method in the controller was bothering me still. > > So i did this thing: > https://github.com/ShinobiDevs/better_strong_params/blob/master/lib/better_strong_params.rb > > (Not as sharp as i would expect, consider this a WIP) > > Does this look like a good direction to go? good enough for a core PR when > i’m done or should i leave it as a gem? > — > Elad Meidar, > shinobidevs.com > > > On Tue, Aug 26, 2014 at 2:48 PM, [email protected] < > [email protected]> wrote: > >> [email protected] >> <https://groups.google.com/forum/?utm_source=digest&utm_medium=email/#!forum/rubyonrails-core/topics%0D%0A> >> Google >> Groups >> <https://groups.google.com/forum/?utm_source=digest&utm_medium=email/#!overview> >> <https://groups.google.com/forum/?utm_source=digest&utm_medium=email/#!overview> >> Today's topic summary >> View all topics >> <https://groups.google.com/forum/?utm_source=digest&utm_medium=email/#!forum/rubyonrails-core/topics%0D%0A> >> >> - why #add_column doesn't support an :index option? >> <#14816e4af26d0e7e_group_thread_0> - 1 Update >> >> why #add_column doesn't support an :index option? >> <http://groups.google.com/group/rubyonrails-core/t/1254ff98693f2d29?utm_source=digest&utm_medium=email> >> powiii <[email protected]>: Aug 25 12:10PM -0700 >> >> I'm curious how come the #add_column method in migrations does not >> support >> an optional :index option, which would also create an index for that >> column. >> >> Since there is the possibility to do ...more >> <http://groups.google.com/group/rubyonrails-core/msg/e9bd3575e59f7114?utm_source=digest&utm_medium=email> >> Back to top <#14816e4af26d0e7e_digest_top> >> You received this digest because you're subscribed to updates for this >> group. You can change your settings on the group membership page >> <https://groups.google.com/forum/?utm_source=digest&utm_medium=email/#!forum/rubyonrails-core/join%0D%0A> >> . >> To unsubscribe from this group and stop receiving emails from it send an >> email to [email protected]. >> > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" 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]. > Visit this group at http://groups.google.com/group/rubyonrails-core. > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" 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]. Visit this group at http://groups.google.com/group/rubyonrails-core. For more options, visit https://groups.google.com/d/optout.
