Bug #55706

Different behavior of levelmedia in TYPO3 6.2

Added by Bjoern Jacob over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-02-05
Priority:Must have Due date:
Assigned To:- % Done:

100%

Category:Content Rendering Spent time: -
Target version:6.2.0
TYPO3 Version:6.2 Is Regression:Yes
PHP Version: Sprint Focus:
Complexity:

Description

I think levelmedia is behaving "strangely" (i.e. different compared to older versions) in TYPO3 6.2beta5. I've tested it with 3 installations.

The TypoScript

lib.getPageRessource = TEXT
lib.getPageRessource {
  data = levelmedia:-1, slide
}
<pre>

returns the following values:

* TYPO3 4.7: filename.jpg (i.e. filename)
* TYPO3 6.1: 63 (i.e. uid of file reference)
* TYPO3 6.2beta5: 35,34,33,22,18,17,16,15,14,10,9,7,12,11,8 (i.e. uids of all file references)

IMHO levelmedia should still return one uid which "fits the rootline".

Related issues

related to Core - Bug #43708: leveluid with slide doesn't work across multiple levels Closed 2012-12-07
related to Core - Bug #55923: intval() to (int) conversion regressions Resolved 2014-02-12
follows Core - Task #54265: Use (int) instead of intval() or (integer) Resolved 2013-12-07

Associated revisions

Revision cb0a33aa
Added by Michiel Roos over 1 year ago

[BUGFIX] Regression: intval patch broke levelmedia

Change-Id: Ib1c877c1353844dfac1955c39c864a0974bd6b78
Resolves: #55706
Releases: 6.2
Reviewed-on: https://review.typo3.org/27573
Reviewed-by: Stefano Kowalke
Reviewed-by: Jo Hasenau
Tested-by: Jo Hasenau
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Markus Klein
Tested-by: Markus Klein

History

#1 Updated by Simon Würstle over 1 year ago

Same probleme here with 6.2beta5. It behaves normal with beta4.

#2 Updated by Ernesto Baschny over 1 year ago

  • Category set to Content Rendering
  • Status changed from New to Accepted
  • Priority changed from Should have to Must have
  • Target version set to 6.2.0
  • Is Regression changed from No to Yes

Problem is reproduceable. Bug introduced in #54265 "(int) to intval()" optimization. Hope that's the last problem found with that big change. :(

#3 Updated by Gerrit Code Review over 1 year 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/27573

#4 Updated by Gerrit Code Review over 1 year 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/27573

#5 Updated by Stefan Neufeind over 1 year ago

Patch is currently under review. Of course you are right this change was not intended.

Just to have this complete: You mentioned that you now get IDs instead of a filename. Since FAL and file-releations you now need to add "treatIdAsReference = 1" to get the filename again.

#6 Updated by Michiel Roos over 1 year ago

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

#7 Updated by Bjoern Jacob over 1 year ago

@Stefan: That's right. Since there's the new FILES cObject I prefer using it instead of the "old" IMAGES cObject. For the last one we need the treatIdAsReference switch. Thx for clarification.

Also available in: Atom PDF