- 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,8 +4,11 @@ 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
     {
+        switch($token) {
+            case "while":
+            case "foreach":
                 $name = "$".uniqid("{$token}_");
                 
                 $stack = array_filter($context->iterationStack ?? [], function($item) {
@@ -24,11 +27,10 @@ abstract class AbstractLoop implements ControlStructure {
                     'token' => "end{$token}",
                 ];
 
-        return "<?php $token ($arguments): \$__loop_index = $name = ( $name ?? 0 ) + 1; ?>";
-    }
+                return "<?php $token ($arguments): $name = ( $name ?? 0 ) + 1; ?>";
 
-    protected function closeLoop(\Picea\Compiler\Context &$context, ?string $arguments, string $token,) : string
-    {
+            case "endwhile":
+            case "endforeach":
                 $last = end($context->iterationStack);
 
                 if ( $last['or'] === false ) {
@@ -43,4 +45,4 @@ abstract class AbstractLoop implements ControlStructure {
                 return $output;
         }
     }
- 
+}

src/ControlStructure/BlockToken.php

@@ -4,25 +4,19 @@ 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":
@@ -46,14 +40,14 @@ class BlockToken implements ControlStructure {
             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;
                     
             case "slot":
-                $def = ( $slotDefinitions[$depth] ?? end($slotDefinitions) );
+                $def = end($slotDefinitions);
                 
                 list($name, $definition) = array_pad(explode(',', $arguments, 2), 2, "");
 
@@ -80,13 +74,12 @@ class BlockToken implements ControlStructure {
                     }
                     
                     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();
@@ -108,6 +101,7 @@ class BlockToken implements ControlStructure {
         }
     }
     
+    
     public function inlineHtml(? object $proxy, string $viewPath, ... $variables) {
         return $this->renderHtml($viewPath, [ 'inlineVariables' => $variables ], $proxy);
     }
@@ -190,7 +184,7 @@ class BlockToken implements ControlStructure {
             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];
@@ -200,6 +194,7 @@ class BlockToken implements ControlStructure {
             {
                 $this->rendering = true;
             }
+            
         };
     }
     
@@ -228,12 +223,7 @@ class BlockToken implements ControlStructure {
             {
                 $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

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) {
@@ -19,7 +19,6 @@ 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) {
@@ -17,7 +17,6 @@ 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

@@ -25,11 +25,9 @@ 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());