Projet

Général

Profil

Bug #22540

amélioration cache des variables de substitution en présence de pages conditionnelles

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
15 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

L'évaluation des conditions de page invalide le cache parce qu'elles ajoutent un "ConditionVars()" qui représente les données live. Comme ces données ne vont pas changer le long de la requête, on peut faire en sorte que le cache soit conservé.


Fichiers

Révisions associées

Révision a6b528ab (diff)
Ajouté par Frédéric Péters il y a environ 6 ans

misc: don't invalidate cache when evaluating conditions (#22540)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 6 ans

#2

Mis à jour par Frédéric Péters il y a environ 6 ans

(non, fait planter certains tests, je creuse)

#3

Mis à jour par Frédéric Péters il y a environ 6 ans

Il s'agit bien sûr d'être égal aux autres objets du même type, pas à tout le monde.

#4

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Ack. Mais en fonction de l'urgence ce serait cool d'avoir des tests sur ça (construction d'un FormDef avec un max de condition de remplissage/condition de saut de page complexes, remplissage d'un champ, passage à la page suivante, puis compter le nombre d'appels à invalidate_cache()).

#5

Mis à jour par Frédéric Péters il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)

Certainement pas urgent au point de se passer de test; j'en ai posé un vérifiant que l'ajout de pages n'ajoutait pas d'appels à invalidate_cache() (et je l'ai exécuté sans le patch pour vérifier que c'était vraiment correct).

commit a6b528ab816297c405543174f7079b5c4f029e72
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Mar 15 14:47:31 2018 +0100

    misc: don't invalidate cache when evaluating conditions (#22540)
#6

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF