Add larastan

app/Console/Commands/ConvertNotifications.php

@@ -11,7 +11,7 @@ class ConvertNotifications extends Command
 
     protected $description = 'Convert notifications to new format';
 
-    public function handle()
+    public function handle(): int
     {
         $array = [];
 

app/Console/Commands/Install.php

@@ -111,6 +111,9 @@ protected function askForStar(): void
         }
     }
 
+    /**
+     * @return array<string, mixed>
+     */
     protected function getUserData(): array
     {
         return [

app/Enums/InboxWorkflow.php

@@ -11,6 +11,9 @@ enum InboxWorkflow: string
     case WithoutBoardOrProject = 'without-board-or-project';
     case WithoutBoard = 'without-board';
 
+    /**
+     * @return Collection<string, string>
+     */
     public static function getSelectOptions(): Collection
     {
         return collect(InboxWorkflow::cases())

app/Exceptions/SsoException.php

@@ -3,10 +3,11 @@
 namespace App\Exceptions;
 
 use Exception;
+use Illuminate\Http\RedirectResponse;
 
 class SsoException extends Exception
 {
-    public function render($request)
+    public function render(): RedirectResponse
     {
         return redirect()->route('login')->withErrors([
             $this->getMessage()

app/Filament/Pages/Colors.php

@@ -40,14 +40,14 @@ public function getHeading(): string|Htmlable
 
     public static function shouldRegisterNavigation(): bool
     {
-        return auth()->user()->hasRole(UserRole::Admin);
+        return auth()->user()?->hasRole(UserRole::Admin) ?? false;
     }
 
     public function mount(): void
     {
         parent::mount();
 
-        abort_unless(auth()->user()->hasRole(UserRole::Admin), 403);
+        abort_unless(auth()->user()?->hasRole(UserRole::Admin) ?? false, 403);
     }
 
     public function form(Form $form): Form

app/Filament/Pages/Settings.php

@@ -5,6 +5,7 @@
 use App\Models\Board;
 use App\Enums\UserRole;
 use App\Models\Project;
+use App\Services\OgImage;
 use Filament\Forms\Get;
 use Filament\Forms\Form;
 use Illuminate\Support\Str;
@@ -53,18 +54,21 @@ public function getHeading(): string|Htmlable
         return trans('settings.title');
     }
 
+    /**
+     * @var Collection<int, string>
+     */
     public Collection $ogImages;
 
     public static function shouldRegisterNavigation(): bool
     {
-        return auth()->user()->hasRole(UserRole::Admin);
+        return auth()->user()?->hasRole(UserRole::Admin) ?? false;
     }
 
     public function mount(): void
     {
         parent::mount();
 
-        abort_unless(auth()->user()->hasRole(UserRole::Admin), 403);
+        abort_unless(auth()->user()?->hasRole(UserRole::Admin) ?? false, 403);
 
         $this->ogImages = collect(Storage::disk('public')->allFiles())
             ->filter(
@@ -275,12 +279,10 @@ public function form(Form $form): Form
                                     Select::make('column_span')
                                         ->label(trans('settings.dashboard-items.column-span'))
                                         ->helperText(trans('settings.dashboard-items.column-span-helper-text'))
-                                        ->options(
-                                            [
-                                                1 => 1,
-                                                2 => 2,
-                                            ]
-                                        )
+                                        ->options(fn () => [
+                                            1 => 1,
+                                            2 => 2,
+                                        ])
                                         ->default(1),
 
                                     Toggle::make('must_have_project')

app/Filament/Pages/System.php

@@ -3,6 +3,7 @@
 namespace App\Filament\Pages;
 
 use App\Enums\UserRole;
+use App\Models\User;
 use Filament\Pages\Page;
 use Filament\Actions\Action;
 use App\Services\SystemChecker;
@@ -36,12 +37,18 @@ public function getHeading(): string|Htmlable
 
     public static function shouldRegisterNavigation(): bool
     {
-        return auth()->user()->hasRole(UserRole::Admin);
+        /** @var User $user */
+        $user = auth()->user();
+
+        return $user->hasRole(UserRole::Admin);
     }
 
     public function mount(): void
     {
-        abort_unless(auth()->user()->hasRole(UserRole::Admin), 403);
+        /** @var User $user */
+        $user = auth()->user();
+
+        abort_unless($user->hasRole(UserRole::Admin), 403);
     }
 
     protected function getHeaderWidgets(): array

app/Filament/Pages/Widgets/System/SystemInfo.php

@@ -11,14 +11,17 @@ class SystemInfo extends Widget
 
     protected int | string | array $columnSpan = 2;
 
+    /**
+     * @var array<string, mixed>
+     */
     public array $version = [
         'remoteVersion' => 0,
         'currentVersion' => 0
     ];
     public bool $isOutOfDate = false;
     public string $phpVersion = '8.1';
 
-    public function mount()
+    public function mount(): void
     {
         $systemChecker = (new SystemChecker);
 

app/Filament/Resources/ChangelogResource.php

@@ -73,7 +73,7 @@ public static function form(Form $form): Form
                     Select::make('user_id')
                         ->label(trans('resources.changelog.author'))
                         ->relationship('user', 'name')
-                        ->default(auth()->user()->id)
+                        ->default(auth()->user()?->id)
                         ->preload()
                         ->required()
                         ->searchable(),

app/Filament/Resources/CommentResource/Pages/EditComment.php

@@ -2,6 +2,7 @@
 
 namespace App\Filament\Resources\CommentResource\Pages;
 
+use App\Models\Comment;
 use Filament\Actions\Action;
 use Filament\Actions\DeleteAction;
 use Filament\Resources\Pages\EditRecord;
@@ -18,8 +19,14 @@ public function getHeaderActions(): array
                   ->label(trans('resources.item.view-public'))
                   ->color('gray')
                   ->openUrlInNewTab()
-                  ->url(fn () => route('items.show', $this->record->item) . '#comment-' . $this->record->id),
+                  ->url(fn () => route('items.show', $this->getCurrentComment()->item) . '#comment-' . $this->getCurrentComment()->id),
             DeleteAction::make(),
         ];
     }
+
+    protected function getCurrentComment(): Comment
+    {
+        /** @var Comment */
+        return $this->record;
+    }
 }

app/Filament/Resources/InboxResource.php

@@ -49,7 +49,8 @@ public static function shouldRegisterNavigation(): bool
 
     public static function getNavigationBadge(): ?string
     {
-        return Item::query()->forInbox()->count();
+        return (string) Item::query()->forInbox()->count();
+
     }
 
     public static function form(Form $form): Form

app/Livewire/Item/VoteButton.php

@@ -2,6 +2,7 @@
 
 namespace App\Livewire\Item;
 
+use App\Models\User;
 use App\Models\Vote;
 use Livewire\Component;
 use Illuminate\Support\Collection;
@@ -12,6 +13,10 @@ class VoteButton extends Component
 {
     public Model $model;
     public Vote|null $vote;
+
+    /**
+     * @var Collection<User>
+     */
     public Collection $recentVoters;
     public int $recentVotersToShow = 5;
     public bool $showSubscribeOption;

app/Livewire/Modals/Item/Search.php

@@ -3,6 +3,7 @@
 namespace App\Livewire\Modals\Item;
 
 use App\Models\Item;
+use Illuminate\Contracts\View\View;
 use Livewire\Component;
 use Filament\Tables\Table;
 use Filament\Forms\Contracts\HasForms;
@@ -29,7 +30,7 @@ public function table(Table $table): Table
             ]);
     }
 
-    public function render()
+    public function render(): View
     {
         return view('livewire.modals.item.search');
     }

app/Livewire/My.php

@@ -2,8 +2,11 @@
 
 namespace App\Livewire;
 
+use App\Models\Item;
 use Closure;
 use Filament\Tables;
+use Illuminate\Contracts\View\View;
+use Illuminate\Database\Eloquent\Model;
 use Livewire\Component;
 use Illuminate\Support\Carbon;
 use Filament\Forms\Contracts\HasForms;
@@ -16,26 +19,39 @@ class My extends Component implements HasTable, HasForms
 {
     use InteractsWithTable, InteractsWithForms;
 
-    public $type = 'default';
+    public string $type = 'default';
 
-    protected function getTableQuery(): Builder
+    /**
+     * @return Builder<Item>|null
+     */
+    protected function getTableQuery(): Builder|null
     {
         if ($this->type == 'default') {
-            return auth()->user()->items()->with('board.project')->getQuery();
+            return auth()->user()?->items()->with('board.project')->getQuery();
         }
 
         if ($this->type == 'commentedOn') {
-            return auth()->user()->commentedItems()->getQuery();
+            return auth()->user()?->commentedItems()->getQuery();
         }
 
-        return auth()->user()->votedItems()->with('board.project')->latest('votes.created_at')->getQuery();
+        return auth()->user()?->votedItems()->with('board.project')->latest('votes.created_at')->getQuery();
     }
 
+    /**
+     * Get per page select options.
+     *
+     * @return int[]
+     */
     protected function getTableRecordsPerPageSelectOptions(): array
     {
         return auth()->user()->per_page_setting ?? [5];
     }
 
+    /**
+     * Get the table columns.
+     *
+     * @return Tables\Columns\TextColumn[]
+     */
     protected function getTableColumns(): array
     {
         return [
@@ -71,7 +87,7 @@ protected function getTableRecordUrlUsing(): ?Closure
         };
     }
 
-    public function render()
+    public function render(): View
     {
         return view('livewire.my');
     }