Revert "[BUGFIX] fix scheduler task logging in case of an Exception" 84/52284/2
authorMarkus Klein <markus.klein@typo3.org>
Fri, 31 Mar 2017 10:15:00 +0000 (11:15 +0100)
committerMarkus Klein <markus.klein@typo3.org>
Fri, 31 Mar 2017 10:15:14 +0000 (12:15 +0200)
This reverts commit 11deab5a29537490ed2b1492f9ac0ca0fb27465c.

Due to a wrong commit message this change is reverted.
There is no backport yet.

Change-Id: I3258a3aed4f1ec4ccb8a9cb9928d1876e6fd67ba
Resolves: #80566
Releases: master
Reviewed-on: https://review.typo3.org/52284
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
typo3/sysext/scheduler/Classes/Task/AbstractTask.php

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