Openid en Plugin ...
Merci pour ton analyse.

Sylvain, tu passes un coup de téléphone à qui tu sais...
On voudrait bien son avis avisé.
Merci.
8-)

rvelices a écrit:

rub a écrit:

rvelices, je vais bientôt commiter sur la partie picture pour le diaporama.
Si tu peux, évites de modifier les tpl liées à picture.php ;-) Merci!

Ha ha ...  je voulais commencer par picture (sans blague) ... Mais je vais commencer par autre chose ...

J'ai bien fait d'en parler...

rvelices a écrit:

J'ai regarde vite jTpl. Il est effectivement plus simple que smarty avec moins de code, mais je trouve que smarty est plus souple en terme d'utilisation (et jTpl n'a pas l'equivalent de "assign_block_vars" par exemple alors que "append" en smarty fait exactement ca).
En plus il me semble que Smarty est bien plus utilise avec plus de developeurs que sur jTpl...

Donc personellement je suis plutot pour Smarty que pour jTpl.

Concernant le assign_block_vars, jTpl permet de définir comme variables es tableaux, associatifs ou non.

Ex emple de Pierrick pour la liste des liens dans PEM:

PHP a écrit:

while ($row = $db->fetch_array($result))
{
  array_push(
    $tpl_links,
    array(
      'name' => $row['name'],
      'url' => $row['url'],
      'description' => $row['description'],
      )
    );
}
$tpl->assign('links', $tpl_links);

jTpl a écrit:

<ul>
  {foreach $links as $link}
  <li><strong><a href="{$link[url]}">{$link[name]}</a></strong>: {$link[description]}</li>
  {/foreach}
</ul>

En fait, jTpl supporte les fonctions PHP, ce qui évite d'en développer d'autres.

Autres exemples pris dans le code de PEM:
- insertion conditionnelle
{if isset($revisions)}
{/if}
- test valeur
{if count($links) > 0}
{else}
{/if}


Concernant "append", comme jTpl ne gère pas les blocs nommés, il "suffit" de faire un array_push quand on le souhaite et de faire "assign" avant l'affichage/la compilation.

Voilà.
J'arrête là mon argumentaire, et voyons voir ce que donne Smarty :-)

rvelices a écrit:

bon voila j'ai appuye commit ... j'espere ne pas avoir fait trop de bobos ... J'ai modifie les templates et le php correspondant pour admin.tpl et admin/plugins.tpl
la revision est ici: http://svn.gna.org/viewcvs/phpwebgaller … p;view=rev

Je vais regarder ca en detail et observer les méthodes que tu utilises.
Je vais essayer de commiter picture/diaporama assez vite, j'ai juste un petit truc à finir mais je ne sais pas quand je pourrais le faire.

J'ai des petites questions:
  o l'ajout du TAG_INPUT_ENABLED est temporaire, le temps de la migration car page_header n'a pas été migré?
  o je n'ai pas compris pourquoi tu as changé le

$template->concat_var_from_handle('ADMIN_CONTENT', 'check_integrity');

par

$template->concat_var('ADMIN_CONTENT', $template->parse('check_integrity', true) );

, c'est pourtant la méthode utilisée partout?
  o finalement, tu détectes si un tpl est smary par contenu et non pas son extension?
  o certains fonctions de présentations (comme le changement de couleur) à chaque ligne d'un tableau ont été déplacés dans le tpl, ca serait bien de faire en même temps un mini doc sur le wiki avec les nouvelles possibilités, les trucs à faire, etc... ca nous permettra ensuite de d'aider à migrer plus rapidement

rvelices a écrit:

si vous etes d'accord, je vais continuer doucement a migrer les tpl ...

Pas de soucis.

Pour savoir où on en est, on ne pourrais mas tagger les tpls non migrés pour savoir d'un coup de grep ce qu'il reste à faire?
Par exemple avec:

<!-- not smarty compatible -->

PS: d'après tes commentaires, sed + grep sera notre grand ami de la migration, je vais m'installer un shell unix sous vista ;-)

[edit]c'est grep pas sed... [/edit]

Dernière modification par rub (2008-02-27 09:22:59)

rvelices a écrit:

rub a écrit:

o finalement, tu détectes si un tpl est smary par contenu et non pas son extension?

je me suis contente d'un preg_match sur le contenu (peu performant)

J'ai taggé dans mon dernier commit, les tpl à migrer.
Il faudrait peut-être faire le preg_match sur le tag? Car je pense qu'il ne fonctionne pas sur les tpl très simple comme plugin.tpl par exemple.

rvelices a écrit:

rub a écrit:

Il faudrait peut-être faire le preg_match sur le tag? Car je pense qu'il ne fonctionne pas sur les tpl très simple comme plugin.tpl par exemple.

Bizarre ... chez moi ca marche (j'ai essaye le event tracer) ....

C'etait juste une supputation sans vérification dans le code.

rvelices a écrit:

Sinon j'ai commite un paquet des modifs... J'essayerai de faire maintenant picture et tout les templates associes (slideshow, picture_content, picture_nav_buttons ...)

J'ai vu ca, tu as bien dépoté!
Si tu n'avances pas aussi vite pour le reste, je testerai bien smarty sur la migration de la NBM, c13y et mail!

Merci pour la correction de </div> dans picture.

rub a écrit:

Si tu n'avances pas aussi vite pour le reste, je testerai bien smarty sur la migration de la NBM, c13y et mail!

Rub, je t'en prie - des que tu peux - ils sont a toi ...