level 5

Author: saeedvaziry

app/Helpers/FTP.php

@@ -6,7 +6,7 @@
 
 class FTP
 {
-    public function connect(string $host, string $port, bool $ssl = false): bool|Connection
+    public function connect(string $host, int $port, bool $ssl = false): bool|Connection
     {
         if ($ssl) {
             return ftp_ssl_connect($host, $port, 5);

app/Models/Backup.php

@@ -46,8 +46,10 @@ public static function boot(): void
     {
         parent::boot();
 
-        static::deleting(function (Backup $backup) {
-            $backup->files()->each(function (BackupFile $file) {
+        static::deleting(function ($backup) {
+            /** @var Backup $backup */
+            $backup->files()->each(function ($file) {
+                /** @var BackupFile $file */
                 $file->delete();
             });
         });

app/Models/Database.php

@@ -41,7 +41,8 @@ public static function boot(): void
         parent::boot();
 
         static::deleting(function (Database $database) {
-            $database->server->databaseUsers()->each(function (DatabaseUser $user) use ($database) {
+            $database->server->databaseUsers()->each(function ($user) use ($database) {
+                /** @var DatabaseUser $user */
                 $databases = $user->databases;
                 if ($databases && in_array($database->name, $databases)) {
                     unset($databases[array_search($database->name, $databases)]);

app/Models/Project.php

@@ -38,7 +38,8 @@ public static function boot(): void
         parent::boot();
 
         static::deleting(function (Project $project) {
-            $project->servers()->each(function (Server $server) {
+            $project->servers()->each(function ($server) {
+                /** @var Server $server */
                 $server->delete();
             });
         });

app/Models/Server.php

@@ -31,7 +31,7 @@
  * @property string $name
  * @property string $ssh_user
  * @property string $ip
- * @property string $local_ip
+ * @property ?string $local_ip
  * @property int $port
  * @property string $os
  * @property string $type
@@ -122,14 +122,16 @@ public static function boot(): void
         static::deleting(function (Server $server) {
             DB::beginTransaction();
             try {
-                $server->sites()->each(function (Site $site) {
+                $server->sites()->each(function ($site) {
+                    /** @var Site $site */
                     $site->queues()->delete();
                     $site->ssls()->delete();
                     $site->deployments()->delete();
                     $site->deploymentScript()->delete();
                 });
                 $server->sites()->delete();
-                $server->logs()->each(function (ServerLog $log) {
+                $server->logs()->each(function ($log) {
+                    /** @var ServerLog $log */
                     $log->delete();
                 });
                 $server->services()->delete();

app/Models/Site.php

@@ -100,7 +100,10 @@ public static function boot(): void
         parent::boot();
 
         static::deleting(function (Site $site) {
-            $site->queues()->each(fn (Queue $queue) => $queue->delete());
+            $site->queues()->each(function ($queue) {
+                /** @var Queue $queue */
+                $queue->delete();
+            });
             $site->ssls()->delete();
             $site->deployments()->delete();
             $site->deploymentScript()->delete();

app/ServerProviders/Custom.php

@@ -63,7 +63,7 @@ public function create(): void
         $storageDisk = Storage::disk(config('core.key_pairs_disk'));
         File::copy(
             storage_path(config('core.ssh_private_key_name')),
-            $storageDisk->path($this->server->id)
+            $storageDisk->path((string) $this->server->id)
         );
         File::copy(
             storage_path(config('core.ssh_public_key_name')),

app/ServerProviders/Hetzner.php

@@ -67,7 +67,7 @@ public function plans(?string $region): array
                 ->json();
 
             return collect($plans['server_types'])->filter(function ($type) use ($region) {
-                return collect($type['prices'])->filter(function ($price) use ($region) {
+                return collect($type['prices'])->contains(function ($price) use ($region) {
                     return $price['location'] === $region;
                 });
             })

app/Web/Pages/Servers/Sites/Widgets/LoadBalancerServers.php

@@ -88,9 +88,11 @@ public function form(Form $form): Form
                                     ->options(function () {
                                         return $this->site->project->servers()
                                             ->where('id', '!=', $this->site->server_id)
+                                            ->whereNotNull('local_ip')
                                             ->get()
-                                            ->mapWithKeys(function (Server $server) {
-                                                return [$server->local_ip => $server->name.' ('.$server->local_ip.')'];
+                                            ->mapWithKeys(function ($server) {
+                                                /** @var Server $server */
+                                                return $server->local_ip ? [$server->local_ip => $server->name.' ('.$server->local_ip.')'] : [];
                                             });
                                     }),
                                 TextInput::make('port')

app/Web/Pages/Servers/Widgets/ServerDetails.php

@@ -63,7 +63,7 @@ public function infolist(Infolist $infolist): Infolist
                                         Notification::make()
                                             ->info()
                                             ->title('Available updates:')
-                                            ->body($record->updates)
+                                            ->body((string) $record->updates)
                                             ->send();
                                     })
                             ),