Remove optimization (covered by Optimization VAR_ACCESS anyway)
authorAndreas Streichardt <andreas.streichardt@100days.de>
Tue, 24 Jan 2012 16:56:32 +0000 (17:56 +0100)
committerAndreas Streichardt <andreas.streichardt@100days.de>
Tue, 24 Jan 2012 16:56:32 +0000 (17:56 +0100)
breaks templates implementing their own getContext()

lib/Twig/Node/Expression/Name.php

index 4b8d541..a8d206f 100644 (file)
@@ -35,29 +35,18 @@ class Twig_Node_Expression_Name extends Twig_Node_Expression
         } elseif ($this->isSpecial()) {
             $compiler->raw($this->specialVars[$name]);
         } else {
-            if (version_compare(phpversion(), '5.4.0RC1', '>=') && ($this->getAttribute('ignore_strict_check') || !$compiler->getEnvironment()->isStrictVariables())) {
-                // PHP 5.4 ternary operator performance was optimized
-                $compiler
-                    ->raw('(isset($context[')
-                    ->string($name)
-                    ->raw(']) ? $context[')
-                    ->string($name)
-                    ->raw('] : null)')
-                ;
-            } else {
-                $compiler
-                    ->raw('$this->getContext($context, ')
-                    ->string($name)
-                ;
-
-                if ($this->getAttribute('ignore_strict_check')) {
-                    $compiler->raw(', true');
-                }
+            $compiler
+                ->raw('$this->getContext($context, ')
+                ->string($name)
+            ;
 
-                $compiler
-                    ->raw(')')
-                ;
+            if ($this->getAttribute('ignore_strict_check')) {
+                $compiler->raw(', true');
             }
+
+            $compiler
+                ->raw(')')
+            ;
         }
     }