ruby-on-rails - Rails 编辑/更新操作不适用于嵌套项

Question

我已经为我的@miniature 模型和它的嵌套模型@scales 准备了新的/创建的操作和表单。我无法正确执行更新/编辑操作。应该很简单，但我很卡。

@miniature 有很多 @scales 到 @sizes。

在我的@miniature模型中我有

    has_many :sizes, dependent: :destroy
    has_many :scales, :through => :sizes
    accepts_nested_attributes_for :sizes, allow_destroy: true

在我的 Controller 中

    def new
        @miniature = Miniature.new 
        @all_scales = Scale.all
        @size = @miniature.sizes.build
    end

    def create
        @miniature = Miniature.new(miniature_params)
        params[:scales][:id].each do |scale|
          if !scale.empty?
            @miniature.sizes.build(:scale_id => scale)
          end
        end
        if @miniature.save
          redirect_to miniature
        else
          render 'new'
        end
    end

private
    def miniature_params
      params.require(:miniature).permit(:name, :release_date, :material, :pcode, :notes,  sizes_attributes: [:id, :scale_id, :miniset_id])
    end

这行得通，但编辑和更新操作不行。我的编辑设置与新设置相同。

def edit
    @miniature = Miniature.find(params[:id])
    @all_scales = Scale.all
    @size = @miniature.sizes.build
end

我原以为更新 miniature params 会更新 @sizes 模型，但事实并非如此

def update
    @miniature = Miniature.find(params[:id])
    if @miniature.update_attributes(miniature_params)
      flash[:success] = "Miniature updated"
      redirect_to @miniature
    else
      render 'edit'
    end
end

它目前更新了 @miniature 但不是 @sizes 信息。我的问题是在编辑中还是在更新中还是在两者中？

表格的相关部分:

<%= f.fields_for(@size) do |sf| %>
      <%= sf.label simple_pluralize(@miniature.scales.count, 'Scale') %>

        <%= collection_select( :scales, :id, @all_scales, :id, :name, 
                   {:selected => @miniature.scales.map(&:id)}, 
                   {class: 'multiselect', multiple: true}) %>
<% end %>

非常感谢任何进一步阅读的帮助或指示。即使只是说“你忽略了这个显而易见的事情，去多做一些阅读/工作”。

我似乎需要一个更类似于我的 create 操作的 if 语句的 update 操作？

JKen13579 的更新

这是提交编辑时来 self 的服务器日志的 PATCH 请求:

Started PATCH "/miniatures/21" for 127.0.0.1 at 2014-04-02 16:00:10 +0100
Processing by MiniaturesController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"jQ79L1Exx83C47jnCF3nsWQ2tV07tRwKfI8wNeLzojo=", "miniature"=>{"name"=>"Test Miniature", "material"=>"Metal", "pcode"=>"123123123123123", "release_date(1i)"=>"2013", "release_date(2i)"=>"2", "release_date(3i)"=>"2", "notes"=>""}, "scales"=>{"id"=>["", "2"]}, "commit"=>"Save changes", "id"=>"21"}
  User Load (0.7ms)  SELECT "users".* FROM "users" WHERE "users"."id" = 4 ORDER BY "users"."id" ASC LIMIT 1
  Miniature Load (0.2ms)  SELECT "miniatures".* FROM "miniatures" WHERE "miniatures"."id" = ? LIMIT 1  [["id", "21"]]
   (0.1ms)  begin transaction
   (0.2ms)  commit transaction
Redirected to http://localhost:3000/miniatures/21
Completed 302 Found in 12ms (ActiveRecord: 1.3ms)

更新观察者

我正在使用 fields_for，因为这是我设法让 new/create 操作与之一起工作的原因。如果有更好的方法，至少据我所知，我不会被束缚。

我的路线有

resources :miniatures do
    collection do
    get :scales
    get :collections
    get :lines
    get :contents
    post :import
    end
    member do
      get :minisets, :setminis
    end
  end

再往下

resources :sizes
resources :scales

我认为我的路由文件通常有点困惑，需要进行重构。

Answer 1

根据服务器日志条目(见下文)，我看到在 params 哈希中，scales(以粗体突出显示)作为单独的传递hash and not within 微型:

Parameters: {"utf8"=>"✓", "authenticity_token"=>"jQ79L1Exx83C47jnCF3nsWQ2tV07tRwKfI8wNeLzojo=", "miniature"=>{"name"=>"Test Miniature", "material"=>"Metal", "pcode"=>"123123123123123", "release_date(1i)"=>"2013", "release_date(2i)"=>"2", "release_date(3i)"=>"2", "notes"=>""}, "scales"=>{"id"=>["", "2"]}, "commit"=>"Save changes", "id"=>"21"}

将 update 操作更改为:

def update
    @miniature = Miniature.find(params[:id])
    if params[:scales][:id]
      ## Convert ["", "1","2","4","8"] to [1,2,4,8]
      params[:scales][:id] = params[:scales][:id].reject(&:empty?).map(&:to_i) 
      ## Get the scale_id from sizes already present in database [1,2,5,6] 
      old_scales = @miniature.sizes.pluck(:scale_id)
      ## Find the new scales to be added [1,2,4,8] - [1,2,5,6] = [4,8]
      new_scales = params[:scales][:id] - old_scales 
      ## Find the old_scales to be deleted [1,2,5,6] - [1,2,4,8] = [5,6]
      old_scales = old_scales - params[:scales][:id] 
      ## Build new_scales [4,8]
      new_scales.each do |scale|
        @miniature.sizes.build(:scale_id => scale)
      end
      ## Delete old_scales [5,6]
      Size.delete_all(:scale_id => old_scales)
    end
    if @miniature.update_attributes(miniature_params)
      flash[:success] = "Miniature updated"
      redirect_to @miniature
    else
      render 'edit'
    end
end

更新 create 操作，使 scale_id 作为 Integer 而不是 String 传递:

def create
    @miniature = Miniature.new(miniature_params)
    if params[:scales][:id]
      ## Convert ["", "1","2","4","8"] to [1,2,4,8]
      params[:scales][:id] = params[:scales][:id].reject(&:empty?).map(&:to_i)
      params[:scales][:id].each do |scale|
        @miniature.sizes.build(:scale_id => scale)
      end
    end
    if @miniature.save
      redirect_to miniature
    else
      render 'new'
    end
end