Project

General

Profile

Actions

Bug #51067

closed

Adjust usage of ContentObjectRenderer->getTreeList()

Added by Anja Leichsenring over 11 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Could have
Category:
Code Cleanup
Target version:
-
Start date:
2013-08-14
Due date:
% Done:

100%

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

Description

In typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer->getQuery() the pid where clause is build using getTreeList() to fetch recursive pids. But for the inclusion of the uppermost pid, string concatenation is used. The method getTreeList() offers the option to include the uppermost pid into the returned result by passing it as a negative value. Method getQuery() should use this opportunity instead of doing its own magic.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #51503: NEWS.txt entry for ContentObjectRenderer::getTreeList()ClosedAnja Leichsenring2013-08-29

Actions
Actions #1

Updated by Gerrit Code Review over 11 years ago

  • Status changed from New to Under Review

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

Actions #2

Updated by Gerrit Code Review about 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23092

Actions #3

Updated by Gerrit Code Review about 11 years ago

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

Actions #4

Updated by Gerrit Code Review about 11 years ago

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

Actions #5

Updated by Gerrit Code Review about 11 years ago

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

Actions #6

Updated by Gerrit Code Review about 11 years ago

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

Actions #7

Updated by Gerrit Code Review about 11 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23092

Actions #8

Updated by Gerrit Code Review about 11 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23092

Actions #9

Updated by Gerrit Code Review about 11 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23092

Actions #10

Updated by Anja Leichsenring about 11 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF