Projet

Général

Profil

Actions

Anomalie #6307

fermé

[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 10 ans. Mis à jour il y a presque 10 ans.

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

100%

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

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.

Mis à jour par Marc Claverie il y a presque 10 ans

  • Version cible mis à 3.2.23

Mis à jour par Éric Brison il y a presque 10 ans

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

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

Mis à jour par Jérôme Augé il y a presque 10 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

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

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

Formats disponibles : Atom PDF