Bug #39428

Layout and partial identifiers should be independent of current controller name

Added by Alexander Berl about 9 years ago. Updated about 7 years ago.

Status:
Resolved
Priority:
Must have
Category:
Core
Start date:
2012-07-31
Due date:
% Done:

100%

Estimated time:
Has patch:
No

Description

Currently, all templates get an identifier made up of package/subpackage, controller, prefix and unique hash.
This leads to all layouts and partials being compiled and cached once for each controller they are used from, even though the contents is independant and hence just a duplicate.

I suggest adding the controller name as part of the prefix in getTemplateIdentifier() only and stripping it from createIdentifierForFile().

#1

Updated by Gerrit Code Review about 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13391

#2

Updated by Karsten Dambekalns about 9 years ago

  • Category set to Core
  • Assignee set to Alexander Berl
  • Target version set to 1.1
#3

Updated by Gerrit Code Review about 9 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13391

#4

Updated by Karsten Dambekalns about 9 years ago

  • Target version changed from 1.1 to 1.1.1
#5

Updated by Karsten Dambekalns almost 9 years ago

  • Target version changed from 1.1.1 to 2052
#6

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13391

#7

Updated by Gerrit Code Review over 8 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13391

#8

Updated by Karsten Dambekalns about 8 years ago

  • Target version changed from 2052 to 1.1.1
#9

Updated by Gerrit Code Review about 7 years ago

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

#10

Updated by Gerrit Code Review about 7 years ago

Patch set 1 for branch 2.2 of project Packages/TYPO3.Fluid has been pushed to the review server.
It is available at http://review.typo3.org/32430

#11

Updated by Alexander Berl about 7 years ago

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

Also available in: Atom PDF