[施設予約] 「重複予約を許可しない」設定にしていても、予約の終了時間を「24:00」にすると重複して予約ができてしまう不具合対応

[akagane99]

概要

施設予約で「重複予約を許可しない」設定にしていても、予約の終了時間を「24:00」にすると重複して予約ができてしまう不具合がありましたので、重複予約チェックを修正しました。

不具合詳細

• 施設管理
  • 「重複予約を許可しない」設定にする
• 施設予約
  • 2025-05-30 14:00-17:00予定あり
  • 2025-05-30 12:00-24:00予定を登録すると、登録できてしまう

※ 終了時間の24:00表示は、施設予約特有の表示で、変換して表示しています。
　システム上では2025-05-30 24:00はイレギュラーな値で、CarbonやDBのdatetime型ですと、正しい値は 翌日の0時 2025-05-31 00:00 になります。
　2025-05-30 24:00のイレギュラーな値で、予約重複チェックを行ったため、うまくチェックが働きませんでした。

対応

• 重複チェックで2025-05-30 24:00等の24:00がきた場合、正しい値の 翌日の0時 2025-05-31 00:00 に変換してチェックするよう対応しました。

レビュー完了希望日

5/30(金)

修正後画面

(参考) 施設管理＞施設変更（重複予約＝許可しない）

(参考) 施設予約＞初期表示（05-30 14:00-17:00予定あり）

施設予約＞予約（05-30 12:00-24:00予定を登録で、重複予約エラーが出ることを確認しました）

簡易テスト

• github actionsで簡易テストを実施しました。
  • https://github.com/opensource-workshop/connect-cms/actions/runs/15311849492

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

無し

チェックリスト

• プルリクエストにわかりやすいタイトルとラベルを付けました。https://github.com/opensource-workshop/connect-cms/wiki/Pull-requests-Rule

[masaton0216]

@akagane99 ご対応ありがとうございます。
そもそも、24時に丸めてしまう処理を除去してしまえば？と考えたのですが、この処理を入れたのにもきっと何か経緯があるんですよね？
3f498e6

24時丸め処理が必要なのであれば、本修正も問題ないとは思いますが、丸め処理が入った経緯ご存知だったらお伺いしてもよいでしょうか？

[akagane99]

@masaton0216
24時表示は、不具合のご連絡頂いたお客様要望で入れたんです。
なので、削るに削れなくて。

[masaton0216]

@akagane99 説明ありがとうございます。
そうしたら仕方ないですね。本修正も妥当なものと思います。
マージさせて頂きます。