[BUGFIX] Optimize tag usage and cache flushing

b13 / menus

TYPO3 Extension for creating fast menus in a fast fashion

GNU General Public License v2.0

54 stars 21 forks source link

[BUGFIX] Optimize tag usage and cache flushing #34

Closed bmack closed 3 years ago

bmack commented 3 years ago

The change

Does not add tags to a specific "page leave" anymore
Flushes the caches only for the tag of "menuId_currentPage" of what is flushed
Checks for "page leaves" on the cache flushing to flush the right "menuId_" tag

This helps to

avoid having lots of tags in the cache backend anymore
reduce the amount of "flushByTag"/"flushByTags" calls when flushing the caches

achimfritz commented 3 years ago

Test: moving a page with subpages does not flush cache expected (ordering of pages are not changed in Menu) (looks like flushing cache of parent page always neccessary?)

bmack commented 3 years ago

Yes, adding tests when flushing via DataHandler Hook should be possible without much effort, will see what I can do and when I have time for it.