Skip to content

fix #4288 【メール】メール送信テスト時にエラーがあった時にエラー出力されない件を修正#4393

Open
kaburk wants to merge 1 commit into
baserproject:5.2.xfrom
kaburk:5.2.x-#4288
Open

fix #4288 【メール】メール送信テスト時にエラーがあった時にエラー出力されない件を修正#4393
kaburk wants to merge 1 commit into
baserproject:5.2.xfrom
kaburk:5.2.x-#4288

Conversation

@kaburk
Copy link
Copy Markdown
Collaborator

@kaburk kaburk commented May 22, 2026

よろしくお願いします。

Copilot AI review requested due to automatic review settings May 22, 2026 10:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

管理画面の「メール送信テスト」実行時にエラーが発生しても内容が表示されない問題(#4288)に対し、APIレスポンスの message に例外メッセージを付与してエラー内容を返すようにする変更です。

Changes:

  • check_sendmail の例外捕捉時に、固定文言に加えて例外メッセージ($e->getMessage())を連結して返却

Comment on lines 104 to +105
$this->setResponse($this->response->withStatus(401));
$message = __d('baser_core', 'ログを確認してください。');
$message = __d('baser_core', 'ログを確認してください。') . $e->getMessage();
Comment on lines 103 to 106
} catch (\Throwable $e) {
$this->setResponse($this->response->withStatus(401));
$message = __d('baser_core', 'ログを確認してください。');
$message = __d('baser_core', 'ログを確認してください。') . $e->getMessage();
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants