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

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

ただいまの
回答率

89.24%

[rails5]パラメータでネストした日時情報をフォームオブジェクトに渡したい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 475

zendendo

score 41

前提・実現したいこと

ruby on railsで、フォームオブジェクトとfields_forを使って
一つのフォーム入力画面から複数の異なったモデルを更新したいと考えています。

ただ、パラメータでネストした(fields_forで子モデルに入力する)日時の情報を FormObjectに渡して正しい日時情報に変換したいのですが、やり方がわからず困っています。

現状のソースコード

ビュー側

 <%= form_for @contract_master, as: :edit_dract_contract_form do |fb| %>

  <div class="form-group">
      <%= fb.label :契約名, class: "label-inline" %>
      <%= fb.text_field :name, class:'form-control' %>
  </div>
    <!-- 子モデルへの入力フォーム -->
    <%= fb.fields_for :temp_cont_item_stock do |f2| %>
      <div class="form-group">
          <%= f2.label :契約受付説明文, class: "label-inline" %><span class="badge badge-pill badge-danger">必須</span>
          <%= f2.text_area :description, class:'form-control', rows:"5" %>
      </div>
      <div class="form-group">
          <%= f2.label :使用期限, class: "label-inline" %>
          <%= f2.datetime_select :expiration_date, start_year: now.year, end_year: now.year + 1000, class:'form-control' %>
      </div>
    <% end %>
<% end %>


コントローラでフォームオブジェクトに渡すストロングパラメータの設定

    def edit_contract_master_params
        params.require(:edit_dract_contract_form).permit(:name, temp_cont_item_stock: [:description, :expiration_date])
    end


生成されたストロングパラメータの内容

 <ActionController::Parameters {"name"=>"テスト契約", "temp_cont_item_stock"=><ActionController::Parameters {"description"=>"契約説明文です", "expiration_date(1i)"=>"2021", "expiration_date(2i)"=>"4", "expiration_date(3i)"=>"18", "expiration_date(4i)"=>"22", "expiration_date(5i)"=>"40"} permitted: true>} permitted: true>


フォームオブジェクトに日時情報(expiration_date)を渡そうとすると5つに分割された状態になってしまいます。
このままでは日時情報をフォームオブジェクト側でうまく受け取ることができません。
そこで5つに分かれてしまった日時情報を一つにまとまった日時情報に変換し直そうとしたのですがうまくいきませんでした。

フォームオブジェクト側

class EditDractContractForm
    include ActiveModel::Model
    attr_accessor :name, :temp_cont_item_stock, :expiration_date

    #年・月・日・時・分に分割されて渡されている日時を一つの日時情報に自動変換する
    def initialize(params = {})

        if params.is_a?(ActionController::Parameters)
            [:expiration_date].each do |attribute|
                datetime_parts = (1..5).map { |i| params.delete("#{attribute}(#{i}i)") }
                params[attribute] = Time.zone.local(*datetime_parts) if datetime_parts.any?
            end
        end
        super
    end
#以下省略

ちなみに、パラメータの日時情報がネストしていないときは、上記のinitializeメソッドで日時情報に変換してくれます。
おそらくパラメータ内のネストした日時情報をinitializeメソッド内で取得すればいいのですが、その方法がわからなくて困っています。

どうすれば、うまく変換できるでしょうか。
回答をよろしくお願いします。

追記(最終的に成功したコード)

回答者からのアドバイスをもとに最終的に成功したコードをここに記します。
(コメント欄だと見ずらいのでここに記します)。

class EditDractContractForm
        include ActiveModel::Model
        attr_accessor :name, :temp_cont_item_stock, :expiration_date

    def initialize(attributes={})#attributesにはパラメータが入る。

        if attributes.present?
            attrs = attributes.to_unsafe_h.dup#受け取ったストロングパラメータをハッシュ化

            if attrs["temp_cont_item_stock"]["expiration_date(1i)"].present?
                        #日時作成
                attrs[:expiration_date] = DateTime.new(
                    attrs["temp_cont_item_stock"]["expiration_date(1i)"].to_i,
                    attrs["temp_cont_item_stock"]["expiration_date(2i)"].to_i,
                    attrs["temp_cont_item_stock"]["expiration_date(3i)"].to_i,
                    attrs["temp_cont_item_stock"]["expiration_date(4i)"].to_i,
                    attrs["temp_cont_item_stock"]["expiration_date(5i)"].to_i
                    )
            end
            super(attrs)
        end
    end

これでフォームオブジェクト内なら、expiration_dateとすれば日時情報を取得できます。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • 退会済みユーザー

    退会済みユーザー

    2019/04/18 23:34

    1. ネストしているときのパラメーター
    2. 最終的にどのような結果になっていればよいのか

    などを書くと解決しやすいと思います。

    キャンセル

回答 1

checkベストアンサー

0

expiration_dateというのはDate型でしょうか?それともDateTime型でしょうか?

  • Date型なら
    datetime_selectdate_selectにすればRailsがうまいこと対応してくれそうな気がします。

  • DateTime型なら
    変換してあげて、superに渡して上げればいいかと

class EditDractContractForm
    include ActiveModel::Model
    attr_accessor :name, :temp_cont_item_stock, :expiration_date

    def initialize(attributes={})
      attrs = attributes.dup # dupしたほうがいいかはわかりませんが


      if attrs.key?("expiration_date(1i)")
        attrs[:expiration_date] = DateTime.new(
          attrs.delete["expiration_date(1i)"].to_i, 
          attrs.delete["expiration_date(2i)"].to_i,
          attrs.delete["expiration_date(3i)"].to_i,
          attrs.delete["expiration_date(4i)"].to_i,
          attrs.delete["expiration_date(5i)"].to_i
        )
      end

      super(attrs)
    end

これで動けば良いですが。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/04 11:02

    hashがネストしていたんですね。

    temp_cont_item_stockを取り出して、attrに入れてあげればうまく行きそうですね。

    キャンセル

  • 2019/05/04 11:27

    最終的には以下のようなコードにすることで分割されていない正しい形の日時情報をフォームオブジェクト側で取得することができました。
    度重なるアドバイスと素早い対応に感謝します。ありがとうございました。

    フォームオブジェクト側の記述
    ##############################################
    class EditDractContractForm
    include ActiveModel::Model
    attr_accessor :name, :temp_cont_item_stock, :expiration_date

    def initialize(attributes={})
    binding.pry#デバック用
    if attributes.present?
    attrs = attributes.to_unsafe_h.dup# dupしたほうがいいかはわかりませんが

    if attrs["temp_cont_item_stock"]["expiration_date(1i)"].present?
    attrs[:expiration_date] = DateTime.new(
    attrs["temp_cont_item_stock"]["expiration_date(1i)"].to_i,
    attrs["temp_cont_item_stock"]["expiration_date(2i)"].to_i,
    attrs["temp_cont_item_stock"]["expiration_date(3i)"].to_i,
    attrs["temp_cont_item_stock"]["expiration_date(4i)"].to_i,
    attrs["temp_cont_item_stock"]["expiration_date(5i)"].to_i
    )
    end
    super(attrs)
    end
    end
    ###############################################

    キャンセル

  • 2019/05/04 11:48

    良いと思います。解決して良かったです。おつかれさまでしたー

    キャンセル

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

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