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
La question "de fond" étant quand même : peut-on séparer fusion et minification? après c'est "facile" de rendre une étape conditionnelle.
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é ;-)
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 ?
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.
rvelices a écrit:
j'ai quand meme l'impression que free veut vraiment se debarasser de ce service gratuit ...
Amen
Bon, c'est sûr qu'il y a encore beaucoup d'utilisateurs Piwigo hébergés sur Free.fr mais qu'ils représentent aussi la plus grosse partie des cas de problèmes d'installation et de migration.
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 ...
Oui, l'astuce du $conf['template_combine_files'] = false permet de ne pas du tout utiliser CssMin.
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...
Je ne sais pas s'il y a une limite de taille des fichiers php chez Free. En tous cas, je n'en ai jamais entendu parler sur le NG de support.
Je teste l'astuce donnée ici : http://fr.piwigo.org/forum/viewtopic.ph … 90#p187990 pour voir.
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 !
Test de mise à jour manuelle chez Free (Php 5.1) avec le nouveau cssmin.class.php :
Parse error: syntax error, unexpected '"', expecting ')' in /mnt/100/sdb/8/d/test/piwigo234/include/cssmin.class.php on line 3642
:-(
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
Oups ! Je n'avais pas tout lu. Si çà fonctionne, tant mieux.
Apparemment il y a une solution :
post:187990