Project

General

Profile

Evolution #4036

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

Added by placido . almost 2 years ago. Updated about 1 year ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
Start date:
10/31/2017
Due date:
% Done:

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 ?

History

#1 Updated by b b almost 2 years ago

  • Target version changed from 3.3 to 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 Updated by kent1 - almost 2 years ago

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 Updated by cedric - about 1 year ago

  • Assignee set to cedric -
  • Target version changed from 3.4 to 3.3

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

#4 Updated by cedric - about 1 year ago

  • Status changed from Nouveau to Fermé
  • Resolution set to fixed

Also available in: Atom PDF