plugin jtpl cycle

[laurentj]

avoir un plugin jtpl pour afficher des choses alternativement, dans une boucle

{cycle 'aa','bb','cc'}

{cycle $tableau}

[nuks]

Je m'en charge dès que j'ai fini les testes unitaires pour {counter} :)

[dubphil]

Replying to laurentj:

avoir un plugin jtpl pour afficher des choses alternativement, dans une boucle

{cycle 'aa','bb','cc'}

{cycle $tableau}

Finalement il fonctionnera comme cela :

{cycle 'aa,bb,cc'} {cycle $tableau} avec $tableau = array('name'=>'myname','values=>'aa,bb,cc'[, 'reset'=>'[true/false]']);

[laurentj]

Quelques petits problèmes dans le fichier :

• problèmes d'indentation sur certaines lignes
• non respect du "coding style" de jelix. Celui du plugin orignal dans smarty était le bon.
• jException n'accepte pas en paramètre un message, mais une clé de locale pour jLocale.

À part ça, je n'aime pas trop l'api. C'est plutôt compliqué à taper... Faudrait penser à un truc plus simple.

[laurentj]

J'ai pensé à une solution, qui repose sur 3 plugins différents :

    {cycle_init 'aa,bb,cc', 'nom'}

Créer un cycle. Premier argument : une chaîne avec des virgules pour séparer les valeurs, ou tout simplement un tableau php contenant les valeurs. Deuxième argument (facultatif), le nom du cycle.

    {cycle 'nom'}

Affiche la valeur cyclique du cycle indiqué. Le nom du cycle étant facultatif. Ce serait bien aussi de pouvoir passer directement un tableau php contenant les valeurs, ce qui évite de faire un cycle_init pour les cycles non nommé.

    {cycle_reset 'nom'}

Pour refaire une remise à zéro du cycle. nom facultatif.

Qu'en penses-tu ?

[bballizlife]

La solution proposée par laurent me paraît simple à utiliser. Je vote pour.

[nuks]

laurentj: Le mieux serait de garder une même structure pour tous les plugins. Pour le miens, tu m'avais proposé {counter_init 'name', 'type', 'start', 'incr'} {counter 'name'} {counter_reset 'name'} {counter_reset_all}

cf: ​http://dev.jelix.org/ticket/375 Donc en gros, le nom avant le cycle et l'ajoutd 'une fonction ycle_reset_all serait un plus ;)

[nuks]

En fait j'avais aussi ajouter un boolean (parametre facultatif évidement) sur {counter 'name'} pour permettre de ne pas afficher le compteur mais de l'augmenter quand même =) Ca donnait ceci:

{counter 'name', bool}

[dubphil]

mince j'avais pas vu ta dernière note avant d'uploader les fichiers je refais ça...

[dubphil]

voilà j'ai changé l'ordre des paramètres pour cycle_init.

[dubphil]

pour l'instant l'option du cycle par default ne marche pas je regarde ça, j'ai corrigé une erreur dans cycle_init

[laurentj]

jtpl_function_common_cycle_init : pour juste tester une virgule, vaut mieux utiliser strpos (et tester la valeur de retour avec false) plutôt que preg_match, c'est plus rapide.

jtpl_function_common_cycle :

      $cycle_array = $cycle_vars['values'];
      $retval = $cycle_array[$cycle_vars['index']];
      if($cycle_vars['index'] >= count($cycle_array) -1){
          $cycle_vars['index'] = 0;
      } else {
          $cycle_vars['index']++;
      }

Pour si peu, je ne suis pas sûr que passer par une variable cycle_array soit utile ici.

Par contre, pour

$tpl->_privateVars['cycle'][$cycle_name]['index'];

pour améliorer la lisibilité, ce serait pas mal de passer par un alias ($index = & $tpl->...).

Pour le reste, c'est ok :-)

[nuks]

A ce propos laurentj, je me pose une question depuis longtemps. Un foreach avec une référence est-il plus rapide qu'un foreach sans référence ? Pourquoi utiliser un foreach sans référence si on ne touche pas à la variable consernée ?

Désolé pour le hs :)

[laurentj]

sans réference est peut être plus rapide en effet, vu qu'il n'y a pas copie de la variable.

[dubphil]

initialisation du cycle

[dubphil]

cycle

[dubphil]

reset du cycle

[dubphil]

pour la localisation des messages d'erreur

[dubphil]

voilà ça devrait être à peu près bon, j'ai aussi corrigé des problèmes de majuscules intempestives dans les messages d'erreur...

[laurentj]

function jtpl_function_common_cycle_init($tpl, $name='', $values='') {
    if($name == ''){
        throw new jException("jelix~errors.tplplugin.block.bad.argument.number", array('cycle_init','1',''));
        return;
    } else {	

Si le paramètre $name est obligatoire, pas besoin de lui donner une valeur par défaut ;-) De plus, après avoir générer une exception, inutile de faire un return (et de faire un else), l'exception arrêtant l'exécution de la fonction.

Il traîne aussi des problèmes d'espaces en fin de ligne, des problèmes d'indentation.

J'ai quand même intégré tes fichiers dans le trunk, et fait moi même les petites corrections. Merci pour le patch :-)

svn 842.

[laurentj]

documentation done: ​http://jelix.org/articles/en/manual-1.1/ref-plugins/tplplugins/functions#common-cycle