- WIP on supporting multiple contexts, better handling of caching for methods / functions (instead of in every files, etc...

docs/02-control-structure-view-include-block.md

@@ -96,9 +96,9 @@ You can do so by using `define` and `slot`.
 ```html
 {% arguments string $name, string $anchor, int $index = 0 %}
 
-{% define "attributes" %}
+{% define slot %}
 
-<a {% slot "attributes" %}{% endslot %} href="{{ $anchor }}" tabindex="{{ $index }}"{% endslot %}>{{ $name }}"</a>
+<a {% slot "attributes" %}href="{{ $anchor }}" tabindex="{{ $index }}"{% endslot %}>{{ $name }}"</a>
 ```
 
 **[HTML]** Would render the same as the `view` example :

src/Builder.php

@@ -21,7 +21,7 @@ class Builder
 
         $replace = [
             '%NAMESPACE%' => $context->namespace,
-            '%USE%' => ( $uses = $context->renderUses() ) ? $uses : false,
+            '%USE%' => ( $uses = $context->renderUses() ) ? "use $uses;" : false,
             '%CLASSNAME%' => $context->className,
             '%PATHNAME%' => $path($context->viewPath),
             '%FULLPATH%' => $path($context->filePath),

src/Compiler/BaseContext.php

@@ -4,7 +4,7 @@ namespace Picea\Compiler;
 
 class BaseContext extends Context {
 
-    public function __construct(?string $defaultNamespace = null) {
+    public function __construct(array|string|null $defaultNamespace = null) {
         if ( $defaultNamespace !== null ) {
             $this->namespace = $defaultNamespace;
         }

src/Compiler/Context.php

@@ -6,7 +6,7 @@ use Picea\Compiler;
 
 abstract class Context {
 
-    public string $namespace = "";
+    public string|array $namespace = "";
 
     public string $extendFrom = "";
 
@@ -57,7 +57,7 @@ abstract class Context {
 
     public function renderUses() : string
     {
-        return implode(PHP_EOL, array_map(fn($use) => "use $use;", $this->useStack ?? []));
+        return implode(",", $this->useStack ?? []);
     }
 
     public function renderFunctions() : string

src/ControlStructure/AbstractLoop.php

@@ -4,43 +4,45 @@ namespace Picea\ControlStructure;
 
 abstract class AbstractLoop implements ControlStructure {
 
-    protected function openLoop(\Picea\Compiler\Context &$context, ?string $arguments, string $token,) : string
+    public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string
     {
-        $name = "$".uniqid("{$token}_");
+        switch($token) {
+            case "while":
+            case "foreach":
+                $name = "$".uniqid("{$token}_");
+                
+                $stack = array_filter($context->iterationStack ?? [], function($item) {
+                    return ! $item['or'];
+                });
+                
+                $count = count($stack);
 
-        $stack = array_filter($context->iterationStack ?? [], function($item) {
-            return ! $item['or'];
-        });
+                if ( $count > 0 ) {
+                    $name .= "[" . end($stack)['uid'] . "]";
+                }
 
-        $count = count($stack);
+                $context->iterationStack[] = [
+                    'or' => false,
+                    'uid' => $name,
+                    'token' => "end{$token}",
+                ];
 
-        if ( $count > 0 ) {
-            $name .= "[" . end($stack)['uid'] . "]";
+                return "<?php $token ($arguments): $name = ( $name ?? 0 ) + 1; ?>";
+
+            case "endwhile":
+            case "endforeach":
+                $last = end($context->iterationStack);
+
+                if ( $last['or'] === false ) {
+                    $output = "<?php $token; ?>";
+                }
+                else {
+                    $output = "<?php endif; if ( isset({$last['uid']}) ) unset({$last['uid']}); ?>";
+                }
+
+                array_pop($context->iterationStack);
+                
+                return $output;
         }
-
-        $context->iterationStack[] = [
-            'or' => false,
-            'uid' => $name,
-            'token' => "end{$token}",
-        ];
-
-        return "<?php $token ($arguments): \$__loop_index = $name = ( $name ?? 0 ) + 1; ?>";
-    }
-
-    protected function closeLoop(\Picea\Compiler\Context &$context, ?string $arguments, string $token,) : string
-    {
-        $last = end($context->iterationStack);
-
-        if ( $last['or'] === false ) {
-            $output = "<?php $token; ?>";
-        }
-        else {
-            $output = "<?php endif; if ( isset({$last['uid']}) ) unset({$last['uid']}); ?>";
-        }
-
-        array_pop($context->iterationStack);
-
-        return $output;
     }
 }
- 

src/ControlStructure/BlockToken.php

@@ -4,27 +4,21 @@ namespace Picea\ControlStructure;
 
 class BlockToken implements ControlStructure {
 
-    public array $token = [ "arguments", "block", "endblock", "/block", "define", "slot", "endslot", "/slot", "using" ];
+    public array $token = [ "arguments", "block", "endblock", "define", "slot", "endslot", "using" ];
 
     public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string
     {
-        static $depth = 0;
         static $slotDefinitions = [];
 
-        # dump($depth, $token, $arguments, $slotDefinitions);
-
         switch($token) {
             case "block":
                 $slotDefinitions[] = $this->slotDefinitions();
-                $depth++;
-
+                
                 return "<?php \$___block = \Picea\ControlStructure\BlockToken::instanciateBlock($arguments); ?>";
-
-            case "/block":
+            
             case "endblock":
-                $depth--;
                 return "<?php echo \$___block->render(\$___class__template); unset(\$___block); ?>";
-
+            
             case "arguments":
                 $class = static::class;
 
@@ -42,19 +36,19 @@ class BlockToken implements ControlStructure {
                         }
                     /*%EXCEPTION_LINE_BASE%*/?>
                 PHP;
-
+            
             case "define":
                 list($name, $definition) = array_pad(explode(',', $arguments, 2), 2, "");
-
-                ( $slotDefinitions[$depth] ?? end($slotDefinitions) )->setDefinition(eval("return $name;"), $definition);
-
+                
+                end($slotDefinitions)->setDefinition(eval("return $name;"), $definition);
+                
                 return <<<PHP
-                <?php \$this->defineSlot($name, function($definition) {}); ?>
+                    <?php \$this->defineSlot($name, function($definition) {}); ?>
                 PHP;
-
+                    
             case "slot":
-                $def = ( $slotDefinitions[$depth] ?? end($slotDefinitions) );
-
+                $def = end($slotDefinitions);
+                
                 list($name, $definition) = array_pad(explode(',', $arguments, 2), 2, "");
 
                 $loops = count($context->iterationStack ?? []) ? ",". implode(', ', array_filter(array_column($context->iterationStack, 'uid'), fn($e) => strpos($e, '[') === false)) : null;
@@ -63,7 +57,7 @@ class BlockToken implements ControlStructure {
                     $slotName = eval("return $name;");
                     $def->currentSlot = $slotName;
                     $def->setDefinitionVars($slotName, $definition);
-
+                    
                     $definition = $def->printDefinition($slotName);
 
                     if ($definition) {
@@ -71,53 +65,53 @@ class BlockToken implements ControlStructure {
                     }
 
                     return <<<PHP
-                    <?php \$this->printSlot($name, function($definition array \$___using = []) use (\$picea $loops) { extract(\$___using, \EXTR_SKIP); ?>
+                        <?php \$this->printSlot($name, function($definition array \$___using = []) use (\$picea $loops) { extract(\$___using, \EXTR_SKIP); ?>
                     PHP;
                 }
                 else {
                     if ($definition) {
                         $definition .= ",";
                     }
-
+                    
                     return <<<PHP
-                    <?php (\$___block ?? \$this)->slotIsSet($name) || (\$___block ?? \$this)->setSlot($name, function($definition array \$___using = []) use (\$picea $loops) { extract(\$___using, \EXTR_SKIP); ?>
+                        <?php (\$___block ?? \$this)->slotIsSet($name) || \$___block->setSlot($name, function($definition array \$___using = []) use (\$picea $loops) { extract(\$___using, \EXTR_SKIP); ?>
                     PHP;
                 }
-
-            case "/slot":
+                    
             case "endslot":
-                $def = ( $slotDefinitions[$depth] ?? end($slotDefinitions) );
-
+                $def =end($slotDefinitions);
+                
                 if ($def->hasDefinitions() ) {
                     $definition = $def->getCurrentSlotDefinitionVars();
-
+                    
                     if ($definition) {
                         $definition .= ",";
                     }
 
                     return <<<PHP
-                    <?php })->call(\$this, $definition array_merge(get_defined_vars(), \$this->using)); ?>
+                        <?php })->call(\$this, $definition array_merge(get_defined_vars(), \$this->using)); ?>
                     PHP;
                 }
                 else {
                     return "<?php }); ?>";
                 }
-
+                
             case "using":
                 return "<?php \$___block->setUsing($arguments); ?>";
         }
     }
-
+    
+    
     public function inlineHtml(? object $proxy, string $viewPath, ... $variables) {
         return $this->renderHtml($viewPath, [ 'inlineVariables' => $variables ], $proxy);
     }
-
+    
     public static function parseSlotArguments(Callable $method) : array
     {
-        #return
+        #return 
     }
-
-
+        
+        
     public static function parseArguments(Callable $method, array $arguments) : array
     {
         try{
@@ -125,10 +119,10 @@ class BlockToken implements ControlStructure {
         }
         catch(\TypeError $ex) {
             throw $ex;
-        }
-
+        } 
+        
         $parameters = [];
-
+        
         foreach((new \ReflectionFunction($method))->getParameters() as $key => $value) {
             if ( isset($arguments[$key]) ) {
                 $parameters[ $value->getName() ] = $arguments[$key];
@@ -143,119 +137,115 @@ class BlockToken implements ControlStructure {
                 $parameters[ $value->getName() ] = null;
             }
         }
-
+        
         return $parameters;
     }
-
+    
     public static function slotDefinitions() {
         return new class() {
-
+            
             public array $definitions = [];
-
+            
             public array $variables = [];
-
+            
             public bool $rendering = false;
-
+            
             public string $currentSlot;
-
-            public function setDefinition(string $name, string $definition) : self
+            
+            public function setDefinition(string $name, string $definition) : self 
             {
                 $this->definitions[$name] = $definition;
-
+                
                 return $this;
             }
-
+            
             public function setDefinitionVars(string $name, string $variables) : self
             {
                 $this->variables[$name] = $variables;
-
+                
                 return $this;
             }
-
+            
             public function getDefinitionVars(string $name) : string
             {
                 return $this->variables[$name] ?? "";
             }
-
+            
             public function getCurrentSlotDefinitionVars() : string
             {
                 return $this->getDefinitionVars($this->currentSlot);
             }
-
+            
             public function hasDefinitions() : bool
             {
                 return count($this->definitions) > 0;
             }
-
+            
             public function printDefinition(string $name) : string
             {
                 if ( ! isset($this->definitions[$name]) ) {
-                    throw new \Exception("Slot definition for `$name` was not found. Have you defined it in your block header using something like '{% define \"$name\", ...\$arguments %}' ?");
+                    throw new \Exception("Slot definition for `$name` was not found. Have you defined it in your block header ?");
                 }
-
+                
                 return $this->definitions[$name];
             }
-
+            
             public function render() : void
             {
                 $this->rendering = true;
             }
+            
         };
     }
-
-    public static function instanciateBlock(string $viewPath, ... $arguments)
+    
+    public static function instanciateBlock(string $viewPath, ... $arguments) 
     {
         return new class($viewPath, ...$arguments) {
-
+            
             public bool $rendering = false;
-
+            
             public string $viewPath;
-
+            
             public array $using = [];
-
+            
             public array $arguments = [];
-
+            
             public array $slots = [];
-
+            
             public array $definition = [];
-
+            
             public function __construct(string $viewPath, ...$arguments) {
                 $this->viewPath = $viewPath;
                 $this->arguments = $arguments;
             }
-
+            
             public function render(object $classTemplate) : string
             {
                 $this->rendering = true;
 
-                if ($this->using['this'] ?? false) {
-                    $thisProxy = $this->using['this'];
-                    unset($this->using['this']);
-                }
-                
-                return $classTemplate->picea->inlineBlock($thisProxy ?? $this, $this->viewPath, ...$this->arguments);
+                return $classTemplate->picea->inlineBlock($this, $this->viewPath, ...$this->arguments);
             }
-
+            
             public function setSlot(string $name, Callable $method) : void
             {
                 $this->slots[$name] = $method;
             }
-
+            
             public function defineSlot(string $name, Callable $method) : void
             {
                 $this->definition[$name] = $method;
             }
-
+            
             public function slotIsSet(string $name) : bool
             {
                 return ! empty($this->slots[$name]);
             }
-
+            
             public function printSlot(string $name, Callable $default)
             {
                 return $this->slotIsSet($name) ? $this->slots[$name] : $default;
             }
-
+            
             public function setUsing(array $variables) {
                 $this->using = $variables;
             }

src/ControlStructure/ForToken.php

@@ -4,7 +4,7 @@ namespace Picea\ControlStructure;
 
 class ForToken implements ControlStructure {
 
-    public array $token = [ "for", "endfor", "/for" ];
+    public array $token = [ "for", "endfor" ];
 
     public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string {
         switch($token) {
@@ -18,8 +18,7 @@ class ForToken implements ControlStructure {
                 ];
 
                 return "<?php for ($arguments): {$uid} = 1; ?>";
-
-            case "/for":
+                
             case "endfor":
                 $last = end($context->iterationStack);
 

src/ControlStructure/ForeachToken.php

@@ -5,17 +5,5 @@ namespace Picea\ControlStructure;
 use DI\Definition\Source\AnnotationBasedAutowiring;
 
 class ForeachToken extends AbstractLoop {
-    public array $token = [ "foreach", "endforeach", "/foreach" ];
-
-    public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string
-    {
-        switch($token) {
-            case "foreach":
-                return $this->openLoop($context, $arguments, $token);
-
-            case "endforeach":
-            case "/foreach":
-                return $this->closeLoop($context, $arguments, $token);
-        }
-    }
+    public array $token = [ "foreach", "endforeach" ];
 }

src/ControlStructure/FunctionToken.php

@@ -33,9 +33,7 @@ class FunctionToken implements ControlStructure {
 
     protected function printFunction($context, ?string $arguments) : string
     {
-        $name = trim(explode('(', $arguments, 2)[0]);
-
-        return "<?php if (! function_exists(__NAMESPACE__ . '\\$name')) { function $arguments { ?>";
+        return "<?php function $arguments { ?>";
     }
 
     protected function printReturn($context, ?string $arguments) : string
@@ -45,6 +43,6 @@ class FunctionToken implements ControlStructure {
 
     protected function printEndFunction($context) : string
     {
-        return "<?php } } ?>";
+        return "<?php } ?>";
     }
 }

src/ControlStructure/IfToken.php

@@ -4,7 +4,7 @@ namespace Picea\ControlStructure;
 
 class IfToken implements ControlStructure {
 
-    public array $token = [ "if", "else", "elseif", "endif", "/if" ];
+    public array $token = [ "if", "else", "elseif", "endif" ];
 
     public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string {
         switch($token) {
@@ -16,8 +16,7 @@ class IfToken implements ControlStructure {
             
             case "elseif":
                 return "<?php elseif ($arguments): ?>";
-
-            case "/if":
+                
             case "endif":
                 return "<?php endif ?>";
         }

src/ControlStructure/SectionToken.php

@@ -11,7 +11,7 @@ enum PrintActionEnum : string {
 
 class SectionToken implements ControlStructure {
 
-    public array $token = [ "section", "endsection", "/section" ];
+    public array $token = [ "section", "endsection" ];
 
     protected PrintActionEnum $action = PrintActionEnum::default;
 
@@ -29,7 +29,6 @@ class SectionToken implements ControlStructure {
                 return $this->printSection($context, $arguments);
 
             case "endsection":
-            case "/endsection":
                 if ( empty($context->sections) ) {
                     throw new \RuntimeException("A section closing tag {% endsection %} was found without an opening {% section %} tag");
                 }

src/ControlStructure/SwitchToken.php

@@ -23,13 +23,11 @@ class SwitchToken implements ControlStructure {
                 }
 
                 return ( $output ?? "" ) . "case $arguments: ?>";
-
+            
             case "endcase":
-            case "/case":
                 return "<?php break; ?>";
                 
             case "endswitch":
-            case "/switch":
                 return "<?php endswitch; ?>"; 
         }
     }

src/ControlStructure/TryCatchToken.php

@@ -1,25 +0,0 @@
-<?php
-
-namespace Picea\ControlStructure;
-
-class TryCatchToken implements ControlStructure {
-
-    public array $token = [ "try", "catch", "finally", "endtry", "/try" ];
-
-    public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string {
-        switch($token) {
-            case "try":
-                return "<?php try { ?>";
-                
-            case "catch":
-                return "<?php } catch($arguments) { ?>";
-
-            case "finally":
-                return "<?php } finally { ?>";
-
-            case "endtry":
-            case "/try":
-                return "<?php } ?>";
-        }
-    }
-}

src/ControlStructure/WhileToken.php

@@ -3,7 +3,7 @@
 namespace Picea\ControlStructure;
 
 class WhileToken extends AbstractLoop {
-    public array $token = [ "do", "while", "endwhile", "/while" ];
+    public array $token = [ "do", "while", "endwhile", ];
 
     public function parse(\Picea\Compiler\Context &$context, ?string $arguments, string $token, array $options = []) : string {
         switch($token) {
@@ -23,13 +23,10 @@ class WhileToken extends AbstractLoop {
                         return "<?php } while($arguments); ?>";
                     }
                 }
-
-                return $this->openLoop($context, $arguments, $token);
-
-            case "endwhile":
-            case "/while":
-                return $this->closeLoop($context, $arguments, $token);
         }
 
+        return parent::parse($context, $arguments, $token);
     }
 }
+
+

src/Extension/UrlExtension.php

@@ -30,8 +30,8 @@ PATTERN;
 
     public array $tokens = [ "url" , "route", "asset", "slug" ];
 
-    ##[\Deprecated]
-    public bool $enforceExistingArguments = true;
+    #[\Deprecated]
+    protected bool $enforceExistingArguments = true;
 
     public function __construct(string $urlBase = "", string $assetToken = "", array $appUrl = [], bool $forceSSL = false) {
         $this->urlBase = trim($urlBase, "/");
@@ -120,7 +120,7 @@ PATTERN;
 
     public function buildRouteUrl(string $name, array $parameters = [], bool $appendVersion = false) : string
     {
-        if ( false !== $route = $this->findRoute($name) ) {
+        if ( false !== ( $route = $this->routes[$name] ?? false ) ) {
             return $this->buildUrl($this->prepareRoute($route, $parameters), $parameters, $appendVersion);
         }
 
@@ -149,28 +149,14 @@ PATTERN;
 
     public function registerRoute(string $name, string $route, string $class, string $method, array $routeMethods) : void
     {
-        $this->routes[] = $array = [
-            'name' => $name,
+        $this->routes[$name] = [
             'route' => $route,
-            'routeMethods' => array_map('strtoupper', $routeMethods),
+            'routeMethods' => $routeMethods,
             'class' => $class,
             'classMethod' => $method,
         ];
 
-        $this->eventExecute(UrlRegisterRouteEvent::class, $name, $array);
-    }
-
-    protected function findRoute(string $name, string $method = "*") : false|array
-    {
-        foreach($this->routes as $route) {
-            if ( $route['name'] === $name ) {
-                if ($method === '*' || in_array(strtoupper($method), $route['routeMethods']) ) {
-                    return $route;
-                }
-            }
-        }
-
-        return false;
+        $this->eventExecute(UrlRegisterRouteEvent::class, $name, $this->routes[$name]);
     }
 
     /**
@@ -289,7 +275,7 @@ PATTERN;
                     if ($default ?? false) {
                         $value = $default;
                     }
-                    elseif ( $this->enforceExistingArguments && ! preg_match(sprintf("/\[\/?%s]/i", $item[0]), $route) ) {
+                    elseif ( strpos($route, "[{$matches[0][0]}]") !== false && $this->enforceExistingArguments) {
                         throw new \RuntimeException(sprintf("Error while preparing route %s : could not match variable '%s' into given arguments ( %s ) from %s::%s", $route, $variable, json_encode($arguments), $routeParam['class'], $routeParam['classMethod']));
                     }
                 }

src/Language/DefaultRegistrations.php

@@ -32,7 +32,6 @@ class DefaultRegistrations implements LanguageRegistration
 
     public function registerControlStructure(Compiler $compiler) : void
     {
-        $compiler->registerControlStructure(new \Picea\ControlStructure\TryCatchToken());
         $compiler->registerControlStructure(new \Picea\ControlStructure\NamespaceToken());
         $compiler->registerControlStructure(new \Picea\ControlStructure\UseToken());
         $compiler->registerControlStructure(new \Picea\ControlStructure\IfToken());