smarty-php/smarty
1
0
Fork 0
mirror of https://github.com/smarty-php/smarty.git synced 2025-08-05 10:54:27 +02:00
Code Issues Packages Projects Releases Wiki Activity

- add concept unique_resource to combat potentially ambiguous template_resource values when custom resource handlers are used (Forum Topic 2012)

Browse Source 
http://www.smarty.net/forums/viewtopic.php?t=2012

thereby DRYed and optimized the resource handler identification method.
This commit is contained in:
rodneyrehm
2011-10-13 13:10:06 +00:00
parent 806c51fdcf
commit 6979b4efb0
11 changed files with 216 additions and 118 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
change_log.txt
View File

@@ -2,6 +2,7 @@
13.10.2011 13.10.2011
- add caching for config files in Smarty_Resource - add caching for config files in Smarty_Resource
- bugfix disable of caching after isCached() call did not work (Forum Topic 20131) - bugfix disable of caching after isCached() call did not work (Forum Topic 20131)
- add concept unique_resource to combat potentially ambiguous template_resource values when custom resource handlers are used (Forum Topic 20128)
11.10.2011 11.10.2011
- add runtime checks for not matching {capture}/{/capture} calls (Forum Topic 20120) - add runtime checks for not matching {capture}/{/capture} calls (Forum Topic 20120)

3
libs/Smarty.class.php
View File

@@ -1161,7 +1161,8 @@ class Smarty extends Smarty_Internal_TemplateBase {
$cache_id = $cache_id === null ? $this->cache_id : $cache_id; $cache_id = $cache_id === null ? $this->cache_id : $cache_id;
$compile_id = $compile_id === null ? $this->compile_id : $compile_id; $compile_id = $compile_id === null ? $this->compile_id : $compile_id;
// already in template cache? // already in template cache?
$_templateId = sha1($this->smarty->joined_template_dir.$template . $cache_id . $compile_id); $unique_template_name = Smarty_Resource::getUniqueTemplateName($this, $template);
$_templateId = sha1($unique_template_name . $cache_id . $compile_id);
if ($do_clone) { if ($do_clone) {
if (isset($this->template_objects[$_templateId])) { if (isset($this->template_objects[$_templateId])) {
// return cached template object // return cached template object

7
libs/sysplugins/smarty_cacheresource_keyvaluestore.php
View File

@@ -176,8 +176,13 @@ abstract class Smarty_CacheResource_KeyValueStore extends Smarty_CacheResource {
if ($tpl->source->exists) { if ($tpl->source->exists) {
$uid = $tpl->source->uid; $uid = $tpl->source->uid;
} }
// TODO: uwe.tews check if $tpl->compile_id vs. $compile_id is correct
// from what I can see $tpl->compile_id would always be null?!
// remove from template cache // remove from template cache
unset($smarty->template_objects[sha1($smarty->joined_template_dir . $tpl->template_resource . $tpl->cache_id . $tpl->compile_id)]); $_templateId = sha1($tpl->source->unique_resource . $tpl->cache_id . $tpl->compile_id);
unset($smarty->template_objects[$_templateId]);
} }
return $uid; return $uid;
} }

18
libs/sysplugins/smarty_config_source.php
View File

@@ -29,8 +29,9 @@ class Smarty_Config_Source extends Smarty_Template_Source {
* @param string $resource full config_resource * @param string $resource full config_resource
* @param string $type type of resource * @param string $type type of resource
* @param string $name resource name * @param string $name resource name
* @param string $unique_resource unqiue resource name
*/ */
public function __construct(Smarty_Resource $handler, Smarty $smarty, $resource, $type, $name) public function __construct(Smarty_Resource $handler, Smarty $smarty, $resource, $type, $name, $unique_resource)
{ {
$this->handler = $handler; // Note: prone to circular references $this->handler = $handler; // Note: prone to circular references
@@ -43,22 +44,9 @@ class Smarty_Config_Source extends Smarty_Template_Source {
$this->resource = $resource; $this->resource = $resource;
$this->type = $type; $this->type = $type;
$this->name = $name; $this->name = $name;
$this->unique_resource = $unique_resource;
} }
/**
* get a Compiled Object of this source
*
* @param Smarty_Internal_Template $_template template objet
* @return Smarty_Template_Compiled compiled object
*
public function getCompiled(Smarty_Internal_Template $_template)
{
$compiled = new Smarty_Template_Compiled($this);
$this->handler->populateCompiledFilepath($compiled, $_template);
return $compiled;
}
*/
/** /**
* <<magic>> Generic setter. * <<magic>> Generic setter.
* *

13
libs/sysplugins/smarty_internal_cacheresource_file.php
View File

@@ -151,13 +151,18 @@ class Smarty_Internal_CacheResource_File extends Smarty_CacheResource {
$smarty->caching = true; $smarty->caching = true;
$tpl = new $smarty->template_class($resource_name, $smarty); $tpl = new $smarty->template_class($resource_name, $smarty);
$smarty->caching = $_save_stat; $smarty->caching = $_save_stat;
// TODO: uwe.tews check if $tpl->compile_id vs. $compile_id is correct
// from what I can see $tpl->compile_id would always be null?!
// remove from template cache
$tpl->source; // have the template registered before unset()
$_templateId = sha1($tpl->source->unique_resource . $tpl->cache_id . $tpl->compile_id);
unset($smarty->template_objects[$_templateId]);
if ($tpl->source->exists) { if ($tpl->source->exists) {
$_resourcename_parts = basename(str_replace('^', '/', $tpl->cached->filepath)); $_resourcename_parts = basename(str_replace('^', '/', $tpl->cached->filepath));
// remove from template cache
unset($smarty->template_objects[sha1($smarty->joined_template_dir.$tpl->template_resource . $tpl->cache_id . $tpl->compile_id)]);
} else { } else {
// remove from template cache
unset($smarty->template_objects[sha1($smarty->joined_template_dir.$tpl->template_resource . $tpl->cache_id . $tpl->compile_id)]);
return 0; return 0;
} }
} }

38
libs/sysplugins/smarty_internal_resource_eval.php
View File

@@ -37,23 +37,45 @@ class Smarty_Internal_Resource_Eval extends Smarty_Resource_Recompiled {
/** /**
* Load template's source from $resource_name into current template object * Load template's source from $resource_name into current template object
* *
* {@internal if source begins with "base64:" or "urlencode:", the source is decoded accordingly}} * @uses decode() to decode base64 and urlencoded template_resources
*
* @param Smarty_Template_Source $source source object * @param Smarty_Template_Source $source source object
* @return string template source * @return string template source
*/ */
public function getContent(Smarty_Template_Source $source) public function getContent(Smarty_Template_Source $source)
{
return $this->decode($source->name);
}
/**
* decode base64 and urlencode
*
* @param string $string template_resource to decode
* @return string decoded template_resource
*/
protected function decode($string)
{ {
// decode if specified // decode if specified
if (($pos = strpos($source->name, ':')) !== false) { if (($pos = strpos($string, ':')) !== false) {
if (!strncmp($source->name, 'base64', 6)) { if (!strncmp($string, 'base64', 6)) {
return base64_decode(substr($source->name, 7)); return base64_decode(substr($string, 7));
} elseif (!strncmp($source->name, 'urlencode', 9)) { } elseif (!strncmp($string, 'urlencode', 9)) {
return urldecode(substr($source->name, 10)); return urldecode(substr($string, 10));
} }
} }
return $source->name; return $string;
}
/**
* modify resource_name according to resource handlers specifications
*
* @param Smarty $smarty Smarty instance
* @param string $resource_name resource_name to make unique
* @return string unique resource name
*/
protected function buildUniqueResourceName(Smarty $smarty, $resource_name)
{
return get_class($this) . '#' .$this->decode($resource_name);
} }
/** /**

11
libs/sysplugins/smarty_internal_resource_stream.php
View File

@@ -60,6 +60,17 @@ class Smarty_Internal_Resource_Stream extends Smarty_Resource_Recompiled {
} }
} }
/**
* modify resource_name according to resource handlers specifications
*
* @param Smarty $smarty Smarty instance
* @param string $resource_name resource_name to make unique
* @return string unique resource name
*/
protected function buildUniqueResourceName(Smarty $smarty, $resource_name)
{
return get_class($this) . '#' . $resource_name;
}
} }
?> ?>

39
libs/sysplugins/smarty_internal_resource_string.php
View File

@@ -37,24 +37,45 @@ class Smarty_Internal_Resource_String extends Smarty_Resource {
/** /**
* Load template's source from $resource_name into current template object * Load template's source from $resource_name into current template object
* *
* {@internal if source begins with "base64:" or "urlencode:", the source is decoded accordingly}} * @uses decode() to decode base64 and urlencoded template_resources
*
* @param Smarty_Template_Source $source source object * @param Smarty_Template_Source $source source object
* @return string template source * @return string template source
* @throws SmartyException if source cannot be loaded
*/ */
public function getContent(Smarty_Template_Source $source) public function getContent(Smarty_Template_Source $source)
{
return $this->decode($source->name);
}
/**
* decode base64 and urlencode
*
* @param string $string template_resource to decode
* @return string decoded template_resource
*/
protected function decode($string)
{ {
// decode if specified // decode if specified
if (($pos = strpos($source->name, ':')) !== false) { if (($pos = strpos($string, ':')) !== false) {
if (!strncmp($source->name, 'base64', 6)) { if (!strncmp($string, 'base64', 6)) {
return base64_decode(substr($source->name, 7)); return base64_decode(substr($string, 7));
} elseif (!strncmp($source->name, 'urlencode', 9)) { } elseif (!strncmp($string, 'urlencode', 9)) {
return urldecode(substr($source->name, 10)); return urldecode(substr($string, 10));
} }
} }
return $source->name; return $string;
}
/**
* modify resource_name according to resource handlers specifications
*
* @param Smarty $smarty Smarty instance
* @param string $resource_name resource_name to make unique
* @return string unique resource name
*/
protected function buildUniqueResourceName(Smarty $smarty, $resource_name)
{
return get_class($this) . '#' .$this->decode($resource_name);
} }
/** /**

6
libs/sysplugins/smarty_internal_template.php
View File

@@ -245,7 +245,8 @@ class Smarty_Internal_Template extends Smarty_Internal_TemplateBase {
public function getSubTemplate($template, $cache_id, $compile_id, $caching, $cache_lifetime, $data, $parent_scope) public function getSubTemplate($template, $cache_id, $compile_id, $caching, $cache_lifetime, $data, $parent_scope)
{ {
// already in template cache? // already in template cache?
$_templateId = sha1($this->smarty->joined_template_dir.$template . $cache_id . $compile_id); $unique_template_name = Smarty_Resource::getUniqueTemplateName($this->smarty, $template);
$_templateId = sha1($unique_template_name . $cache_id . $compile_id);
if (isset($this->smarty->template_objects[$_templateId])) { if (isset($this->smarty->template_objects[$_templateId])) {
// clone cached template object because of possible recursive call // clone cached template object because of possible recursive call
$tpl = clone $this->smarty->template_objects[$_templateId]; $tpl = clone $this->smarty->template_objects[$_templateId];
@@ -606,7 +607,8 @@ class Smarty_Internal_Template extends Smarty_Internal_TemplateBase {
// cache template object under a unique ID // cache template object under a unique ID
// do not cache eval resources // do not cache eval resources
if ($this->source->type != 'eval') { if ($this->source->type != 'eval') {
$this->smarty->template_objects[sha1($this->smarty->joined_template_dir.$this->template_resource . $this->cache_id . $this->compile_id)] = $this; $_templateId = sha1($this->source->unique_resource . $this->cache_id . $this->compile_id);
$this->smarty->template_objects[$_templateId] = $this;
} }
return $this->source; return $this->source;

14
libs/sysplugins/smarty_internal_utility.php
View File

@@ -189,16 +189,22 @@ class Smarty_Internal_Utility {
$smarty->caching = false; $smarty->caching = false;
$tpl = new $smarty->template_class($resource_name, $smarty); $tpl = new $smarty->template_class($resource_name, $smarty);
$smarty->caching = $_save_stat; $smarty->caching = $_save_stat;
// TODO: uwe.tews check if $tpl->compile_id vs. $compile_id is correct
// from what I can see $tpl->compile_id would always be null?!
// remove from template cache
$tpl->source; // have the template registered before unset()
$_templateId = sha1($tpl->source->unique_resource . $tpl->cache_id . $tpl->compile_id);
unset($smarty->template_objects[$_templateId]);
if ($tpl->source->exists) { if ($tpl->source->exists) {
$_resource_part_1 = basename(str_replace('^', '/', $tpl->compiled->filepath)); $_resource_part_1 = basename(str_replace('^', '/', $tpl->compiled->filepath));
$_resource_part_1_length = strlen($_resource_part_1); $_resource_part_1_length = strlen($_resource_part_1);
// remove from template cache
unset($smarty->template_objects[sha1($smarty->joined_template_dir.$tpl->template_resource . $tpl->cache_id . $tpl->compile_id)]);
} else { } else {
// remove from template cache
unset($smarty->template_objects[sha1($smarty->joined_template_dir.$tpl->template_resource . $tpl->cache_id . $tpl->compile_id)]);
return 0; return 0;
} }
$_resource_part_2 = str_replace('.php','.cache.php',$_resource_part_1); $_resource_part_2 = str_replace('.php','.cache.php',$_resource_part_1);
$_resource_part_2_length = strlen($_resource_part_2); $_resource_part_2_length = strlen($_resource_part_2);
} else { } else {

150
libs/sysplugins/smarty_resource.php
View File

@@ -91,6 +91,19 @@ abstract class Smarty_Resource {
// intentionally left blank // intentionally left blank
} }
/**
* modify resource_name according to resource handlers specifications
*
* @param Smarty $smarty Smarty instance
* @param string $resource_name resource_name to make unique
* @return string unique resource name
*/
protected function buildUniqueResourceName(Smarty $smarty, $resource_name)
{
return get_class($this) . '#' . $smarty->joined_template_dir . '#' . $resource_name;
}
/** /**
* populate Compiled Object with compiled filepath * populate Compiled Object with compiled filepath
* *
@@ -381,6 +394,54 @@ abstract class Smarty_Resource {
throw new SmartyException('Unkown resource type \'' . $resource_type . '\''); throw new SmartyException('Unkown resource type \'' . $resource_type . '\'');
} }
/**
* extract resource_type and resource_name from template_resource and config_resource
*
* @note "C:/foo.tpl" was forced to file resource up till Smarty 3.1.3 (including).
* @param string $resource_name template_resource or config_resource to parse
* @param string $default_resource the default resource_type defined in $smarty
* @param string &$name the parsed resource name
* @param string &$type the parsed resource type
* @return void
*/
protected static function parseResourceName($resource_name, $default_resource, &$name, &$type)
{
$parts = explode(':', $resource_name, 2);
if (!isset($parts[1]) || !isset($parts[0][1])) {
// no resource given, use default
// or single character before the colon is not a resource type, but part of the filepath
$type = $default_resource;
$name = $resource_name;
} else {
$type = $parts[0];
$name = $parts[1];
}
}
/**
* modify resource_name according to resource handlers specifications
*
* @param Smarty $smarty Smarty instance
* @param string $resource_name resource_name to make unique
* @return string unique resource name
*/
/**
* modify template_resource according to resource handlers specifications
*
* @param string $smarty Smarty instance
* @param string $template_resource template_resource to extracate resource handler and name of
* @return string unique resource name
*/
public static function getUniqueTemplateName($smarty, $template_resource)
{
self::parseResourceName($template_resource, $smarty->default_resource_type, $name, $type);
// TODO: optimize for Smarty's internal resource types
$resource = Smarty_Resource::load($smarty, $type);
return $resource->buildUniqueResourceName($smarty, $name);
}
/** /**
* initialize Source Object for given resource * initialize Source Object for given resource
* *
@@ -398,41 +459,23 @@ abstract class Smarty_Resource {
$template_resource = $_template->template_resource; $template_resource = $_template->template_resource;
} }
if (($pos = strpos($template_resource, ':')) === false) { // parse resource_name, load resource handler, identify unique resource name
// no resource given, use default self::parseResourceName($template_resource, $smarty->default_resource_type, $name, $type);
$resource_type = $smarty->default_resource_type; $resource = Smarty_Resource::load($smarty, $type);
$resource_name = $template_resource; $unique_resource_name = $resource->buildUniqueResourceName($smarty, $name);
} else {
// get type and name from path
$resource_type = substr($template_resource, 0, $pos);
$resource_name = substr($template_resource, $pos +1);
if (strlen($resource_type) == 1) {
// 1 char is not resource type, but part of filepath
$resource_type = 'file';
$resource_name = $template_resource;
}
}
$resource = Smarty_Resource::load($smarty, $resource_type);
// TODO: modify $template_resource here
// check runtime cache // check runtime cache
$_cache_key_dir = $smarty->joined_template_dir; $_cache_key = 'template|' . $unique_resource_name;
$_cache_key = 'template|' . $template_resource; if (isset(self::$sources[$_cache_key])) {
if (!isset(self::$sources[$_cache_key_dir])) { return self::$sources[$_cache_key];
self::$sources[$_cache_key_dir] = array();
}
if (isset(self::$sources[$_cache_key_dir][$_cache_key])) {
return self::$sources[$_cache_key_dir][$_cache_key];
} }
// create source // create source
$source = new Smarty_Template_Source($resource, $smarty, $template_resource, $resource_type, $resource_name); $source = new Smarty_Template_Source($resource, $smarty, $template_resource, $type, $name, $unique_resource_name);
$resource->populate($source, $_template); $resource->populate($source, $_template);
// runtime cache // runtime cache
self::$sources[$_cache_key_dir][$_cache_key] = $source; self::$sources[$_cache_key] = $source;
return $source; return $source;
} }
@@ -448,45 +491,30 @@ abstract class Smarty_Resource {
$config_resource = $_config->config_resource; $config_resource = $_config->config_resource;
$smarty = $_config->smarty; $smarty = $_config->smarty;
if (($pos = strpos($config_resource, ':')) === false) { // parse resource_name
// no resource given, use default self::parseResourceName($config_resource, $smarty->default_config_type, $name, $type);
$resource_type = $smarty->default_config_type;
$resource_name = $config_resource; // make sure configs are not loaded via anything smarty can't handle
} else { if (isset($_incompatible_resources[$type])) {
// get type and name from path throw new SmartyException ("Unable to use resource '{$type}' for config");
$resource_type = substr($config_resource, 0, $pos);
$resource_name = substr($config_resource, $pos +1);
if (strlen($resource_type) == 1) {
// 1 char is not resource type, but part of filepath
$resource_type = 'file';
$resource_name = $config_resource;
}
} }
if (isset($_incompatible_resources[$resource_type])) { // load resource handler, identify unique resource name
throw new SmartyException ("Unable to use resource '{$resource_type}' for config"); $resource = Smarty_Resource::load($smarty, $type);
} $unique_resource_name = $resource->buildUniqueResourceName($smarty, $name);
$resource = Smarty_Resource::load($smarty, $resource_type);
// TODO: modify $config_resource here
// check runtime cache // check runtime cache
$_cache_key_dir = $smarty->joined_config_dir; $_cache_key = 'config|' . $unique_resource_name;
$_cache_key = 'config|' . $config_resource; if (isset(self::$sources[$_cache_key])) {
if (!isset(self::$sources[$_cache_key_dir])) { return self::$sources[$_cache_key];
self::$sources[$_cache_key_dir] = array();
}
if (isset(self::$sources[$_cache_key_dir][$_cache_key])) {
return self::$sources[$_cache_key_dir][$_cache_key];
} }
// create source // create source
$source = new Smarty_Config_Source($resource, $smarty, $config_resource, $resource_type, $resource_name); $source = new Smarty_Config_Source($resource, $smarty, $config_resource, $type, $name, $unique_resource_name);
$resource->populate($source, null); $resource->populate($source, null);
// runtime cache // runtime cache
self::$sources[$_cache_key_dir][$_cache_key] = $source; self::$sources[$_cache_key] = $source;
return $source; return $source;
} }
@@ -550,6 +578,12 @@ class Smarty_Template_Source {
*/ */
public $name = null; public $name = null;
/**
* Unique Resource Name
* @var string
*/
public $unique_resource = null;
/** /**
* Source Filepath * Source Filepath
* @var string * @var string
@@ -594,8 +628,9 @@ class Smarty_Template_Source {
* @param string $resource full template_resource * @param string $resource full template_resource
* @param string $type type of resource * @param string $type type of resource
* @param string $name resource name * @param string $name resource name
* @param string $unique_resource unqiue resource name
*/ */
public function __construct(Smarty_Resource $handler, Smarty $smarty, $resource, $type, $name) public function __construct(Smarty_Resource $handler, Smarty $smarty, $resource, $type, $name, $unique_resource)
{ {
$this->handler = $handler; // Note: prone to circular references $this->handler = $handler; // Note: prone to circular references
@@ -609,6 +644,7 @@ class Smarty_Template_Source {
$this->resource = $resource; $this->resource = $resource;
$this->type = $type; $this->type = $type;
$this->name = $name; $this->name = $name;
$this->unique_resource = $unique_resource;
} }
/** /**