Project

General

Profile

Actions
Copy link

Bug #83328

closed

Wrong result with QueryGenerator->getTreeList()

Added by Florian Rival over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2017-12-14
Due date:
% Done:

100%

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

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

bug_83328.diff (780 Bytes) bug_83328.diff Florian Rival, 2017-12-14 16:15

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Task #85024: Add functional tests for QueryGeneratorTestClosedAlexander Schnitzler2018-05-16

Actions
Actions
Copy link
 #1

Updated by Florian Rival over 6 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Florian Rival over 6 years ago

The patch to correct this problem.

Actions
Copy link
 #3

Updated by Susanne Moog over 6 years ago

  • Category set to Backend API
Actions
Copy link
 #4

Updated by Marieke de Bruin about 6 years ago

This bug is also present in CMS7

Actions
Copy link
 #5

Updated by Gerrit Code Review about 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

Actions
Copy link
 #6

Updated by Gerrit Code Review about 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

Actions
Copy link
 #7

Updated by Gerrit Code Review about 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

Actions
Copy link
 #8

Updated by Gerrit Code Review about 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

Actions
Copy link
 #9

Updated by Gerrit Code Review about 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

Actions
Copy link
 #10

Updated by Gerrit Code Review about 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

Actions
Copy link
 #11

Updated by Gerrit Code Review about 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

Actions
Copy link
 #12

Updated by Gerrit Code Review about 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

Actions
Copy link
 #13

Updated by Gerrit Code Review about 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

Actions
Copy link
 #14

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

Actions
Copy link
 #15

Updated by Tymoteusz Motylewski almost 6 years ago

Is there an easy way to reproduce the issue from the backend?

Actions
Copy link
 #16

Updated by Florian Rival almost 6 years ago

I don't know if there's an easy way to reproduce this bug from the backend.

Actions
Copy link
 #17

Updated by Guido Schmechel almost 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
Actions
Copy link
 #18

Updated by Alexander Schnitzler almost 6 years ago

  • Related to Task #85024: Add functional tests for QueryGeneratorTest added
Actions
Copy link
 #19

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

Actions
Copy link
 #20

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

Actions
Copy link
 #21

Updated by Guido S. almost 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #22

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

Actions
Copy link
 #23

Updated by Guido S. almost 6 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #24

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF