Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tagplugin cache #1

Merged
merged 4 commits into from
Mar 5, 2012
Merged

Tagplugin cache #1

merged 4 commits into from
Mar 5, 2012

Conversation

demental
Copy link
Owner

Salut mister,
J'ai créé pour M une branche tagplugin_cache avec la chose dedans. Y'a le fichier de test mais on peut pas le lancer direct au niveau du framework, l'environnement phpunit est en place uniquement dans un projet...Mais tout est vert, promis.
Par contre j'ai fait ensorte que l'appli plante au premier hasTag() si le champ tagplugin_cache est absent de la table taggée.

Effectivement niveau requêtes quand on fait beaucoup de hasTag ça économise pas mal, genre sur du listing de commandes où, quelques requêtes économisées.
Tu me diras si tu vois des schmilblicks qui collent pas... je vais pas passer sur le master de suite

A+

@demental
Copy link
Owner Author

demental commented Mar 2, 2012

En même temps j'hésite à aller plus loin:

En cas d'ajout de tag, concaténer plutôt que régénérer.

@demental
Copy link
Owner Author

demental commented Mar 5, 2012

Un autre point sur lequel je souhaiterais travailler dans ce PR:

Sur les formulaires de recherche du backoffice, on affiche les tags utilisés dans la table correspondant au module.

ça ne prend pas en compte le plugin falsedelete.
J'ai essayé de le faire avec une jointure, mais laisse tomber la neige et en plus ça doit être un gouffre de perf.

Bref l'idée serait qu'au delete, on remplit le cache du plugin. Et que la plugin falsedelete ait une méthode undelete qui broadcasterait un event "undelete" que le plugin tag implémenterait (il reconstituerait les relations dans ce cas).

A voir, sachant qu'avec cette méthode, on perdrait quand même l'info concernant la date d'application du tag. Mais c'est mineur me semble-til

@demental
Copy link
Owner Author

demental commented Mar 5, 2012

Bon, je réalise cette feature sur une prochaine branche, celle-ci est bonne à passer

demental pushed a commit that referenced this pull request Mar 5, 2012
@demental demental merged commit 8c557a4 into master Mar 5, 2012
@demental demental deleted the tagplugin_cache branch December 17, 2014 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant