TYPO3 Core

Thanks for the patch, will create one on current master and push it to Gerrit.

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12543

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12543

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12543

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12543

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12543

Error still exists in TYPO3 6.1.8..

Error still there in 6.1.9, indeed in GraphicalFunctions.php in line 2285

The error is still there in 6.2.4 and refers to typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php line 2216.
But it also depends on the PHP version.

I've called the imageMagickConvert function by this line:

TYPO3\CMS\Core\Imaging\GraphicalFunctions::imageMagickConvert("fileadmin/introduction/downloads/TYPO3_Logo_Guide_v3.pdf", "jpg", "1", "1", "", 0, "", 1)

If I use the following line, no error occurs:

TYPO3\CMS\Core\Imaging\GraphicalFunctions::imageMagickConvert("fileadmin/introduction/downloads/TYPO3_Logo_Guide_v3.pdf", "jpg", "1", "1", "", 0, array(), 1)

The only difference is the next to last parameter, not "" but array().

Maybe this helps to find a solution for core, or is a hint for extension-developers running into this error.

API says that the parameter needs to be an array. And core seems to have correct usage of the API in all places. So it would be up to extension-developers to please adjust their API-calls if they call it with a string instead. (We could add checked for is_array() in the function - but imho we can't check for correct API-usage in each and every place ...).