Project

General

Profile

Anomalie #4050

Minifier ne fonctionne pas sur les fichier générés par #PRODUIRE

Added by phe nix over 2 years ago. Updated almost 2 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
SPIP - 3.3
Start date:
11/25/2017
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

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 2 years ago

Et si tu rajoutes un supprimer_timestamp https://www.spip.net/fr_article6184.html (comme l'a fait marcimat dernièrement) ?

#3 Updated by phe nix over 2 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 2 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 2 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 2 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 2 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 👺鬼 almost 2 years ago

Je peux fermer donc ?

#9 Updated by phe nix almost 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 - almost 2 years ago

  • Status changed from Nouveau to Fermé
  • Target version changed from 3.2 to 3.3
  • Resolution set to fixed

Also available in: Atom PDF