Piwigo.org

P@t · 2008-03-01 16:01:50

Merci radu pour ton dernier commit ;-)
Le set_filenames re-fonctionne avec des chemins relatifs...
J'en avais pas dormi de la nuit! (non, j'exagère un poil...)

Bon, sinon, j'ai livré LocalFiles Editor avec pas mal de changements...
J'ai utilisé par exemple le {html_options}
C'est vraiment le top ce moteur de template!
Sinon, l'affichage des fichiers default est amélioré.
Bref, je pars en HS la... rub, tu peux mettre en place sur la démo?

EDIT: a ben non, je déconne, ca n'a rien changé ;-)
Tu pourras nous faire ca quand meme?

Dernière modification par P@t (2008-03-01 16:02:41)

rvelices · 2008-03-01 17:19:10

P@t a écrit:
Merci radu pour ton dernier commit ;-)
Le set_filenames re-fonctionne avec des chemins relatifs...
J'en avais pas dormi de la nuit! (non, j'exagère un poil...)

EDIT: a ben non, je déconne, ca n'a rien changé ;-)
Tu pourras nous faire ca quand meme?

C'est vraiment embetant si on reste comme on est ? J'ai reflechi et je peux faire la modif pour avoir ce vieux fonctionnement (bizarre d'apres moi pour avoir 2 comportements differents pour ./admin.tpl et admin.tpl) mais ca ne marchera pas pour l'utilisation a l'interieur de smarty du tag {include}. Et en fait j'aimerai pousser un peu pour l'utilisation des include (quand c'est possible).

P@t · 2008-03-01 17:57:49

rvelices a écrit:
C'est vraiment embetant si on reste comme on est ? J'ai reflechi et je peux faire la modif pour avoir ce vieux fonctionnement (bizarre d'apres moi pour avoir 2 comportements differents pour ./admin.tpl et admin.tpl) mais ca ne marchera pas pour l'utilisation a l'interieur de smarty du tag {include}. Et en fait j'aimerai pousser un peu pour l'utilisation des include (quand c'est possible).

C'est toi le chef!
Moi j'aimais bien ce fonctionnement, mais on s'adaptera ;-)

PS: pour info, je suis en train de migrer Plugins Manager pour la 1.8.
Une fois fait, il faudra que tu nous donnes ton avis, avant de commiter ca.

PS2: d'ailleur, petite question: les données récupérées sur PEM ne sont pas en utf8. Il y a un moyen pratique de convertir ca?

Dernière modification par P@t (2008-03-01 18:06:06)

rub · 2008-03-01 18:54:23

P@t a écrit:
rvelices a écrit:
C'est vraiment embetant si on reste comme on est ? J'ai reflechi et je peux faire la modif pour avoir ce vieux fonctionnement (bizarre d'apres moi pour avoir 2 comportements differents pour ./admin.tpl et admin.tpl) mais ca ne marchera pas pour l'utilisation a l'interieur de smarty du tag {include}. Et en fait j'aimerai pousser un peu pour l'utilisation des include (quand c'est possible).
C'est toi le chef!
Moi j'aimais bien ce fonctionnement, mais on s'adaptera ;-)

Je suis du même avis que rvelices, c'était un comportement bizarre.

De toute faon dans les plugins, on utilise dirname(__FILE__) pour les php alors pourquoi pas pour les tpl.
Après, il suffit de faire un define pour faire dirname(__FILE__) qu'une seule fois!

rub · 2008-03-01 18:59:37

rvelices a écrit:
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...

Ok, c'est pour le debugging, no soucis!

rvelices a écrit:
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 ?

Bien le assign au niveau du parse, ca permet que tout soit mieux iniatilisé!

rub · 2008-03-01 19:01:06

P@t a écrit:
Bref, je pars en HS la... rub, tu peux mettre en place sur la démo?

Fait!
Sinon, je le fais au moins un jour par jour pour des commits important... donc en ce moment, ca sera tous les jours...

edit: petit soucis au niveau de c13y avec cette version de php => cf message dans intro

Dernière modification par rub (2008-03-01 19:16:29)

P@t · 2008-03-02 00:20:10

P@t a écrit:
PS2: d'ailleur, petite question: les données récupérées sur PEM ne sont pas en utf8. Il y a un moyen pratique de convertir ca?

Je me réponds à moi meme: fonction utf8_encode()
C'était facile ;-)

P@t · 2008-03-02 02:09:53

Voila une version de Plugins Manager compatible 1.8 (et smarty bien sur!)
Si vous avez le temps de jeter un oeil avant que j'intègre ca en standard (rub, vincent, radu?)
Merci ;-)

PS: je viens de mettre à jour l'archive, vu qu'entre temps, le header est passé en smarty...
Ca va à une vitesse! ;-)

Dernière modification par P@t (2008-03-02 02:14:58)

rub · 2008-03-02 09:43:55

P@t a écrit:
Voila une version de Plugins Manager compatible 1.8 (et smarty bien sur!)
Si vous avez le temps de jeter un oeil avant que j'intègre ca en standard (rub, vincent, radu?)
Merci ;-)

Integre ca en standard tout de suite, c'est plus simple pour la synchro et pour les diff avec svn.
De plus, je pense que tu ne vas l'intégrer en plugin mais directement dans le core de pwg?

P@t a écrit:
PS: je viens de mettre à jour l'archive, vu qu'entre temps, le header est passé en smarty...
Ca va à une vitesse! ;-)

Et encore, rvelices a dit qu'il allait continuer moins vite...

rub · 2008-03-02 09:46:24

Sinon, j'avais posé une petite question dans un commit pour les footer_elements.
J'ai eu ma réponse avec les headers!
Il suffit de faire des

Code:

$template->append('footer_elements', ...);

Attention au s qui n'était pas présent dans les version 1.7!

rub · 2008-03-02 09:52:44

J'ai encore très peu testé smarty mais ce qui est sur, c'est que ca simplifie le code php. Notamment aux niveaux des tableaux et des listes.
Et que l'on sépare bien les données à afficher et la façon de les afficher!

plg · 2008-03-02 16:57:19

rub a écrit:
J'ai encore très peu testé smarty mais ce qui est sur, c'est que ca simplifie le code php. Notamment aux niveaux des tableaux et des listes.
Et que l'on sépare bien les données à afficher et la façon de les afficher!

Alors c'est une étape significative de plus dans l'évolution technique de PWG. C'est une bonne chose de séparer le "métier" et la "présentation" (même si au début, le PHP générait directement le HTML, mais c'était il y a très très longtemps, il faudra que je fasse une génèse de tout ça un jour)

rub · 2008-03-02 18:35:09

Voila, je viens de faire ma 1ere migration de tpl avec c13y.
J'ai fait une migration simple en gardant le code identique mais en supprimant le code php générant du HTML (nl2br et ajout particulier de br).
Je vais faire une nouvelle version, ou, je pense, tout ce que je fais pour la présentation ne sera plus dans le php mais dans le tpl.
En gros, il s'agit de la visibilité des boutons et de l'affichage de tel ou tel libellé. (par l'usage simple de {assign})
A vous de me dire, après si j'en fais trop ou non!

P@t · 2008-03-02 18:52:15

Bon, ben moi je viens de faire mon premier gros commit, et en plus, tout en smarty ;-)
Alors n'hésitez pas à aller jeter un oeil et à corriger!

PS: les spécialistes de la langue de shakespeare sont invités à corriger le fichier anglais (que j'ai écris en franglais)

PS2: arg...j'ai oublié de rester à 79 colonnes dans le fichier functions_plugins.inc.php...
Je corrigearai ca ;-)

Dernière modification par P@t (2008-03-02 19:23:27)

rvelices · 2008-03-02 22:11:57

P@t a écrit:
Bon, ben moi je viens de faire mon premier gros commit, et en plus, tout en smarty ;-)
Alors n'hésitez pas à aller jeter un oeil et à corriger!

He he bienvenue. Si je peut me permettre 2 commentaires:
- la fonction check_server_plugins -> t'aurais pu passer le fs_plugins comme variable en reference a la fonction a la place d'utiliser une variable globale dans la fonction ?
- je pense que les modifs css ne devrait pas aller en template-common mais plutot en admin.css ?

Sinon yes !!! super boulot....

Dernière modification par rvelices (2008-03-02 22:12:37)

Piwigo.org

Annonce

#91 2008-03-01 16:01:50

Re: Evolution des templates

#92 2008-03-01 17:19:10

Re: Evolution des templates

P@t a écrit:

#93 2008-03-01 17:57:49

Re: Evolution des templates

rvelices a écrit:

#94 2008-03-01 18:54:23

Re: Evolution des templates

P@t a écrit:

rvelices a écrit:

#95 2008-03-01 18:59:37

Re: Evolution des templates

rvelices a écrit:

rub a écrit:

rvelices a écrit:

rub a écrit:

#96 2008-03-01 19:01:06

Re: Evolution des templates

P@t a écrit:

#97 2008-03-02 00:20:10

Re: Evolution des templates

P@t a écrit:

#98 2008-03-02 02:09:53

Re: Evolution des templates

#99 2008-03-02 09:43:55

Re: Evolution des templates

P@t a écrit:

P@t a écrit:

#100 2008-03-02 09:46:24

Re: Evolution des templates

Code:

#101 2008-03-02 09:52:44

Re: Evolution des templates

#102 2008-03-02 16:57:19

Re: Evolution des templates

rub a écrit:

#103 2008-03-02 18:35:09

Re: Evolution des templates

#104 2008-03-02 18:52:15

Re: Evolution des templates

#105 2008-03-02 22:11:57

Re: Evolution des templates

P@t a écrit:

Pied de page des forums