Вложенные атрибуты RoR создают дубликаты при редактировании

Я пытаюсь следовать за Райаном Бейтсом RailsCast #196: Форма вложенной модели, часть 1 . Есть два очевидных отличия от версии Райана: 1) я использую встроенные леса, а не изящные, как он, и 2) я запускаю рельсы 4 (я действительно не знаю, какую версию Райанс использует в своем гипсе). , но это не 4).

Итак, вот что я сделал

rails new survey2
cd survey2
bundle install
rails generate scaffold survey name:string
rake db:migrate
rails generate model question survey_id:integer content:text
rake db:migrate

Затем я добавил ассоциации к моделям вот так

class Question < ActiveRecord::Base
  belongs_to :survey
end

и так

class Survey < ActiveRecord::Base
  has_many :questions
  accepts_nested_attributes_for :questions
end

Затем я добавил часть вложенного представления

<%= form_for(@survey) do |f| %>
  <!-- Standard rails 4 view stuff -->

  <div class="field">
    <%= f.label :name %><br>
    <%= f.text_field :name %>
  </div>
  <div class="field">
    <%= f.fields_for :questions do |builder| %>
      <div>
        <%= builder.label :content, "Question" %><br/>
        <%= builder.text_area :content, :rows => 3 %>
      </div>
    <% end %>
  </div>
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

и, наконец, контроллер, чтобы при создании нового опроса создавались 3 вопроса.

class SurveysController < ApplicationController
  before_action :set_survey, only: [:show, :edit, :update, :destroy]

  # Standard rails 4 index and show 

  # GET /surveys/new
  def new
    @survey = Survey.new
    3.times { @survey.questions.build }
    Rails.logger.debug("New method executed")
  end

  # GET /surveys/1/edit
  def edit
  end

  # Standard rails 4 create

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

  # Standard rails 4 destroy

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_survey
      @survey = Survey.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def survey_params
      params.require(:survey).permit(:name, questions_attributes: [:content])
    end
end

Итак, создание нового опроса с тремя вопросами — это нормально. Однако, если я попытаюсь отредактировать один из опросов, исходные три вопроса будут сохранены, а будут созданы еще три дополнительных. Таким образом, вместо 3 вопросов для отредактированного опроса у меня теперь 6. Я добавил

Rails.logger.debug("New method executed")

к новому методу в контроллере, и, насколько я могу судить, он не выполняется, когда я выполняю операцию редактирования. Может ли кто-нибудь сказать мне, что я делаю неправильно?

Любая помощь приветствуется!


person conciliator    schedule 22.09.2013    source источник
comment
Можете ли вы добавить действия редактирования и обновления в код вашего контроллера, чтобы мы могли проверить наличие там ошибок?   -  person Almaron    schedule 22.09.2013
comment
Конечно! Я предполагаю, что что-то потерялось на алтаре краткости.   -  person conciliator    schedule 22.09.2013


Ответы (2)


Итак, я понял это. Мне пришлось добавить :id к разрешенным параметрам в методе survey_params. Теперь это выглядит так:

# Never trust parameters from the scary internet, only allow the white list through.
def survey_params
  params.require(:survey).permit(:name, questions_attributes: [:id, :content])
end

который отлично работает. Я новичок в RoR, поэтому, пожалуйста, отнеситесь к моему анализу с недоверием, но я предполагаю, что новый идентификатор создается, а не передается в действие обновления. Надеюсь, это поможет кому-то еще.

person conciliator    schedule 22.09.2013
comment
Полезный совет, спасибо! (редактировать: просто говорю, потому что большинство людей не отвечают на свой вопрос, когда выясняют это..) - person Robin Kanters; 06.11.2013
comment
Конечно! Я обращался к SO так много раз, что было бы справедливо попытаться что-то вернуть. :) - person conciliator; 06.11.2013
comment
также новичок в RoR. ты спас меня! я потратил 2 дня на эти вложенные формы, и я просто пытаюсь сделать это относительно простое приложение, лол. - person dtc; 11.12.2013
comment
Приятно слышать - рад, что смог помочь! :) - person conciliator; 12.12.2013
comment
СПАСИБО! Это утомляло быть в течение нескольких дней и заняло много поиска. - person ; 03.03.2014
comment
Конечно @timmillwood! Всегда приятно кому-то время от времени помогать. :) - person conciliator; 03.03.2014
comment
Спасибо бро, реально большая помощь. - person JellyFishBoy; 15.06.2017

Используя cocoon gem на Rails 4, я по-прежнему получал повторяющиеся поля даже после добавления :id в список разрешенных при редактировании. Заметил еще следующее

Unpermitted parameters: _destroy
Unpermitted parameters: _destroy

Поэтому я добавил поле :_destroy к разрешенному полю model_attributes:, и после этого все заработало гладко.

Например...

def survey_params
  params.require(:survey).permit(:name, questions_attributes: [:id, :content, :_destroy])
end
person cevaris    schedule 01.05.2015