ruby on rails - modelo - Rails 4 NO actualizando atributos anidados

formularios anidados rails

Question

Problema: en lugar de actualizar los atributos anidados, se crean sobre los atributos anidados existentes cuando #update acción #update del features_controller.rb asociado

Causa probable: creo que el problema radica en mi falta de comprensión en el form_for Rails. Creo que el desglose está en mis vistas, cómo renderizo los atributos anidados persistentes y / o cómo no puedo especificar el ID del atributo anidado, lo que hace que se cree simplemente uno nuevo.

feature.rb

class Feature < ActiveRecord::Base
...
has_many :scenarios
accepts_nested_attributes_for :scenarios,
allow_destroy: true,
reject_if: :all_blank
...
end

features_controller.rb

def update
...
project = Project.find(params[:project_id])
@feature = Feature.find(params[:id])
if @feature.update_attributes(feature_params)
# checking feature_params looks good...
# feature_params[''scenarios''] => { <correct object hash> }
redirect_to project
else
render :edit
end
end
...
private
def feature_params
params.require(:feature).permit(:title, :narrative, :price, :eta, scenarios_attributes[:description, :_destroy])
end

_form.html.haml (simplificado)

= form_for [@project, @feature] do |f|
...
- if @feature.new_record? -# if we are creating new feature
= f.fields_for :scenarios, @feature.scenarios.build do |builder|
= builder.label :description, "Scenario"
= builder.text_area :description, rows: "3", autocomplete: "off"
- else -# if we are editing an existing feature
= f.fields_for :scenarios do |builder|
= builder.label :description, "Scenario"
= builder.text_area :description, rows: "3", autocomplete: "off"

Estoy seguro de que hay una mejor manera de lograr el if @feature.new_record? comprobar. También estoy usando algunos enlaces de Javascript para crear formas de atributos anidados dinámicos (que he omitido ), muy influenciados por el Formulario de modelo anidado de Railscast # 196 (revisado)

Me encantaría una implementación realmente agradable de Rails-y para tratar con este tipo de formas anidadas.

Answer 1

Como @ Philip7899 mencionó como un comentario en la respuesta aceptada, permitir que el usuario establezca la id significa que podrían "robar" los registros secundarios que pertenecen a otro usuario.

Sin embargo , Rails accepts_nested_attributes_for realidad comprueba el id y plantea:

ActiveRecord::RecordNotFound:
Couldn''t find Answer with ID=5 for Questionnaire with ID=5

Básicamente, las identificaciones se buscan en la asociación de niños (de nuevo, como dijo @glampr). Por lo tanto, el registro hijo que pertenece a otro usuario no se encuentra.

En última instancia, 401 es el estado de respuesta (a diferencia del 404 habitual de ActiveRecord::RecordNotFound )

Sigue algunos códigos que usé para probar el comportamiento.

let :params do
{
id: questionnaire.id,
questionnaire: {
participation_id: participation.id,
answers_attributes: answers_attributes
}
}
end
let :evil_params do
params.tap do |params|
params[:questionnaire][:answers_attributes][''0''][''id''] = another_participant_s_answer.id.to_s
end
end
it "doesn''t mess with other people''s answers" do
old_value = another_participant_s_answer.value
put :update, evil_params
expect(another_participant_s_answer.reload.value).to eq(old_value) # pass
expect(response.status).to eq(401) # pass
end

En conclusión , agregar el id a los parámetros permitidos como se indicó anteriormente es correcto y seguro .

Carriles fascinantes.

Answer 2

Intente agregar :id a la parte :scenario_attributes de su método feature_params . Solo tienes el campo de descripción y la capacidad de permitir una destrucción.

def feature_params
# added => before nested attributes
params.require(:feature).permit(:id, :title, :narrative, :price, :eta, scenarios_attributes => [:id, :description, :_destroy])
end

Como lo sugirió @vinodadhikary, ya no necesita verificar si la función es un nuevo registro, ya que Rails, específicamente utilizando el método form_for , lo hará por usted.

Actualizar:

No necesita definir if @feature.new_record? ... else if @feature.new_record? ... else en tu forma. Rails lo cuidará cuando uses form_for . Rails verifica si la acción se create o update función de object.persisted? Entonces, puedes actualizar tu formulario a:

= form_for [@project, @feature] do |f|
...
= f.fields_for :scenarios, @feature.scenarios.build do |builder|
= builder.label :description, "Scenario"
= builder.text_area :description, rows: "3", autocomplete: "off"