Skip to content

[施設予約] 繰り返し予約後、特定操作で500エラーになる - #2200

Merged
masaton0216 merged 1 commit into
masterfrom
OW-2575
May 28, 2025
Merged

[施設予約] 繰り返し予約後、特定操作で500エラーになる#2200
masaton0216 merged 1 commit into
masterfrom
OW-2575

Conversation

@masaton0216

Copy link
Copy Markdown
Contributor

背景

  • 顧客より問い合わせあり、後述の再現手順により500エラーが発生する
[2025-05-27 09:05:02] production.ERROR: Trying to get property 'id' of non-object {"userId":1,"exception":"[object] (ErrorException(code: 0): Trying to get property 'id' of non-object at C:\\sites\\connect_cms_php_7427\\htdocs\\app\\Plugins\\User\\Reservations\\ReservationsPlugin.php:1922)

修正概要

  • 直接的な原因は$input_earliestがnullの為、reservations_inputsテーブル更新処理でエラーになっています。
  • 後続の処理は繰り返し関係を保持する為の処理になりますが、$input_earliestが存在しない場合は後続処理の必要もないと判断して、nullチェックによるアーリーリターンを追加しています。※ここの判断もし間違ってたら突っ込んでください。

参考(再現手順)

  1. 繰り返し予約で3日間連続予約
  2. 1日目と3日目の時間を「編集」-「この予定のみ編集する」で変更
  3. さらに2日目の時間を「編集」-「この予定のみ編集する」で変更しようとするとシステムエラー
  4. システムエラーは出るが2日目の予約自体は変更されている。

レビュー完了希望日

6月3日(火)

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

なし

チェックリスト

@masaton0216
masaton0216 requested a review from akagane99 May 27, 2025 04:45
@masaton0216 masaton0216 self-assigned this May 27, 2025
@masaton0216 masaton0216 added the bug バグ・不具合連絡 label May 27, 2025
@masaton0216

Copy link
Copy Markdown
Contributor Author

@akagane99 レビューお願いできますでしょうか?難しい場合はその旨、返信ください。

@akagane99

akagane99 commented May 27, 2025

Copy link
Copy Markdown
Contributor

@masaton0216 明日5/28で良ければ時間とれるので、見て見ますね

@akagane99 akagane99 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認はしてませんが、目検してOKでした。

@akagane99

Copy link
Copy Markdown
Contributor

@masaton0216
レビューOKです。
マージ&ブランチの削除を、お時間あるときにお願いします。

@masaton0216

Copy link
Copy Markdown
Contributor Author

@akagane99 ありがとうございます!

@masaton0216
masaton0216 merged commit afbeec7 into master May 28, 2025
@masaton0216
masaton0216 deleted the OW-2575 branch May 28, 2025 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug バグ・不具合連絡

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants