Bug #83328
closedWrong result with QueryGenerator->getTreeList()
100%
Description
When retrieving tree list with QueryGenerator->getTreeList(), the result is often wrong.
Example with following tree :
A
!-> B > D> C -> E
!
!
$queryGenerator->getTreeList(#pid-A, 3, 0);
Result : A, B, C, D, E (OK)
$queryGenerator->getTreeList(#pid-A, 3, 1);
Result : B, E instead of B, C, D, E
$queryGenerator->getTreeList(#pid-A, 3, 2);
Empty result instead of D, E
The problem is because when concatenated pid, a comma is missing.
Files
Updated by Florian Rival almost 7 years ago
- File bug_83328.diff bug_83328.diff added
The patch to correct this problem.
Updated by Gerrit Code Review over 6 years ago
- Status changed from New 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/56139
Updated by Gerrit Code Review over 6 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/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Tymoteusz Motylewski over 6 years ago
Is there an easy way to reproduce the issue from the backend?
Updated by Florian Rival over 6 years ago
I don't know if there's an easy way to reproduce this bug from the backend.
Updated by Guido Schmechel over 6 years ago
There is no easy way. My steps:
- Build a small rootline in the backend
- Build or use an extension which calls getTreeList() standalone
Updated by Alexander Schnitzler over 6 years ago
- Related to Task #85024: Add functional tests for QueryGeneratorTest added
Updated by Gerrit Code Review over 6 years ago
Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56139
Updated by Gerrit Code Review over 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56984
Updated by Guido S. over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b2ebd75481b5810e4d33b5b15b6548dbad61d35a.
Updated by Gerrit Code Review over 6 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57026
Updated by Guido S. over 6 years ago
- Status changed from Under Review to Resolved
Applied in changeset ae609b912a42e35a79d37647b4d068b6a643380c.