chore: bumped

Author: roadiz-ci

.github/workflows/run-test.yml

@@ -20,7 +20,7 @@ jobs:
         runs-on: ubuntu-latest
         strategy:
             matrix:
-                php-version: ['8.3', '8.4']
+                php-version: ['8.1', '8.2', '8.3']
         steps:
             - uses: shivammathur/setup-php@v2
               with:
@@ -36,5 +36,7 @@ jobs:
                       ${{ runner.os }}-php-${{ matrix.php-version }}-
             - name: Install Dependencies
               run: composer install --no-scripts --no-ansi --no-interaction --no-progress
+            - name: Run PHP Code Sniffer
+              run: vendor/bin/phpcs -p ./src
             - name: Run PHPStan
               run: vendor/bin/phpstan analyse --no-progress -c phpstan.neon

LICENSE.md

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright © 2025 Ambroise Maupate
+Copyright © 2024 Ambroise Maupate
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the “Software”), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
 

Makefile

@@ -0,0 +1,4 @@
+test:
+	vendor/bin/phpcbf --report=full --report-file=./report.txt -p ./
+	vendor/bin/phpstan analyse -c phpstan.neon
+

composer.json

@@ -3,12 +3,12 @@
     "description": "Roadiz sub-package which generates Typescript interfaces skeleton based on your schema",
     "type": "library",
     "require": {
-        "roadiz/nodetype-contracts": "^3.1.1",
-        "symfony/http-foundation": "7.4.*"
+        "roadiz/nodetype-contracts": "~1.1.2",
+        "symfony/http-foundation": "6.4.*"
     },
     "require-dev": {
-        "phpstan/phpstan": "^2.1.36",
-        "phpstan/phpdoc-parser": "<2"
+        "phpstan/phpstan": "^1.5.3",
+        "squizlabs/php_codesniffer": "^3.5"
     },
     "license": "MIT",
     "authors": [
@@ -26,8 +26,8 @@
     },
     "extra": {
         "branch-alias": {
-            "dev-master": "2.7.x-dev",
-            "dev-develop": "2.8.x-dev"
+            "dev-master": "2.3.x-dev",
+            "dev-develop": "2.4.x-dev"
         }
     }
 }

phpcs.xml.dist

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ruleset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="vendor/squizlabs/php_codesniffer/phpcs.xsd">
+
+    <arg name="basepath" value="."/>
+    <arg name="cache" value=".phpcs-cache"/>
+    <arg name="colors"/>
+    <arg name="extensions" value="php"/>
+
+    <rule ref="PSR12">
+        <exclude name="Generic.Files.LineLength"/>
+    </rule>
+    <file>./src</file>
+    <exclude-pattern>*/node_modules</exclude-pattern>
+    <exclude-pattern>*/.AppleDouble</exclude-pattern>
+    <exclude-pattern>*/vendor</exclude-pattern>
+</ruleset>

src/DeclarationGeneratorFactory.php

@@ -4,7 +4,6 @@
 
 namespace RZ\Roadiz\Typescript\Declaration;
 
-use RZ\Roadiz\Contracts\NodeType\NodeTypeClassLocatorInterface;
 use RZ\Roadiz\Contracts\NodeType\NodeTypeFieldInterface;
 use RZ\Roadiz\Contracts\NodeType\NodeTypeInterface;
 use RZ\Roadiz\Typescript\Declaration\Generators\AbstractFieldGenerator;
@@ -16,14 +15,15 @@
 use RZ\Roadiz\Typescript\Declaration\Generators\ScalarFieldGenerator;
 use Symfony\Component\HttpFoundation\ParameterBag;
 
-final readonly class DeclarationGeneratorFactory
+final class DeclarationGeneratorFactory
 {
-    public function __construct(
-        private ParameterBag $nodeTypesBag,
-        private NodeTypeClassLocatorInterface $nodeTypeClassLocator,
-    ) {
+    public function __construct(private readonly ParameterBag $nodeTypesBag)
+    {
     }
 
+    /**
+     * @return ParameterBag
+     */
     public function getNodeTypesBag(): ParameterBag
     {
         return $this->nodeTypesBag;
@@ -34,20 +34,29 @@ public function getHumanBool(bool $bool): string
         return $bool ? 'true' : 'false';
     }
 
+    /**
+     * @param NodeTypeInterface $nodeType
+     *
+     * @return NodeTypeGenerator
+     */
     public function createForNodeType(NodeTypeInterface $nodeType): NodeTypeGenerator
     {
         return new NodeTypeGenerator(
             $nodeType,
-            $this,
-            $this->nodeTypeClassLocator
+            $this
         );
     }
 
+    /**
+     * @param NodeTypeFieldInterface $field
+     *
+     * @return AbstractFieldGenerator
+     */
     public function createForNodeTypeField(NodeTypeFieldInterface $field): AbstractFieldGenerator
     {
         return match (true) {
             $field->isDocuments() => new DocumentsFieldGenerator($field, $this->nodeTypesBag),
-            $field->isNodes() => new NodeReferencesFieldGenerator($this->nodeTypeClassLocator, $field, $this->nodeTypesBag),
+            $field->isNodes() => new NodeReferencesFieldGenerator($field, $this->nodeTypesBag),
             $field->isChildrenNodes() => new ChildrenNodeFieldGenerator($field, $this->nodeTypesBag),
             $field->isMultiple(), $field->isEnum() => new EnumFieldGenerator($field, $this->nodeTypesBag),
             default => new ScalarFieldGenerator($field, $this->nodeTypesBag),

src/Generators/AbstractFieldGenerator.php

@@ -12,10 +12,19 @@ abstract class AbstractFieldGenerator
     // Four spaces
     public const INDENTATION_MARK = '    ';
 
+    protected NodeTypeFieldInterface $field;
+    protected ParameterBag $nodeTypesBag;
+
+    /**
+     * @param NodeTypeFieldInterface $field
+     * @param ParameterBag $nodeTypesBag
+     */
     public function __construct(
-        protected readonly NodeTypeFieldInterface $field,
-        protected readonly ParameterBag $nodeTypesBag,
+        NodeTypeFieldInterface $field,
+        ParameterBag $nodeTypesBag
     ) {
+        $this->field = $field;
+        $this->nodeTypesBag = $nodeTypesBag;
     }
 
     protected function getNullableAssertion(): string
@@ -27,18 +36,18 @@ abstract protected function getType(): string;
 
     private function getDeclaration(): string
     {
-        return static::INDENTATION_MARK.
-            $this->field->getVarName().
-            $this->getNullableAssertion().': '.
+        return static::INDENTATION_MARK .
+            $this->field->getVarName() .
+            $this->getNullableAssertion() . ': ' .
             $this->getType();
     }
 
     public function getContents(): string
     {
         return implode(PHP_EOL, [
-            $this->getIntroduction(),
-            $this->getDeclaration(),
-        ]);
+                $this->getIntroduction(),
+                $this->getDeclaration()
+            ]);
     }
 
     protected function getIntroductionLines(): array
@@ -51,14 +60,19 @@ protected function getIntroductionLines(): array
         }
 
         if (!empty($this->field->getGroupName())) {
-            $lines[] = 'Group: '.$this->field->getGroupName();
+            $lines[] = 'Group: ' . $this->field->getGroupName();
         }
 
         return $lines;
     }
 
+    /**
+     * @return string
+     */
     public function getIntroduction(): string
     {
-        return implode(PHP_EOL, array_map(fn (string $line) => static::INDENTATION_MARK.'// '.$line, $this->getIntroductionLines()));
+        return implode(PHP_EOL, array_map(function (string $line) {
+            return static::INDENTATION_MARK . '// ' . $line;
+        }, $this->getIntroductionLines()));
     }
 }

src/Generators/ChildrenNodeFieldGenerator.php

@@ -6,28 +6,23 @@
 
 final class ChildrenNodeFieldGenerator extends AbstractFieldGenerator
 {
-    #[\Override]
     public function getContents(): string
     {
         return implode(PHP_EOL, [
             $this->getIntroduction(),
         ]);
     }
-
-    #[\Override]
     protected function getIntroductionLines(): array
     {
         $lines = [
-            'This node-type uses "blocks" which are available through parent RoadizNodesSources.blocks',
+            'This node-type uses "blocks" which are available through parent RoadizNodesSources.blocks'
         ];
-        if (!empty($this->field->getDefaultValuesAsArray())) {
-            $lines[] = 'Possible block node-types: '.json_encode($this->field->getDefaultValuesAsArray());
+        if (!empty($this->field->getDefaultValues())) {
+            $lines[] = 'Possible block node-types: ' . $this->field->getDefaultValues();
         }
-
         return $lines;
     }
 
-    #[\Override]
     protected function getType(): string
     {
         return '';

src/Generators/DeclarationGenerator.php

@@ -4,7 +4,6 @@
 
 namespace RZ\Roadiz\Typescript\Declaration\Generators;
 
-use RZ\Roadiz\Contracts\NodeType\NodeTypeClassLocatorInterface;
 use RZ\Roadiz\Contracts\NodeType\NodeTypeInterface;
 use RZ\Roadiz\Typescript\Declaration\DeclarationGeneratorFactory;
 
@@ -16,12 +15,12 @@ final class DeclarationGenerator
     private array $nodeTypes;
 
     /**
+     * @param DeclarationGeneratorFactory $generatorFactory
      * @param NodeTypeInterface[] $nodeTypes
      */
     public function __construct(
         private readonly DeclarationGeneratorFactory $generatorFactory,
-        private readonly NodeTypeClassLocatorInterface $nodeTypeClassLocator,
-        array $nodeTypes = [],
+        array $nodeTypes = []
     ) {
         if (empty($nodeTypes)) {
             $this->nodeTypes = array_unique($this->generatorFactory->getNodeTypesBag()->all());
@@ -42,12 +41,14 @@ public function getContents(): string
 
         $blocks[] = $this->getAllTypesInterface();
 
-        return implode(PHP_EOL.PHP_EOL, $blocks);
+        return implode(PHP_EOL . PHP_EOL, $blocks);
     }
 
     private function getAllTypesInterface(): string
     {
-        $nodeTypeNames = array_map($this->nodeTypeClassLocator->getSourceEntityClassName(...), $this->nodeTypes);
+        $nodeTypeNames = array_map(function (NodeTypeInterface $nodeType) {
+            return $nodeType->getSourceEntityClassName();
+        }, $this->nodeTypes);
 
         $nodeTypeNames = implode(' | ', $nodeTypeNames);
 

src/Generators/DocumentsFieldGenerator.php

@@ -6,13 +6,11 @@
 
 final class DocumentsFieldGenerator extends AbstractFieldGenerator
 {
-    #[\Override]
     protected function getNullableAssertion(): string
     {
         return ''; // always available even if empty
     }
 
-    #[\Override]
     protected function getType(): string
     {
         return 'Array<RoadizDocument>';