Piwigo Bugtracker

Viewing Issue Advanced Details Jump to Notes ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0000718 [Piwigo] other feature N/A 2007.07.09 16:58 2007.07.10 07:08
Reporter Pat View Status public  
Assigned To rub
Priority normal Resolution fixed Platform
Status closed   OS
Projection none   OS Version
ETA none Fixed in Version 1.7.1 Product Version 1.7.0
  Target Version Product Build
Summary 0000718: Ajout d'une fonction "concat_var_from_handle"
Description Permet à plusieurs plugins d'insérer du code HTML au meme endroit dans un template en concaténant les valeurs.
Steps To Reproduce
Additional Information Code de la fonction à insérer dans template.php:

function concat_var_from_handle($varname, $handle)
    {
      if (!$this->loadfile($handle))
      {
        die("Template->assign_var_from_handle(): Couldn't load template file for handle $handle");
      }

      trigger_action('loc_before_tpl_concat_var_from_handle', 'comonindex', array(&$template));

      // Compile it, with the "no echo statements" option on.
      $_str = "";
      $code = $this->compile($this->uncompiled_code[$handle], true, '_str');

      // evaluate the variable assignment.
      eval($code);
      // assign the value of the generated variable to the given varname.
      $this->concat_var($varname, $_str);

      return true;
    }
Tags No tags attached.
browser any
Database engine and version
PHP version
Web server
Attached Files

- Relationships

-  Notes
(0001910)
rvelices (developer)
2007.07.09 22:39
edited on: 2007.07.09 22:39

Je ne vois pas bien l'interet de cette fonction. C'est l'equivalent de:
$template->concat_var( 'X', $template->parse($handle, true) );

ou j'ai rate qq chose ?

(0001911)
rvelices (developer)
2007.07.09 23:01

et ca me fait penser quand j'ai ajoute le 2eme parametre a la fonction parse, je voulais changer assign_var_from_handle($varname, $handle) dans

$this->assign_var($varname, $this->parse($handle, true) );

mais je l'avais oublie ...
(0001912)
rub (developer)
2007.07.09 23:47

Non, tu n'as rien raté, j'ai donné la même ligne de code dans http://forum.phpwebgallery.net/viewtopic.php?pid=64691#p64691 [^]

Pourquoi avoir un concat_var_from_handle car on a un assign_var et un concat_var alors autant avoir la paire pour assign_var_from_handle et concat_var_from_handle.
On garde une logique identique et sachant aussi que le concat_var(de parse) n'est pas évident au 1er abord pour le développeurs de plugins.

Par contre, un peu de simplication de code, pourquoi pas! (Je mettais posé la même question!)
(0001913)
rub (developer)
2007.07.10 00:09

[Subversion] r2056

Ca te convient comme ca? Ou je fais marche arrière?
(0001914)
rvelices (developer)
2007.07.10 00:21

oui c'est super car + simple.
juste une remarque - j'avais mis toujours des loc_begin_xxx et non loc_before_xxx ... c'est pas important mais ca serait bien d'avoir une certaine coherence.
(0001915)
rub (developer)
2007.07.10 07:06

Pas de soucis pour les loc_negin, surtout qu'il faut mieux les renommer maintenant qu'après la sortie de la 1.7.
(0001916)
rub (developer)
2007.07.10 07:07

Exemple de code compatible 1.7.0 et >= 1.7.1:

if (method_exists($template, 'concat_var_from_handle'))
{
  $template->concat_var_from_handle('PLUGIN_INDEX_CONTENT_END', 'comonindex');
}
else
{
  $template->concat_var('PLUGIN_INDEX_CONTENT_END', $template->parse('comonindex', true));
}
(0001917)
rub (developer)
2007.07.10 07:08

[Subversion] r2056 & [Subversion] r2057 & [Subversion] r2058

- Issue History
Date Modified Username Field Change
2007.07.09 16:58 Pat New Issue
2007.07.09 16:58 Pat browser => any
2007.07.09 17:10 rub Status new => assigned
2007.07.09 17:10 rub Assigned To => rub
2007.07.09 17:11 rub Additional Information Updated
2007.07.09 22:39 rvelices Note Added: 0001910
2007.07.09 22:39 rvelices Note Edited: 0001910
2007.07.09 23:01 rvelices Note Added: 0001911
2007.07.09 23:47 rub Note Added: 0001912
2007.07.10 00:09 rub Note Added: 0001913
2007.07.10 00:21 rvelices Note Added: 0001914
2007.07.10 07:06 rub Note Added: 0001915
2007.07.10 07:07 rub Note Added: 0001916
2007.07.10 07:08 rub Status assigned => closed
2007.07.10 07:08 rub Note Added: 0001917
2007.07.10 07:08 rub Resolution open => fixed
2007.07.10 07:08 rub Fixed in Version => 1.7.1


Mantis 1.1.6[^]
Copyright © 2000 - 2008 Mantis Group
Contact
Powered by Mantis Bugtracker