Skip to content
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

Added Exception Message to Taks Notification Email #43666

Open
wants to merge 12 commits into
base: 5.1-dev
Choose a base branch
from
Prev Previous commit
phpcs fix
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
  • Loading branch information
brbrbr and richard67 committed Jun 18, 2024
commit 03eef8e8aa37003cbef0ec1ecaf8ddc44b118908
Original file line number Diff line number Diff line change
Expand Up @@ -242,9 +242,8 @@ private function getDataFromTask(Task $task): array
'EXEC_DATE_TIME' => $lockOrExecTime,
'TASK_OUTPUT' => $task->getContent()['output_body'] ?? '',
'TASK_EXCEPTION' => ($task->getContent()['exception']?->getMessage() ?? '')
."\n\n". // This might result in one or two empty strings with new Lines. Not too visible in email or HTML, so not really worth the effort to add a lot of extra code to optimize this.
Path::removeRoot($task->getContent()['exception']?->getTraceAsString() ?? '')
,
. "\n\n" // This might result in one or two empty strings with new Lines. Not too visible in email or HTML, so not really worth the effort to add a lot of extra code to optimize this.
. Path::removeRoot($task->getContent()['exception']?->getTraceAsString() ?? ''),
];
}

Expand Down