#76 2008-02-29 13:51:02

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

rvelices a écrit:

rub a écrit:

Après picture, tu vas faire quoi?
Moi, je reste sur mes choix, P@t veut faire intro et toi?

je pense continuer avec les templates de la page index, mais je vais aller plus dooucement :-)

Tu avais pris un bon rythme pourtant... ;-)

Hors ligne

#77 2008-02-29 13:52:50

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

rvelices a écrit:

ddtddt a écrit:

S'il y en a un "facile" je veux bien regarder !

admin/group_perm et/ou admin/group_list ?

Et ensuite, je pense que si tu veux faire un bien formateur, c'est admin/user_list.php ;-)

Hors ligne

#78 2008-02-29 20:17:09

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

P@t a écrit:

rub a écrit:

P@t a écrit:

rub, pour le site démo, voila le premier plugin smarty ;-)
LocalFiles Editor 1.8.beta
Il faudra que je pense à le commiter...

Avec mon commit 2226, ta version ne doit plus être bonne à cause des tabsheets!

C'est bon, j'ai mis à jour l'archive...
J'essairai de faire un code propre et de commiter tout ca.

C'est mis...
Ca fonctionne sous xampp chez moi, par contre sur la démo, il y a un soucis! /////

Hors ligne

#79 2008-02-29 20:28:24

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

rub a écrit:

C'est mis...
Ca fonctionne sous xampp chez moi, par contre sur la démo, il y a un soucis! /////

Arf... le soucis de la fonction realpath qui ne renvoi plus de / à la fin en php 5.2.5
L'archive est corrigée.

J'en reviens donc au problème cité plus haut...
Avec smarty, on ne peut pas utiliser un chemin relatif pour un set_filename.
Du coup, on est obligé de faire ca:

$template->set_filenames(array('plugin_admin_content' => rtrim(realpath(LOCALEDIT_PATH), '\\/') . '/admin.tpl'));

C'est pas le top, non?

Dernière modification par P@t (2008-02-29 20:35:54)


P@t

Hors ligne

#80 2008-02-29 20:39:47

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

P@t a écrit:

J'en reviens donc au problème cité plus haut...
Avec smarty, on ne peut pas utiliser un chemin relatif pour un set_filename.
Du coup, on est obligé de faire ca:

$template->set_filenames(array('plugin_admin_content' => rtrim(realpath(LOCALEDIT_PATH), '\\/') . '/admin.tpl'));

C'est pas le top, non?

C'est pas plus simple de faire dirname(__FILE__)/admin.tpl pour obtenir un chemin absolu ?
Sinon effectivement - le chemin relatif est maintenant toujours par rapport au repertoire theme.

Hors ligne

#81 2008-02-29 20:50:06

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

rvelices a écrit:

C'est pas plus simple de faire dirname(__FILE__)/admin.tpl pour obtenir un chemin absolu ?
Sinon effectivement - le chemin relatif est maintenant toujours par rapport au repertoire theme.

Oui, en effet, ca marche aussi...
Mais ca me chagrine un peu de ne pas pouvoir utiliser un chemin relatif...

Pourquoi ne pas rajouter dans la fonction set_filenames un truc du genre:

if (preg_match('/^[a-z_][^:]/i', $filename) )
{
   $filename = $this->root.'/'.$filename;
}

Comme pour l'ancien template... ;-)

Dernière modification par P@t (2008-02-29 20:54:06)


P@t

Hors ligne

#82 2008-02-29 20:58:04

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

P@t a écrit:

rub a écrit:

C'est mis...
Ca fonctionne sous xampp chez moi, par contre sur la démo, il y a un soucis! /////

Arf... le soucis de la fonction realpath qui ne renvoi plus de / à la fin en php 5.2.5
L'archive est corrigée.

Done. C'est bon.

Hors ligne

#83 2008-02-29 21:22:12

ddtddt
Équipe Piwigo
Quetigny (21) - France
2007-07-27
16915

Re: Evolution des templates

rvelices a écrit:

ddtddt a écrit:

S'il y en a un "facile" je veux bien regarder !

admin/group_perm et/ou admin/group_list ?

je regarde c'est 2 là


Vous aimez Piwigo alors n'hésitez pas à participer avec nous, plus d'infos sur la page "Contribuer à Piwigo". Si vous n'avez pas beaucoup de temps et que vous souhaitez nous soutenir vous pouvez aussi le faire par un don.

Hors ligne

#84 2008-02-29 21:53:18

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

P@t a écrit:

Pourquoi ne pas rajouter dans la fonction set_filenames un truc du genre:

if (preg_match('/^[a-z_][^:]/i', $filename) )
{
   $filename = $this->root.'/'.$filename;
}

Comme pour l'ancien template... ;-)

on peut le faire, mais dans l'ancien template assign( './admin.tpl' ) et assign( 'admin.tpl' ) ont 2 resultats differents: le premier est par rapport au script php d'origine et le 2eme par rapport au repertoire template.

Hors ligne

#85 2008-02-29 22:09:22

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

rvelices a écrit:

on peut le faire, mais dans l'ancien template assign( './admin.tpl' ) et assign( 'admin.tpl' ) ont 2 resultats differents: le premier est par rapport au script php d'origine et le 2eme par rapport au repertoire template.

Exactement ;-)
Et cela me parait etre une excellente solution!
C'est possible docteur?


P@t

Hors ligne

#86 2008-03-01 14:47:11

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

Par rapport au commit 2231, on ne devrait pas "ne pas toucher" à la partie smarty pour faciliter l'intégration de leurs nouvelles versions?


Et concernant le TAG_INPUT_ENABLED (c'est vrai que le page_header est mieux placé maintenant dans l'admin), pour éviter de la déclarer 2 dois (dans admin.php et page_header.php), on ne pourrait pas l'inclure dans la class template?
Soit avec un assign dans le constructeur comme pour themeconf?
Soit par l'ajout d'une fonction comme pour @translate?
Soit autrement?

Hors ligne

#87 2008-03-01 15:13:40

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12672

Re: Evolution des templates

(Bonjour à tous, longue absence, désolé, tout ça, merci à Mathias de m'avoir notifié de la discussion)

J'ai le sentiment qu'au point où nous en sommes, la discussion n'est plus "doit on passer à un moteur de template plus puissant ?" ni même "quel moteur de template plus puissant choisit-on ?" mais bien "en route vers Smarty".

Comme le dit mathiasm, j'ai passé PEM sur jTpl mi-2006 et c'est largement mieux que ce qu'il y avait avant. J'aurais pu passer au même moteur que PWG, mais j'avais envie de tester autre chose. J'aime bien jTpl, j'aime le confort qu'il apporte par rapport au moteur utilisé dans PWG. Comme dit rvelices, si on est d'accord avec jTpl, on est d'accord avec Smarty (et vice-versa). En effet, ils sont tous les 2 basés sur le même principe de fonctionnement, peut-être que Smarty sépare encore davantage la logique de présentation en limitant le nombre de fonctions accessibles. Bref, c'est plutôt un détail pour moi.

Smarty a l'énorme avantage d'être connu et donc de potentiellement attirer des développeurs (de plugin par exemple). Smarty a l'inconvénient d'être devenu une petite usine à gaz. C'est ce point qui pourrait me contrarier en terme de performances.

rvelices, ce qui serait intéressant, ce serait d'avoir une idée de la perte de performances entre une 1.7.1 et une Butterfly sur picture.php et index.php, les 2 pages principales.

En tout cas, on va dans le bon sens. Si cette discussion revient aujourd'hui, c'est qu'on ressent plus que jamais les limitations du moteur de template actuel (devrais-je déjà dire "l'ancien" ?). Merci rvelices d'être à nouveau "moteur" sur ce projet :-)


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#88 2008-03-01 15:33:34

vimages
Membre
2004-03-27
2429

Re: Evolution des templates

salut zOrglub !!
ça fait plaisir d'avoir des nouvelles de temps en temps..

quoi de neuf ?
tu as vu si tes petits travaillent bien...

à bientôt.
éric.

Hors ligne

#89 2008-03-01 15:52:33

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

rub a écrit:

Par rapport au commit 2231, on ne devrait pas "ne pas toucher" à la partie smarty pour faciliter l'intégration de leurs nouvelles versions?

Oui, oui - mais comme on fait parfois les choses a notre sauce differente de smarty - je voulais juste avoir les infos de debugging sur les perfs... Mais t'as parfaitement raison sur le fond...

rub a écrit:

Et concernant le TAG_INPUT_ENABLED (c'est vrai que le page_header est mieux placé maintenant dans l'admin), pour éviter de la déclarer 2 dois (dans admin.php et page_header.php), on ne pourrait pas l'inclure dans la class template?
Soit avec un assign dans le constructeur comme pour themeconf?
Soit par l'ajout d'une fonction comme pour @translate?
Soit autrement?

Peut etre dans la classe template comme ROOT_URL ?

Hors ligne

#90 2008-03-01 16:00:27

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

z0rglub a écrit:

(Bonjour à tous, longue absence, désolé, tout ça, merci à Mathias de m'avoir notifié de la discussion)

rvelices, ce qui serait intéressant, ce serait d'avoir une idée de la perte de performances entre une 1.7.1 et une Butterfly sur picture.php et index.php, les 2 pages principales.

Salut aussi a toi apres longtemps ... :-)
En fait je ne m'attend pas a voir une perte de perf (a l'exception de la premiere fois que les templates sont compiles)... mais on verra bien le moment venu...

Hors ligne

Pied de page des forums

Propulsé par FluxBB

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