Skip to content

Commit

Permalink
- correct generated code on empty() and isset() call, observe change …
Browse files Browse the repository at this point in the history 
…PHP behaviour since PHP 5.5

    #347 (reverted from commit cfd8bf3)
  • Loading branch information
@uwetews
uwetews committed Apr 17, 2017
1 parent f7bdfe4 commit 29388b8
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 73 deletions.
4 changes: 0 additions & 4 deletions change_log.txt
View file
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
===== 3.1.32 - dev ===
17.4.2017
- correct generated code on empty() and isset() call, observe change PHP behaviour since PHP 5.5
https://github.com/smarty-php/smarty/issues/347

14.4.2017
- merge pull requests https://github.com/smarty-php/smarty/pull/349, https://github.com/smarty-php/smarty/pull/322 and
https://github.com/smarty-php/smarty/pull/337 to fix spelling and annotation
Expand Down
2 changes: 1 addition & 1 deletion libs/Smarty.class.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ class Smarty extends Smarty_Internal_TemplateBase
/**
* smarty version
*/
const SMARTY_VERSION = '3.1.32-dev-1';
const SMARTY_VERSION = '3.1.32-dev';

/**
* define variable scopes
Expand Down
111 changes: 43 additions & 68 deletions libs/sysplugins/smarty_internal_templatecompilerbase.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -280,10 +280,7 @@ abstract class Smarty_Internal_TemplateCompilerBase
*
* @var array
*/
public $plugin_search_order = array('function',
'block',
'compiler',
'class');
public $plugin_search_order = array('function', 'block', 'compiler', 'class');

/**
* General storage area for tag compiler plugins
Expand Down Expand Up @@ -317,8 +314,7 @@ abstract protected function doCompile($_content, $isTemplateSource = false);
public function __construct(Smarty $smarty)
{
$this->smarty = $smarty;
$this->nocache_hash = str_replace(array('.',
','), '_', uniqid(rand(), true));
$this->nocache_hash = str_replace(array('.', ','), '_', uniqid(rand(), true));
}

/**
Expand Down Expand Up @@ -391,8 +387,7 @@ public function compileTemplateSource(Smarty_Internal_Template $template, $nocac
// add file dependency
if ($this->smarty->merge_compiled_includes || $this->template->source->handler->checkTimestamps()) {
$this->parent_compiler->template->compiled->file_dependency[ $this->template->source->uid ] =
array($this->template->source->filepath,
$this->template->source->getTimeStamp(),
array($this->template->source->filepath, $this->template->source->getTimeStamp(),
$this->template->source->type,);
}
$this->smarty->_current_file = $this->template->source->filepath;
Expand Down Expand Up @@ -458,7 +453,7 @@ public function postFilter($code)
public function preFilter($_content)
{
// run pre filter if required
if ($_content !== '' &&
if ($_content != '' &&
((isset($this->smarty->autoload_filters[ 'pre' ]) || isset($this->smarty->registered_filters[ 'pre' ])))
) {
return $this->smarty->ext->_filterHandler->runFilter('pre', $_content, $this->template);
Expand Down Expand Up @@ -513,8 +508,7 @@ private function compileTag2($tag, $args, $parameter)
$this->has_output = false;
// log tag/attributes
if (isset($this->smarty->_cache[ 'get_used_tags' ])) {
$this->template->_cache[ 'used_tags' ][] = array($tag,
$args);
$this->template->_cache[ 'used_tags' ][] = array($tag, $args);
}
// check nocache option flag
foreach ($args as $arg) {
Expand All @@ -524,7 +518,7 @@ private function compileTag2($tag, $args, $parameter)
}
} else {
foreach ($arg as $k => $v) {
if (($k === "'nocache'" || $k === 'nocache') && (trim($v, "'\" ") === 'true')) {
if (($k === "'nocache'" || $k === 'nocache') && (trim($v, "'\" ") == 'true')) {
$this->tag_nocache = true;
}
}
Expand All @@ -533,9 +527,7 @@ private function compileTag2($tag, $args, $parameter)
// compile the smarty tag (required compile classes to compile the tag are auto loaded)
if (($_output = $this->callTagCompiler($tag, $args, $parameter)) === false) {
if (isset($this->parent_compiler->tpl_function[ $tag ]) ||
(isset ($this->template->smarty->ext->_tplFunction) &&
$this->template->smarty->ext->_tplFunction->getTplFunction($this->template, $tag) !== false)
) {
(isset ($this->template->smarty->ext->_tplFunction) && $this->template->smarty->ext->_tplFunction->getTplFunction($this->template, $tag) !== false)) {
// template defined by {template} tag
$args[ '_attr' ][ 'name' ] = "'" . $tag . "'";
$_output = $this->callTagCompiler('call', $args, $parameter);
Expand Down Expand Up @@ -565,7 +557,7 @@ private function compileTag2($tag, $args, $parameter)
}
}
// not an internal compiler tag
if (strlen($tag) < 6 || substr($tag, - 5) !== 'close') {
if (strlen($tag) < 6 || substr($tag, - 5) != 'close') {
// check if tag is a registered object
if (isset($this->smarty->registered_objects[ $tag ]) && isset($parameter[ 'object_method' ])) {
$method = $parameter[ 'object_method' ];
Expand All @@ -584,12 +576,11 @@ private function compileTag2($tag, $args, $parameter)
}
}
// check if tag is registered
foreach (array(Smarty::PLUGIN_COMPILER,
Smarty::PLUGIN_FUNCTION,
Smarty::PLUGIN_BLOCK,) as $plugin_type) {
foreach (array(Smarty::PLUGIN_COMPILER, Smarty::PLUGIN_FUNCTION, Smarty::PLUGIN_BLOCK,) as $plugin_type)
{
if (isset($this->smarty->registered_plugins[ $plugin_type ][ $tag ])) {
// if compiler function plugin call it now
if ($plugin_type === Smarty::PLUGIN_COMPILER) {
if ($plugin_type == Smarty::PLUGIN_COMPILER) {
$new_args = array();
foreach ($args as $key => $mixed) {
if (is_array($mixed)) {
Expand All @@ -602,19 +593,18 @@ private function compileTag2($tag, $args, $parameter)
$this->tag_nocache = true;
}
return call_user_func_array($this->smarty->registered_plugins[ $plugin_type ][ $tag ][ 0 ],
array($new_args,
$this));
array($new_args, $this));
}
// compile registered function or block function
if ($plugin_type === Smarty::PLUGIN_FUNCTION || $plugin_type === Smarty::PLUGIN_BLOCK) {
if ($plugin_type == Smarty::PLUGIN_FUNCTION || $plugin_type == Smarty::PLUGIN_BLOCK) {
return $this->callTagCompiler('private_registered_' . $plugin_type, $args, $parameter,
$tag);
}
}
}
// check plugins from plugins folder
foreach ($this->plugin_search_order as $plugin_type) {
if ($plugin_type === Smarty::PLUGIN_COMPILER &&
if ($plugin_type == Smarty::PLUGIN_COMPILER &&
$this->smarty->loadPlugin('smarty_compiler_' . $tag) &&
(!isset($this->smarty->security_policy) ||
$this->smarty->security_policy->isTrustedTag($tag, $this))
Expand Down Expand Up @@ -671,7 +661,7 @@ private function compileTag2($tag, $args, $parameter)
}
if ($found) {
// if compiler function plugin call it now
if ($plugin_type === Smarty::PLUGIN_COMPILER) {
if ($plugin_type == Smarty::PLUGIN_COMPILER) {
$new_args = array();
foreach ($args as $key => $mixed) {
if (is_array($mixed)) {
Expand All @@ -681,8 +671,7 @@ private function compileTag2($tag, $args, $parameter)
}
}
return call_user_func_array($this->default_handler_plugins[ $plugin_type ][ $tag ][ 0 ],
array($new_args,
$this));
array($new_args, $this));
} else {
return $this->callTagCompiler('private_registered_' . $plugin_type, $args, $parameter,
$tag);
Expand Down Expand Up @@ -734,8 +723,7 @@ private function compileTag2($tag, $args, $parameter)
$this->tag_nocache = true;
}
return call_user_func_array($this->smarty->registered_plugins[ Smarty::PLUGIN_COMPILER ][ $tag ][ 0 ],
array($args,
$this));
array($args, $this));
}
if ($this->smarty->loadPlugin('smarty_compiler_' . $tag)) {
$plugin = 'smarty_compiler_' . $tag;
Expand Down Expand Up @@ -764,7 +752,7 @@ private function compileTag2($tag, $args, $parameter)
*/
public function compileVariable($variable)
{
if (strpos($variable, '(') === 0) {
if (strpos($variable, '(') == 0) {
// not a variable variable
$var = trim($variable, '\'');
$this->tag_nocache = $this->tag_nocache |
Expand Down Expand Up @@ -805,32 +793,28 @@ public function compilePHPFunctionCall($name, $parameter)
$func_name = strtolower($name);
$par = implode(',', $parameter);
$parHasFuction = strpos($par, '(') !== false;
if ($func_name === 'isset') {
if (count($parameter) === 0) {
if ($func_name == 'isset') {
if (count($parameter) == 0) {
$this->trigger_template_error('Illegal number of paramer in "isset()"');
}
if ($parHasFuction) {
$pa = array();
foreach ($parameter as $p) {
$pa[] = (strpos($p, '(') === false) ? ('isset(' . $p . ')') : ('(' . $p . ' !== null )');
}
return "(" . implode(' && ', $pa) . ")";
$prefixVar = $this->getNewPrefixVariable();
$this->appendPrefixCode("<?php $prefixVar" . '=' . $par . ';?>');
$isset_par = $prefixVar;
} else {
$isset_par = str_replace("')->value", "',null,true,false)->value", $par);
}
return $name . "(" . $isset_par . ")";
} elseif (in_array($func_name, array('empty',
'reset',
'current',
'end',
'prev',
'next'))) {
if (count($parameter) !== 1) {
$this->trigger_template_error("Illegal number of paramer in '{$func_name()}'");
} elseif (in_array($func_name, array('empty', 'reset', 'current', 'end', 'prev', 'next'))) {
if (count($parameter) != 1) {
$this->trigger_template_error('Illegal number of paramer in "empty()"');
}
if ($func_name === 'empty') {
if ($parHasFuction && version_compare(PHP_VERSION, '5.5.0', '<')) {
return '(' . $parameter[ 0 ] . ' == false )';
if ($func_name == 'empty') {
if ($parHasFuction) {
$prefixVar = $this->getNewPrefixVariable();
$this->appendPrefixCode("<?php $prefixVar" . '=' . $par . ';?>');

return $func_name . '(' . $prefixVar . ')';
} else {
return $func_name . '(' .
str_replace("')->value", "',null,true,false)->value", $parameter[ 0 ]) . ')';
Expand Down Expand Up @@ -858,7 +842,7 @@ public function compilePHPFunctionCall($name, $parameter)
*/
public function processText($text)
{
if ((string) $text !== '') {
if ((string) $text != '') {
$store = array();
$_store = 0;
if ($this->parser->strip) {
Expand All @@ -874,7 +858,7 @@ public function processText($text)
$text = substr_replace($text, $replace, $match[ 0 ][ 1 ] - $_offset, $_length);

$_offset += $_length - strlen($replace);
++ $_store;
$_store ++;
}
}
$expressions = array(// replace multiple spaces between tags by a single space
Expand All @@ -883,8 +867,7 @@ public function processText($text)
'#(:SMARTY@!@|>)[\040\011]*[\n]\s*(?=@!@SMARTY:|<)#s' => '\1\2',
// remove multiple spaces between attributes (but not in attribute values!)
'#(([a-z0-9]\s*=\s*("[^"]*?")|(\'[^\']*?\'))|<[a-z0-9_]+)\s+([a-z/>])#is' => '\1 \5',
'#>[\040\011]+$#Ss' => '> ',
'#>[\040\011]*[\n]\s*$#Ss' => '>',
'#>[\040\011]+$#Ss' => '> ', '#>[\040\011]*[\n]\s*$#Ss' => '>',
$this->stripRegEx => '',);

$text = preg_replace(array_keys($expressions), array_values($expressions), $text);
Expand All @@ -897,7 +880,7 @@ public function processText($text)
$text = substr_replace($text, $replace, $match[ 0 ][ 1 ] + $_offset, $_length);

$_offset += strlen($replace) - $_length;
++ $_store;
$_store ++;
}
}
} else {
Expand Down Expand Up @@ -925,7 +908,6 @@ public function processText($text)
*/
public function callTagCompiler($tag, $args, $param1 = null, $param2 = null, $param3 = null)
{
/* @var Smarty_Internal_CompileBase $tagCompiler */
$tagCompiler = $this->getTagCompiler($tag);
// compile this tag
return $tagCompiler === false ? false : $tagCompiler->compile($args, $this, $param1, $param2, $param3);
Expand Down Expand Up @@ -993,7 +975,7 @@ public function getPlugin($plugin_name, $plugin_type)
}
}
if (isset($function)) {
if ($plugin_type === 'modifier') {
if ($plugin_type == 'modifier') {
$this->modifier_plugins[ $plugin_name ] = true;
}

Expand All @@ -1015,7 +997,7 @@ public function getPlugin($plugin_name, $plugin_type)
$this->parent_compiler->template->compiled->required_plugins[ 'compiled' ][ $plugin_name ][ $plugin_type ][ 'function' ] =
$function;
}
if ($plugin_type === 'modifier') {
if ($plugin_type == 'modifier') {
$this->modifier_plugins[ $plugin_name ] = true;
}

Expand All @@ -1042,12 +1024,8 @@ public function getPluginFromDefaultHandler($tag, $plugin_type)
$callback = null;
$script = null;
$cacheable = true;
$result = call_user_func_array($this->smarty->default_plugin_handler_func, array($tag,
$plugin_type,
$this->template,
&$callback,
&$script,
&$cacheable,));
$result = call_user_func_array($this->smarty->default_plugin_handler_func,
array($tag, $plugin_type, $this->template, &$callback, &$script, &$cacheable,));
if ($result) {
$this->tag_nocache = $this->tag_nocache || !$cacheable;
if ($script !== null) {
Expand All @@ -1069,9 +1047,7 @@ public function getPluginFromDefaultHandler($tag, $plugin_type)
}
}
if (is_callable($callback)) {
$this->default_handler_plugins[ $plugin_type ][ $tag ] = array($callback,
true,
array());
$this->default_handler_plugins[ $plugin_type ][ $tag ] = array($callback, true, array());

return true;
} else {
Expand Down Expand Up @@ -1260,8 +1236,7 @@ public function trigger_template_error($args = null, $line = null, $tagline = nu
$line = (int) $line;
}

if (in_array($this->template->source->type, array('eval',
'string'))) {
if (in_array($this->template->source->type, array('eval', 'string'))) {
$templateName = $this->template->source->type . ':' . trim(preg_replace('![\t\r\n]+!', ' ',
strlen($lex->data) > 40 ?
substr($lex->data, 0, 40) .
Expand Down Expand Up @@ -1350,7 +1325,7 @@ public function isVariable($value)
*/
public function getNewPrefixVariable()
{
++ self::$prefixVariableNumber;
self::$prefixVariableNumber ++;
return $this->getPrefixVariable();
}

Expand Down

0 comments on commit 29388b8

Please sign in to comment.