Rails：fields_forを使って関連先のテーブルへアップロードされたファイルを保存する方法

Question

Railsにお詳しい方のお知恵をお借しください……

親テーブル(workitems)の編集フォームからアップロードされたファイルを、
子テーブル(destination)へ保存しようとしているのですが、なかなか上手く行きません。

アップロードされるファイルはCSV(text/csv)のみです。

paperclipが有効という情報を見つけ、あちこち参考にしつつ試したのですが、
「contenttypeが不正」というエラーが回避できず……

仕方なく自力でbinary型の列へ保存する方法を行おうとしてるのですが、
こちらでも躓いてしまい途方に暮れております。

Model(~/app/models/workitem.rb , ~/app/models/destination.rb)

class Workitem < ActiveRecord::Base
  # rails g scaffold workitem name:string
  has_one :destination, :dependent => :destroy
  accepts_nested_attributes_for :destination
end

class Destination < ActiveRecord::Base
  # rails g model destination name:string workitem_id:integer upload_file:binary
  belongs_to :workitem
end

View(~/app/views/workitems/_form.html.erb)

<%= form_for @workitem, multipart: true do |f| %>
  <%= f.fields_for @workitem.destination do |dest_f| %>
    <%= dest_f.file_field :upload_file %>
  <% end %>
<% end %>

Controller(~/app/controllers/workitems_controller.rb)

class WorkitemsController < ApplicationController
  def new
    @workitem = Workitem.new
    @workitem.build_destination
  end

  def create
    # あれこれ試してみて自力でパラメータを作ればどうにかなるかと以下のコードを書きました

    # workitem_paramsを退避。ファイルがアップロードされなければ
    # こいつがこのまま使用されるので問題ないはず
    temp_workitem_params = workitem_params
    destination_attributes = workitem_params[:destination_attributes]
    upload_file = destination_attributes[:upload_file]
    # => ActionDispath::Http::UploadedFile

    if upload_file != nil
      # ファイルがアップロードされてれば
      if @workitem.destination != nil
        # 既存のアップロードファイルがあれば消す
        @workitem.destination.destroy
      end

      # アップロードされたファイルの情報をworkitem_param[:destination_attributes]に仕込む
      content = {}
      content[:filename]    = upload_file.original_filename
      content[:upload_file] = upload_file.read

      temp_workitem_params.delete('destination_attributes')
      temp_workitem_params['destination_attributes'] = content
    end

    @workitem = Workitem.new(temp_workitem_params)
    respond_to do |format|
      if @workitem.save
        # この中身はそのまま
      end
    end
  end

  def update
    # create と同じく、workitem_paramsからDispatch::Http::UploadedFileの情報を展開し、
    # destination_attributesの中身を差し替えて@workitem.updateを行っています
  end

  private
  def workitem_params
    params.require(:workitem).permit(:name, destination_attributes: [ :upload_file, :filename, :workitem_id ])
  end
end

上記のような構成にてデータの更新を行おうとすると、
ActiveModel::ForbiddenAttributesError という例外が発生してしまいます。

ストロングパラメータはテーブルのものと同じにしてあるのですが、
どこが原因なのか皆目検討もつかず……

サーバ環境は

• CentOS 6.6
• ruby 2.0.0p645
• Rails 4.2.1

となっております。

workitemが登録・更新されるタイミングでファイルのアップロードも行いたいため
色々と調べてはみたのですが、これ以上はどうしもうよなく……
不足している情報があれば分かる範囲でお出ししますので、よろしくお願いいたします。

Answer 1

workitem_paramsから:destination_attributesだけを取り出してDestinationを登録、workitem_paramsがら:destination_attributesを消す という力技で解決いたしました。

class WorkitemsController < ApplicationController
  def new
    @workitem = Workitem.new
    @workitem.build_destination
  end

  def edit
    @workitem.build_destination unless @workitem.destination
  end

  def create
    @destination = nil

    temp_workitem_params   = workitem_params
    destination_attributes = temp_workitem_params[:destination_attributes]
    upload_file            = destination_attributes[:upload_file]
    if upload_file != nil
      content = {}
      content[:filename   ] = upload_file.original_filename
      content[:upload_file] = upload_file.read
      @destination = Destination.new(content)
    end
    # 後述する@workitem.saveに影響するのでdestination_attributesを削除
    temp_workitem_params.delete('destination_attributes');

    @workitem = Workitem.new(workitem_params)
    respond_to do |format|
      if @workitem.save
        if @destination
          @destination.workitem_id = @workitem.id
          @destination.save
        end
      end
    end
  end

  def update
    temp_workitem_params   = workitem_params
    destination_attributes = temp_workitem_params[:destination_attributes]
    upload_file            = destination_params[:upload_file]
    if upload_file != nil
      if @workitem.destination != nil
        @workitem.destination.destroy
      end

      content = {}
      content[:filename   ] = upload_file.original_filename
      content[:upload_file] = upload_file.read
      @destination = Destination.new(content)
      @destination.workitem_id = @workitem.id
      @destination.save
    end
    # 後述する@workitem.updateに影響するのでdestination_attributesを削除
    temp_workitem_params.delete('destination_attributes');

    respond_to do |format|
      if @workitem.update(temp_workitem_params)
      end
    end
  end
end

なんと泥臭い……(´・ω・｀)

Answer 2

もしかして、

 @workitem = Workitem.new(temp_workitem_params)

のところでコケてませんか？

だとしたら、attr_acessibleの設定がないだけなのでは?

class Workitem < ActiveRecord::Base
  # rails g scaffold workitem name:string
  has_one :destination, :dependent => :destroy
  accepts_nested_attributes_for :destination
  attr_accessible :name
end

class Destination < ActiveRecord::Base
  # rails g model destination name:string workitem_id:integer upload_file:binary
  belongs_to :workitem
  attr_accessible :upload_file, :filename
end

こんな感じ。

Comments

[Uchikoba]

ご回答ありがとうございます。

Rails4では、attr_accessibleではなく下記のように、ストロングパラメータによるパーミッション設定を行わねばならないようでして、設定そのものはできていると思います。

```lang-Ruby
class WorkitemsController < ApplicationController
private
def workitem_params
params.require(:workitem).permit(:name, destination_attributes: [ :upload_file, :filename, :workitem_id ])
end
end
```

[rifuch]

あ、Rails4でしたね・・・すみません。