Project

General

Profile

Actions

Bug #78071

closed

Language menu TypoScript configurations throws exception if cHashIncludePageId is set

Added by Thorsten Lübkemann about 8 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Caching
Target version:
-
Start date:
2016-09-27
Due date:
% Done:

0%

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

Description

In TYPO3 6.2.27 with cHashIncludePageId set to true adding any parameter to lib.langMenu.addQueryString.exclude will throw an exception on some pages.

If I set [FE][cHashIncludePageId] in TYPO3 6.2.27 and use the language menu with:

lib.langMenu = HMENU
lib.langMenu.addQueryString = 1
lib.langMenu.addQueryString.exclude = L,cHash

and the URL contains a parameter which isn't excluded from the cache hash generation (like a parameter from an Extbase extension)

then

typo3/sysext/frontend/Classes/ContentObject/Menu/AbstractMenuContentObject.php on line 415 will call the method getQueryArguments() of typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer. On line 6544 of that file the 'id' gets added to the $exclude array.

After that the AbstractMenuContentObject calls the method analyzeCacheHashRequirements() in line 416, which calls the method getRelevantParameters() of typo3/sysext/frontend/Classes/page/CacheHashCalculator.php. This method throws an exception on line 128 because the 'id' parameter isn't set anymore.


Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #77956: Exception in language menu when using addQueryString.exclude and cHashIncludePageIdClosed2016-09-15

Actions
Actions #1

Updated by Stephan Großberndt about 8 years ago

  • Status changed from New to Closed
  • PHP Version deleted (7.0)

This is the better bug description but a patch already exists for the other one, so closing this. (btw. PHP7 is not supported for TYPO3 6.2)

Actions #2

Updated by Stephan Großberndt about 8 years ago

Thorsten, could you please try if the patch works for you and add a verify +1 if so? Thanks!

https://review.typo3.org/#/c/49943

Actions

Also available in: Atom PDF