[FIX] phpstan findings

Author: elcreator

core/includes/aliases.inc.php

@@ -38,4 +38,5 @@ class Helper extends \EvolutionCMS\Facades\HelperProcessor {}
 class UserManager extends \EvolutionCMS\UserManager\Facades\UserManager {}
 class DocumentManager extends \EvolutionCMS\DocumentManager\Facades\DocumentManager {}
 class Tailwind extends \EvolutionCMS\Facades\Tailwind {}
+class DLTemplate extends \EvolutionCMS\Parser {}
 }

core/src/AbstractLaravel.php

@@ -14,6 +14,7 @@
 use Symfony\Component\Finder\Finder;
 use Illuminate\Log\LogServiceProvider;
 
+/** @phpstan-consistent-constructor */
 abstract class AbstractLaravel extends Container implements ApplicationContract
 {
     /**

core/src/AliasLoader.php

@@ -30,10 +30,10 @@ class AliasLoader
     /**
      * Create a new AliasLoader instance.
      *
-     * @param  array  $aliases
+     * @param array $aliases
      * @return void
      */
-    private function __construct($aliases)
+    final private function __construct(array $aliases)
     {
         $this->aliases = $aliases;
     }

core/src/Core.php

@@ -29,6 +29,7 @@
 use UrlProcessor;
 
 /**
+ * @phpstan-consistent-constructor
  * @see: https://github.com/laravel/framework/blob/5.6/src/Illuminate/Foundation/Bootstrap/LoadConfiguration.php
  * @property Mail $mail
  *      $this->loadExtension('MODxMailer');
@@ -6638,7 +6639,9 @@ public function terminating($callback)
     {}
 
     public function hasDebugModeEnabled()
-    {}
+    {
+        return false;
+    }
 
     public function getFallbackLocale()
     {}

core/src/Database.php

@@ -185,7 +185,9 @@ public function setDebug($flag)
     {
         parent::setDebug($flag);
         $driver = $this->getDriver();
+        /* @phpstan-ignore-next-line class.notFound deprecated */
         if ($driver instanceof Drivers\IlluminateDriver) {
+            /* @phpstan-ignore-next-line method.notFound deprecated */
             if ($this->isDebug()) {
                 $driver->getConnect()->enableQueryLog();
             } else {

core/src/HelperProcessor.php

@@ -51,7 +51,7 @@ public function phpThumb($input = '', $options = '', $webp = true)
         }
 
         if (empty($input) || !file_exists(MODX_BASE_PATH . $input)) {
-            $input = isset($noImage) ? $noImage : $phpThumbNoImagePath . 'noimage.png';
+            $input = $phpThumbNoImagePath . 'noimage.png';
         }
 
         /**

core/src/Models/SiteContent.php

@@ -170,6 +170,10 @@ class SiteContent extends Eloquent\Model
         'alias_visible' => 'int',
     ];
 
+    /*
+     * @phpstan-consistent-constructor
+     * @param  array<string, mixed>  $attributes
+     */
     public function __construct(array $attributes = [])
     {
         $position = $this->getPositionColumn();

core/src/Providers/FsServiceProvider.php

@@ -3,6 +3,10 @@
 use EvolutionCMS\ServiceProvider;
 use Helpers\FS;
 
+/**
+ * @deprecated
+ * @TODO: remove in 3.7
+ */
 class FsServiceProvider extends ServiceProvider
 {
     /**
@@ -13,6 +17,7 @@ class FsServiceProvider extends ServiceProvider
     public function register()
     {
         $this->app->bind('FS', function ($modx) {
+            /* @phpstan-ignore-next-line class.notFound */
             return FS::getInstance();
         });
 

core/src/Providers/ModResourceServiceProvider.php

@@ -3,6 +3,10 @@
 use EvolutionCMS\ServiceProvider;
 use modResource;
 
+/**
+ * @deprecated
+ * @TODO: remove in 3.7
+ */
 class ModResourceServiceProvider extends ServiceProvider
 {
     /**
@@ -13,6 +17,7 @@ class ModResourceServiceProvider extends ServiceProvider
     public function register()
     {
         $this->app->bind('modResource', function ($modx) {
+            /* @phpstan-ignore-next-line class.notFound */
             return new modResource($modx);
         });
 

core/src/Providers/ModUsersServiceProvider.php

@@ -3,6 +3,10 @@
 use EvolutionCMS\ServiceProvider;
 use modUsers;
 
+/**
+ * @deprecated
+ * @TODO: remove in 3.7
+ */
 class ModUsersServiceProvider extends ServiceProvider
 {
     /**
@@ -13,6 +17,7 @@ class ModUsersServiceProvider extends ServiceProvider
     public function register()
     {
         $this->app->bind('modUsers', function ($modx) {
+            /* @phpstan-ignore-next-line class.notFound */
             return new modUsers($modx);
         });