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

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

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!

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

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

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?

Faut voir lesquelles et comment!

Hum... peut-etre pas toutes, mais la plupart ;-)