TYPO3 Core

Since once again I had a hard time finding out why an Extbase CommandController failed:

As found out by Alexander, if a task throws an exception, TYPO3\CMS\Extbase\Scheduler\Task catches it, logs it and returns false.

Logging is only done if [SYS][systemLog] has a value set (file|mail|syslog|error_log) - which is not the case in a default installation - not even in development context!
So normally this information is just silently dropped, thus I would not consider this as a feature request but as a bug.

Logging to the systemLog is fine, but the information should be thrown again afterwards, otherwise the only message you will see when executing a Task with an error:

Execution of task "Extbase CommandController Task (extbase)" failed with the following message: Task failed to execute successfully. Class: TYPO3\CMS\Extbase\Scheduler\Task, UID: 3

Oh, really, it failed with the message "failed to execute"? Very helpful..

Using

throw $e

return FALSE;

Execution of task "Extbase CommandController Task (extbase)" failed with the following message: PHP Warning: mssql_pconnect(): message: Cannot open user default database. Login failed. (severity 11) in xxx line 123

I tried to commit this as a change but I could not upload (maybe due to not having signed the CLA?).

If someone from the extbase team could commit this please?

[BUGFIX] Rethrow exceptions in task execution of SchedulerTask

Rethrow exceptions from tasks inheriting from Extbase-SchedulerTask 
instead of just returning false.

The logging in TYPO3\CMS\Extbase\Scheduler\Task happens only if 
[SYS][systemLog] has a value set (file|mail|syslog|error_log) - which is
not the case in a default installation. So this information is just 
silently dropped at the moment.

When returning false the unhelpful message in the BE log is "Task failed
to execute successfully." Rethrowing the exception gives the reason of 
failure instead as well when executing the task manually from the 
BE-scheduler as in the BE-log when executed via cli_dispatch scheduler

Resolves: #49071
Releases: master, 6.2

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36985

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36985

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37355