featdd / dpn_glossary

Glossary extension for TYPO3
http://typo3.org/extensions/repository/view/dpn_glossary
GNU General Public License v2.0
20 stars 31 forks source link

[Deprecation] parsing of terms not working with cached pages #125

Closed rr-it closed 6 years ago

rr-it commented 6 years ago

The parsing of terms is broken by commit https://github.com/featdd/dpn_glossary/commit/3907441377fc5ebb631fac40bcb5d2c381f7a1e4 .

On first page load terms are correctly parsed and replaced. On second page load - this time as cached page by TYPO3 - none of the terms are replaced.

TYPO3: v7.6.31 dpn_glossary: v2.7.5

featdd commented 6 years ago

There seems different beheviour in typo3 versions and setup, in 8.7.x it didn't.

Had fixed this for 9 compatibility, could you test if this is also valid for 7.6?

greetings Daniel

rr-it commented 6 years ago

I did a test with both "contentPostProc-cached" and "contentPostProc-output" combined (https://github.com/featdd/dpn_glossary/commit/5b1080c5b8d8cbe3c48d5c11f9f025593ad80ef3): it works!


As far as I see, TYPO3 7.6 behaves like this.

contentPostProc-all fires:

contentPostProc-cached fires:

contentPostProc-output fires:

My conclusion is to use both "contentPostProc-cached" and "contentPostProc-output" combined.

Drawbacks - unsolvable with availible hooks:

featdd commented 6 years ago

hi @rr-it thanks for testing, guess I'll cherry pick this and push this into the next update.

featdd commented 6 years ago

merged into master 27682036c7c40dc4e1ccab7bff8273d69c0a92f1