質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.50%

accepts_nested_attributes_forを利用したform(check_box)のEdit画面でチェックが消えてしまう

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 1,631

score 300

前提・実現したいこと

accepts_nested_attributes_forでネストされたフォームからデータを入力できるのですが、エディット画面では、checkboxの情報を取得できません。

入力された値を正しくEditでも表示され、その後もUpdate処理できるようにしたいです。

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

https://teratail.com/questions/14954

ここで書かれているような実装をしているのですが、私の場合は、cocoonというgemを使っており、ネストされたオブジェクトの特定が上手くいっていません。

該当のソースコード

event.rb

has_many :tickets
accepts_nested_attributes_for :tickets, allow_destroy: true
ticket.rb
belongs_to :event
form

<%
if @event.id.nil?
  resource_info = [@group, @event]
else
  resource_info = [@event]
end
%>

<%= form_for( resource_info,:html => {:multipart => true, :class => "form-horizontal h-adr"}) do |f| %>

<div class="row ">
    <div class="col-md-3" >
        <div class="gray"><span class="label-title">タイトル</span></div>
    </div>    
    <div class="col-md-9">
    <%= f.text_field :title, :class =>"form-control" %>
    </div> 
</div>

<div class="row ">
    <div class="col-md-3" >
        <div class="gray"><span class="label-title">サブタイトル</span></div>
    </div>    
    <div class="col-md-9">
    <%= f.text_field :subtitle, :class =>"form-control" %>
    </div> 
</div>

<div class="row ">
    <div class="col-md-3" >
        <div class="gray"><span class="label-title">概要</span></div>
    </div>    
    <div class="col-md-9">


    <%= f.text_area :description, rows:15,:id => "event_description" ,:class =>"form-control" %>
    </div> 
</div>

<div class="row ">
    <div class="col-md-3" >
        <div class="gray"><span class="label-title">場所</span></div>
    </div>   
    <div class="col-md-9" >   
        <div class="row">
            <div class="col-md-3">  
            <%= f.label :zipcode,"郵便番号"%>
            <%= f.text_field :zipcode ,:placeholder =>"郵便番号", :id => "event_zipcode", :class =>"form-control p-postal-code" %>
            </div>
        </div>
        <%= f.label :address,"住所"%>
        <div class="row">
            <div class="col-md-3"> 
            <%= f.text_field :pref ,:placeholder =>"都道府県 ", :class =>"form-control p-region" %>
            </div>
            <div class="col-md-3">    

            <%= f.text_field :city ,:placeholder =>"市区 ", :id => "address_city", :class =>"form-control p-locality" %> 
            </div>
            <div class="col-md-6">
            <%= f.text_field :address02,:placeholder =>"住所 ",:id => "address_street",:class =>"p-street-address p-extended-address form-control" %>


            </div>
        </div>

    </div> 
</div>

<div class="row ">
    <div class="col-md-12">

        <div id="tickets">
        <%= f.fields_for :tickets do |builder| %>
        <%= render "ticket_fields", f: builder %>
        <% end %>
        <div class="links text-center">
        <%= link_to_add_association "追加",f, :tickets, :class => "btn btn-info btn-lg" %> 
        </div> 

        </div>        
    </div> 
</div>

<div class="col-md-4 col-md-offset-4 mt20">

<%= f.submit "保存" ,:id =>"eventsubmit" ,:class => "btn btn-lg btn-success btn-block" ,:disabled => "disabled" %>
</div>
<% end %>
</div>
ticket_fields.html.erb
cocoonのpartialの部分です。

<div class="nested-fields">
<h3>tickets</h3>


<div class="row text-center">
    <div class="col-md-3">
    <span class="label-title">title</span>
    </div>
    <div class="col-md-9">
    <div class='input-group'>       
    <%= f.text_field :title, :class => "form-control" %
    </div>
    </div>   
</div>     


<div class="row text-center">
    <div class="col-md-3">
    <span class="label-title">対象者</span>
    </div>

    <div class="col-md-9"> 
    <%= f.hidden_field :id %>

<% students = { '小学生' => 1, '中学生' => 2, '高校生' => 3 } %>

<% students.each do |st,i| %>
  <%= f.check_box :student_type, { multiple: true , include_hidden: false }, i %>
  <%= f.label :student_type, st, :value => i %>
<% end %>    

   </div>     
</div>  


<%= link_to_remove_association 'remove this', f %>
</div>

上記のようなフォームから、下記のようにDBに格納されたことは確認できました。
student_type: "[\"1\", \"2\"]"

これをeditで確認すると、このままでは、checked: の規定がないため、何も表示されません。

試したこと

https://teratail.com/questions/14954

を確認し、checked: を実装しようとしたのですが、Ticketからfindすることが出来ていません。

パラメータを確認すると、

Parameters: {"utf8"=>"✓", "authenticity_token"=>" 省略 , "tickets_attributes"=>{"0"=>{"id"=>"24", "_destroy"=>"false"}}, ""}, "button"=>"", "id"=>"23"}

上記の通りだったので、Ticket.find(params[:tickets_attributes[:id]] を試したのですが、うまく取得できませんでした。

どのようにすればcheckedの判定ができますでしょうか。

教えてください

補足情報(言語/FW/ツール等のバージョンなど)

Ruby 2.3
rails 4.2.1
cocoon 1.2.9

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • hana-da

    2016/10/14 19:05

    ticket_fields.html.erb の form_for の部分を公開してください

    キャンセル

  • Yuinyan

    2016/10/14 19:22

    ご連絡有難うございます。form_forの部分を記載しました。不足がありましたらご連絡下さい。

    キャンセル

  • hana-da

    2016/10/14 19:24

    あ、すみません。form_forで囲まれている部分全体です。他にもevent.rb や ticket.rb その他関連するコントローラなども開示すると解決が早いと思います。その際、みやすいように ``` を分けてもらえるとさらによいです。

    キャンセル

  • Yuinyan

    2016/10/14 19:27

    すみません、少々お待ちください。

    キャンセル

回答 1

checkベストアンサー

0

上記のようなフォームから、下記のようにDBに格納されたことは確認できました。 
student_type: "[\"1\", \"2\"]"

ここがとても気になります。
テーブル(モデル)設計を間違えていませんか?

記載されている情報から判断すると

  • Eventは複数のTicketを持つ事ができて、Ticketには複数のstudent_typeを指定できる。
  • student_typeは複数のTicketから指定される事がある

という事になるので

class Event
  has_many :tickets
end

class Ticket
  belongs_to :event
  has_many :ticket_student_types
end

class TicketStudentType
  belongs_to :ticket
  belongs_to :student_type
end

class StudentType
  has_many :tickets
end

になると思うのですが。。。いかがでしょう?


今は恐らくticketsテーブルのstudent_typeカラム(属性)がstring(文字列)で定義されているので折角formで複数指定できるcheckboxをつくってsubmitしても

  • paramsに入るところまではちゃんと ["1", "2"] という形で配列に入っているのに
  • Ticket の student_type に入ると to_s されて "[\"1\", \"2\"]" という形の文字列になってしまっています

という事で f.checkbox のことろで event.tickets の student_type を参照した際 "1" や "2" といった文字列が出てくれば簡単に比較して checked をつける事ができますが、"[\"1\", \"2\"]"という誤った形式の文字列で保存されているため容易には比較できず checked をつける事も非常に難しいです。

これを無理矢理配列に戻したりしてcheckedをつける方法はありますが、間違いを正す方が今後のためですし、正さないとstudent_typeについてrailsの便利機能(小学生対象のTicketをまとめて取得するなど)も使えなくなるのでここには書きません。

まずはテーブル(モデル)を正しい形に直しましょう。
そうすれば、それほど悩まずともできるはずです。
やり方がわからない、うまくいかないのはテーブル(モデル)の設計が間違えているからです。


Parameters: {"utf8"=>"✓", "authenticity_token"=>" 省略 , "tickets_attributes"=>{"0"=>{"id"=>"24", "_destroy"=>"false"}}, ""}, "button"=>"", "id"=>"23"}

上記の通りだったので、Ticket.find(params[:tickets_attributes[:id]] を試したのですが、うまく取得できませんでした。

これは色々おかしいところが多いです。

  • find() の ( に対応する ) がない
  • Symbol である :tickets_attributes に [:id] がついている

のでエラーになります。

Ticket.find(params[:tickets_attributes][:id]) と書けば文法上は大丈夫ですが
params[:tickets_attributes] は {"0"=>{"id"=>"24", "_destroy"=>"false"}}, ""} という Hash なので :tickets_attributes の直下に :id はなく nil が返ります。

"24" がほしいのであれば params[:tickets_attributes][0][:id] で取得する必要があります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/10/15 11:54

    有難うございます。後程時間をかけて確認させて頂きます。

    ちなみに下記をつけることで何とかやりたいことはできることができました。(おっしゃられる通り後々他の部分で影響がでるかもしれないので、モデルの部分はいずれにせよ見直しを考えてみます。)

    <% checked = f.object.payment_type.presence || [] %>

    また単純に一つのカラムに[]を指定して入れているのですが、これがあまりよくないのでしょうか。
    ここのカラムは確かに string でしたのでここが問題になるのでしょうか。。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る