Anomalie #4050
Minifier ne fonctionne pas sur les fichier générés par #PRODUIRE
0%
Description
Hello,
Soit le code suivant :
$js = '<script async src="'.minifier(produire_fond_statique('javascript/spip.foundation.js'), 'js').'" type="text/javascript"></script>';
Il ne sera pas minifier, car produire créer un chemin local/cache-js/jsdyn-javascript_spip_foundation_js-caab56e4.js?1511547016
Or, l'expression régulière dans compresseur_fonctions, ligne 52 n'accepte pas les querystrings :
preg_match(',\.' . $format . '$,i', $source, $r)
Du coup impossible de minifier :)
History
#1
Updated by RealET 🔸 over 3 years ago
Et si tu rajoutes un supprimer_timestamp https://www.spip.net/fr_article6184.html (comme l'a fait marcimat dernièrement) ?
#2
Updated by RealET 🔸 over 3 years ago
#3
Updated by phe nix over 3 years ago
C'est une idée, seulement, elle fait vraiment trop rustine à mon goût.
Perso, j'ai contourné le problème autrement : https://zone.spip.org/trac/spip-zone/changeset/107737/_plugins_/foundation_6/trunk
Je pense que l'expression régulière devrait prendre en compte les querystring, ce sera beaucoup plus propre :)
#4
Updated by b b over 3 years ago
Vu que le ticket liste deux méthodes pour contourner cette limitation pour ce cas d'usage, je pense qu'on peut fermer le ticket non ?
#5
Updated by phe nix over 3 years ago
Si l'utilisation d'une rustine est normal et qu'il ne faut pas améliorer l'expression régulière, oui, on peut fermer le ticket :)
#6
Updated by b b over 3 years ago
N'hésite pas à fournir un patch si tu considères que c'est bloquant et améliorable :)
#7
Updated by b b over 3 years ago
- Subject changed from Minifier ne fonctionne plus du les fichier "produit" to Minifier ne fonctionne pas sur les fichier générés par #PRODUIRE
#8
Updated by erational 👺鬼 over 2 years ago
Je peux fermer donc ?
#9
Updated by phe nix over 2 years ago
erational 鬼 a écrit :
Je peux fermer donc ?
Je ne sais pas, le bug a été corrigé ?
J'ai rien vu passé, mais j'avoue que je ne suis plus attentif a ce qui se passe sur SPIP.
#10
Updated by cedric - over 2 years ago
- Status changed from Nouveau to Fermé
- Target version changed from 3.2 to 3.3
- Resolution set to fixed