Projet

Général

Profil

Evolution #4036

[Compresseur] Rechercher la présence événtuelle de .min.(js|css) dans le nom de la source

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
31/10/2017
Echéance:
% réalisé:

0%

Resolution:
fixed

Description

Je propose un amélioration de la fonction minifier, lorqu'elle est appliquée à un fichier.

La plupart des plugins qui embarquent des bibliothèques tierces fournissent leur sources en format avec commentaires (plugin.js) et minifié (plugin.min.js).
On peut parler de convention tacite concernant l'extension ".min.js"

Le traitement dérogatoire serait le suivant :

A : La source a-t-elle une extension min.(css|js) ?

Oui : on renvoie tel quel
Non : ->B

B : Existe-t-il un fichier nommé source.min.(css|js) dans le même répertoire que la source ?

Oui : on renvoit le fichier trouvé
Non : Alors on lance le compresseur sur la source et on renvoie

Je vois 2 avantages à cette évolution :
- Un léger gain de performance : on économise un processus de compression superflu en échange d'une recherche de fichier dans un dossier unique (même pas besoin de find_in_path)
- On se fie sur la version minifiée livrée par le tiers (supposée fiable), et ainsi on évite d'éventuels problèmes qui peuvent surgir lorsque qu'on applique le compresseur sur un fichier déjà minifié.

Vos avis ?

Historique

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

  • Version cible changé de 3.3 à 3.4

L'idée me semble intéressante, même si elle risque de complexifier le comportement du compresseur. Voyons ce qu'en pense le reste de l'équipe, en attendant je bascule vers la branche 3.4.

#2 Mis à jour par kent1 - il y a plus d'un an

b b a écrit :

L'idée me semble intéressante, même si elle risque de complexifier le comportement du compresseur. Voyons ce qu'en pense le reste de l'équipe, en attendant je bascule vers la branche 3.4.

Pour le point 1 je pense que c'est bien, ce serait plutôt pas mal même

Le point 2 me semble plus tiré par les cheveux perso

#3 Mis à jour par cedric - il y a 6 mois

  • Assigné à mis à cedric -
  • Version cible changé de 3.4 à 3.3

Moi je trouve que les 2 idées sont bonnes !

#4 Mis à jour par cedric - il y a 6 mois

  • Statut changé de Nouveau à Fermé
  • Resolution mis à fixed

Formats disponibles : Atom PDF