Annonce

Écrire une réponse

Veuillez écrire votre message et l'envoyer

Cliquez dans la zone sombre de l'image pour envoyer votre message.

Retour

Résumé de la discussion (messages les plus récents en premier)

rvelices
2012-06-26 18:40:37

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

plg
2012-06-26 15:09:09

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.

plg
2012-06-26 15:08:32

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
2012-06-26 15:00:06

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 ?

plg
2012-06-26 14:45:19

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.

plg
2012-06-24 20:40:20

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.

rvelices
2012-06-24 20:23:24

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

Code:

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 ...

plg
2012-06-24 18:32:44

Oui, l'astuce du $conf['template_combine_files'] = false permet de ne pas du tout utiliser CssMin.

Eric
2012-06-24 18:27:05

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...

Eric
2012-06-24 18:09:39

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.

plg
2012-06-24 18:07:05

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 !

Eric
2012-06-24 17:46:36

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

:-(

plg
2012-06-24 14:18:13

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

Eric
2012-06-22 19:04:53

Oups ! Je n'avais pas tout lu. Si çà fonctionne, tant mieux.

Gotcha
2012-06-22 18:59:30

Apparemment il y a une solution :

post:187990

Pied de page des forums

Propulsé par FluxBB

github twitter newsletter Faire un don Piwigo.org © 2002-2024 · Contact