Rails 4 удаляет вложенные атрибуты, работает при создании, но не при редактировании / обновлении
Так что я работаю с этимRailscast.
И я знаю, что в Rails 4 произошли некоторые изменения для параметров Strong.
Я четыре раза проверил свою реализацию, но не вижу, где я иду не так. Как и на данный момент, отметка поля «уничтожить» при первоначальной отправке пациента (т. Е. Метод создания) работает как задумано, и удалит любое лекарство, которое имеет отмеченный флажок, и разрешит любое, которое не имеет (из трех входов формы) это обеспечивает).
Однако, когда я впоследствии редактирую этого пациента, любые лекарства, которыене Проверка на удаление дублируется (поэтому я получаю больше лекарств, чем я начал), и все, которые проверены на удаление, похоже, не меняются.
Поэтому, если к двум лекарствам прикреплены «Med1» и «Med2», и я редактирую пациента, если оба помечены для удаления, я все равно получу «Med1» и «Med2». Если только «Med1» помечен для удаления, я получу «Med1» и «Med2» и дополнительный «Med2». Если ни один из них не помечен для удаления, я получу по два «Med1» и «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
Я был очень осторожен и более миллиона раз проверял: _destroy флаг, разрешенный через строгие параметры, но все еще без кубиков.
Любая помощь оценена, должно быть что-то очевидное, чего я просто не вижу.
РЕДАКТИРОВАТЬ
Меняя это ...
# 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
К этому...
# Never trust parameters from the scary internet, only allow the white list through.
def patient_params
params.require(:patient).permit!
end
Кажется, работает правильно, так что я все еще уверен, что это связано с сильными параметрами, но последний менее безопасен, я уверен, и не лучшая практика.