Bug #51752
closedFAL ArrayIterator::seek() warning in ElementBrowser
Added by Marcin Sągol about 11 years ago. Updated about 6 years ago.
100%
Description
Hi, when working with content elements that use images (copy/paste), i have often this errors in Log module:
Core: Error handler (BE): PHP Warning: ArrayIterator::seek() expects parameter 1 to be long, string given in /typo3/sysext/core/Classes/Resource/Driver/LocalDriver.php line 321
Core: Error handler (BE): PHP Warning: rtrim() expects parameter 1 to be string, object given in /typo3/sysext/recordlist/Classes/Browser/ElementBrowser.php line 1893
Updated by Benjamin Rau about 11 years ago
I can confirm this problem!
What i did to get the php warning messages:- Added content element of type "Only images"
- Added two images to this ce
- Checked if ce is shown correctly in FE (it does)
- Copied ce
- Pasted on same page on another colPos
- Went to Log
My setup:
PHP 5.4
TYPO3 6.1.3
Relevant output:
PHP Warning: ArrayIterator::seek() expects parameter 1 to be long, string given in typo3/sysext/core/Classes/Resource/Driver/LocalDriver.php line 325
Core: Error handler (BE): PHP Warning: rtrim() expects parameter 1 to be string, object given in typo3/sysext/recordlist/Classes/Browser/ElementBrowser.php line 1893
Output long: https://gist.github.com/benjaminrau/9367799064572d800e8f
Updated by Ernesto Baschny about 11 years ago
- Subject changed from FAL related error reported in Log module to FAL ArrayIterator::seek() warning in ElementBrowser
- Status changed from New to Accepted
- Priority changed from Should have to Must have
- Target version deleted (
next-patchlevel) - TYPO3 Version changed from 6.1 to 6.0
- Complexity set to easy
This was half-fixed in #39235 before the 6.0 release, but there are still two other occurrences in the ElementBrowser that make wrong use of the getFiles() API (passing the extensionList).
Updated by Andrea Herzog-Kienast about 11 years ago
Can confirm this problem too in TYPO3 6.1.3 and 6.1.4. So TYPO3 in this version is imho not usable.
But interesting is, that one time I was able to insert images in this 6.1 project. But I still do not know since when this problem exists. Images I put in, before this problem occurs, are still there.
Really big problem,
Best,
Andrea
Updated by Dennis Dittrich about 11 years ago
Ernesto Baschny wrote:
This was half-fixed in #39235 before the 6.0 release, but there are still two other occurrences in the ElementBrowser that make wrong use of the getFiles() API (passing the extensionList).
Thank You.
I use this changeset to make it usable...
Updated by Ernesto Baschny about 11 years ago
- Is Regression set to No
Would you be so kind and posting a review request with the changes you made for latest "master"? Read http://wiki.typo3.org/CWT for details on how to do that. Thanks!
Updated by Markus Klein about 11 years ago
The commit linked by Dennis is the fix from Andreas, which is in the Core since 6.0. I'm still having this issue here on master.
Updated by Gerrit Code Review about 11 years ago
- Status changed from Accepted 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/25359
Updated by Gerrit Code Review about 11 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Steffen Ritter about 11 years ago
- Target version set to next-patchlevel
Updated by Gerrit Code Review almost 11 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Florian Boos almost 11 years ago
The warning in the description of this issue can also be reproduced on Typo3 6.1 when opening a file browser in extension records.
E.g. if you edit a record for tt_news or tt_address in the Backend and click on the FileBrowser Link to insert a image, the above warning is also shown in the log. I tested this issue in one of my extensions, tt_news and tt_address.
Best regards, Florian
Updated by Gerrit Code Review almost 11 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Gerrit Code Review almost 11 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Gerrit Code Review almost 11 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Gerrit Code Review almost 11 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Gerrit Code Review almost 11 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25359
Updated by Markus Klein almost 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 52585063a5240b11aa1d541c699e240de8261813.
Updated by Gerrit Code Review almost 11 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26475
Updated by Gerrit Code Review almost 11 years ago
Patch set 1 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26476
Updated by Gerrit Code Review almost 11 years ago
Patch set 2 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26476
Updated by Gerrit Code Review almost 11 years ago
Patch set 2 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26475
Updated by Markus Klein almost 11 years ago
- Status changed from Under Review to Resolved
Applied in changeset e6bfc6e740007049de1b695b82fde753b9028e83.