[BUGFIX] Fix scheduler task logging in case of an exception 92/52292/2
authorChristian Wolff <chris@connye.com>
Fri, 31 Mar 2017 07:23:49 +0000 (09:23 +0200)
committerMarkus Klein <markus.klein@typo3.org>
Fri, 31 Mar 2017 11:35:25 +0000 (13:35 +0200)
Correctly log the message if a Scheduler task throws an exception.
The exception code is now part of the logged message too.

Resolves: #80566
Releases: master, 7.6
Change-Id: Ic4eb25608094869733b4a23057d209fab1a1bccf
Reviewed-on: https://review.typo3.org/52292
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
typo3/sysext/scheduler/Classes/Task/AbstractTask.php

index 526ee3d..e11e76a 100644 (file)
@@ -451,8 +451,9 @@ abstract class AbstractTask
                 }
                 if ($failure instanceof \Exception) {
                     // Log failed execution
-                    $logMessage = 'Task failed to execute successfully. Class: ' . get_class($this) . ', UID: ' . $this->taskUid . '. ' . $failure->getMessage();
-                    $this->scheduler->log($logMessage, 1, $failure->getCode());
+                    $logMessage = 'Task failed to execute successfully. Class: ' . get_class($this)
+                        . ', UID: ' . $this->taskUid . ', Code: ' . $failure->getCode() . ', ' . $failure->getMessage();
+                    $this->scheduler->log($logMessage, 1);
                     // Do not serialize the complete exception or the trace, this can lead to huge strings > 50MB
                     $failureString = serialize([
                         'code' => $failure->getCode(),