strong params

[rlmattax]

Do you guys have any suggestions on how to make the strong params suck less?

On the activity model:
galleryable :briefing_gallery, :debriefing_gallery

In the activities controller:
params.require(:activity).permit([:name, :description, :start_at, :end_at,
:briefing_gallery_attributes=>{:asset_attachments_attributes=>[:id,:asset_id,:sort_order]},
:debriefing_gallery_attributes=>{:asset_attachments_attributes=>[:id,:asset_id,:sort_order]}])

[johnkoht]

I've been thinking about this a bit. I think we need to abstract the strong params into a class, e.g. Assetabler::Gallery::permitted_params so you could call:

params.require(:activity).permit([
  :name,
  :description, 
  :start_at,
  :end_at,
  briefing_gallery_attributes: Assetabler::Gallery::permitted_params,
  debriefing_gallery_attributes: Assetabler::Gallery::permitted_params
])

We could add another for an asset: Assetabler::Asset::permitted_params

This is similar to how spree handles it -- https://github.com/spree/spree/blob/master/core/lib/spree/core/controller_helpers/strong_parameters.rb

[rlmattax]

I was thinking very similarly. I might try it in my own branch.