Projet

Général

Profil

Anomalie #6307

[performance] API setStyle déclenche systématiquement la régénération de l'autoloader

Ajouté par Jérôme Augé il y a environ 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
14/03/2016
Echéance:
% réalisé:

100%

Version source:
Solution proposée:
Principaux fichiers impactés:
Complexité:
Contrôle:
Thème:
Socle technique
Régression:
Non
Jalons: 3.2 R17

Description

L'utilisation de setStyle déclenche la régénération de l'autoloader.

Cela fait que lors d'une mise à jour d'un module, wstart va régénérer deux fois le cache de l'autoloader : une première fois par wstart lui même, et une seconde fois par setStyle appelé par wstart.

Déjà que la régénération du cache de l'autoloader peut être longue, on la subit en plus 2 fois à chaque mise à jour de modules.

Après investigation il s'avère que le problème se situe dans less.php, qui déclenche la régénération en cherchant une classe "px" qui n'existe pas (effet de bord d'un appel, à property_exists() sur une chaîne contenant "px"). Cela peut-être corrigé en patchant less.php : https://github.com/oyejorge/less.php/pull/308

Si ce n'est pas corrigé par less.php, il faudra voir pour implémenter des méthodes pour désactiver (unregister ?) notre autoloader avant d'appeler less.php.

Historique

#1 Mis à jour par Marc Claverie il y a presque 3 ans

  • Version cible mis à 3.2.23

#2 Mis à jour par Éric Brison il y a presque 3 ans

  • Assigné à mis à Jérôme Augé
  • Statut changé de Nouveau à Assigné

Je crois que less.php à déjà corrigé ce problème

#3 Mis à jour par Jérôme Augé il y a presque 3 ans

- La correction sur "less.php" a été effectivement intégrée sur la branche "master" mais pas taggé ni releasé....
- "dynacase-less-installer" a été modifié pour désactiver et réactiver explicitement l'autoloader : #6320

#4 Mis à jour par Jérôme Augé il y a presque 3 ans

  • Statut changé de Assigné à Fermé
  • % réalisé changé de 0 à 100

Formats disponibles : Atom PDF