Bug #51752

FAL ArrayIterator::seek() warning in ElementBrowser

Added by Marcin Sągol almost 9 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2013-09-04
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
5.5
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

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


Related issues

Related to TYPO3 Core - Bug #39235: Warning: ArrayIterator::seek() expects parameter 1 to be long, string givenClosedAndreas Wolf2012-07-25

Actions
Related to TYPO3 Core - Bug #47648: PHP Warning: rtrim() expects parameter 1 to be string in ElementBrowser.php line 1867Closed2013-04-27

Actions
Related to TYPO3 Core - Bug #52969: Uncaught exception if editor got no file mountClosedMarkus Klein2013-10-19

Actions
Related to TYPO3 Core - Bug #53580: Cleanup ElementBrowserClosedMarkus Klein2013-11-13

Actions
Related to TYPO3 Core - Bug #54025: Fix usage of file filter in ElementBrowser->TBE_expandFolder()Closed2013-11-27

Actions
Has duplicate TYPO3 Core - Bug #47649: LocalDriver.php line 312 - PHP Warning: ArrayIterator::seek() expects parameter 1 to be long, stringClosed2013-04-27

Actions
#1

Updated by Benjamin Rau almost 9 years ago

I can confirm this problem!

What i did to get the php warning messages:
  1. Added content element of type "Only images"
  2. Added two images to this ce
  3. Checked if ce is shown correctly in FE (it does)
  4. Copied ce
  5. Pasted on same page on another colPos
  6. 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

#2

Updated by Ernesto Baschny almost 9 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).

#3

Updated by Andrea Herzog-Kienast almost 9 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

#4

Updated by Dennis Dittrich almost 9 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...

http://forge.typo3.org/projects/typo3v4-core/repository/revisions/7472f53a064bde013b938d4d28051ba211aeb5a3/diff/typo3/sysext/recordlist/Classes/Browser/ElementBrowser.php

#5

Updated by Ernesto Baschny almost 9 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!

#6

Updated by Markus Klein almost 9 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.

#7

Updated by Gerrit Code Review over 8 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

#8

Updated by Gerrit Code Review over 8 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

#9

Updated by Steffen Ritter over 8 years ago

  • Target version set to next-patchlevel
#10

Updated by Gerrit Code Review over 8 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

#11

Updated by Florian Boos over 8 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

#12

Updated by Gerrit Code Review over 8 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

#13

Updated by Gerrit Code Review over 8 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

#14

Updated by Gerrit Code Review over 8 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

#15

Updated by Gerrit Code Review over 8 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

#16

Updated by Gerrit Code Review over 8 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

#17

Updated by Markus Klein over 8 years ago

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

Updated by Gerrit Code Review over 8 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

#19

Updated by Gerrit Code Review over 8 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

#20

Updated by Gerrit Code Review over 8 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

#21

Updated by Gerrit Code Review over 8 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

#22

Updated by Markus Klein over 8 years ago

  • Status changed from Under Review to Resolved
#23

Updated by Benni Mack almost 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF