Projet

Général

Profil

Anomalie #4050

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

Ajouté par phe nix il y a plus d'un an. Mis à jour il y a 6 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
Début:
25/11/2017
Echéance:
% réalisé:

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 :)

Historique

#1 Mis à jour par RealET 🔸 il y a plus d'un an

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

#3 Mis à jour par phe nix il y a plus d'un an

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 Mis à jour par b b il y a plus d'un an

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 Mis à jour par phe nix il y a plus d'un an

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 Mis à jour par b b il y a plus d'un an

N'hésite pas à fournir un patch si tu considères que c'est bloquant et améliorable :)

#7 Mis à jour par b b il y a plus d'un an

  • Sujet changé de Minifier ne fonctionne plus du les fichier "produit" à Minifier ne fonctionne pas sur les fichier générés par #PRODUIRE

#8 Mis à jour par erational 👺鬼 il y a 6 mois

Je peux fermer donc ?

#9 Mis à jour par phe nix il y a 6 mois

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 Mis à jour par cedric - il y a 6 mois

  • Statut changé de Nouveau à Fermé
  • Version cible changé de 3.2 à 3.3
  • Resolution mis à fixed

Formats disponibles : Atom PDF