makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-28 14:18:52 +00:00
Code Issues Projects Releases Packages Wiki Activity

[feature/twig] Change style->set_style to accept a list of base directories

Browse source 
set_style now accepts an array containing a list of paths, e.g. array(
'ext/foo/bar/styles', 'styles'). Default: array('styles')

Using this option allows us to set the style based on the user's preferred
style (including the full tree), but use one or more base directories to
add the paths from.

The main use for this ability is so that extensions can call set_style,
including their path and the phpBB styles path (or any others) and have
their template files loaded from those directories (in the order given).

PHPBB3-11598
This commit is contained in:
Nathaniel Guse 2013-07-04 10:12:09 -05:00
parent f1717412f3
commit 36f25ea09b
3 changed files with 58 additions and 37 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

87
phpBB/includes/style/style.php
View file

@ -85,28 +85,56 @@ class phpbb_style
}
/**
* Set style location based on (current) user's chosen style.
* Get the style tree of the style preferred by the current user
*
* @return array Style tree, most specific first
*/
public function set_style()
public function get_user_style()
{
$style_path = $this->user->style['style_path'];
$style_dirs = ($this->user->style['style_parent_id']) ? array_reverse(explode('/', $this->user->style['style_parent_tree'])) : array();
return array_merge(array(
$this->user->style['style_path'],
),
($this->user->style['style_parent_id']) ? array_reverse(explode('/', $this->user->style['style_parent_tree'])) : array()
);
}
$names = array($style_path);
foreach ($style_dirs as $dir)
{
$names[] = $dir;
}
// Add 'all' path, used as last fallback path by events and extensions
//$names[] = 'all';
/**
* Set style location based on (current) user's chosen style.
*
* @param array $style_directories The directories to add style paths for
* E.g. array('ext/foo/bar/styles', 'styles')
* Default: array('styles') (phpBB's style directory)
* @return bool true
*/
public function set_style($style_directories = array('styles'))
{
$this->names = $this->get_user_style();
$paths = array();
foreach ($names as $name)
foreach ($style_directories as $directory)
{
$paths[] = $this->get_style_path($name);
foreach ($this->names as $name)
{
$path = $this->get_style_path($name, $directory);
if (is_dir($path))
{
$paths[] = $path;
}
}
}
return $this->set_custom_style($style_path, $paths, $names);
$this->provider->set_styles($paths);
$this->locator->set_paths($this->provider);
foreach ($paths as &$path)
{
$path .= '/template/';
}
$this->template->set_style_names($this->names, $paths, ($style_directories === array('styles')));
return true;
}
/**
@ -118,6 +146,7 @@ class phpbb_style
* @param array or string $paths Array of style paths, relative to current root directory
* @param array $names Array of names of templates in inheritance tree order, used by extensions. If empty, $name will be used.
* @param string $template_path Path to templates, relative to style directory. False if path should be set to default (templates/).
* @return bool true
*/
public function set_custom_style($name, $paths, $names = array(), $template_path = false)
{
@ -138,28 +167,15 @@ class phpbb_style
if ($template_path !== false)
{
$this->locator->set_template_path($template_path);
$appended_paths = array();
foreach ($paths as $path)
{
$appended_paths[] = $path . '/' . $template_path;
}
$this->template->set_style_names($names, $appended_paths);
}
else
foreach ($paths as &$path)
{
$this->locator->set_default_template_path();
$appended_paths = array();
foreach ($paths as $path)
{
$appended_paths[] = $path . '/template/';
}
$this->template->set_style_names($names, $appended_paths);
$path .= '/' . (($template_path !== false) ? $template_path : 'template/');
}
$this->template->set_style_names($names, $paths);
return true;
}
@ -167,11 +183,14 @@ class phpbb_style
* Get location of style directory for specific style_path
*
* @param string $path Style path, such as "prosilver"
* @param string $style_base_directory The base directory the style is in
* E.g. 'styles', 'ext/foo/bar/styles'
* Default: 'styles'
* @return string Path to style directory, relative to current path
*/
public function get_style_path($path)
public function get_style_path($path, $style_base_directory = 'styles')
{
return $this->phpbb_root_path . 'styles/' . $path;
return $this->phpbb_root_path . trim($style_base_directory, '/') . '/' . $path;
}
/**

2
phpBB/includes/template/template.php
View file

@ -41,7 +41,7 @@ interface phpbb_template
* @param array $style_paths List of style paths in inheritance tree order
* @return phpbb_template $this
*/
public function set_style_names(array $style_names, array $style_paths = array());
public function set_style_names(array $style_names, array $style_paths);
/**
* Clears all variables and blocks assigned to this template.

6
phpBB/includes/template/twig/twig.php
View file

@ -181,9 +181,11 @@ class phpbb_template_twig implements phpbb_template
*
* @param array $style_names List of style names in inheritance tree order
* @param array $style_paths List of style paths in inheritance tree order
* @param bool $is_core True if the style names are the "core" styles for this page load
* Core means the main phpBB template files
* @return phpbb_template $this
*/
public function set_style_names(array $style_names, array $style_paths = array())
public function set_style_names(array $style_names, array $style_paths, $is_core = false)
{
$this->style_names = $style_names;
@ -191,7 +193,7 @@ class phpbb_template_twig implements phpbb_template
$this->twig->getLoader()->setPaths($style_paths);
// Core style namespace from phpbb_style::set_style()
if (isset($this->user->style['style_path']) && ($style_names === array($this->user->style['style_path']) || $style_names[0] == $this->user->style['style_path']))
if ($is_core)
{
$this->twig->getLoader()->setPaths($style_paths, 'core');
}