Check Util folder with phpstan (#63)

* add phpstan config and deps

* fix phpstan errors, add phpstan to ci pipeline

.github/workflows/ci.yml

@@ -63,3 +63,20 @@ jobs:
         run: |
           composer global require php-coveralls/php-coveralls
           php-coveralls --coverage_clover=build/logs/clover.xml -v
+
+  phpstan:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: 8.0
+
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Install the dependencies
+        run: composer install --no-interaction
+
+      - name: PHPStan
+        run: vendor/bin/phpstan analyse -c phpstan.neon

composer.json

@@ -31,7 +31,9 @@
     "heimrichhannot/contao-test-utilities-bundle": "^0.1",
     "phpunit/phpunit": "^8.0 || ^9.0",
     "php-coveralls/php-coveralls": "^2.0",
-    "symfony/phpunit-bridge": "^3.2 || ^4.0 || ^5.0"
+    "symfony/phpunit-bridge": "^3.2 || ^4.0 || ^5.0",
+    "phpstan/phpstan": "^1.10",
+    "phpstan/phpstan-symfony": "^1.2"
   },
   "conflict": {
     "heimrichhannot/contao-encore-bundle": "<1.5"

phpstan.neon

@@ -0,0 +1,19 @@
+parameters:
+  level: 2
+  paths:
+    - src/Util
+    - tests/Util
+#  universalObjectCratesClasses:
+#    - Contao\LayoutModel
+#    - Contao\Model
+#    - Contao\Template
+#    - PHPUnit\Framework\MockObject\MockObject
+#  ignoreErrors:
+#    - message: '#^Call to an undefined method PHPUnit\\Framework\\MockObject\\MockObject\:\:setName\(\)\.$#'
+#      path: tests/EventListener/Contao/ParseTemplateListenerTest.php
+#    - message: '#^Call to an undefined method Symfony\\Component\\Config\\Definition\\Builder\\NodeDefinition\:\:getChildNodeDefinitions\(\)\.$#'
+#      path: tests/DependencyInjection/ConfigurationTest.php
+#    - message: '#^Call to an undefined method Symfony\\Component\\Config\\Definition\\NodeInterface\:\:getChildren\(\)\.$#'
+#      path: tests/DependencyInjection/ConfigurationTest.php
+includes:
+  - vendor/phpstan/phpstan-symfony/extension.neon

src/Util/Model/ModelUtil.php

@@ -1,7 +1,7 @@
 <?php
 
 /*
- * Copyright (c) 2022 Heimrich & Hannot GmbH
+ * Copyright (c) 2023 Heimrich & Hannot GmbH
  *
  * @license LGPL-3.0-or-later
  */
@@ -81,7 +81,7 @@ public function findModelInstancesBy(string $table, $columns, $values, array $op
         ];
         $options = array_merge($defaults, $options);
 
-        /** @var string|null $modelClass */
+        /* @var string|null $modelClass */
         if (!($modelClass = $this->framework->getAdapter(Model::class)->getClassFromTable($table))) {
             return null;
         }

src/Util/Request/RequestUtil.php

@@ -1,7 +1,7 @@
 <?php
 
 /*
- * Copyright (c) 2022 Heimrich & Hannot GmbH
+ * Copyright (c) 2023 Heimrich & Hannot GmbH
  *
  * @license LGPL-3.0-or-later
  */
@@ -42,7 +42,7 @@ public function __construct(ModelUtil $modelUtil, RequestStack $requestStack, ar
     /**
      * Return the current page model.
      *
-     * @return PageModel|Model|null the current page model or null if no page context
+     * @return PageModel|null the current page model or null if no page context
      *
      * @See AbstractContentElementController::getPageModel()
      */
@@ -84,7 +84,7 @@ public function getCurrentPageModel(): ?PageModel
     /**
      * Return the root page of the current page.
      *
-     * @return PageModel|Model|null the root page model or null if not exist
+     * @return PageModel|null the root page model or null if not exist
      */
     public function getCurrentRootPageModel(PageModel $currentPage = null): ?PageModel
     {

src/Util/Request/UrlUtil.php

@@ -90,8 +90,8 @@ public function addQueryStringParameterToUrl(string $parameter, string $url = ''
      * Options:
      * - removeLeadingSlash: (boolean) Remove leading slash from path
      *
-     * @param string                          $url     The url that should be made relative
-     * @param array{removeLeadingSlash: bool} $options Pass additional options
+     * @param string $url     The url that should be made relative
+     * @param array  $options Pass additional options
      *
      * @throws InvalidUrlException
      */
@@ -101,6 +101,7 @@ public function makeUrlRelative(string $url, array $options = []): string
             'removeLeadingSlash' => false,
         ], $options);
 
+        /** @var array|false $urlParts */
         $urlParts = parse_url($url);
 
         if (false === $urlParts) {

src/Util/Type/StringUtil.php

@@ -1,7 +1,7 @@
 <?php
 
 /*
- * Copyright (c) 2022 Heimrich & Hannot GmbH
+ * Copyright (c) 2023 Heimrich & Hannot GmbH
  *
  * @license LGPL-3.0-or-later
  */
@@ -235,8 +235,6 @@ public function pregReplaceLast(string $regExp, string $subject, string $replace
 
     /**
      * Convert an xml string to array.
-     *
-     * @return mixed
      */
     public function convertXmlToArray(string $xmlData): ?array
     {

src/Util/Ui/AccordionUtil.php

@@ -1,7 +1,7 @@
 <?php
 
 /*
- * Copyright (c) 2022 Heimrich & Hannot GmbH
+ * Copyright (c) 2023 Heimrich & Hannot GmbH
  *
  * @license LGPL-3.0-or-later
  */
@@ -18,6 +18,12 @@ class AccordionUtil
      */
     private $contaoFramework;
 
+    /** @var array */
+    private $accordionStartStopCache = [];
+
+    /** @var array */
+    private $accordionSingleCache;
+
     public function __construct(ContaoFramework $contaoFramework)
     {
         $this->contaoFramework = $contaoFramework;