Piwigo.org

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

Faut voir lesquelles et comment!

rub · 2008-04-29 23:29:57

Code:

Warning: include(./include/template.php) [function.include]: failed to open stream: No such file or directory in D:\Dev\pwg_photograph\upgrade_feed.php on line 29

Warning: include() [function.include]: Failed opening './include/template.php' for inclusion (include_path='.;C:\xampp\php\pear\') in D:\Dev\pwg_photograph\upgrade_feed.php on line 29

0 upgrades to apply

lors de l'upgrade_feed.php

P@t · 2008-05-21 16:34:15

Après une discussion coté privé, mathias et vincent pensent (comme moi) qu'il faudrait ne pas tester les variables non-assignées dans les templates.
Je trouve que ca alourdit pas mal le script, on perd en lisibilité.

Est-ce qu'on pourrait désactiver cette option du debugging de smarty?

rub · 2008-05-21 17:34:14

P@t a écrit:
Après une discussion coté privé, mathias et vincent pensent (comme moi) qu'il faudrait ne pas tester les variables non-assignées dans les templates.
Je trouve que ca alourdit pas mal le script, on perd en lisibilité.

Quelles discussion en privé?
Des exemples ou c'est lourd?

Pourquoi vouloir interdire? C'est assez utile dans certains cas!

P@t · 2008-05-21 17:58:43

rub a écrit:
Quelles discussion en privé?

La discussion sur PWG Stuffs et le passage en class (CSS) du block content dans les templates.
Mathias et Vincent ne conprenaient pas l'utilité du {if !empty($PLUGIN_INDEX_CONTENT_BEGIN)}{$PLUGIN_INDEX_CONTENT_BEGIN}{/if} par exemple

rub a écrit:
Des exemples ou c'est lourd?

Ben déjà sur le index.tpl, il y en a pas mal...
Quand je diss "lourd", c'est visuellement quand on re garde le fichier template.
Et du coup, ca peut géner les développeurs de template.
Mais bon, c'est mon avis perso à moi ;-)

rub a écrit:
Pourquoi vouloir interdire? C'est assez utile dans certains cas!

Ben, quelle utilité dans le cas ci-dessus???

rub · 2008-05-21 18:27:38

En parlant de non assignés, je penses que tu parlais du isset.
C'est différent mais c'est presque pareil...

Effectivement pour

Code:

{if !empty($PLUGIN_INDEX_ACTIONS)}{$PLUGIN_INDEX_ACTIONS}{/if}

ce n'est pas utile mais pour

Code:

      {if isset($U_MODE_FLAT) }
      <li><a href="{$U_MODE_FLAT}" title="{'mode_flat_hint'|@translate}" rel="nofollow"><img src="{$ROOT_URL}{$themeconf.icon_dir}/flat.png" class="button" alt="{'mode_flat_hint'|@translate}"></a></li>
      {/if}

C'est utilise.

C'est pourquoi, je penses qu'il ne faut pas désactiver pas empty ou isset...

Juste à nous de mettre dans les normes qu'un test qui ne sert à afficher que la variable sans ajout n'est pas à faire!

Dernière modification par rub (2008-05-22 07:05:45)

P@t · 2008-05-21 19:11:47

rub a écrit:
C'est pourquoi, je penses qu'il ne faut désactiver empty ou isset...

Oula non, je ne veux rien désactiver du tout!
J'aimerai juste mettre à off l'option de smarty qui génére une erreur si une variable n'est pas assignée au template (si $conf['debug_template'] = true)

Je suis bien d'ccord que c'est utile dans beaucoup de cas...
Mais je serais pour enlever les isset ou empty quand ce n'est pas nécessaire...

Cf message de radu:

rvelices a écrit:
P@t a écrit:
rvelices a écrit:
Sinon c'est reparti pour les erreurs de user_list ... J'en ai un paquet. Un exemple est ADVISER_YES qui n'est pas sette dans le template ... (Notice: Undefined index: ADVISER_YES in D:\pwg\bsf\_data\templates_c\%%D1^D1A^D1A29EED%%user_list.tpl.php)
Heu... c'est normal, non?
Est-ce qu'il faut que toutes les variables du templates soient attribuées???
Ce n'était pas le cas avec l'ancien template...
En Smarty on peut le mettre a On, a Off ou les laisser tel que defini en php.ini. J'ai choisi le dernier cas car ca m'a beaucoup aide dans la migration d'avoir des erreurs quand les variables ne sont pas definies...
Personellement je suis pour le laisser comme caet s'assurer que toutes les verifs sont faites. Si vous voulez le mettre a Off, pas de probleme mais ca sera ainsi dans tout les cas (y compris conf[debug_template]=true)

EDIT: je rectifie - rub a raison les warnings sont la seulement si debug_template = true ...

Dernière modification par P@t (2008-05-21 19:12:18)

mathiasm · 2008-05-21 23:19:59

P@t a écrit:
J'aimerai juste mettre à off l'option de smarty qui génére une erreur si une variable n'est pas assignée au template (si $conf['debug_template'] = true)

rvelices a écrit:
EDIT: je rectifie - rub a raison les warnings sont la seulement si debug_template = true ...

Donc on est d'accord: on vire les if qui ne sont pas nécessaire et on laisse l'option pour le debugging; et on met dans le wiki comment désactiver l'option si c'est trop verbeux.
Non ?

P@t · 2008-05-21 23:23:15

mathiasm a écrit:
P@t a écrit:
J'aimerai juste mettre à off l'option de smarty qui génére une erreur si une variable n'est pas assignée au template (si $conf['debug_template'] = true)
rvelices a écrit:
EDIT: je rectifie - rub a raison les warnings sont la seulement si debug_template = true ...
Donc on est d'accord: on vire les if qui ne sont pas nécessaire et on laisse l'option pour le debugging; et on met dans le wiki comment désactiver l'option si c'est trop verbeux.
Non ?

Ben non... car si on laisse l'option, on a plein de messages d'avertissement dans la page elle-meme
Donc ca va etre pratique pour bosser dessus!

Dernière modification par P@t (2008-05-21 23:23:57)

rvelices · 2008-05-22 01:48:54

Ayant migre une cinquantaine des templates je peux dire que ces warnings ont ete bien utiles, donc mon avis perso est qu'il faut les laisser ca peut vraiment aider pour le debug. (C'est le meme systeme qu'en php)

Sinon si c'est plus joli on peut ecrire aussi qq chose comme

Code:

{$PLUGIN_INDEX_ACTIONS|@default}
a la place de
{if !empty($PLUGIN_INDEX_ACTIONS)}{$PLUGIN_INDEX_ACTIONS}{/if}

rub · 2008-05-22 07:08:14

houlala, j'avais oublié des négations dans mes posts et j'avais mal compris que c'était pour le debug...

rvelices a écrit:
Ayant migre une cinquantaine des templates je peux dire que ces warnings ont ete bien utiles, donc mon avis perso est qu'il faut les laisser ca peut vraiment aider pour le debug. (C'est le meme systeme qu'en php)

Sinon si c'est plus joli on peut ecrire aussi qq chose comme
Code:
{$PLUGIN_INDEX_ACTIONS|@default}
a la place de
{if !empty($PLUGIN_INDEX_ACTIONS)}{$PLUGIN_INDEX_ACTIONS}{/if}

Je suis de ton avis:
+1
+1

VDigital · 2008-05-22 07:37:53

{$PLUGIN_INDEX_ACTIONS|@default}

Merci rvelices !!!
8-)

P@t · 2008-05-22 11:39:55

Et ben voila! Je vote pour!
Comme ca, tout le monde est content...
Merci radu ;-)

Piwigo.org

Annonce

#211 2008-03-27 19:01:01

Re: Evolution des templates

#212 2008-04-29 23:29:57

Re: Evolution des templates

Code:

#213 2008-05-21 16:34:15

Re: Evolution des templates

#214 2008-05-21 17:34:14

Re: Evolution des templates

P@t a écrit:

#215 2008-05-21 17:58:43

Re: Evolution des templates

rub a écrit:

rub a écrit:

rub a écrit:

#216 2008-05-21 18:27:38

Re: Evolution des templates

Code:

Code:

#217 2008-05-21 19:11:47

Re: Evolution des templates

rub a écrit:

rvelices a écrit:

P@t a écrit:

rvelices a écrit:

#218 2008-05-21 23:19:59

Re: Evolution des templates

P@t a écrit:

rvelices a écrit:

#219 2008-05-21 23:23:15

Re: Evolution des templates

mathiasm a écrit:

P@t a écrit:

rvelices a écrit:

#220 2008-05-22 01:48:54

Re: Evolution des templates

Code:

#221 2008-05-22 07:08:14

Re: Evolution des templates

rvelices a écrit:

Code:

#222 2008-05-22 07:37:53

Re: Evolution des templates

#223 2008-05-22 11:39:55

Re: Evolution des templates

Pied de page des forums