Rails - Issue with permitting params & require - ruby-on-rails

Question

I'm having an issue with submitting my params,
portrait and portrait_tag are passed through my form and are siblings,
how would I go permitting both of these?
Output
{"utf8"=>"✓", "_method"=>"patch", "authenticity_token"=>"", "portrait"=>{"artist_image"= "", #original_filename="rubytraining.jpg", #content_type="image/jpeg"}, "portrait_tag"=>{"tag_ids"=>["", "1", "2", "3", "4"]}, "commit"=>"Save changes", "controller"=>"admin/portraits", "action"=>"update", "id"=>"72"}
I've tried the following
private
def portrait_params
params.require(:portrait).permit(:id, :artist_image)
params.require(:portrait_tag).permit(:id, :tag => [])
end
These work separately but overwrite one another when added together
controller
def update
#portrait = Portrait.where(artist_id: 33, id: params[:id]).take
if #portrait.update(portrait_params)
redirect_to :edit_admin_portrait, flash: {notice: "Successfully updated your information"}
else
flash[:system] = #portrait.errors.full_messages
p #portrait.errors.full_messages
render :edit
end
end
private
def portrait_params
params.require(:portrait).permit(:id, :artist_image)
params.require(:portrait_tag).permit(:id, :tag => [])
end
Edit Form
%h1 Edit Portrait
= form_for [:admin, #portraits] do |f|
- if flash[:system].present?
- flash[:system].each do |e|
%div= e
- if flash[:notice].present?
%div= flash[:notice]
= f.file_field :artist_image
= collection_select :portrait_tag, :tag_ids, Tag.all, :id, :name, {:selected => #portraits.tag_ids}, { :multiple => true}
= f.submit "Save changes", class: "btn btn-primary"

Answer 1

the tags are an association of the portrait (portrait_tags).
Finally, the core of the problem. In this case, you should be using accepts_nested_attributes_for. It will allow you to post attributes for tags inside your :portrait params.
So your strong params method would look like this:
def portrait_params
params.require(:portrait).permit(:id, :artist_image, portrait_tags_attributes: [:id, :tag])
end
Of course, amend your forms accordingly.

Answer 2

You are returning only last line of portrait_params.
params.require(:portrait_tag).permit(:id, :tag => [])
I'd do it like this:
Make two private methods with params(portrait, portrait_tag) , they will return you Hash'es, then you can merge them. It is not clear to have method portrait_params which return you
portrait_tag params if they are not in nested attributes.
If they are you can use but you have to add this to model
method accepts_nested_attributes_for
and then you can do it like this
def portrait_params
params.require(:portrait).permit(:id, :artist_image, portrait_tags_attributes: [:id, :tag]))
end

Answer 3

You can use nested_attributes to update parent and child same time. But you have to made some changes, for instance you need to change params structure.
Ex:
params.require(:portrait).permit(:attr1,:att2,..., portrait_attributes: [:attr1, :attr2])
Ref: http://api.rubyonrails.org/classes/ActiveRecord/NestedAttributes/ClassMethods.html