From 04ad327bd52d031cf9ac775465845ece195c34ea Mon Sep 17 00:00:00 2001
From: Nicos Panayides <nicosp@gmail.com>
Date: Thu, 20 Jun 2024 13:39:07 +0300
Subject: [PATCH 1/2] Update brick/var-exporter to 0.5.0 and fix deprecations.

---
 composer.json                  |  2 +-
 src/CodeGen/CodeParser.php     | 27 ++++++++++++++-------------
 src/View/Helper/BakeHelper.php |  4 ++--
 3 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/composer.json b/composer.json
index 2e718fc1..94e9c468 100644
--- a/composer.json
+++ b/composer.json
@@ -21,7 +21,7 @@
     },
     "require": {
         "php": ">=8.1",
-        "brick/varexporter": "^0.4.0",
+        "brick/varexporter": "^0.5.0",
         "cakephp/cakephp": "^5.0.3",
         "cakephp/twig-view": "^2.0.0",
         "nikic/php-parser": "^4.13.2 || ^5.0.0"
diff --git a/src/CodeGen/CodeParser.php b/src/CodeGen/CodeParser.php
index 42434ecf..528acf0c 100644
--- a/src/CodeGen/CodeParser.php
+++ b/src/CodeGen/CodeParser.php
@@ -17,18 +17,19 @@
 namespace Bake\CodeGen;
 
 use PhpParser\Error;
-use PhpParser\Lexer\Emulative;
 use PhpParser\Node;
 use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\GroupUse;
 use PhpParser\Node\Stmt\Namespace_;
 use PhpParser\Node\Stmt\Use_;
-use PhpParser\Node\Stmt\UseUse;
+use PhpParser\Node\UseItem;
 use PhpParser\NodeAbstract;
 use PhpParser\NodeTraverser;
+use PhpParser\NodeVisitor;
 use PhpParser\NodeVisitorAbstract;
 use PhpParser\Parser;
 use PhpParser\ParserFactory;
+use PhpParser\PhpVersion;
 
 /**
  * @internal
@@ -65,12 +66,8 @@ class CodeParser extends NodeVisitorAbstract
      */
     public function __construct()
     {
-        $this->parser = (new ParserFactory())->create(
-            ParserFactory::PREFER_PHP7,
-            new Emulative([
-                'usedAttributes' => ['comments', 'startLine', 'endLine', 'startFilePos', 'endFilePos'],
-            ])
-        );
+        $version = PhpVersion::fromComponents(8,1);
+        $this->parser = (new ParserFactory())->createForVersion($version);
         $this->traverser = new NodeTraverser();
         $this->traverser->addVisitor($this);
     }
@@ -137,7 +134,11 @@ public function enterNode(Node $node)
                 throw new ParseException('Multiple use statements per line are not supported, update your file');
             }
 
-            [$alias, $target] = $this->normalizeUse(current($node->uses));
+            if ($node->uses === []) {
+                throw new ParseException('Use statement without uses!');
+            }
+
+            [$alias, $target] = $this->normalizeUse($node->uses[0]);
             switch ($node->type) {
                 case Use_::TYPE_NORMAL:
                     $this->parsed['imports']['class'][$alias] = $target;
@@ -150,7 +151,7 @@ public function enterNode(Node $node)
                     break;
             }
 
-            return NodeTraverser::DONT_TRAVERSE_CHILDREN;
+            return NodeVisitor::DONT_TRAVERSE_CHILDREN;
         }
 
         if ($node instanceof GroupUse) {
@@ -203,7 +204,7 @@ public function enterNode(Node $node)
                 $methods
             );
 
-            return NodeTraverser::DONT_TRAVERSE_CHILDREN;
+            return NodeVisitor::DONT_TRAVERSE_CHILDREN;
         }
 
         return null;
@@ -230,11 +231,11 @@ protected function getNodeCode(NodeAbstract $node): string
     }
 
     /**
-     * @param \PhpParser\Node\Stmt\UseUse $use Use node
+     * @param \PhpParser\Node\UseItem $use Use item
      * @param string|null $prefix Group use prefix
      * @return array{string, string}
      */
-    protected function normalizeUse(UseUse $use, ?string $prefix = null): array
+    protected function normalizeUse(UseItem $use, ?string $prefix = null): array
     {
         $name = (string)$use->name;
         if ($prefix) {
diff --git a/src/View/Helper/BakeHelper.php b/src/View/Helper/BakeHelper.php
index 3f87c325..0777f4a5 100644
--- a/src/View/Helper/BakeHelper.php
+++ b/src/View/Helper/BakeHelper.php
@@ -98,7 +98,7 @@ public function exportArray(array $var, int $indentLevel = 0, bool $inline = tru
     {
         $options = 0;
         if ($inline) {
-            $options = VarExporter::INLINE_NUMERIC_SCALAR_ARRAY;
+            $options = VarExporter::INLINE_SCALAR_LIST;
         }
 
         return $this->exportVar($var, $indentLevel, $options);
@@ -369,7 +369,7 @@ public function getValidationMethods(string $field, array $rules): array
                 return $this->exportVar(
                     $item,
                     is_array($item) ? 3 : 0,
-                    VarExporter::INLINE_NUMERIC_SCALAR_ARRAY
+                    VarExporter::INLINE_SCALAR_LIST
                 );
             }, $rule['args']);
             $validationMethods[] = sprintf(

From f0cbbf01cdafd8890349b7c780c1111facd7ff62 Mon Sep 17 00:00:00 2001
From: Nicos Panayides <nicosp@gmail.com>
Date: Thu, 20 Jun 2024 13:44:05 +0300
Subject: [PATCH 2/2] Code style fix

---
 src/CodeGen/CodeParser.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/CodeGen/CodeParser.php b/src/CodeGen/CodeParser.php
index 528acf0c..594335ca 100644
--- a/src/CodeGen/CodeParser.php
+++ b/src/CodeGen/CodeParser.php
@@ -66,7 +66,7 @@ class CodeParser extends NodeVisitorAbstract
      */
     public function __construct()
     {
-        $version = PhpVersion::fromComponents(8,1);
+        $version = PhpVersion::fromComponents(8, 1);
         $this->parser = (new ParserFactory())->createForVersion($version);
         $this->traverser = new NodeTraverser();
         $this->traverser->addVisitor($this);