🌍
Français
Et ben voila! Je vote pour!
Comme ca, tout le monde est content...
Merci radu ;-)
{$PLUGIN_INDEX_ACTIONS|@default}
Merci rvelices !!!
8-)
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 commeCode:
{$PLUGIN_INDEX_ACTIONS|@default} a la place de {if !empty($PLUGIN_INDEX_ACTIONS)}{$PLUGIN_INDEX_ACTIONS}{/if}
Je suis de ton avis:
+1
+1
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
{$PLUGIN_INDEX_ACTIONS|@default}
a la place de
{if !empty($PLUGIN_INDEX_ACTIONS)}{$PLUGIN_INDEX_ACTIONS}{/if}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!
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 ?
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 ...
En parlant de non assignés, je penses que tu parlais du isset.
C'est différent mais c'est presque pareil...
Effectivement pour
{if !empty($PLUGIN_INDEX_ACTIONS)}{$PLUGIN_INDEX_ACTIONS}{/if}ce n'est pas utile mais pour
{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!
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???
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!
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?
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
Faut voir lesquelles et comment!
P@t a écrit:
Qu'est-ce qu'on fait du fichier functions_html.inc.php
Je propose qu'on passe toutes ces fonctions en fonction de template
Qu'en pensez vous?
Pour etre honete je ne vois pas l'interet de le changer en utilisant les fonctions de Smarty. C'est la meme chose: une fonction php appelee du php (aujourd'hui) versus une fonction php appelee du template...
Hum... peut-etre pas toutes, mais la plupart ;-)