diff options
| author | Nathaniel Guse <nathaniel.guse@gmail.com> | 2013-07-02 10:13:16 -0500 |
|---|---|---|
| committer | Nathaniel Guse <nathaniel.guse@gmail.com> | 2013-07-02 10:13:16 -0500 |
| commit | 156d5c671fb9db28fbbdcacb974093c2c52b0f5b (patch) | |
| tree | 7e6cf4ae101e15793c8b6dfabc5de2ec874c78ee /phpBB/includes/template | |
| parent | 883b0a9f8c3e102db8965069899dc2b97d1eb1ff (diff) | |
| download | forums-156d5c671fb9db28fbbdcacb974093c2c52b0f5b.tar forums-156d5c671fb9db28fbbdcacb974093c2c52b0f5b.tar.gz forums-156d5c671fb9db28fbbdcacb974093c2c52b0f5b.tar.bz2 forums-156d5c671fb9db28fbbdcacb974093c2c52b0f5b.tar.xz forums-156d5c671fb9db28fbbdcacb974093c2c52b0f5b.zip | |
[feature/twig] INCLUDEJS now uses the definition class
This had to be done because, like DEFINE, setting variables to $context
only affected the local file and any children, not parent templates.
PHPBB3-11598
Diffstat (limited to 'phpBB/includes/template')
| -rw-r--r-- | phpBB/includes/template/twig/definition.php | 19 | ||||
| -rw-r--r-- | phpBB/includes/template/twig/node/includejs.php | 4 |
2 files changed, 21 insertions, 2 deletions
diff --git a/phpBB/includes/template/twig/definition.php b/phpBB/includes/template/twig/definition.php index 110437eb32..6557b209eb 100644 --- a/phpBB/includes/template/twig/definition.php +++ b/phpBB/includes/template/twig/definition.php @@ -47,4 +47,23 @@ class phpbb_template_twig_definition return $this; } + + /** + * Append to a variable + * + * @param string $name + * @param string $value + * @return phpbb_template_twig_definition + */ + public function append($name, $value) + { + if (!isset($this->definitions[$name])) + { + $this->definitions[$name] = ''; + } + + $this->definitions[$name] .= $value; + + return $this; + } } diff --git a/phpBB/includes/template/twig/node/includejs.php b/phpBB/includes/template/twig/node/includejs.php index f4c26affa4..bba5d4ef91 100644 --- a/phpBB/includes/template/twig/node/includejs.php +++ b/phpBB/includes/template/twig/node/includejs.php @@ -31,9 +31,9 @@ class phpbb_template_twig_node_includejs extends Twig_Node $config = $this->environment->get_phpbb_config(); $compiler - ->write("\$context['SCRIPTS'] .= '<script type=\"text/javascript\" src=\"' . ") + ->write("\$context['definition']->append('SCRIPTS', '<script type=\"text/javascript\" src=\"' . ") ->subcompile($this->getNode('expr')) - ->raw(" . '?assets_version=" . $config['assets_version'] . "\"></script>';\n\n") + ->raw(" . '?assets_version=" . $config['assets_version'] . "\"></script>');\n") ; } } |