Skip to content

Conversation

@masaton0216
Copy link
Contributor

@masaton0216 masaton0216 commented Oct 28, 2025

概要

新規インストール時、マイグレーション実行後にシーダーが正常に動作せず、500エラーが発生する問題を修正しました。

問題の原因

  • 2025_10_09_160056_add_mail_oauth2_to_configs.php でconfigsテーブルに直接レコードを挿入していた
  • マイグレーション実行後、configsテーブルに1件レコードが存在する状態になる
  • DefaultConfigsTableSeeder の条件 if (Configs::count() == 0) が false となり、基本設定が投入されない
  • 基本設定レコード(base_site_name等)が存在しないため、画面表示時に「Attempt to read property 'value' on null」エラーが発生

対応内容

  • 新しいマイグレーション 2025_10_29_000000_remove_mail_auth_method_from_migration.php を追加
  • マイグレーションで作成した mail_auth_method レコードを削除
  • 以降は MailAuthMethod::smtp をデフォルト値として動作(コードは既に対応済み)
  • 管理画面でメール認証方式を変更すると updateOrCreate で自動作成される

影響範囲

  • 既存環境: mail_auth_method レコードが削除されますが、コードがデフォルト値で動作するため機能に影響なし
  • 新規インストール: configsテーブルが空の状態でシーダーが実行され、正常にインストール完了

レビュー完了希望日

できるだけ早く

関連PR/Issues

参考情報

  • MailServiceProvider.phpSystemManage.php では既に getConfigsValue() の第3引数でデフォルト値を指定済み
  • メール設定画面で認証方式を変更すると updateOrCreate でレコードが作成される

DB変更

なし(マイグレーションによるデータ削除のみ)

チェックリスト

  • コーディング規約に従っている
  • 既存機能への影響を確認した
  • テストを実施した
  • ドキュメントの更新が不要であることを確認した

@masaton0216 masaton0216 changed the title [基盤] 新規インストール時にエラーが発生する問題を修正しました [コア] 新規インストール時にエラーが発生する問題を修正しました Oct 28, 2025
@masaton0216 masaton0216 changed the title [コア] 新規インストール時にエラーが発生する問題を修正しました [Configsデータ] 新規インストール時にエラーが発生する問題を修正しました Oct 28, 2025
@masaton0216 masaton0216 self-assigned this Oct 28, 2025
@masaton0216 masaton0216 added the bug バグ・不具合連絡 label Oct 28, 2025
@masaton0216 masaton0216 merged commit bdbb3c3 into master Oct 28, 2025
1 check passed
@masaton0216 masaton0216 deleted the fix-issues-2293 branch October 28, 2025 12:55
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