plg a écrit:

La question "de fond" étant quand même : peut-on séparer fusion et minification?

Oui parce que les balises speciales @import et url(...) sont gérés par piwigo et non par le minimifacteur

rvelices a écrit:

plg a écrit:

On pourrait par exemple, ne faire par défaut QUE la fusion des fichiers et en option désactivée par défaut la minification.

Pourquoi ne pas faire la minification seulement si php version >= 5.2 sans option ?

Parce que je n'y ai pas pensé ;-)

rvelices, pourrait-on, le temps de trouver mieux, ne plus minifier les CSS et juste les fusionner ?

Même si le soucis ne concerne principalement que les abonnées Free.fr, ils ne sont pas les seuls à avoir une version de PHP inférieure à 5.2. De plus, je me demande vraiment si le minify des fichiers CSS est si utile que ça en terme de performances.

On pourrait par exemple, ne faire par défaut QUE la fusion des fichiers et en option désactivée par défaut la minification.

donc il doit y avoir qq chose que php 5.1 n'aime pas ... j'avais vu que css min utilise "type hinting" pour les parametres

public static function minify($source, array $filters = null, array $plugins = null)

mais je croyais que c'est compatible php 5.1. Si ce n'est pas ca ca doit etre un truc semblable autour du language...

dans ce cas je dirais que soit rollback soit tant pis pour free. A noter aussi que wordpress necessite php 5.2.4 - http://wordpress.org/about/requirements/ et j'ai quand meme l'impression que free veut vraiment se debarasser de ce service gratuit ...

Eric a écrit:

Je teste l'astuce donnée ici : http://fr.piwigo.org/forum/viewtopic.ph … 90#p187990 pour voir.

Testé et validé ! J'ai remis le fichier cssmin.class.php par défaut et ajouté la ligne $conf['template_combine_files'] = false; dans le config.inc.php. Pas eu besoin de supprimer le contenu des template-extension. La mise à jour s'est déroulée correctement :

temps total de la mise à jour : 0.151 s
    temps total des requêtes SQL : 0.118 s
    nombre de requêtes SQL : 107

Bon, en même temps c'était une galerie de test ne contenant que 4 photos. Mais çà fonctionne...

OK, c'est quand même une information intéressante : ça plante à la ligne 3642, alors qu'elle n'a rien de spéciale cette ligne. Par contre, je me demande s'il n'y a pas une limite sur Free.fr pour la taille d'un fichier PHP : 3642 lignes, c'est monstrueux et en tout c'est 5085 lignes ce fichier !

OK, j'ai un peu investigué sur le sujet.

D'abord j'ai écrit une grosse bêtise : le problème de licence, ce n'est pas sur cssmin, mais sur jsmin (et mistic100 a déjà trouvé un remplaçant pour Piwigo 2.5).

Ensuite, cssmin est distribué sous 2 formes : minifiée (tout sur une seule ligne) ou non. De plus, rvelices a mis à jour CSSmin pour Piwigo 2.4 (pour d'excellentes raison, pas juste pour se faire plaisir ;-)

* en Piwigo 2.3, on utilisait CssMin 2.0.1.0064, non minifié
* en Piwigo 2.4, on utilise CssMin 3.0.1, minifié

Pouvez vous remplacer include/cssmin.class.php par http://code.google.com/p/cssmin/downloa … v3.0.1.php ? (en gardant bien le nom "cssmin.class.php")

Je cherche si CssMin 3.0.1 requiert une version spécifiquement récente de PHP

Apparemment il y a une solution :

post:187990