- revert bugfix compiling {section} did create warning

change_log.txt

@@ -1,4 +1,7 @@
  ===== 3.1.30-dev ===== (xx.xx.xx)
+ 27.01.2016
+ - revert bugfix compiling {section} did create warning
+
  26.01.2016
   - improvement observe Smarty::$_CHARSET in debugging console https://github.com/smarty-php/smarty/issues/169
   - bugfix compiling {section} did create warning

libs/Smarty.class.php

@@ -121,7 +121,7 @@ class Smarty extends Smarty_Internal_TemplateBase
     /**
      * smarty version
      */
-    const SMARTY_VERSION = '3.1.30-dev/19';
+    const SMARTY_VERSION = '3.1.30-dev/20';
 
     /**
      * define variable scopes

libs/sysplugins/smarty_internal_compile_private_foreachsection.php

@@ -14,42 +14,9 @@
  * @package    Smarty
  * @subpackage Compiler
  */
-abstract class Smarty_Internal_Compile_Private_ForeachSection extends Smarty_Internal_CompileBase
+class Smarty_Internal_Compile_Private_ForeachSection extends Smarty_Internal_CompileBase
 {
 
-    /**
-     * Valid properties of $smarty.xxx variable
-     *
-     * @var array
-     */
-    public $nameProperties = array();
-
-    /**
-     * Name of this tag
-     *
-     * @var string
-     */
-    public $tagName = '';
-
-    /**
-     * {section} tag has no item properties
-     *
-     * @var array
-     */
-    public $itemProperties = null;
-
-    /**
-     * {section} tag has always name attribute
-     *
-     * @var bool
-     */
-    public $isNamed = true;
-
-    /**
-     * @var array
-     */
-    public $matchResults = array();
-
     /**
      * Preg search pattern
      *
@@ -72,37 +39,45 @@ abstract class Smarty_Internal_Compile_Private_ForeachSection extends Smarty_Int
     private $startOffset = 0;
 
     /**
-     * Compiles code for the {$smarty.foreach.xxx} or {$smarty.section.xxx}tag
+     * Name of this tag
      *
-     * @param  array                            $args      array with attributes from parser
+     * @var string
-     * @param \Smarty_Internal_TemplateCompiler $compiler  compiler object
-     * @param  array                            $parameter array with compilation parameter
-     *
-     * @return string compiled code
-     * @throws \SmartyCompilerException
      */
-    public function compileSpecialVariable($args, Smarty_Internal_TemplateCompiler $compiler, $parameter = null)
+    public $tagName = '';
-    {
+
-        $tag = strtolower(trim($parameter[ 0 ], '"\''));
+    /**
-        $name = isset($parameter[ 1 ]) ? $compiler->getId($parameter[ 1 ]) : false;
+     * Valid properties of $smarty.xxx variable
-        if (!$name) {
+     *
-            $compiler->trigger_template_error("missing or illegal \$smarty.{$tag} name attribute", null, true);
+     * @var array
-        }
+     */
-        $property = isset($parameter[ 2 ]) ? strtolower($compiler->getId($parameter[ 2 ])) : false;
+    public $nameProperties = array();
-        if (!$property || !in_array($property, $this->nameProperties)) {
+
-            $compiler->trigger_template_error("missing or illegal \$smarty.{$tag} property attribute", null, true);
+    /**
-        }
+     * {section} tag has no item properties
-        $tagVar = "'__smarty_{$tag}_{$name}'";
+     *
-        return "(isset(\$_smarty_tpl->tpl_vars[{$tagVar}]->value['{$property}']) ? \$_smarty_tpl->tpl_vars[{$tagVar}]->value['{$property}'] : null)";
+     * @var array
-    }
+     */
+    public $itemProperties = null;
+
+    /**
+     * {section} tag has always name attribute
+     *
+     * @var bool
+     */
+    public $isNamed = true;
+
+    /**
+     * @var array
+     */
+    public $matchResults = array();
 
     /**
      * Scan sources for used tag attributes
      *
      * @param  array                                $attributes
-     * @param \Smarty_Internal_TemplateCompiler $compiler
+     * @param \Smarty_Internal_TemplateCompilerBase $compiler
      */
-    public function scanForProperties($attributes, Smarty_Internal_TemplateCompiler $compiler)
+    public function scanForProperties($attributes, Smarty_Internal_TemplateCompilerBase $compiler)
     {
         $this->propertyPreg = '~(';
         $this->startOffset = 0;
@@ -120,6 +95,10 @@ abstract class Smarty_Internal_Compile_Private_ForeachSection extends Smarty_Int
         $this->propertyPreg .= ')\W~i';
         // Template source
         $this->matchTemplateSource($compiler);
+        // Parent template source
+        $this->matchParentTemplateSource($compiler);
+        // {block} source
+        $this->matchBlockSource($compiler);
     }
 
     /**
@@ -151,16 +130,6 @@ abstract class Smarty_Internal_Compile_Private_ForeachSection extends Smarty_Int
         $this->propertyPreg .= '))';
     }
 
-    /**
-     * Find matches in template source
-     *
-     * @param \Smarty_Internal_TemplateCompiler $compiler
-     */
-    public function matchTemplateSource(Smarty_Internal_TemplateCompiler $compiler)
-    {
-        $this->matchProperty($compiler->parser->lex->data);
-    }
-
     /**
      * Find matches in source string
      *
@@ -177,4 +146,75 @@ abstract class Smarty_Internal_Compile_Private_ForeachSection extends Smarty_Int
             }
         }
     }
+
+    /**
+     * Find matches in template source
+     *
+     * @param \Smarty_Internal_TemplateCompilerBase $compiler
+     */
+    public function matchTemplateSource(Smarty_Internal_TemplateCompilerBase $compiler)
+    {
+        $this->matchProperty($compiler->parser->lex->data);
+    }
+
+    /**
+     * Find matches in all parent template source
+     *
+     * @param \Smarty_Internal_TemplateCompilerBase $compiler
+     */
+    public function matchParentTemplateSource(Smarty_Internal_TemplateCompilerBase $compiler)
+    {
+        // search parent compiler template source
+        $nextCompiler = $compiler;
+        while ($nextCompiler !== $nextCompiler->parent_compiler) {
+            $nextCompiler = $nextCompiler->parent_compiler;
+            if ($compiler !== $nextCompiler) {
+                // get template source
+                $_content = $nextCompiler->template->source->getContent();
+                if ($_content != '') {
+                    // run pre filter if required
+                    if ((isset($nextCompiler->smarty->autoload_filters['pre']) ||
+                            isset($nextCompiler->smarty->registered_filters['pre']))) {
+                        $_content = $nextCompiler->smarty->ext->_filter_Handler->runFilter('pre', $_content, $nextCompiler->template);
+                    }
+                    $this->matchProperty($_content);
+                }
+            }
+        }
+    }
+
+    /**
+     * Find matches in {block} tag source
+     *
+     * @param \Smarty_Internal_TemplateCompilerBase $compiler
+     */
+    public function matchBlockSource(Smarty_Internal_TemplateCompilerBase $compiler)
+    {
+
+    }
+
+    /**
+     * Compiles code for the {$smarty.foreach.xxx} or {$smarty.section.xxx}tag
+     *
+     * @param  array                                $args      array with attributes from parser
+     * @param \Smarty_Internal_TemplateCompilerBase $compiler  compiler object
+     * @param  array                                $parameter array with compilation parameter
+     *
+     * @return string compiled code
+     * @throws \SmartyCompilerException
+     */
+    public function compileSpecialVariable($args, Smarty_Internal_TemplateCompilerBase $compiler, $parameter)
+    {
+        $tag = strtolower(trim($parameter[ 0 ], '"\''));
+        $name = isset($parameter[ 1 ]) ? $compiler->getId($parameter[ 1 ]) : false;
+        if (!$name) {
+            $compiler->trigger_template_error("missing or illegal \$smarty.{$tag} name attribute", null, true);
+        }
+        $property = isset($parameter[ 2 ]) ? strtolower($compiler->getId($parameter[ 2 ])) : false;
+        if (!$property || !in_array($property, $this->nameProperties)) {
+            $compiler->trigger_template_error("missing or illegal \$smarty.{$tag} property attribute", null, true);
+        }
+        $tagVar = "'__smarty_{$tag}_{$name}'";
+        return "(isset(\$_smarty_tpl->tpl_vars[{$tagVar}]->value['{$property}']) ? \$_smarty_tpl->tpl_vars[{$tagVar}]->value['{$property}'] : null)";
+    }
 }