Allow arrays in extension attributes

Author: shochdoerfer

src/bitExpert/PHPStan/Magento/Autoload/DataProvider/ExtensionAttributeDataProvider.php

@@ -113,7 +113,7 @@ protected function getAttrType(DOMElement $attr): string
         $cleanType = str_replace('[]', '', $type);
 
         $primitiveTypes = ['float', 'int', 'string', 'bool', 'boolean'];
-        return in_array(strtolower($cleanType), $primitiveTypes, true) ? $cleanType : '\\'.$cleanType;
+        return in_array(strtolower($cleanType), $primitiveTypes, true) ? $type : '\\'.$type;
     }
 
     /**

src/bitExpert/PHPStan/Magento/Autoload/ExtensionAutoloader.php

@@ -18,6 +18,7 @@
 use Laminas\Code\Generator\DocBlock\Tag\ReturnTag;
 use Laminas\Code\Generator\DocBlockGenerator;
 use Laminas\Code\Generator\MethodGenerator;
+use Laminas\Code\Generator\ParameterGenerator;
 use PHPStan\Cache\Cache;
 
 class ExtensionAutoloader implements Autoloader
@@ -88,6 +89,13 @@ public function getFileContents(string $className): string
              * @see \Magento\Framework\Api\Code\Generator\ExtensionAttributesGenerator::_getClassMethods
              */
 
+            // treat array types properly in the generated code. Similar to Magento core MyInterface[] type gets
+            // converted to just an array
+            $paramType = $type;
+            if (strpos($type, '[]') !== false) {
+                $paramType = '?array';
+            }
+
             $generator->addMethodFromGenerator(
                 MethodGenerator::fromArray([
                     'name' => 'get' . ucfirst($propertyName),
@@ -101,7 +109,7 @@ public function getFileContents(string $className): string
             $generator->addMethodFromGenerator(
                 MethodGenerator::fromArray([
                     'name' => 'set' . ucfirst($propertyName),
-                    'parameters' => [$propertyName],
+                    'parameters' => [new ParameterGenerator($propertyName, $paramType)],
                     'docblock' => DocBlockGenerator::fromArray([
                         'tags' => [
                             new ParamTag($propertyName, [$type]),

src/bitExpert/PHPStan/Magento/Autoload/ExtensionInterfaceAutoloader.php

@@ -19,6 +19,7 @@
 use Laminas\Code\Generator\DocBlockGenerator;
 use Laminas\Code\Generator\InterfaceGenerator;
 use Laminas\Code\Generator\MethodGenerator;
+use Laminas\Code\Generator\ParameterGenerator;
 use PHPStan\Cache\Cache;
 
 class ExtensionInterfaceAutoloader implements Autoloader
@@ -104,6 +105,13 @@ public function getFileContents(string $interfaceName): string
              * @see \Magento\Framework\Api\Code\Generator\ExtensionAttributesGenerator::_getClassMethods
              */
 
+            // treat array types properly in the generated code. Similar to Magento core MyInterface[] type gets
+            // converted to just an array
+            $paramType = $type;
+            if (strpos($type, '[]') !== false) {
+                $paramType = '?array';
+            }
+
             $generator->addMethodFromGenerator(
                 MethodGenerator::fromArray([
                     'name' => 'get' . ucfirst($propertyName),
@@ -117,7 +125,7 @@ public function getFileContents(string $interfaceName): string
             $generator->addMethodFromGenerator(
                 MethodGenerator::fromArray([
                     'name' => 'set' . ucfirst($propertyName),
-                    'parameters' => [$propertyName],
+                    'parameters' => [new ParameterGenerator($propertyName, $paramType)],
                     'docblock' => DocBlockGenerator::fromArray([
                         'tags' => [
                             new ParamTag($propertyName, [$type]),