strongparameterで値をハッシュに変換できない

Question

###現状とエラー
現在railsを使ってアプリを製作中です。
updateアクションを実行したいのですが、unable to convert unpermitted parameters to hashのエラーで失敗してしまいます。

###コード
エラーコード
ストロングパラメータの行でエラーが起きています

ActionController::UnfilteredParameters - unable to convert unpermitted parameters to hash:
  app/controllers/incomes_controller.rb:41:in `block in incomes_params'
  app/controllers/incomes_controller.rb:40:in `incomes_params'
  app/controllers/incomes_controller.rb:11:in `create

モデル
Incomeモデル：category,price,date,memoの４つのカラムを持っています

class Income < ApplicationRecord 
    default_scope -> { order(date: :desc) }
    validates :category, presence:true
    validates :price, presence:true
    validates :date, presence:true
    validates :memo, presence:true, length: { maximum: 40 }
end

Income_collectionモデル：Incomeをまとめて作成、変更するためのモデルです

class IncomeCollection < ApplicationRecord
end
class IncomeCollection
    include ActiveModel::Conversion
    extend ActiveModel::Naming
    extend ActiveModel::Translation
    include ActiveModel::AttributeMethods
    include ActiveModel::Validations
    INCOME_NUM = 5
    attr_accessor :collection  # ここに作成したモデルが格納される
  

    def initialize(attributes = [])
      if attributes.present?
        self.collection = attributes.map do |value|
          Income.new(
            category: value['category'],
            price: value['price'],
            date: value['date'],
            memo: value['memo']
          )
        end
      else
        self.collection = INCOME_NUM.times.map{ Income.new }
      end
    end
  
    # レコードが存在するか確認するメソッド
    def persisted?
      false
    end
  
  
    # コレクションをDBに保存するメソッド
    def save
      is_success = true
      ActiveRecord::Base.transaction do
        collection.each do |result|
          # バリデーションを全てかけたいからsave!ではなくsaveを使用
          is_success = false unless result.save
        end
        # バリデーションエラーがあった時は例外を発生させてロールバックさせる
        raise ActiveRecord::RecordInvalid unless is_success
      end
      rescue
        p 'エラー'
      ensure
        return is_success
    end
  
  end

コントローラー
Incomesコントローラー

class IncomesController < ApplicationController
    def edit
      @incomes = Income.all
    end
    
    def update
      @incomes = incomes_params.to_unsafe_h.map do |id, income_param|
        income = Income.find(id)
        income.update_attributes(income_param)
        income
      end
      redirect_to incomes_path

    end

    private
    
    def incomes_params
      params.require(:incomes).map do |p|
         ActionController::Parameters.new(p.to_hash).permit(:category, :price, :date, :memo)
       end
    end

end

ビュー
editページ

<%= form_tag income_path, method: :put do %>
  <% @incomes.each do |income| %>
    <%= fields_for "incomes[]", income do |fi| %>
        <section class = "row incomes_row">
            <div class="incomes_item col-sm-3">
            <%= fi.label :カテゴリ %>
            <%= fi.text_field :category, class:"form" %>
            </div>
            
            <div class="incomes_item col-sm-3">
            <%= fi.label :金額 %>
            <%= fi.text_field :price %>
            </div>

            <div class="incomes_item col-sm-3">
            <%= fi.label :日付 %>
            <%= fi.text_field :date %>
            </div>

            <div class="incomes_item col-sm-3">
            <%= fi.label :メモ %>
            <%= fi.text_field :memo %>
            </div>
            </br>
      </section>
      <br />
    <% end %>
  <% end %>
  <%= submit_tag %>
<% end %>

###原因予想
income_collectionモデルからインスタンスを作っている事が原因で
上手くパラメータを渡せていないかなと思っています。

• githubの同じような事例を試したのですがダメでした。
• ストロングパラメータのpermitをpermit!にしても変化がありませんでした。

原因がお分かりの方がいらっしゃいましたらご指摘をお願いいたします。

###開発環境
Rails 5.2.4.1

Answer 1

とりあえずパラメータ全体を.permit!すれば.to_hashできます。
が、params.require(:incomes)の中身がよくわからないのでセキュリティー面は保証できません。

params.require(:incomes).permit!.map do |p| # 質問とは関係ないけど p 変数を作るのは p メソッドがある関係上よくない。
  ActionController::Parameters.new(p.to_hash).permit(:category, :price, :date, :memo)
end

Comments

[ketyan]

Mugheartさん、ご返信有り難うございます。

いろいろと試してみたところ、mapメソッドを取り除いたところpermit!
を使わずにアプリが動きました。
動いた理由を調べてみようと思います。

ご回答頂きありがとうございました。