Feature #45883
closed
Adding getPageOverlay_postProcess hook
0%
Description
Hi,
I need to modify attributes of page records. The best point for this is the getPageOverlay() method in t3lib_pageSelect. It has already a preProcess hook, where I could modify eg. the title of the page. But after this hook some translations are performed and merged with my modified array, which will overwrite my changes with the translated attributes.
So, the preProcess hook works fine with page records in default language. But when I want to modify also translated pages, I have to write a xclass, because no postProcess is existing here.
Furthermore the getRecordOverlay() method has a pre and postProcess hook, so it seems to be a bit inconsistent.
I wish there would be such postProcess hook in getPageOverlay() method, too.
Thanks!
Updated by Gerrit Code Review over 11 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 https://review.typo3.org/18515
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18515
Updated by Mathias Schreiber almost 10 years ago
Hey Armin,
would you continue to work on this or abandon the patch?
Thanks in advance
Updated by Alexander Opitz over 9 years ago
Patch was from Georg.
@Georg Ringer
would you continue to work on this or abandon the patch?
Updated by Gerrit Code Review about 9 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/18515
Updated by Gerrit Code Review almost 9 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/18515
Updated by
Updated by Benni Mack over 5 years ago
- Related to Epic #88027: Properly handle Hooks/Signals and Events added
Updated by Susanne Moog over 4 years ago
- Status changed from New to Closed
As this is a technical task and has not gotten priority for a long long time (in between the class names, locations and even the hook system changed) I'm going to close this issue now. If an event in that location is still needed (and there is no other mechanism that can be used in the meantime) please reopen the ticket once work starts again.
Updated by Peter Schuhmann over 2 years ago
hello everyone. was just working on something similar: I want to manual set specific values from the original page to the overlay since the overlay does not have the value. Like "use default language" except the field is background field. I totally agree that the behavior is inconsistent here since there is a getPageOverlay_preProcess hook but no getPageOverlay_postProcess which would be the right behavior and would propably take like 5 Minutes to implement. I would love to see that implemented.