Bug #31295

t3lib_beFunc::thumbCode doesn't respect "file_refernce" types

Added by Tolleiv Nietsch about 9 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2011-10-25
Due date:
% Done:

100%

TYPO3 Version:
4.5
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The upload folder detection in t3lib_befunc::thumbCode doesn't check the "internal_type" => "file_refernce" setting and might prefix the filepath by accident.

Related: http://bugs.typo3.org/view.php?id=18113

Associated revisions

Revision 3693037c (diff)
Added by Tolleiv Nietsch about 7 years ago

[BUGFIX] Handle file_references properly in t3lib_befunc::thumbCode

Group fields with internal_type "file_reference" reference the files
directly. The filepaths don't need any additional uploadFolder prefixes.

Due to the FAL changes in that method, this patch is not relevant for 6.0.

Change-Id: I7428fb9561180dd51cc1f45a9f6c012626b72fa8
Resolves: #31295
Releases: 4.5
Reviewed-on: https://review.typo3.org/21934
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

History

#1 Updated by Mr. Hudson about 9 years ago

  • Status changed from New to Under Review

Patch set 1 of change If7492ff3aef2caaa249743cb242ed3e0c0ec8dd6 has been pushed to the review server.
It is available at http://review.typo3.org/6334

#2 Updated by Mr. Hudson about 9 years ago

Patch set 2 of change If7492ff3aef2caaa249743cb242ed3e0c0ec8dd6 has been pushed to the review server.
It is available at http://review.typo3.org/6334

#3 Updated by Gerrit Code Review almost 9 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6334

#4 Updated by Gerrit Code Review almost 9 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6334

#5 Updated by Ernesto Baschny almost 9 years ago

  • Target version changed from 4.5.8 to 4.5.12

#6 Updated by Gerrit Code Review over 8 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6334

#7 Updated by Gerrit Code Review over 8 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6334

#8 Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/11353

#9 Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/21934

#10 Updated by Tolleiv Nietsch about 7 years ago

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

#11 Updated by Riccardo De Contardi about 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF