Piwigo Bugtracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000718Piwigootherpublic2007.07.09 16:582007.07.10 07:08
ReporterPat 
Assigned Torub 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version1.7.0 
Target VersionFixed in Version1.7.1 
Summary0000718: Ajout d'une fonction "concat_var_from_handle"
DescriptionPermet à plusieurs plugins d'insérer du code HTML au meme endroit dans un template en concaténant les valeurs.
Additional InformationCode 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;
    }
TagsNo tags attached.
browserany
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


Copyright © 2000 - 2015 MantisBT Team
Contact
Powered by Mantis Bugtracker