Bump up phpstan level

.github/workflows/ci.yml

@@ -18,4 +18,5 @@ jobs:
 
   cs-stan:
     uses: ADmad/.github/.github/workflows/cs-stan.yml@master
-    secrets: inherit
+    secrets:
+      CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

phpstan.neon

@@ -2,7 +2,7 @@ includes:
 	- phpstan-baseline.neon
 
 parameters:
-	level: 6
+	level: 8
 	paths:
 		- src
 	excludePaths:

src/Action/AddAction.php

@@ -128,6 +128,7 @@ protected function _post(): ?Response
         }
 
         $saveCallback = [$this->_model(), $subject->saveMethod];
+        /** @phpstan-ignore argument.type */
         if (call_user_func($saveCallback, $subject->entity, $subject->saveOptions)) {
             return $this->_success($subject);
         }

src/Action/EditAction.php

@@ -124,6 +124,7 @@ protected function _put(string|int|null $id = null): ?Response
         );
 
         $this->_trigger('beforeSave', $subject);
+        /** @phpstan-ignore argument.type */
         if (call_user_func([$this->_model(), $this->saveMethod()], $entity, $this->saveOptions())) {
             return $this->_success($subject);
         }

src/Controller/Component/CrudComponent.php

@@ -607,7 +607,10 @@ public function defaults(string $type, array|string $name, mixed $config = null)
             return null;
         }
 
-        /** @psalm-suppress PossiblyInvalidArgument */
+        /**
+         * @psalm-suppress PossiblyInvalidArgument
+         * @phpstan-ignore argument.type
+         */
         return $this->getConfig(sprintf('%s.%s', $type, $name));
     }
 
@@ -690,6 +693,7 @@ public function getSubject(array $additional = []): Subject
      */
     protected function _loadListeners(): void
     {
+        /** @var string $name */
         foreach (array_keys($this->getConfig('listeners')) as $name) {
             $this->_loadListener($name);
         }

src/Controller/ControllerTrait.php

@@ -14,6 +14,7 @@
  * For full copyright and license information, please see the LICENSE.txt
  *
  * @property \Crud\Controller\Component\CrudComponent $Crud
+ * @phpstan-ignore trait.unused
  */
 trait ControllerTrait
 {

src/Error/ExceptionRenderer.php

@@ -131,8 +131,16 @@ protected function _getQueryLog(): array
         $queryLog = [];
         $sources = ConnectionManager::configured();
         foreach ($sources as $source) {
-            $logger = ConnectionManager::get($source)->getDriver()->getLogger();
-            if ($logger && method_exists($logger, 'getLogs')) {
+            $driver = ConnectionManager::get($source)->getDriver();
+            if (!method_exists($driver, 'getLogger')) {
+                continue;
+            }
+
+            $logger = $driver->getLogger();
+            if (method_exists($logger, 'getLogs')) {
+                /**
+                 * @var \Crud\Log\QueryLogger $logger
+                 */
                 $queryLog[$source] = $logger->getLogs();
             }
         }

src/Listener/ApiListener.php

@@ -281,6 +281,7 @@ protected function _ensureData(Subject $subject): void
                 }
 
                 if (method_exists($subject->entity, $valuePath)) {
+                    /** @phpstan-ignore argument.type */
                     $data = Hash::insert($data, $keyPath, call_user_func([$subject->entity, $valuePath]));
                 } elseif (isset($subject->entity->{$valuePath})) {
                     $data = Hash::insert($data, $keyPath, $subject->entity->{$valuePath});

src/Listener/ApiQueryLogListener.php

@@ -9,6 +9,7 @@
 use Cake\Datasource\Exception\MissingDatasourceConfigException;
 use Cake\Event\EventInterface;
 use Crud\Log\QueryLogger;
+use Psr\Log\LoggerAwareInterface;
 
 /**
  * When loaded Crud API will include query logs in the response
@@ -64,8 +65,10 @@ public function setupLogging(EventInterface $event): void
 
         foreach ($connections as $connectionName) {
             try {
-                $connection = $this->_getSource($connectionName);
-                $connection->getDriver()->setLogger(new QueryLogger());
+                $driver = $this->_getSource($connectionName)->getDriver();
+                if ($driver instanceof LoggerAwareInterface) {
+                    $driver->setLogger(new QueryLogger());
+                }
             } catch (MissingDatasourceConfigException $e) {
                 //Safe to ignore this :-)
             }
@@ -99,8 +102,16 @@ protected function _getQueryLogs(): array
 
         $queryLog = [];
         foreach ($sources as $source) {
-            $logger = $this->_getSource($source)->getDriver()->getLogger();
+            $driver = $this->_getSource($source)->getDriver();
+            if (!method_exists($driver, 'getLogger')) {
+                continue;
+            }
+
+            $logger = $driver->getLogger();
             if (method_exists($logger, 'getLogs')) {
+                /**
+                 * @var \Crud\Log\QueryLogger $logger
+                 */
                 $queryLog[$source] = $logger->getLogs();
             }
         }

src/Traits/FindMethodTrait.php

@@ -66,6 +66,7 @@ protected function _findRecord(string|int|null $id, Subject $subject): EntityInt
         /**
          * @psalm-suppress PossiblyInvalidArgument
          * @psalm-suppress InvalidArrayOffset
+         * @phpstan-ignore argument.type
          */
         $query->where([current($query->aliasField($repository->getPrimaryKey())) => $id]);