Bug #91421

PageRepository::getMenu() without uid in field list returns broken array

Added by Thomas Löffler 9 days ago. Updated 7 days ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2020-05-16
Due date:
% Done:

0%

TYPO3 Version:
9
PHP Version:
7.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

When not providing "uid" in the fields list to select, the returned array for that menu is broken.

Reproduce

1. Create a page with several subpages
2. Use PageRepository in extension code
3. Use getMenu() or any other function which uses protected function getSubpagesForPages()
4. Provide "fields" string without "uid", e.g. only "title"

Expectation

An array with all subpages of the given page, like this:

10 => ['title' => 'Subpage 1'],
11 => ['title' => 'Subpage 2'],
12 => ['title' => 'Subpage 3']

Current result

It gives you an array with empty index and only the last page of that menu:

 => ['title' => 'Subpage 3']

History

#1 Updated by Gerrit Code Review 8 days 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/c/Packages/TYPO3.CMS/+/64506

#2 Updated by Gerrit Code Review 8 days 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/c/Packages/TYPO3.CMS/+/64506

#3 Updated by Gerrit Code Review 8 days 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/c/Packages/TYPO3.CMS/+/64506

#4 Updated by Gerrit Code Review 7 days 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/c/Packages/TYPO3.CMS/+/64506

Also available in: Atom PDF