undefined method `id' for "数値":String

Question

前提・実現したいこと

Ruby 2.5.1
Rails 5.2.4.1
JQuery 4.3.5
active_hash 3.1.0
ancestry 3.0.7

某フリマアプリを模したアプリを開発中なのですが、出品機能で、商品情報をデータベースに登録をしたい。

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

submitボタンを押すと、

  NoMethodError in ItemsController#create
  undefined method `id' for "6":String

=>    @item = Item.new(item_params)
    
      if @item.save!

  app/controllers/items_controller.rb:68:in `create'
     

この"6"というのは、paramsの中のconditionの値なのですが、integer型で保存されるはずが、string型になっているし、そもそもid というメソッドはcondition には使っていないし、他のfieldにも使っていないはず。。

該当のソースコード

paramsのimagesの記述は割愛

Parameters:

{"utf8"=>"✓",
 "authenticity_token"=>"6EPm8+xm+3boKu3Zx4u3eooAgiUncilmAVXHTnGu6HTk6HnSpJoo7JvcNxgedr83SVz1biocFprLjYFJX/ZWzg==",
 "item"=>
  {"images_attributes"=>
    {"0"=>
      {"content"=>
        #<ActionDispatch::Http::UploadedFile:0x00007fc25a0625d8
         @content_type="image/jpeg",
         @headers=
          "Content-Disposition: form-data; name=\"item[images_attributes][0][content]\"; filename=\"\xE3\x82\xBF\xE3\x82\x99\xE3\x82\xA6\xE3\x83\xB3\xE3\x83\xAD\xE3\x83\xBC\xE3\x83\x88\xE3\x82\x99 (1).jpeg\"\r\n" +
          "Content-Type: image/jpeg\r\n",
         @original_filename="ダウンロード (1).jpeg",
         @tempfile=#<File:/var/folders/mk/8f7vv9q95xdf64s6pz1dk_vr0000gn/T/RackMultipart20200315-58575-1ujrxyn.jpeg>>}},
   "name"=>"aaaa",
   "description"=>"bbbbb",
   "parent_name"=>"レディース",
   "brand_id"=>"3",
   "condition"=>"6",
   "fee"=>"1",
   "shipping_days"=>"1",
   "area"=>"秋田県",
   "price"=>"5000"},
 "category_id"=>"21",
 "commit"=>"出品する"}

<ActionController::Parameters {"name"=>"hhhh", "description"=>"gagagga", 
"condition"=>"6", "price"=>"4444", "fee"=>"1", "brand_id"=>"1", "area"=>"青森県",
 "shipping_days"=>"1", "images_attributes"=>省略,,, permitted: true>} permitted: true>, 
"user_id"=>1, "category_id"=>"101"} permitted: true>

items_controller.rb

def new
    @item = Item.new
    @brands = Brand.all
    @category_parent_array = ["指定なし"]
    Category.where(ancestry: nil).each do |parent|
      @category_parent_array << parent.name
    end
    @item.images.build
  end

  def create
    @item = Item.new(item_params)
    if @item.save!
      @image = @item.images.create
      redirect_to :root
    else
      render :new
    end
  end

private
  def item_params
    params.require(:item).permit(
      :name, :description, :condition, :price, 
      :fee, :brand_id, :area, :shipping_days, 
      images_attributes: [:content, :id, :_destroy]
      ).merge(user_id: current_user.id, category_id: params[:category_id], brand_id: params[:item][:brand_id])
end

_form.html.haml

.wrapper.name-wrapper
        %label.wrapper__label
          商品名：
          %span.required
            ※必須
          = f.text_field :name, placeholder: "40字まで", class: "name-wrapper__name form-control", required: "required"
      .wrapper.description-wrapper
        %label.wrapper__label
          商品説明：
          %span.required
            ※必須
          = f.text_area :description, placaeholder: "色・素材・重さ・定価・注意点などを書きましょう（1000文字まで）", class: "name-wrapper__description form-control", required: "required", size: "40 x 10"
      .form-separator

      .wrapper.category-wrapper
        = f.label :category_id , class: 'wrapper__label category-wrapper-label', id: "wrapper__label--category" do
          カテゴリー：
          %span.required ※必須
          .category-wrapper-box
            .category-wrapper-select
              .category-wrapper-select__box
                = f.select :parent_name, @category_parent_array, {}, {class: 'category-wrapper__category form-control', id: 'parent_category'}


      .wrapper.brand-wrapper
        %label.wrapper__label
          ブランド：
          = f.select :brand_id, options_for_select(@brands.map{|b| [b.name, b.id, {}]}), {prompt: "指定なし"}, class: "brand-wrapper__brand form-control"
          
      .wrapper.condition-wrapper
        %label.wrapper__label
          商品の状態：
          %span.required
            ※必須
          = f.collection_select :condition, Condition.all, :id, :name, {selected: 1}, {class: "condition-wrapper__condition form-control", required: "required"}
      .form-separator

      .wrapper.fee-wrapper
        %label.wrapper__label
          配送料の負担：
          %span.required
            ※必須
          = f.collection_select :fee, Fee.all, :id, :name, {selected: 1}, {class: "fee-wrapper__fee form-control", required: "required"}
      
      .wrapper.shipping_days-wrapper
        %label.wrapper__label
          発送日の目安：
          %span.required
            ※必須
          = f.collection_select :shipping_days, ShippingDay.all, :id, :name, {selected: 1}, {class: "shipping_days-wrapper__shipping-days form-control", required: "required"}
      .wrapper.area-wrapper
        %label.wrapper__label
          発送元の地域：
          %span.required
            ※必須
          = f.collection_select :area, Prefecture.all, :name, :name, {selected: 1}, {class: "area-wrapper__area form-control", required: "required"}
      .form-separator

      .wrapper.price-wrapper
        %label.wrapper__label
          価格：
          %span.required
            ※必須
          = f.number_field :price, class: "price-wrapper__price form-control", required: "required"

      .wrapper.submit-wrapper
        = f.submit "出品する", class: "btn submit-wrapper__submit-btn"

item.rb

class Item < ApplicationRecord 
  belongs_to :brand, optional: true
  belongs_to :user, optional: true
  belongs_to :category,  optional: true
  has_many :images, dependent: :destroy
  accepts_nested_attributes_for :images, allow_destroy: true

  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to_active_hash :prefecture
  belongs_to_active_hash :fee
  belongs_to_active_hash :shipping_day
  belongs_to_active_hash :condition
end

condition.rb

class Condition < ActiveHash::Base
  self.data = [
    { id: 1, name: '新品、未使用'}, {id: 2, name: '未使用に近い'}, {id: 3, name: '目立った傷や汚れなし'},
    { id: 4, name: 'やや傷や汚れあり'}, {id: 5, name: '傷や汚れあり'}, {id: 6, name: '全体的に状態が悪い'}
  ]
end

試したこと

ビューファイルでのconditionのフォームのvalue属性の確認をしたが、数字になっている。
Conditionモデルも記述ミスもない。
もしかしたら、paramsの中のparent_nameの値が、邪魔しているのかな？

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

condition・prefecture・fee・shipping_dayなどは、active_hashをitemモデルに紐付けて使っています。

categoryのselectタグは、jqueryで動的に出てくるようにしています。

Comments

[winterboum]

エラーメッセージを全文載せてください

[SomaSekimoto]

修正しました。

Answer 1

まず　「string型になっているし」　ですが、paramsの値はみな string です。
newなりcreateなりで渡された先で、columの型に応じて変換されます。
ので、confitinがIntegerなら "6" が 6 に変換されて格納されます。
でも
condition は belongs_to_active_hash で ActiveHash ですね。んで id を持っています。
Itemのmigrationがないのですが多分 condition　を integerで定義していると思います。ではなく condition_id を定義しなければなりません。

Comments

[SomaSekimoto]

返信ありがとうございます。
paramsの値が全てstringだということは知りませんでした。
となると、feeカラムもshipping_daysカラムも名前変更しなければいけないということですね。
ありがとうございます！！