Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 28, 2025

Conversation

masaton0216
Copy link
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
Contributor Author

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

@akagane99
Copy link
Contributor

akagane99 commented May 27, 2025

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

Copy link
Contributor

@akagane99 akagane99 left a comment

Choose a reason for hiding this comment

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

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

@akagane99
Copy link
Contributor

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

@masaton0216
Copy link
Contributor Author

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

@masaton0216 masaton0216 merged commit afbeec7 into master May 28, 2025
1 check passed
@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