Mmh, en effet, je vois le problème... En fait, $VALUE n'est pas interprété (car la syntaxe est {VALUE,$MAVAR} et non {$VALUE,MAVAR})... En attendant, pourrais tu éviter de l'utiliser dans un foreach ? J'essaierais de corriger ce bug dans la prochaine version... (Pour l'utiliser dans une boucle, je te recommande les blocs à la place, en attendant...)
C'est noté. Pour le moment j'ai résolu le problème en passant par une autre méthode (relative à mon site, ça ne sert à rien que je la donne).
La prochaine fois je passerai par un bloc (je ne les ai jamais utilisé, mais ça va venir vite du coup ^^).
Merci à toi.
Tout d'abord, félicitation pour le très bon moteur de template!
J'ai un petit problème relié à ce sujet. Je me suis créé un filtre qui converti un timestamp en date lisible. Le filtre fonctionne sur une variable normale, mais quand j'utilise une variable sous la forme {VALUE,VARIABLE['timestamp']|monFiltre} j'obtiens une erreur :
Parse error: parse error, unexpected '}', expecting ',' or ';' in C:\***\***\cache\index.tpl.php on line 21
J'ai fouillé un peu sur le forum. J'ai updaté avec la version 1.5.1 (en supprimant bien sure la cache). Toujours une erreur.
Ah enfin quelqu'un qui a ce cas, pour m'aider à débugger. :D
Pourrais-tu essayer ceci, en remplacant toute la méthode _filter dans Talus_TPL_Compiler :
Code PHP<?php // ... /** * Filtre une var avec un ou plusieurs des filtres de Talus_TPL_Filters. (cf ligne 73) * * @param array $match Capture de la regex * @see Talus_TPL_Filters * @return string */ private function _filters(array $match){ $i = 0; $return = ''; $toPrint = false; $var = "{{$match[1]}}"; $filters = array_reverse(array_filter(explode('|', $match[2]))); /* * Variable à traiter ; si la présence d'un $ est détectée, on ne fait * rien que la mettre en forme entre des {}, sinon on improvise. * * C'est à dire si elle est de la forme QQCH,VAR (foreach, fonctions), * il faut placer le $ juste après la virgule, sinon le placer au début. */ if (strpos($match[1], '$') === false) { $cap = array(); if (preg_match('`^(?:KEY|VALUE|GLOB),`', $match[1], $cap)) { $var = '{' . preg_replace('`^(?:KEY|VALUE|GLOB),`', '$0\$', $match[1]) . '}'; } else { $var = "{\${$match[1]}}"; } // -- On indique bien qu'il s'agit bien d'un affichage du contenu. $toPrint = true; } foreach ($filters as &$filter) { // -- Filtre non déclaré ? if (!method_exists('Talus_TPL_Filters', $filter)){ echo "Talus_TPL_Compiler :: Le filtre \"$filter\" n'existe pas, et sera donc ignoré.\n"; continue; } // -- Ajout du filtre, incrémentation du nombre de ( $return .= 'Talus_TPL_Filters::' . $filter . '('; $i++; } // -- Association de la variable, fermeture des différentes ( ouvertes $return .= $var . str_repeat(')', $i); /* * Si, comme on l'a vu tout à l'heure, il ne s'agit pas d'un retour du * contenu de la variable mais de son affichage, on fait un echo. Sinon, * on laisse la variable telle quelle. */ if ($toPrint){ $return = '<?php echo ' . $return . '; ?>'; } return $return; }
Et dis moi si ca va mieux (pour la prochaine version, en principe) ? Et même mieux, fourni moi le résultat du cache après (dans la zone là) ?
Par contre, note qu'avec cette solution, impossible de passer des array dans les filtres (que ce soit avec des variables ou non). Enfin si ca, déjà, ca marche, alors j'essaierais de faire marcher avec les array. Si ce n'est pas trop lourd après, évidemment...
Dernière édition le 16/06/2009, à 21:52, par Talus
Pingax, je sais ça! Cependant ce code est généré par le moteur de template! Le } est présente dans le code de mon template, mais ne l'est plus dans la page en cache quand je remplace la méthode!
Par contre, note qu'avec cette solution, impossible de passer des array dans les filtres (que ce soit avec des variables ou non). Enfin si ca, déjà, ca marche, alors j'essaierais de faire marcher avec les array. Si ce n'est pas trop lourd après, évidemment...
Pourrais-tu essayer de faire sans array ? {VALUE,QQCH|date}, au lieu de {VALUE,QQCH['cle']|date} ? Et au passage, {VAR|date}.
Faut déjà vérifier que ca marche pour les vars simples, je regarderais plus tard pour les array :p
De toute manière, j'ai transformé le tout avec les blocs et ça marche tempête. Cependant, qu'elle est la manière la plus optimisé pour passer le résultat d'une requête MySQL à un bloc?
Voici un exemple tiré de mes codes, est-ce la façon la plus propre?
Oui, les blocs sont la facon la plus propre de faire si c'est pour parcourir des données (car itération), car tu as accès à d'autres trucs qui sont bien symathique, comme par exemple les stats (combien de tours à faire ? Est-ce le premier ? Est-ce le dernier ? Quel tour en cours ?). Le foreach est mieux dans les autres cas, c'est à dire aprcours d'un array en lui même.
Mais comme la modif marche pour les variables normales, je voudrais aussi que tu testes pour les variables de type {TYPE,VAR}. Par exemple, {VALUE,VAR} ou bien {KEY,VAR} ou encore {GLOB,VAR] (dans le cadre d'une fonction). Après, je pourrais essayer de l'attaquer aux arrays. Tu pourrais donc me dire ce qui se passe dans ce cas ? N'ayant pas de quoi tester sous la main, ca me permet de tester le but de la manip... :)
Dernière édition le 17/06/2009, à 19:54, par Talus
Home
» Talus' Works » Talus' TPL » Rapports de Bugs » [Filtres] Parser les {X,Y|filtre} » Lecture du Sujet » Page 1 
Ce sujet est considéré comme "Résolu".
#1 - Le 11/03/2009, à 13:54
