チェックボックスの値を配列から文字列に変換して表示したい

Question

前提・実現したいこと

プログラミング言語(languageカラム)をチェックボックスにて複数選択し、取得した値を文字列で表示したいです。
複数選択のため配列にして保存しているので、文字列に変換するコードをコントローラに記述すると保存されなくなります。
languageカラムはstring型にしています。

該当のソースコード

company/new.html.erb

       <%= form_for(@company ,url:  admins_companies_path) do |f| %>
         <div class="company-post__title">
           <h2>掲載企業を追加</h2>
         </div>
           <%= f.label :language %>
           <% ["PHP","Ruby","JAVA"].each do |language| %>
           <%= f.check_box :language, {multiple: true}, language, nil  %>
           <%= language %>
           <% end %>
         <%= f.submit 'Create Company' %>
         <% end %>

Admins::CompaniesController

class Admins::CompaniesController < ApplicationController
  before_action :set_user, except:[:new, :create, :index]
  before_action :language_string, only: [:create, :update]

~省略~

  def create
    # params[:company][:language] = params[:company][:language].join
    company = Company.new(company_params)
    if company.save
      # binding.pry
      flash[:notice] = 'successfully'
      redirect_to admins_prefectures_path
    end
  end

~省略~


  private

  def set_user
    @company = Company.find(params[:id])
  end

  def language_string
    params[:company][:language] = params[:company][:language].join("/")
  end

  def company_params
    params.require(:company).permit(:company_name, :description, :image, :prefecture_id, :address, :phone_number, :url, language: [])
  end
end

試したこと

binding.pryで見てみると、パラメーターは渡っているようです。

From: /app_name/app/controllers/admins/companies_controller.rb:20 Admins::CompaniesController#create:

    15: def create
    16:   # params[:company][:language] = params[:company][:language].join
    17:   company = Company.new(company_params)
    18:   if company.save
    19:     binding.pry
 => 20:     flash[:notice] = 'successfully'
    21:     redirect_to admins_prefectures_path
    22:   end
    23: end

[1] pry(#<Admins::CompaniesController>)> params
=> <ActionController::Parameters {"utf8"=>"✓", "authenticity_token"=>"6ojp57p89gjRZWkOJQ9dv0aQvrPbLvYZuDMs+TVABV1GYtS9/ctIbC97GJTuRh7cLeZi2i1H8Gc0wjapOhF0qQ==", "company"=><ActionController::Parameters {"prefecture_id"=>"1", "company_name"=>"ディーエ", "address"=>"東京都渋谷区", "phone_number"=>"000000", "language"=>["PHP", "Ruby", "JAVA"], "description"=>"test", "image"=>"{}"} permitted: false>, "commit"=>"Create Company", "controller"=>"admins/companies", "action"=>"create"} permitted: false>
[2] pry(#<Admins::CompaniesController>)> params[:company][:language] 
=> ["PHP", "Ruby", "JAVA"]

before_action :language_string, only: [:create, :update]を記述しなければ値を保存できます。
上記を記述すると、languageのデータは保存されませんが、エラーは出現しません。

Answer 1

before_action :language_string, only: [:create, :update]

def language_string
  params[:company][:language] = params[:company][:language].join("/")
end

ここでparams[:company][:language] は文字列になっているのだから、

def company_params
  params.require(:company).permit(:company_name, :description, :image, :prefecture_id, :address, :phone_number, :url, language: [])
end

permit で配列 language: [] を指定するのは変です。

Comments

[shin342]

おっしゃる通り、language: []の指定を削除したらうまくいきました。
ありがとうございました！