Project

General

Profile

Actions

Bug #51752

closed

FAL ArrayIterator::seek() warning in ElementBrowser

Added by Marcin Sągol over 10 years ago. Updated over 5 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 6 (0 open6 closed)

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
Actions #1

Updated by Benjamin Rau over 10 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

Actions #2

Updated by Ernesto Baschny over 10 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).

Actions #3

Updated by Andrea Herzog-Kienast over 10 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

Actions #4

Updated by Dennis Dittrich over 10 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

Actions #5

Updated by Ernesto Baschny over 10 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!

Actions #6

Updated by Markus Klein over 10 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.

Actions #7

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

Actions #8

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

Actions #9

Updated by Steffen Ritter over 10 years ago

  • Target version set to next-patchlevel
Actions #10

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

Actions #11

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

Actions #12

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

Actions #13

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

Actions #14

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

Actions #15

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

Actions #16

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

Actions #17

Updated by Markus Klein over 10 years ago

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

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

Actions #19

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

Actions #20

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

Actions #21

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

Actions #22

Updated by Markus Klein over 10 years ago

  • Status changed from Under Review to Resolved
Actions #23

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF