search

featdd / dpn_glossary

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

Use lazy-loading for properties of term #187

Closed julianhofmann closed 1 year ago

julianhofmann commented 1 year ago

I#m focused on the parser part of the extension, less on the plugins... but for the parser, performance can be increased by not resolving all relations of the model. For tooltips, there's no need for having media or descriptions. The relations could be loaded via lazy-loading when accessing the property.

Maybe, even a fully simplified model could be used here: no need for medadescription, seotitle,... And for the plugin, this simplified model could be extended...

Still my use-case: 1500 terms, 2500 synonyms. So, every avoidable query counts.

featdd commented 1 year ago

Hi @julianhofmann,

I merged your pull request here: 738d6c07bba8884271b0d93c7b7f218001416560 Thanks for your work.

Greetings Daniel