トランザクション例外処理がうまく動きません

Question

現在、セレクトボックスの選択肢により、異なる更新処理を作っています。
下のようなセレクトボックスですが、
選択肢は「なし」「承認」「否認」「申請中」でカラム名はindicater_replyです
右のチェックボックスのカラム名はchangeで、チェックボックスにチェックをし、セレクトの選択肢を選んで更新するといった動きです。

「なし」「承認」「否認」の場合は問題なくカラムが更新され、flashが出るのですが
チェックをしない場合やセレクトが「申請中」の場合も「なし」「承認」「否認」と同じflashが出ます。
コントローラーの８行目で、チェックボックスにチェックがある、かつ「なし」「承認」「否認」の場合しか進行しない内容になっているのに、
なぜ、例外のエラーに飛ばないのかがわかりません。

恐れ入りますが、方法を教えて頂けますでしょうか。

 def update_overtime_notice
    ActiveRecord::Base.transaction do 
      o1 = 0
      o2 = 0
      o3 = 0
      overtime_notice_params.each do |id, item|
        if item[:indicater_reply].present?
          if (item[:change] == "1") && (item[:indicater_reply] == "なし" || item[:indicater_reply] == "承認" || item[:indicater_reply] == "否認")
          attendance = Attendance.find(id)
          user = User.find(attendance.user_id)
            if item[:indicater_reply] == "なし" 
              o1+= 1
              item[:overtime_finished_at] = nil
              item[:tomorrow] = nil
              item[:overtime_work] = nil
              item[:indicater_check] = nil

            elsif item[:indicater_reply] == "承認"
                  item[:indicater_check] = nil
              o2 += 1
          attendance.indicater_check_anser = "承認しました"
            elsif item[:indicater_reply] == "否認"
                  item[:indicater_check] = nil
              o3 += 1
          attendance.indicater_check_anser = "否認しました"
            end
            attendance.update_attributes!(item)
          end
        end
      end
      flash[:success] = "【残業申請】　#{o1}件なし,　#{o2}件承認,　#{o3}件否認しました"
        redirect_to user_url(params[:user_id])
    end  
  rescue ActiveRecord::RecordInvalid 
      flash[:danger] = "無効な入力データがあった為、更新をキャンセルしました。"
      redirect_to edit_overtime_notice_user_attendance_url(@user,item)
  end
  

Answer 1

もうひとつ問題があります
if (item[:change] == "1") && a || b || c
の部分。

if (item[:change] == "1") && ( a || b || c )

にしないと。

Comments

[zoff77]

かっこですかね？？
８行目ですか？ご指摘のように(item[:indicater_reply] == "なし" || item[:indicater_reply] == "承認" || item[:indicater_reply] == "否認")なので（）はついていますが、他の部分ですか？

[winterboum]

あ、ごめんなさい。錯覚してました。

Answer 2

なぜ、例外のエラーに飛ばないのかがわかりません。

逆に「なぜ例外に飛ぶ」と思ったのでしょうか？条件に当たらない場合、単に処理をしないだけで、例外を起こす要素はないかと思います。

Comments

[zoff77]

そうなんですね。例外処理 = エラーに飛ぶと思っていました。
「申請中」でupdateしたらエラーを出すにはどうしたら良いでしょうか？

[maisumakun]

モデルにそのようなバリデーションを書いてください。

[zoff77]

検討したのですが、コードがうまく書けませんでした。
カスタムメソッド
def check_indicater_reply
if change == "1" && indicater_reply == "申請中"
errors.add(:indicater_reply, "申請内容を決定して下さい")
end
を書き、validate :check_indicater_reply, on: :update_overtime_notice

にしましたが変わりなしでした。。。。
申し訳ないですが、術を教えてもらえないでしょうか。
因みに、セレクトはenumをs使っています