Cleanup. Improves.

src/Fenom.php

@@ -405,9 +405,6 @@ class Fenom
         $this->_provider = $provider;
     }
 
-    public function setCachePerms() {
-    }
-
     /**
      * Set compile directory
      *

src/Fenom/Compiler.php

@@ -9,11 +9,9 @@
  */
 namespace Fenom;
 
+use Doctrine\Instantiator\Exception\InvalidArgumentException;
 use Fenom\Error\InvalidUsageException;
 use Fenom\Error\UnexpectedTokenException;
-use Fenom\Tokenizer;
-use Fenom\Template;
-use Fenom\Scope;
 
 /**
  * Compilers collection
@@ -116,7 +114,7 @@ class Compiler
      * @param Tag $scope
      * @return string
      */
-    public static function tagElse(Tokenizer $tokens, Tag $scope)
+    public static function tagElse($tokens, Tag $scope)
     {
         $scope["else"] = true;
         return '} else {';
@@ -247,8 +245,8 @@ class Compiler
             "last"  => false,
             "step"  => 1,
             "to"    => false,
-            "max"   => false,
-            "min"   => false
+//            "max"   => false,
+//            "min"   => false
         );
         $scope["after"] = $before = $body = array();
         $i              = array('', '');
@@ -263,6 +261,9 @@ class Compiler
         $p   = $scope->tpl->parseParams($tokens, $p);
 
         if (is_numeric($p["step"])) {
+            if($p['to'] === false) {
+                throw new InvalidArgumentException("Invalid value for parameter 'to'");
+            }
             if ($p["step"] > 0) {
                 $condition = "$var <= {$p['to']}";
                 if ($p["last"]) {
@@ -312,7 +313,7 @@ class Compiler
      * @param Tag $scope
      * @return string
      */
-    public static function forElse(Tokenizer $tokens, Tag $scope)
+    public static function forElse($tokens, Tag $scope)
     {
         $scope["no-break"] = $scope["no-continue"] = true;
         $scope["else"]     = true;
@@ -436,7 +437,7 @@ class Compiler
      * @param Tag $scope
      * @return string
      */
-    public static function switchClose(Tokenizer $tokens, Tag $scope)
+    public static function switchClose($tokens, Tag $scope)
     {
         self::_caseResort($scope);
         $expr    = $scope["var"];
@@ -776,7 +777,7 @@ class Compiler
      * @param Tag $scope
      * @return string
      */
-    public static function setClose(Tokenizer $tokens, Tag $scope)
+    public static function setClose($tokens, Tag $scope)
     {
         return $scope["name"] . '=' . $scope["value"] . ';';
     }
@@ -787,7 +788,7 @@ class Compiler
      * @param Tag $scope
      * @return string
      */
-    public static function filterOpen(Tokenizer $tokens, Tag $scope)
+    public static function filterOpen($tokens, Tag $scope)
     {
         $scope["filter"] = $scope->tpl->parseModifier($tokens, "ob_get_clean()");
         return "ob_start();";
@@ -962,7 +963,7 @@ class Compiler
      * @param Tokenizer $tokens
      * @param Tag $scope
      */
-    public static function macroClose(Tokenizer $tokens, Tag $scope)
+    public static function macroClose($tokens, Tag $scope)
     {
         if ($scope["recursive"]) {
             $scope["macro"]["recursive"] = true;
@@ -1017,7 +1018,7 @@ class Compiler
      * @param Tokenizer $tokens
      * @param Tag $tag
      */
-    public static function ignoreOpen(Tokenizer $tokens, Tag $tag)
+    public static function ignoreOpen($tokens, Tag $tag)
     {
         $tag->tpl->ignore('ignore');
     }

src/Fenom/Template.php

@@ -1472,8 +1472,6 @@ class Template extends Render
                 if ($tokens->is("=")) {
                     $tokens->next();
                     $params[$key] = $this->parseExpr($tokens);
-                } else {
-                    $params[$key] = 'true';
                 }
             } elseif ($tokens->is(Tokenizer::MACRO_SCALAR, '"', T_VARIABLE, "[", '(')) {
                 $params[] = $this->parseExpr($tokens);