Bug #35260

Missing selection field in t3lib_TCEmain::getPreviousLocalizedRecordUid()

Added by Francois Suter over 10 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Should have
Category:
DataHandler aka TCEmain
Target version:
Start date:
2012-03-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.7
PHP Version:
5.3
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

To fix #30469 a new method was introduced in t3lib_TCEmain to fetch the "neighbouring" translated record, if any. For tt_content this method checks for the "colPos" field in order to compare elements from the same content area. But "colPos" is not among the fields selected in the SQL query.

The solution is simply to add "colPos" in the list of fields to select when the table being handled is "tt_content".


Related issues

Related to TYPO3 Core - Bug #30469: Sorting of localized tt_content elements is wrong after copyingClosedStanislas Rolland2011-09-30

Actions
Related to TYPO3 Core - Bug #37221: Follow up #35260: Missing selection field in t3lib_TCEmain::getPreviousLocalizedRecordUid()ClosedFrancois Suter2012-05-16

Actions
#1

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Accepted to Under Review

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

#2

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

#3

Updated by Gerrit Code Review over 10 years ago

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

#4

Updated by Gerrit Code Review over 10 years ago

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

#5

Updated by Francois Suter over 10 years ago

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

Updated by Felix Nagel over 10 years ago

This patch is faulty. Please take a look here: #37221

#7

Updated by Riccardo De Contardi almost 5 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF