Bug #55763

using object as string in ResourceStorage->getUniqueName

Added by Tymoteusz Motylewski almost 6 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2014-02-07
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

Description

I got this error while attaching file in RTE
"Object of class TYPO3\CMS\Core\Resource\Folder could not be converted to string"

it is caused by the call
if (!$this->driver->fileExistsInFolder($theDestFile, $folder)) {
which should look like:
if (!$this->driver->fileExistsInFolder($theDestFile, $folder->getIdentifier())) {

because $folder is an object.


Related issues

Duplicated by TYPO3 Core - Bug #55766: PHP Catchable Fatal Error: Object of class TYPO3\CMS\Core\Resource\Folder could not be converted to string in LocalDriver.php line 677 Closed 2014-02-07

Associated revisions

Revision 3119e34a (diff)
Added by Tymoteusz Motylewski almost 6 years ago

[BUGFIX] Pass path instead of object to fileExistsInFolder

In ResourceStorage in the method getUniqueName second argument
passed to fileExistsInFolder should be string instead of object.

This fixes an exception thrown when trying to insert magic image
in the RTE.

Resolves: #55763
Releases: 6.2
Change-Id: I4ed624d4ea5a3602936cf8493052ef111a391990
Reviewed-on: https://review.typo3.org/27412
Reviewed-by: Michiel Roos
Reviewed-by: Marcin SÄ…gol
Reviewed-by: Frans Saris
Tested-by: Frans Saris
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

History

#1 Updated by Gerrit Code Review almost 6 years ago

  • Status changed from New to Under Review

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

#2 Updated by Tymoteusz Motylewski almost 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#3 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF