Fixes for static analysis

2025-05-24 02:14:21 -04:00 · 2024-10-02 10:05:28 +10:00 · 2024-10-02 10:05:28 +10:00 · 002e4a7aab
commit 002e4a7aab
parent 42733f2679
7 changed files with 35 additions and 171 deletions
--- a/app/Console/Commands/CheckDb.php
+++ b/app/Console/Commands/CheckDb.php
@ -1,143 +0,0 @@
-<?php
-/**
- * Invoice Ninja (https://invoiceninja.com).
- *
- * @link https://github.com/invoiceninja/invoiceninja source repository
- *
- * @copyright Copyright (c) 2024. Invoice Ninja LLC (https://invoiceninja.com)
- *
- * @license https://www.elastic.co/licensing/elastic-license
- */
-
-namespace App\Console\Commands;
-
-use App\Models\Account;
-use App\Models\Activity;
-use App\Models\Backup;
-use App\Models\Client;
-use App\Models\ClientContact;
-use App\Models\ClientGatewayToken;
-use App\Models\Company;
-use App\Models\CompanyGateway;
-use App\Models\CompanyLedger;
-use App\Models\CompanyToken;
-use App\Models\CompanyUser;
-use App\Models\Credit;
-use App\Models\CreditInvitation;
-use App\Models\Design;
-use App\Models\Document;
-use App\Models\Expense;
-use App\Models\ExpenseCategory;
-use App\Models\Gateway;
-use App\Models\GroupSetting;
-use App\Models\Invoice;
-use App\Models\InvoiceInvitation;
-use App\Models\Payment;
-use App\Models\Paymentable;
-use App\Models\PaymentHash;
-use App\Models\Product;
-use App\Models\Project;
-use App\Models\Quote;
-use App\Models\QuoteInvitation;
-use App\Models\RecurringInvoice;
-use App\Models\RecurringInvoiceInvitation;
-use App\Models\Subscription;
-use App\Models\SystemLog;
-use App\Models\Task;
-use App\Models\TaskStatus;
-use App\Models\TaxRate;
-use App\Models\User;
-use App\Models\Vendor;
-use App\Models\VendorContact;
-use App\Models\Webhook;
-use Illuminate\Console\Command;
-
-/**
- * Class CheckDb.
- */
-class CheckDb extends Command
-{
-    protected $signature = 'ninja:check-db';
-
-    protected $description = 'Check MultiDB';
-
-    protected $log = '';
-
-    private $entities = [
-        Account::class,
-        Activity::class,
-        Backup::class,
-        Client::class,
-        ClientContact::class,
-        ClientGatewayToken::class,
-        Company::class,
-        CompanyGateway::class,
-        CompanyLedger::class,
-        CompanyToken::class,
-        CompanyUser::class,
-        Credit::class,
-        CreditInvitation::class,
-        Design::class,
-        Document::class,
-        Expense::class,
-        ExpenseCategory::class,
-        Gateway::class,
-        GroupSetting::class,
-        Invoice::class,
-        InvoiceInvitation::class,
-        Payment::class,
-        Paymentable::class,
-        PaymentHash::class,
-        Product::class,
-        Project::class,
-        Quote::class,
-        QuoteInvitation::class,
-        RecurringInvoice::class,
-        RecurringInvoiceInvitation::class,
-        Subscription::class,
-        SystemLog::class,
-        Task::class,
-        TaskStatus::class,
-        TaxRate::class,
-        User::class,
-        Vendor::class,
-        VendorContact::class,
-        WebHook::class,
-    ];
-
-    public function handle()
-    {
-        $this->LogMessage('Checking - V5_DB1');
-
-        foreach ($this->entities as $entity) {
-            $count_db_1 = $entity::on('db-ninja-01')->count();
-            $count_db_2 = $entity::on('db-ninja-02a')->count();
-
-            $diff = $count_db_1 - $count_db_2;
-
-            if ($diff != 0) {
-                $this->logMessage("{$entity} DB1: {$count_db_1} - DB2: {$count_db_2} - diff = {$diff}");
-            }
-        }
-
-        $this->LogMessage('Checking - V5_DB2');
-
-        foreach ($this->entities as $entity) {
-            $count_db_1 = $entity::on('db-ninja-02')->count();
-            $count_db_2 = $entity::on('db-ninja-01a')->count();
-
-            $diff = $count_db_1 - $count_db_2;
-
-            if ($diff != 0) {
-                $this->logMessage("{$entity} DB1: {$count_db_1} - DB2: {$count_db_2} - diff = {$diff}");
-            }
-        }
-    }
-
-    private function logMessage($str)
-    {
-        $str = date('Y-m-d h:i:s').' '.$str;
-        $this->info($str);
-        $this->log .= $str."\n";
-    }
-}
--- a/app/Http/Middleware/SetWebDb.php
+++ b/app/Http/Middleware/SetWebDb.php
@ -1,4 +1,13 @@
 <?php
+/**
+ * Invoice Ninja (https://invoiceninja.com).
+ *
+ * @link https://github.com/invoiceninja/invoiceninja source repository
+ *
+ * @copyright Copyright (c) 2024. Invoice Ninja LLC (https://invoiceninja.com)
+ *
+ * @license https://www.elastic.co/licensing/elastic-license
+ */

 namespace App\Http\Middleware;

--- a/app/Http/Middleware/ThrottleRequestsWithPredis.php
+++ b/app/Http/Middleware/ThrottleRequestsWithPredis.php
@ -1,4 +1,13 @@
 <?php
+/**
+ * Invoice Ninja (https://invoiceninja.com).
+ *
+ * @link https://github.com/invoiceninja/invoiceninja source repository
+ *
+ * @copyright Copyright (c) 2024. Invoice Ninja LLC (https://invoiceninja.com)
+ *
+ * @license https://www.elastic.co/licensing/elastic-license
+ */

 namespace App\Http\Middleware;

@ -14,7 +23,7 @@ class ThrottleRequestsWithPredis extends \Illuminate\Routing\Middleware\Throttle
     *
     * @var \Illuminate\Contracts\Redis\Factory
     */
-    protected $redis;
+    protected $redis; /** @phpstan-ignore-line */

    /**
     * The timestamp of the end of the current duration by key.
@ -38,9 +47,12 @@ class ThrottleRequestsWithPredis extends \Illuminate\Routing\Middleware\Throttle
    /** @phpstan-ignore-next-line */
    public function __construct(RateLimiter $limiter, Redis $redis)
    {
-        parent::__construct($limiter);
+        
        /** @phpstan-ignore-next-line */
-        $this->redis = \Illuminate\Support\Facades\Redis::connection('sentinel-cache');
+        parent::__construct($limiter); /** @phpstan-ignore-line */
+        
+        /** @phpstan-ignore-next-line */
+        $this->redis = \Illuminate\Support\Facades\Redis::connection('sentinel-cache'); /** @phpstan-ignore-line */
    }

    /**
--- a/app/Models/Account.php
+++ b/app/Models/Account.php
@ -55,9 +55,9 @@ use Laracasts\Presenter\PresentableTrait;
 * @property string|null $referral_code
 * @property int|null $created_at
 * @property int|null $updated_at
- * @property int $is_scheduler_running
+ * @property bool $is_scheduler_running
 * @property int|null $trial_duration
- * @property int $is_onboarding
+ * @property bool $is_onboarding
 * @property object|null $onboarding
 * @property bool $is_migrated
 * @property string|null $platform
--- a/config/database.php
+++ b/config/database.php
@ -98,22 +98,6 @@ return [
            'options'        => [],
        ],

-        'db-ninja-01a' => [
-            'driver'         => 'mysql',
-            'host'           => env('DB_HOST1', env('DB_HOST', '127.0.0.1')),
-            'database'       => env('DB_DATABASE2', env('DB_DATABASE', 'forge')),
-            'username'       => env('DB_USERNAME2', env('DB_USERNAME', 'forge')),
-            'password'       => env('DB_PASSWORD2', env('DB_PASSWORD', '')),
-            'port'           => env('DB_PORT1', env('DB_PORT', '3306')),
-            'charset'        => 'utf8mb4',
-            'collation'      => 'utf8mb4_unicode_ci',
-            'prefix'         => '',
-            'prefix_indexes' => true,
-            'strict'         => env('DB_STRICT', false),
-            'engine'         => 'InnoDB ROW_FORMAT=DYNAMIC',
-            'options'        => [],
-        ],
-
        'db-ninja-02' => [
            'driver'         => 'mysql',
            'host'           => env('DB_HOST2', env('DB_HOST', '127.0.0.1')),
@ -130,13 +114,13 @@ return [
            'options'        => [],
        ],

-        'db-ninja-02a' => [
+        'db-ninja-03' => [
            'driver'         => 'mysql',
-            'host'           => env('DB_HOST2', env('DB_HOST', '127.0.0.1')),
-            'database'       => env('DB_DATABASE1', env('DB_DATABASE', 'forge')),
-            'username'       => env('DB_USERNAME1', env('DB_USERNAME', 'forge')),
-            'password'       => env('DB_PASSWORD1', env('DB_PASSWORD', '')),
-            'port'           => env('DB_PORT2', env('DB_PORT', '3306')),
+            'host'           => env('DB_HOST3', env('DB_HOST', '127.0.0.1')),
+            'database'       => env('DB_DATABASE3', env('DB_DATABASE', 'forge')),
+            'username'       => env('DB_USERNAME3', env('DB_USERNAME', 'forge')),
+            'password'       => env('DB_PASSWORD3', env('DB_PASSWORD', '')),
+            'port'           => env('DB_PORT3', env('DB_PORT', '3306')),
            'charset'        => 'utf8mb4',
            'collation'      => 'utf8mb4_unicode_ci',
            'prefix'         => '',
--- a/phpstan.neon
+++ b/phpstan.neon
@ -6,6 +6,7 @@ parameters:
    level: 5
    paths:
        - app
+        - Modules
    excludePaths:
        analyseAndScan:
            - 'vendor'
@ -16,6 +17,7 @@ parameters:
            - 'app/DataMapper/Analytics/*'
            - 'app/PaymentDrivers/Authorize/*'
            - 'app/PaymentDrivers/AuthorizePaymentDriver.php'
+            - 'app/Http/Middleware/ThrottleRequestsWithPredis.php'
            - 'app/Utils/Traits/*'
    universalObjectCratesClasses:
        - App\DataMapper\Tax\RuleInterface