ruby on rails - Rails 4 borra atributos anidados, funciona en crear pero no en editar/actualizar

ruby-on-rails ruby-on-rails-3

Question

Así que estoy trabajando desde este Railscast .

Y soy consciente de que ha habido algunos cambios en Rails 4 para los parámetros Strong.

Primera pregunta relevante .

Segunda pregunta relevante

He revisado cuádruple mi implementación, pero no puedo ver dónde me equivoco. Tal como está en este momento, marcar la casilla "destruir" cuando se envía al paciente inicialmente (es decir, el método de creación) funciona según lo previsto, y eliminará cualquier medicamento que tenga una casilla marcada y permitirá que no lo haga (de las tres entradas del formulario proporciona).

Sin embargo, cuando posteriormente edito a ese paciente, todos los medicamentos que no se verifican para ser eliminados se duplican (por lo que termino con más medicamentos adjuntos de los que comencé), y los que se verifican para su eliminación no parecen cambiar.

Entonces, si hay dos medicamentos adjuntos "Med1" y "Med2", y edito al paciente, si ambos están marcados para su eliminación, terminaré con "Med1" y "Med2". Si solo se marca "Med1" para su eliminación, terminaré con "Med1" y "Med2" y un "Med2" adicional. Si ninguno de los dos está marcado para su eliminación, terminaré con dos de cada "Med1" y "Med2".

#patient.rb
class Patient < ActiveRecord::Base
has_many :procedures
has_many :medications, dependent: :destroy
has_many :previous_operations, dependent: :destroy

accepts_nested_attributes_for :medications, :allow_destroy => true, :reject_if => lambda { |a| a[:name].blank? },
end

#views/patients/_form.html.erb
<%= form_for(@patient) do |f| %>
  <% if @patient.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(@patient.errors.count, "error") %> prohibited this patient from being saved:</h2>

      <ul>
      <% @patient.errors.full_messages.each do |msg| %>
        <li><%= msg %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <%= f.fields_for :medications do |builder| %>
    <%= render "medication_fields", :f => builder %>
  <% end %>

  <div class="field">
    <%= f.label :first_name %><br>
    <%= f.text_field :first_name %>
  </div>
  <div class="field">
    <%= f.label :last_name %><br>
    <%= f.text_field :last_name %>
  </div>
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

#views/patients/medications_fields.html
<div class="field">
  <%= f.label :name %><br>
  <%= f.text_field :name %>
</div>
<div class="field">
  <%= f.label :_destroy, "Remove Medication" %>
  <%= f.check_box :_destroy %>
</div>

#controllers/patients_controller.rb
class PatientsController < ApplicationController
  before_action :set_patient, only: [:show, :edit, :update, :destroy]

  # GET /patients
  # GET /patients.json
  def index
    @patients = Patient.all
  end

  # GET /patients/1
  # GET /patients/1.json
  def show
  end

  # GET /patients/new
  def new
    @patient = Patient.new
    3.times { @patient.medications.build }
  end

  # GET /patients/1/edit
  def edit
  end

  # POST /patients
  # POST /patients.json
  def create
    @patient = Patient.new(patient_params)

    respond_to do |format|
      if @patient.save
        format.html { redirect_to @patient, notice: ''Patient was successfully created.'' }
        format.json { render action: ''show'', status: :created, location: @patient }
      else
        format.html { render action: ''new'' }
        format.json { render json: @patient.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /patients/1
  # PATCH/PUT /patients/1.json
  def update
    respond_to do |format|
      if @patient.update(patient_params)
        format.html { redirect_to @patient, notice: ''Patient was successfully updated.'' }
        format.json { head :no_content }
      else
        format.html { render action: ''edit'' }
        format.json { render json: @patient.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /patients/1
  # DELETE /patients/1.json
  def destroy
    @patient.destroy
    respond_to do |format|
      format.html { redirect_to patients_url }
      format.json { head :no_content }
    end
    flash[:notice] = "Patient was successfully deleted."
  end

  private

    # Never trust parameters from the scary internet, only allow the white list through.
    def patient_params
      params.require(:patient).permit(:first_name, :last_name, medications_attributes: [:name, :_destroy])
    end

end

He sido súper cuidadoso y he comprobado más de un millón de veces que se permite: _destroy flag a través de parámetros fuertes, pero aún no hay dados.

Cualquier ayuda apreciada, debe ser algo obvio que simplemente no puedo ver.

EDITAR

Cambiando esto ...

    # Never trust parameters from the scary internet, only allow the white list through.
    def patient_params
      params.require(:patient).permit(:first_name, :last_name, medications_attributes: [:name, :_destroy])
    end

A esto...

    # Never trust parameters from the scary internet, only allow the white list through.
    def patient_params
      params.require(:patient).permit!
    end

Parece funcionar correctamente, así que todavía estoy seguro de que es algo que tiene que ver con parámetros sólidos, pero estoy seguro de que este último es menos seguro y no es la mejor práctica.

Answer 1

cuando estas haciendo

# Never trust parameters from the scary internet, only allow the white list through.
def patient_params
  params.require(:patient).permit!
end

Permite todos tus atributos y no es recomendable . Es más un hack que una solución.

Si miras tu pregunta

Rails 4 deleting nested attributes, works on create but not on edit/update

y si miras a tus params permitidos

# Never trust parameters from the scary internet, only allow the white list through.
def patient_params
  params.require(:patient).permit(:first_name, :last_name, medications_attributes: [:name, :_destroy])
end

Esto funcionará para crear pacientes, pero no para actualizarlos o editarlos, porque cuando creas un nuevo registro, no es necesario que permitas la identificación, pero cuando quieres actualizar o editar un registro también necesitas que se permita su identificación .

Fijar:

Simplemente pase el atributo id a los atributos permitidos y funcionará para usted

# Never trust parameters from the scary internet, only allow the white list through.
def patient_params
  params.require(:patient).permit(:id, :first_name, :last_name, medications_attributes: [:id,:name, :_destroy])
end