rails ステータスアップデートの時に、前チェックを皆さんはやりますか？

Question

rails ステータスアップデートの時に、前チェックを皆さんはやりますか？

たとえは下記のように審査するステータスがあったとします。

• 前提条件

■審査ステータスのモデル
status
 name: 文字列  

■審査ステータス一覧

審査中、審査完了、キャンセル、契約

※捕捉情報
- ステータスはenumで管理しないの？は今回はスルーでお願いします。
- ステータスは定数で管理

質問詳細

キャンセルにupdateするときに、キャンセルに更新できるのは、審査中のステータスだけのようなチェック入れますか？

def update_judgement_status_to_cancel
 #変更がなかったらreturn
 return false unless saved_change_to_attribute?(:status)
　
 after_status = self.saved_changes[:status][1]

 # 審査中のステータスだけキャンセルにできる
 self.update(name:"キャンセル") if after_status == "審査中"
end

上記のように、チェック入れますか？

ビューの実装で、キャンセルのボタンを押すのに制限がもともと入っているので、updateの処理のところでもいるのでしょうか？

Answer 1

キャンセルにupdateするときに、キャンセルに更新できるのは、審査中のステータスだけのようなチェック入れますか？

基本的には入れたほうがいいです（内輪しか使わないような画面なら手を抜くこともあるかもしれませんが）。フォームで除外していても、攻撃者はフォームで生成し得ないパターンのリクエストを作成可能です。

---

質問とは別件でしょうが、いったん成約になった後にキャンセルが発生した場合はどう処理するのでしょうか。