collection_check_boxesのデータがテーブルに保存されず困っています

Question

前提・実現したいこと

Ruby on Rails
ActiveHashモデルのデータを用いてcollection_check_boxesで選択した複数の値を保存させたいです。

機能の目的は、自宅にある調味料をユーザーデータとして登録させたいというものです。
調味料のデータは頻繁に変わるものではないためActiveHashを用いて実装しようとしています。

発生している問題・エラーメッセージ

エラーメッセージは出ずに、createアクションで定義した保存できた場合、の条件分岐処理としてトップページに戻っても保存はできていません。

MySQLの調味料テーブル(seasonings)を確認すると、user_idはちゃんと紐付いた状態でレコードは追加されているのですが、collection_check_boxesで選択した値はsalt_sugar_idカラムに保存されずNULLになっています。

試したこと

crateアクションをbinding.pryで止めてターミナルログを確認し、salt_sugar_idカラムのパラメーターに問題があるとわかり、以下のことを試しました。

• salt_sugar_idカラムに関係する記述すべてに誤字脱字はないか確認しました
• バリデーションでnumericality: { other_than: 1 } をしている状態で、salt_sugar_idの中に空白""の値も入っているのも原因かと推測し、SaltSugarモデルself.dataのデフォルト部分を削除してみました
• 複数の値を保存させたいことから、belongs_toのアソシエーションも間違っている可能性を考え、has_manyで検証しましたがNoMethodErrorになります。
• permit!メソッドにしてみました。ターミナルログにはパラメーターのエラー文は出なくなったのですが、MySQLに値は保存されず保存できなかったときのページ遷移になります。

以下の情報以外に必要次項等ありましたらすぐに補足させていただきます。
直接的ではなくても、考え方やヒントだけでもご教示いただけたら嬉しいです。
宜しくお願い致します。

Started POST "/seasonings" for ::1 at 
Processing by SeasoningsController#create as HTML
  Parameters: {"authenticity_token"=>"HgwBaS0yU0S6Qf1kCr3IQ/k2X0OK9KlIxBz237TYrXLEjll3/02oSDL6qsJqGasb3XOUHcmhvX/ct8DegVOPPQ==", "seasoning"=>{"salt_sugar_id"=>["", "7", "8"]}, "commit"=>"登録"}
Unpermitted parameter: :salt_sugar_id
  User Load (0.2ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 2 ORDER BY `users`.`id` ASC LIMIT 1
  ↳ app/controllers/seasonings_controller.rb:25:in `seasoning_params'
   (0.2ms)  BEGIN
  ↳ app/controllers/seasonings_controller.rb:13:in `create'
  User Load (0.4ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 2 LIMIT 1
  ↳ app/controllers/seasonings_controller.rb:13:in `create'
   (0.2ms)  ROLLBACK
  ↳ app/controllers/seasonings_controller.rb:13:in `create'
  Rendering seasonings/new.html.erb within layouts/application
  Rendered shared/_header.html.erb (Duration: 0.2ms | Allocations: 133)
  Rendered shared/_footer.html.erb (Duration: 0.0ms | Allocations: 5)
  Rendered seasonings/new.html.erb within layouts/application (Duration: 1.6ms | Allocations: 1372)
[Webpacker] Everything's up-to-date. Nothing to do
Completed 200 OK in 53ms (Views: 23.3ms | ActiveRecord: 1.0ms | Allocations: 36486)

該当のソースコード

カラム名はsalt_sugarのみ抜粋し他は省略させていただいてます
config/routes.rb

Rails.application.routes.draw do
  devise_for :users
  root to: 'seasonings#index'
  get 'rakuten_search' => 'seasonings#index'
  resources :seasonings, only:[:index, :new, :create]
  resources :users, only:[:show]
end

app/controllers/seasonings_controller.rb

class SeasoningsController < ApplicationController

  def index
      @categories = RakutenWebService::Recipe.large_categories
      @recipes = @categories.first.ranking
  end

  def new
    @seasoning = Seasoning.new
  end

  def create
    @seasoning = Seasoning.create(seasoning_params)
    if @seasoning.save
      binding.pry
      redirect_to action: :index
    else
      render :new
    end
  end

  private
  
  def seasoning_params
    params.require(:seasoning).permit(:salt_sugar_id).merge(user_id: current_user.id)
  end

end

app/models/seasoning.rb

class Seasoning < ApplicationRecord
  belongs_to :user
  
  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to_active_hash :salt_sugar

  validates :salt_sugar_id, numericality: { other_than: 1 } 

end

app/models/user.rb

class User < ApplicationRecord
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  
  has_one :seasoning
  has_one_attached :image

  with_options presence: true do
    validates :nickname
    validates :password, length: { minimum: 8 }, format: { with: /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i }
  end
end

app/models/salt_sugar.rb ActiveHash model

class SaltSugar < ActiveHash::Base
  self.data = [
    # { id: 1, name: '--' },
    { id: 2, name: '塩' },
    { id: 3, name: '岩塩' },
    { id: 4, name: '藻塩' },
    { id: 5, name: '白砂糖' },
    { id: 6, name: '黒糖' },
    { id: 7, name: 'きび砂糖' },
    { id: 8, name: 'オリゴ糖' },
  ]
 
   include ActiveHash::Associations
   has_many :seasonings
 
end

app/seasonings/new.html.rb

<div id='home', class='big-bg'>
  <%= render 'shared/header' %>

  <div class='main-content'>
    <div class='form-box'>
      <div class='list'>
        <h3><%= link_to "uchino_aji", "/", class: 'main-subtitle' %></h3><br />
        <h2>うちの調味料登録</h2>

        <%= form_with model: @seasoning, url: seasonings_path, method: :post, local: true do |f| %>
          
          <div class="form">
            <span class='optional'>(任意)</span>
            <%= f.label "塩・砂糖", class: 'form-label' %><br />
            <%= f.collection_check_boxes(:salt_sugar_id, SaltSugar.all, :id, :name, {}, {class: 'check_box', id:'salt-sugar'}) %>
          </div>

          <div class="actions">
            <%= f.submit "登録", class: 'recipe-detail-btn' %>
          </div>
        <% end %>

      </div>
    </div>
  </div>
</div>
<%= render 'shared/footer' %>

db/migrate/XXXXX_craate_seasonings.rb

class CreateSeasonings < ActiveRecord::Migration[6.0]
  def change
    create_table :seasonings do |t|
      t.references :user, null:false, foreign_key: true
      t.integer :salt_sugar_id      
      t.timestamps
    end
  end
end

補足情報（FW/ツールのバージョンなど）

GitHub(エラー部分は反映できていない状態です)
rails 6.0.0
mysql2 0.5.3
gem 'active_hash'

Answer 1

ログに以下の記述がありますので、params[:salt_sugar_id] が弾かれてます。

Unpermitted parameter: :salt_sugar_id

以下のようにすると:salt_sugar_idが通ります。

params.require(:seasoning).permit(:salt_sugar_id => []).merge(user_id: current_user.id)

ただ、seasoning の salt_sugar_id カラムに配列は保存できないと思います。

選択された salt_sugar_id の数だけ seasoning を作成するのであれば、
salt_sugar_id を ループで回して seasoning を create します。

seasoning がひとつで、それに紐づけられる salt_sugar が複数(seasoning has_many :salt_sugars)ならば
中間テーブルを作成する必要があります。

Comments

[退会済みユーザー]

neko_daisukiさん
こんなにも早く具体的な解説までしていただけたことに感激です
ありがとうございます
教えていただいた書き方の意味も調べながら、後者の複数紐づける方向でやってみて、改めてご報告いたします

[退会済みユーザー]

neko_daisukiさん
配列を許可するpermitの書き方勉強になりました。ありがとうございます。
実際にやってみて仰ったように配列では保存できないことを学べました。

中間テーブル作成ではカラムをbigint型にしてくださいというエラーが起きマイグレーションを失敗してしまいます。
下記を参考にいくつか修正を試してみました。
[referencesを使用した外部キー設定で沼った](https://qiita.com/Jwataru/items/7b840be38d4279224245)
[作成されたテーブルにFK制約を張れない問題の解決策](https://qiita.com/nametaketakewo/items/d9d26fb301a09d7e6c2c)

`ターミナルエラー表示`

```
-- create_table(:seasoning_salt_sugars)
rails aborted!
StandardError: An error has occurred, all later migrations canceled:

Column `seasoning_id` on table `seasoning_salt_sugars` does not match column `id` on `seasonings`, which has type `bigint(20)`. To resolve this issue, change the type of the `seasoning_id` column on `seasoning_salt_sugars` to be :bigint. (For example `t.bigint :seasoning_id`).
Original message: Mysql2::Error: Cannot add foreign key constraint
```

試したこと
- ターミナルの提案通り中間テーブルカラムをbigint型にし、外部キー制約はdef change外に記述しました

```ruby
class CreateSeasoningSaltSugars < ActiveRecord::Migration[6.0]
def change
create_table :seasoning_salt_sugars do |t|
t.bigint :seasoning_id
t.bigint :salt_sugar_id

t.timestamps
end

add_foreign_key :seasonings, :salt_sugars
end
end
```

- 以上でも同じエラーのため、
　t.references :seasoning, type: :bigint, foreign_key: true
　t.references :salt_sugar, type: :bigint, foreign_key: true
にしても変わらなかったです。

- seasoningテーブルのsugar_salt_idカラムを
　t.bigint :salt_sugar_idに修正もしてみました。

salt_sugar_idはActiveHashを用いてテーブルを作っておらず、seasoningテーブルのカラムとして存在している状態でも中間テーブルを作成することは可能でしょうか。salt_sugar_id以外にも調味料分類用に13種類ActiveHashでモデルを作成しております。

ActiveHashで種類別カテゴリーを作るのを諦めて、（何十個にもなる膨大なカラム数になってでも）seasoningテーブルのカラムに白砂糖、黒糖、きび砂糖・・・赤味噌、白味噌、合わせ味噌・・・作るようにしたり、
または塩_砂糖テーブル、味噌テーブル・・・とカテゴリ別にテーブルを作って全てにseasoningテーブルとの中間テーブルを作成したりしたほうがよいのでしょうか…

[neko_daisuki]

ActiveHashを使っているのでsalt_sugar_id には外部キー制約がつけられないと思います。

t.references :seasoning, foreign_key: true
t.references :salt_sugar

これは通らないでしょうか？

[退会済みユーザー]

迅速なご対応ありがとうございます！
ご提示いただいた形でマイグレーション無事通りました！
検証不足にも関わらず見当違いなことまで考えてしまいお恥ずかしい限りです。
設計からやり直しかと焦って視野が狭くなっていたところ、ActiveHashに中間テーブルを使えることを知れてよかったです…！
ここからまた自身でアソシエーションやビューへの反映等を実践してみて、教えていただいた書き方と中間テーブルへの理解を深めたいと思います。
neko_daisukiさん改めて、ありがとうございます！！