Project

General

Profile

Actions

Feature #64258

closed

PageRepository::getPageOverlay() for multiple pages at once

Added by Christian Weiske almost 10 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
Start date:
2015-01-13
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
hard
Sprint Focus:

Description

TYPO3\CMS\Frontend\Page\PageRepository::getMenu() calls getPageOverlay() for each single page, generating an additional SQL query for each page.

It would be more efficient if only one SQL query would be made, fetching the overlays for all pages at once. The method implementing this could then be used by getMenu().

This would reduce the number of SQL queries dramatically for big menus.


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #64257: PageRepository::getMenu(): Support multiple uidClosed2015-01-13

Actions
Related to TYPO3 Core - Bug #66557: getPageOverlay Hook - does not use getUserObj (Exception when using '&' before object)Closed2015-04-23

Actions
Related to TYPO3 Core - Bug #76031: getPageOverlay Hook does not loop through pages by referenceClosed2016-05-03

Actions
Actions #1

Updated by Mathias Schreiber almost 10 years ago

  • Target version set to 7.2 (Frontend)
Actions #2

Updated by Mathias Schreiber almost 10 years ago

  • Complexity set to hard
Actions #3

Updated by Gerrit Code Review almost 10 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 http://review.typo3.org/36054

Actions #4

Updated by Gerrit Code Review almost 10 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36054

Actions #5

Updated by Gerrit Code Review almost 10 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36054

Actions #6

Updated by Gerrit Code Review almost 10 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36054

Actions #7

Updated by Anonymous almost 10 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF