Bug #55706

Different behavior of levelmedia in TYPO3 6.2

Added by Bjoern Jacob almost 7 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Content Rendering
Target version:
Start date:
2014-02-05
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

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 TYPO3 Core - Bug #43708: leveluid with slide doesn't work across multiple levelsClosed2012-12-07

Actions
Related to TYPO3 Core - Bug #55923: intval() to (int) conversion regressionsClosed2014-02-12

Actions
Follows TYPO3 Core - Task #54265: Use (int) instead of intval() or (integer)ClosedJo Hasenau2013-12-07

Actions
#1

Updated by Simon Würstle almost 7 years ago

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

#2

Updated by Ernesto Baschny almost 7 years 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 almost 7 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/27573

#4

Updated by Gerrit Code Review almost 7 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/27573

#5

Updated by Stefan Neufeind almost 7 years 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 almost 7 years ago

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

Updated by Bjoern Jacob almost 7 years 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.

#9

Updated by Riccardo De Contardi about 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF