受け取ったパラメータをnilで更新したいです

Question

AのユーザーがBのユーザーに下記のパラメーターを送り（申請）
受け取ったBのユーザーが

def overtime_params
    
      params.require(:attendance).permit(:overtime_finished_at, :tomorrow, :overtime_work,:indicater_check,:indicater_check_superior)
    end

受け取ったBのユーザーが、下記のモーダルにて確認を押して確認を押して内容を知らせるとい機能です。

指示者確認で「なし」と選択すると、Aのユーザーの申請自体がなかったことにしたいので、
送ったパラメーターをnilにして、上記のモーダルを消したいです。
考えたのですが、そのようなロジックが思いつきませんので、方法を教えて欲しいです。

<div class="modal-body">
      <%= form_with(model: @user, url: "#" , method: :patch , local: true) do |f| %>
        <% @attendances.each do |id, attendance|%>
          <% user = User.find(id) %>
          <h3 align="center" class="modal-title"><%= "【#{user.name}からの残業申請】" %></h3>
      
          <table class="table table-bordered table-condensed overtime_notice-table">

            <thead class="overtime_notice">
              <th>日付</th>
              <th>曜日</th>
              <th>終了予定時間</th>
              <th>指定勤務終了時間</th>
              <th>時間外時間</th>
              <th>業務処理内容</th>
              <th>指示者確認
                <div class="maru size_small black">       
                  <div class="letter3">印</div>
                </div> 
              </th>
              <th>変更</th>
              <th>勤務を確認する</th>
            </thead>

            <% attendance.each do |day| %>
              <%= f.fields_for "attendance[]", day do |d| %>
              
                <tbody class="line">
                  <% css_class = 
                  case $days_of_the_week[day.worked_on.wday]
                    when '土'
                      'text-primary'
                    when '日'
                      'text-danger'
                    end
                    %>
                  <td><%= l(day.worked_on, format: :short) %></td>
                  <td class="<%= css_class %>"><%= $days_of_the_week[day.worked_on.wday] %></td>
                  <td><%= day.overtime_finished_at.try(:strftime,"%H:%M") %></td>
                  <td><%= format_basic_info(user.designation_work_end_time) %></td>
                  <td><%= overtime_worked_on(day.overtime_finished_at, user.designation_work_end_time, day.tomorrow) %></td>
                  <td><%= day.overtime_work %></td>
                  <td><%= d.select :indicater_reply ,{'なし':1, '申請中':2, '承認':3, '否認':4}, { prompt: false}, { class: 'form-control input-sm'} %></td>
                  <td><%= d.check_box :change,id: "change" %></td>
                  <td><%= link_to "確認", verifacation_user_path(user.id, worked_on: day.worked_on), class: "btn btn-primary" %></td>
                </tbody>
                
              <% end %>
            <% end %>
          </table>
          <div class="center">
            <%= f.submit "変更を送信する", class: "btn btn-primary" %>
          </div>
        <% end %>
      <% end %>
    </div>
  </div>
</div>

create_table "attendances", force: :cascade do |t|
    t.date "worked_on"
    t.time "started_at"
    t.time "finished_at"
    t.string "note"
    t.integer "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.datetime "overtime_finished_at"
    t.boolean "tomorrow", default: false
    t.string "overtime_work"
    t.string "indicater_check"
    t.string "indicater_check_superior"
    t.string "indicater_reply"
    t.boolean "change"
    t.boolean "verification"
    t.index ["user_id"], name: "index_attendances_on_user_id"
  end

class AttendancesController < ApplicationController
 
  before_action :set_user, only: [:edit_one_month]
  before_action :logged_in_user, only: [:update, :edit_one_month]
  before_action :admin_user, only: [:index,:destroy,:edit_basic_info]
  before_action :set_one_month, only: [:edit_one_month]

  UPDATE_ERROR_MSG = "勤怠登録に失敗しました。やり直してください。"


  # 申請お知らせモーダル
  
  def edit_overtime_notice
    @user = User.find(params[:user_id])
    @attendances = Attendance.where(indicater_check_superior: "申請中", indicater_check: @user.name).order(:user_id, :worked_on).group_by(&:user_id)
  end


# お知らせモーダル更新
  def update_overtime_notice
  end 




# 申請モーダル
  def edit_overtime_request
    @user = User.find(params[:user_id])
    @attendance = Attendance.find(params[:id])
    @superior = User.where(superior: true).where.not( id: current_user.id )
  end

  
  def update_overtime_request
    @user = User.find(params[:user_id])
    @attendance = Attendance.find(params[:id])
    if @attendance.update_attributes(overtime_params)
      flash[:success] = "残業申請を受け付けました"
      redirect_to user_url(@user)
    end  
  end



  def update
    @user = User.find(params[:user_id])
    @attendance = Attendance.find(params[:id])
    if @attendance.started_at.nil?
      if @attendance.update_attributes(started_at: Time.current.change(sec: 0))
        flash[:info] = "おはようございます！"
      else
        flash[:danger] = UPDATE_ERROR_MSG
      end
    elsif @attendance.finished_at.nil?
      if @attendance.update_attributes(finished_at: Time.current.change(sec: 0))
        flash[:info] = "お疲れ様でした。"
      else
        flash[:danger] = UPDATE_ERROR_MSG
      end
    end
    redirect_to @user
  end

  def edit_one_month
  end 

  def update_one_month
    ActiveRecord::Base.transaction do # トランザクションを開始します。
      attendances_params.each do |id, item|
        attendance = Attendance.find(id)
        attendance.update_attributes!(item)
      end
    end
    flash[:success] = "1ヶ月分の勤怠情報を更新しました。"
    redirect_to user_url(date: params[:date])
  rescue ActiveRecord::RecordInvalid # トランザクションによるエラーの分岐です。
    flash[:danger] = "無効な入力データがあった為、更新をキャンセルしました。"
    redirect_to attendances_edit_one_month_user_url(date: params[:date])
  end


  
  
  


private
    # 1ヶ月分の勤怠情報を扱います。
    def attendances_params
      params.require(:user).permit(attendances: [:started_at, :finished_at, :note])[:attendances]
    end


    # 申請モーダルの情報
    def overtime_params
      params.require(:attendance).permit(:overtime_finished_at, :tomorrow, :overtime_work,:indicater_check,:indicater_check_superior)
    end

    # 申請お知らせモーダルの情報
    def overtime_notice_params
      params.require(:attendance).permit(:indicater_reply, :change, :verification, :indicater_check_superior)
    end


    


  end   

Comments

[no1knows]

> 指示者確認で「なし」と選択すると、Aのユーザーの申請自体がなかったことにしたい
とは、申請を削除するということでしょうか？
またコントローラーはすべて記載ください。

[zoff77]

はい。選択肢には「なし」「申請中」「承認」「拒否」があり、「なし」は申請自体をなかったことし、取り消すという動きです。よって、申請したユーザーが送ったパラメーターをnilで更新すれば良いと考えましたが、消去でも同じですよね？
コントローラーすいませんでした。。。

Answer 1

変更を送信　したらモーダルは消えるのでしょうから、なし にして送信したら何もせずに戻ったら？

サーバーに戻すのがいやでしたら、indicater_reply　の on change を拾って、そこが なし　な行をクリアするというJSを書くことになります。

あ。もしかして、入力データをクリアしたあたいでDBも修正するのでしょうか？
ですと「何もせずに」ではなくて　params上のその人のデータをclearすることになりますね。

Comments

[no1knows]

方針としてはwinterboumさんと同じなので補足させていただきます。

> 指示者確認で「なし」と選択すると、Aのユーザーの申請自体がなかったことにしたいので、
送ったパラメーターをnilにして、上記のモーダルを消したいです。

ということですが、データの持ち方などを考えると登録済みのレコードのindicater_replyカラムを4にする形が適切な流れかと思います。

上記を前提として、UI側を考えると下記2つの選択肢が良いかと思いますがいかがでしょうか？

①「なし」にして「変更を送信する」ボタンを押したら、モーダルが消える。
② 申請を取り消すボタンを各行に追加する。
（もし、「なし」を選択⇒「変更を送信する」ボタンを押す⇒モーダルが消えるという流れが手間だと考えているのであれば・・・）

[zoff77]

ありがとうございます!
①がイメージ通りですので、一度試してみます！！
ありがとうございました

[no1knows]

ごめんなさい。なしと拒否がごっちゃになっていました。
違いがよく理解できていませんが・・・①のような形で実装いただければと思います。