Code analyses

.gitignore

@@ -43,6 +43,7 @@ Homestead.json
 Homestead.yaml
 .phpstorm.meta.php
 _ide_helper.php
+_ide_helper_models.php
 
 # OSX
 #

app/Actions/Fortify/CreateNewUser.php

@@ -16,8 +16,9 @@ class CreateNewUser implements CreatesNewUsers
     /**
      * Validate and create a newly registered user.
      *
-     * @param array $input
+     * @param  array  $input<string, string>
      * @return \App\Models\User
+     *
      * @throws \Illuminate\Validation\ValidationException
      */
     public function create(array $input): User

app/Actions/Fortify/PasswordValidationRules.php

@@ -9,7 +9,7 @@ trait PasswordValidationRules
     /**
      * Get the validation rules used to validate passwords.
      *
-     * @return array
+     * @return array<int, \Illuminate\Validation\Rules\Password|string>
      */
     protected function passwordRules(): array
     {

app/Actions/Fortify/ResetUserPassword.php

@@ -13,9 +13,10 @@ class ResetUserPassword implements ResetsUserPasswords
     /**
      * Validate and reset the user's forgotten password.
      *
-     * @param mixed $user
-     * @param array $input
+     * @param  mixed  $user
+     * @param  array  $input<string string>
      * @return void
+     *
      * @throws \Illuminate\Validation\ValidationException
      */
     public function reset($user, array $input)

app/Actions/Fortify/UpdateUserPassword.php

@@ -13,9 +13,10 @@ class UpdateUserPassword implements UpdatesUserPasswords
     /**
      * Validate and update the user's password.
      *
-     * @param mixed $user
-     * @param array $input
+     * @param  mixed  $user
+     * @param  array  $input
      * @return void
+     *
      * @throws \Illuminate\Validation\ValidationException
      */
     public function update($user, array $input)
@@ -24,7 +25,7 @@ public function update($user, array $input)
             'current_password' => ['required', 'string'],
             'password' => $this->passwordRules(),
         ])->after(function ($validator) use ($user, $input) {
-            if (!isset($input['current_password']) || !Hash::check($input['current_password'], $user->password)) {
+            if (! isset($input['current_password']) || ! Hash::check($input['current_password'], $user->password)) {
                 $validator->errors()->add('current_password', __('Le mot de passe fourni ne correspond pas à votre mot de passe actuel.'));
             }
         })->validateWithBag('updatePassword');

app/Actions/Fortify/UpdateUserProfileInformation.php

@@ -12,9 +12,10 @@ class UpdateUserProfileInformation implements UpdatesUserProfileInformation
     /**
      * Validate and update the given user's profile information.
      *
-     * @param mixed $user
-     * @param array $input
+     * @param  mixed  $user
+     * @param  array  $input<string, string>
      * @return void
+     *
      * @throws \Illuminate\Validation\ValidationException
      */
     public function update($user, array $input)

app/Console/Commands/Cleanup/DeleteOldUnverifiedUsers.php

@@ -12,16 +12,18 @@ class DeleteOldUnverifiedUsers extends Command
 
     protected $description = 'Removed all unverified users.';
 
-    public function handle()
+    public function handle(): void
     {
         $this->info('Deleting old unverified users...');
 
         $query = User::query()
             ->whereNull('email_verified_at')
             ->where('created_at', '<', now()->subDays(10));
 
-        if ($query->get()->isNotEmpty()) {
-            foreach ($query->get() as $user) {
+        $users = $query->get();
+
+        if ($users->isNotEmpty()) {
+            foreach ($users as $user) {
                 $user->notify((new SendEMailToDeletedUser())->delay(now()->addMinutes(5)));
             }
         }

app/Console/Commands/CreateAdminUser.php

@@ -2,27 +2,18 @@
 
 namespace App\Console\Commands;
 
+use App\Models\User;
 use Illuminate\Console\Command;
 use Illuminate\Database\QueryException;
 use Illuminate\Support\Facades\Hash;
 
 class CreateAdminUser extends Command
 {
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
     protected $signature = 'lcm:admin';
 
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
     protected $description = 'Create user with admin role and all permissions.';
 
-    public function handle()
+    public function handle(): void
     {
         $this->info('Create Admin User.');
         $this->createUser();
@@ -51,10 +42,9 @@ protected function createUser(): void
             'password' => Hash::make($password),
             'email_verified_at' => now()->toDateTimeString(),
         ];
-        $model = config('auth.providers.users.model');
 
         try {
-            $user = tap((new $model)->forceFill($userData))->save();
+            $user = User::query()->create($userData);
 
             $user->assignRole('admin');
         } catch (\Exception | QueryException $e) {

app/Console/Commands/UpdateUserDiscussionsPoints.php

@@ -17,6 +17,7 @@ public function handle()
         $this->info('Updating users discussions reputations...');
 
         foreach (Discussion::all() as $discussion) {
+            // @phpstan-ignore-next-line
             givePoint(new DiscussionCreated($discussion), $discussion->author);
         }
 

app/Console/Commands/UpdateUserPostsPoints.php

@@ -16,7 +16,8 @@ public function handle()
     {
         $this->info('Updating users posts reputations...');
 
-        foreach (Article::all() as $article) {
+        foreach (Article::published()->get() as $article) {
+            // @phpstan-ignore-next-line
             givePoint(new PostCreated($article), $article->author);
         }
 

app/Console/Commands/UpdateUserThreadsPoints.php

@@ -17,6 +17,7 @@ public function handle()
         $this->info('Updating users threads reputations...');
 
         foreach (Thread::all() as $thread) {
+            // @phpstan-ignore-next-line
             givePoint(new ThreadCreated($thread), $thread->author);
         }
 

app/Console/Kernel.php

@@ -40,7 +40,7 @@ protected function schedule(Schedule $schedule)
      */
     protected function commands()
     {
-        $this->load(__DIR__ . '/Commands');
+        $this->load(__DIR__.'/Commands');
 
         require base_path('routes/console.php');
     }

app/Contracts/ReplyInterface.php

@@ -7,15 +7,15 @@
 
 interface ReplyInterface
 {
-    public function subject(): string;
+    public function subject(): int|string;
 
     public function latestReplies(int $amount = 5): Collection;
 
     public function replies(): MorphMany;
 
     public function isConversationOld(): bool;
 
-    public function replyAbleSubject(): string;
+    public function replyAbleSubject(): int|string;
 
     public function getPathUrl(): string;
 }

app/Exceptions/Handler.php

@@ -10,7 +10,7 @@ class Handler extends ExceptionHandler
     /**
      * A list of the exception types that are not reported.
      *
-     * @var array
+     * @var array<int, class-string<\Throwable>>
      */
     protected $dontReport = [
         //
@@ -19,7 +19,7 @@ class Handler extends ExceptionHandler
     /**
      * A list of the inputs that are never flashed for validation exceptions.
      *
-     * @var array
+     * @var array<int, string>
      */
     protected $dontFlash = [
         'current_password',
@@ -32,7 +32,7 @@ class Handler extends ExceptionHandler
      *
      * @return void
      */
-    public function register()
+    public function register(): void
     {
         $this->reportable(function (Throwable $e) {
             if ($this->shouldReport($e) && app()->bound('sentry')) {