Install Rector

filamentphp · Feb 4, 2025 · fc3ce96 · fc3ce96
1 parent 3c56920
commit fc3ce96
Show file tree

Hide file tree

Showing 66 changed files with 136 additions and 125 deletions.
diff --git a/packages/actions/database/migrations/create_exports_table.php b/packages/actions/database/migrations/create_exports_table.php
@@ -11,7 +11,7 @@
      */
     public function up(): void
     {
-        Schema::create('exports', function (Blueprint $table) {
+        Schema::create('exports', function (Blueprint $table): void {
             $table->id();
             $table->timestamp('completed_at')->nullable();
             $table->string('file_disk');

diff --git a/packages/actions/database/migrations/create_failed_import_rows_table.php b/packages/actions/database/migrations/create_failed_import_rows_table.php
@@ -11,7 +11,7 @@
      */
     public function up(): void
     {
-        Schema::create('failed_import_rows', function (Blueprint $table) {
+        Schema::create('failed_import_rows', function (Blueprint $table): void {
             $table->id();
             $table->json('data');
             $table->foreignId('import_id')->constrained()->cascadeOnDelete();

diff --git a/packages/actions/database/migrations/create_imports_table.php b/packages/actions/database/migrations/create_imports_table.php
@@ -11,7 +11,7 @@
      */
     public function up(): void
     {
-        Schema::create('imports', function (Blueprint $table) {
+        Schema::create('imports', function (Blueprint $table): void {
             $table->id();
             $table->timestamp('completed_at')->nullable();
             $table->string('file_name');

diff --git a/packages/actions/src/AssociateAction.php b/packages/actions/src/AssociateAction.php
@@ -84,7 +84,7 @@ protected function setUp(): void
                 /** @var BelongsTo $inverseRelationship */
                 $inverseRelationship = $table->getInverseRelationshipFor($record);
 
-                $this->process(function () use ($inverseRelationship, $record, $relationship) {
+                $this->process(function () use ($inverseRelationship, $record, $relationship): void {
                     $inverseRelationship->associate($relationship->getParent());
                     $record->save();
                 }, [

diff --git a/packages/actions/src/AttachAction.php b/packages/actions/src/AttachAction.php
@@ -86,7 +86,7 @@ protected function setUp(): void
                 $this->record($record);
             }
 
-            $this->process(function () use ($data, $record, $relationship) {
+            $this->process(function () use ($data, $record, $relationship): void {
                 $relationship->attach(
                     $record,
                     Arr::only($data, $relationship->getPivotColumns()),

diff --git a/packages/actions/src/Concerns/CanBeMounted.php b/packages/actions/src/Concerns/CanBeMounted.php
@@ -29,7 +29,7 @@ public function mountUsing(?Closure $callback): static
      */
     public function fillForm(array | Closure $data): static
     {
-        $this->mountUsing(function (?Schema $form) use ($data) {
+        $this->mountUsing(function (?Schema $form) use ($data): void {
             $form?->fill($this->evaluate($data));
         });
 

diff --git a/packages/actions/src/Concerns/CanExportRecords.php b/packages/actions/src/Concerns/CanExportRecords.php
@@ -108,7 +108,7 @@ protected function setUp(): void
             ...$action->getExporter()::getOptionsFormComponents(),
         ]);
 
-        $this->action(function (ExportAction | ExportBulkAction $action, array $data, Component $livewire) {
+        $this->action(function (ExportAction | ExportBulkAction $action, array $data, Component $livewire): void {
             $exporter = $action->getExporter();
 
             if ($livewire instanceof HasTable) {

diff --git a/packages/actions/src/Concerns/InteractsWithActions.php b/packages/actions/src/Concerns/InteractsWithActions.php
@@ -211,7 +211,7 @@ public function callMountedAction(array $arguments = []): mixed
             if ($this->mountedActionHasSchema(mountedAction: $action)) {
                 $action->callBeforeFormValidated();
 
-                $schema->getState(afterValidate: function (array $state) use ($action, $schemaState) {
+                $schema->getState(afterValidate: function (array $state) use ($action, $schemaState): void {
                     $action->callAfterFormValidated();
 
                     $action->formData([

diff --git a/packages/actions/src/EditAction.php b/packages/actions/src/EditAction.php
@@ -54,7 +54,7 @@ protected function setUp(): void
         });
 
         $this->action(function (): void {
-            $this->process(function (array $data, HasActions & HasSchemas $livewire, Model $record, ?Table $table) {
+            $this->process(function (array $data, HasActions & HasSchemas $livewire, Model $record, ?Table $table): void {
                 $relationship = $table?->getRelationship();
 
                 $translatableContentDriver = $livewire->makeFilamentTranslatableContentDriver();

diff --git a/packages/actions/src/Exports/Concerns/CanFormatState.php b/packages/actions/src/Exports/Concerns/CanFormatState.php
@@ -82,7 +82,7 @@ public function formatState(mixed $state): mixed
         }
 
         if (filled($suffix)) {
-            $state = $state . $suffix;
+            $state .= $suffix;
         }
 
         return $state;

diff --git a/packages/actions/src/Exports/Downloaders/CsvDownloader.php b/packages/actions/src/Exports/Downloaders/CsvDownloader.php
@@ -17,7 +17,7 @@ public function __invoke(Export $export): StreamedResponse
             abort(404);
         }
 
-        return response()->streamDownload(function () use ($disk, $directory) {
+        return response()->streamDownload(function () use ($disk, $directory): void {
             echo $disk->get($directory . DIRECTORY_SEPARATOR . 'headers.csv');
 
             flush();

diff --git a/packages/actions/src/Exports/Downloaders/XlsxDownloader.php b/packages/actions/src/Exports/Downloaders/XlsxDownloader.php
@@ -31,7 +31,7 @@ public function __invoke(Export $export): StreamedResponse
 
         $csvDelimiter = $export->exporter::getCsvDelimiter();
 
-        $writeRowsFromFile = function (string $file) use ($csvDelimiter, $disk, $writer) {
+        $writeRowsFromFile = function (string $file) use ($csvDelimiter, $disk, $writer): void {
             $csvReader = CsvReader::createFromStream($disk->readStream($file));
             $csvReader->setDelimiter($csvDelimiter);
             $csvResults = Statement::create()->process($csvReader);
@@ -41,7 +41,7 @@ public function __invoke(Export $export): StreamedResponse
             }
         };
 
-        return response()->streamDownload(function () use ($disk, $directory, $fileName, $writer, $writeRowsFromFile) {
+        return response()->streamDownload(function () use ($disk, $directory, $fileName, $writer, $writeRowsFromFile): void {
             $writer->openToBrowser($fileName);
 
             $writeRowsFromFile($directory . DIRECTORY_SEPARATOR . 'headers.csv');

diff --git a/packages/actions/src/Exports/Jobs/CreateXlsxFile.php b/packages/actions/src/Exports/Jobs/CreateXlsxFile.php
@@ -52,7 +52,7 @@ public function handle(): void
 
         $csvDelimiter = $this->exporter::getCsvDelimiter();
 
-        $writeRowsFromFile = function (string $file, ?Style $style = null) use ($csvDelimiter, $disk, $writer) {
+        $writeRowsFromFile = function (string $file, ?Style $style = null) use ($csvDelimiter, $disk, $writer): void {
             $csvReader = CsvReader::createFromStream($disk->readStream($file));
             $csvReader->setDelimiter($csvDelimiter);
             $csvResults = Statement::create()->process($csvReader);

diff --git a/packages/actions/src/Exports/Jobs/ExportCsv.php b/packages/actions/src/Exports/Jobs/ExportCsv.php
@@ -95,7 +95,7 @@ public function handle(): void
 
         $filePath = $this->export->getFileDirectory() . DIRECTORY_SEPARATOR . str_pad(strval($this->page), 16, '0', STR_PAD_LEFT) . '.csv';
 
-        DB::transaction(function () use ($csv, $filePath, $processedRows, $successfulRows) {
+        DB::transaction(function () use ($csv, $filePath, $processedRows, $successfulRows): void {
             $this->export::query()
                 ->whereKey($this->export->getKey())
                 ->lockForUpdate()

diff --git a/packages/actions/src/Exports/Jobs/PrepareCsvExport.php b/packages/actions/src/Exports/Jobs/PrepareCsvExport.php
@@ -108,7 +108,7 @@ public function handle(): void
         // in case it contains attributes that are not serializable, such as binary columns.
         $this->export->unsetRelation('user');
 
-        $dispatchRecords = function (array $records) use ($exportCsvJob, &$page, &$totalRows) {
+        $dispatchRecords = function (array $records) use ($exportCsvJob, &$page, &$totalRows): void {
             $recordsCount = count($records);
 
             if (($totalRows + $recordsCount) > $this->export->total_rows) {

diff --git a/packages/actions/src/ImportAction.php b/packages/actions/src/ImportAction.php
@@ -91,7 +91,7 @@ protected function setUp(): void
                 ->placeholder(__('filament-actions::import.modal.form.file.placeholder'))
                 ->acceptedFileTypes(['text/csv', 'text/x-csv', 'application/csv', 'application/x-csv', 'text/comma-separated-values', 'text/x-comma-separated-values', 'text/plain', 'application/vnd.ms-excel'])
                 ->rules($action->getFileValidationRules())
-                ->afterStateUpdated(function (FileUpload $component, Component $livewire, Set $set, ?TemporaryUploadedFile $state) use ($action) {
+                ->afterStateUpdated(function (FileUpload $component, Component $livewire, Set $set, ?TemporaryUploadedFile $state) use ($action): void {
                     if (! $state instanceof TemporaryUploadedFile) {
                         return;
                     }
@@ -179,7 +179,7 @@ protected function setUp(): void
                 ->visible(fn (Get $get): bool => $get('file') instanceof TemporaryUploadedFile),
         ], $action->getImporter()::getOptionsFormComponents()));
 
-        $this->action(function (ImportAction $action, array $data) {
+        $this->action(function (ImportAction $action, array $data): void {
             /** @var TemporaryUploadedFile $csvFile */
             $csvFile = $data['file'];
 
@@ -271,7 +271,7 @@ protected function setUp(): void
                     filled($jobBatchName = $importer->getJobBatchName()),
                     fn (PendingBatch $batch) => $batch->name($jobBatchName),
                 )
-                ->finally(function () use ($authGuard, $columnMap, $import, $jobConnection, $options) {
+                ->finally(function () use ($authGuard, $columnMap, $import, $jobConnection, $options): void {
                     $import->touch('completed_at');
 
                     event(new ImportCompleted($import, $columnMap, $options));
@@ -373,7 +373,7 @@ protected function setUp(): void
 
                     $csv->insertAll($exampleRows);
 
-                    return response()->streamDownload(function () use ($csv) {
+                    return response()->streamDownload(function () use ($csv): void {
                         $csv->setOutputBOM(Bom::Utf8);
 
                         echo $csv->toString();
@@ -601,7 +601,7 @@ public function getFileValidationRules(): array
         $fileRules = [
             'extensions:csv,txt',
             File::types(['csv', 'txt'])->rules([
-                function (string $attribute, mixed $value, Closure $fail) {
+                function (string $attribute, mixed $value, Closure $fail): void {
                     $csvStream = $this->getUploadedFileStream($value);
 
                     if (! $csvStream) {

diff --git a/packages/actions/src/Imports/Http/Controllers/DownloadImportFailureCsv.php b/packages/actions/src/Imports/Http/Controllers/DownloadImportFailureCsv.php
@@ -53,7 +53,7 @@ public function __invoke(Request $request, Import $import): StreamedResponse
                 'error' => $failedImportRow->validation_error ?? __('filament-actions::import.failure_csv.system_error'),
             ]));
 
-        return response()->streamDownload(function () use ($csv) {
+        return response()->streamDownload(function () use ($csv): void {
             foreach ($csv->chunk(1000) as $offset => $chunk) {
                 echo $chunk;
 

diff --git a/packages/actions/src/Imports/ImportColumn.php b/packages/actions/src/Imports/ImportColumn.php
@@ -405,7 +405,7 @@ public function getDataValidationRules(): array
         $rules = $this->evaluate($this->dataValidationRules);
 
         if ($this->hasRelationship()) {
-            $rules[] = function (string $attribute, mixed $state, Closure $fail) {
+            $rules[] = function (string $attribute, mixed $state, Closure $fail): void {
                 if (blank($state)) {
                     return;
                 }

diff --git a/packages/actions/src/Imports/Jobs/ImportCsv.php b/packages/actions/src/Imports/Jobs/ImportCsv.php
@@ -74,7 +74,7 @@ public function handle(): void
 
         $rows = is_array($this->rows) ? $this->rows : unserialize(base64_decode($this->rows));
 
-        DB::transaction(function () use (&$processedRows, $rows, &$successfulRows) {
+        DB::transaction(function () use (&$processedRows, $rows, &$successfulRows): void {
             foreach ($rows as $row) {
                 $row = $this->utf8Encode($row);
 

diff --git a/packages/actions/src/ReplicateAction.php b/packages/actions/src/ReplicateAction.php
@@ -52,7 +52,7 @@ protected function setUp(): void
         });
 
         $this->action(function () {
-            $result = $this->process(function (array $data, Model $record) {
+            $result = $this->process(function (array $data, Model $record): void {
                 $this->replica = $record->replicate($this->getExcludedAttributes());
 
                 $this->replica->fill($data);

diff --git a/packages/actions/src/RestoreBulkAction.php b/packages/actions/src/RestoreBulkAction.php
@@ -64,7 +64,7 @@ protected function setUp(): void
         $this->modalIcon(FilamentIcon::resolve('actions::restore-action.modal') ?? Heroicon::OutlinedArrowUturnLeft);
 
         $this->action(fn () => $this->processIndividualRecords(
-            static function (Model $record) {
+            static function (Model $record): void {
                 if (! method_exists($record, 'restore')) {
                     return;
                 }

diff --git a/packages/forms/src/Components/CheckboxList.php b/packages/forms/src/Components/CheckboxList.php
@@ -54,7 +54,7 @@ protected function setUp(): void
 
         $this->default([]);
 
-        $this->afterStateHydrated(static function (CheckboxList $component, $state) {
+        $this->afterStateHydrated(static function (CheckboxList $component, $state): void {
             if (is_array($state)) {
                 return;
             }
@@ -196,7 +196,7 @@ public function relationship(string | Closure | null $name = null, string | Clos
             );
         });
 
-        $this->saveRelationshipsUsing(static function (CheckboxList $component, ?array $state) use ($modifyQueryUsing) {
+        $this->saveRelationshipsUsing(static function (CheckboxList $component, ?array $state) use ($modifyQueryUsing): void {
             $relationship = $component->getRelationship();
 
             if ($modifyQueryUsing) {

diff --git a/packages/forms/src/Components/Concerns/CanBeValidated.php b/packages/forms/src/Components/Concerns/CanBeValidated.php
@@ -537,7 +537,7 @@ public function unique(string | Closure | null $table = null, string | Closure |
     public function distinct(bool | Closure $condition = true): static
     {
         $this->rule(static function (Field $component, mixed $state) {
-            return function (string $attribute, mixed $value, Closure $fail) use ($component, $state) {
+            return function (string $attribute, mixed $value, Closure $fail) use ($component, $state): void {
                 if (blank($state)) {
                     return;
                 }

diff --git a/packages/forms/src/Components/Concerns/CanFixIndistinctState.php b/packages/forms/src/Components/Concerns/CanFixIndistinctState.php
@@ -14,7 +14,7 @@ public function fixIndistinctState(bool | Closure $condition = true): static
         $this->distinct($condition);
         $this->live(condition: $condition);
 
-        $this->afterStateUpdated(static function (Component $component, mixed $state, Set $set) use ($condition) {
+        $this->afterStateUpdated(static function (Component $component, mixed $state, Set $set) use ($condition): void {
             if (! $component->evaluate($condition)) {
                 return;
             }

diff --git a/packages/forms/src/Components/MorphToSelect.php b/packages/forms/src/Components/MorphToSelect.php
@@ -85,7 +85,7 @@ public function getDefaultChildComponents(): array
                 ->native($this->isNative())
                 ->required($isRequired)
                 ->live()
-                ->afterStateUpdated(function (Set $set) use ($keyColumn) {
+                ->afterStateUpdated(function (Set $set) use ($keyColumn): void {
                     $set($keyColumn, null);
                     $this->callAfterStateUpdated();
                 }),
@@ -112,7 +112,7 @@ public function getDefaultChildComponents(): array
                     $this->isLive(),
                     fn (Select $component) => $component->live(onBlur: $this->isLiveOnBlur()),
                 )
-                ->afterStateUpdated(function () {
+                ->afterStateUpdated(function (): void {
                     $this->callAfterStateUpdated();
                 }),
         ];

diff --git a/packages/forms/src/Components/Repeater.php b/packages/forms/src/Components/Repeater.php
@@ -913,21 +913,21 @@ public function relationship(string | Closure | null $name = null, ?Closure $mod
         $this->relationship = $name ?? $this->getName();
         $this->modifyRelationshipQueryUsing = $modifyQueryUsing;
 
-        $this->afterStateHydrated(function (Repeater $component) {
+        $this->afterStateHydrated(function (Repeater $component): void {
             if (! is_array($component->hydratedDefaultState)) {
                 return;
             }
 
             $component->mergeHydratedDefaultStateWithItemsState();
         });
 
-        $this->loadStateFromRelationshipsUsing(static function (Repeater $component) {
+        $this->loadStateFromRelationshipsUsing(static function (Repeater $component): void {
             $component->clearCachedExistingRecords();
 
             $component->fillFromRelationship();
         });
 
-        $this->saveRelationshipsUsing(static function (Repeater $component, HasForms $livewire, ?array $state) {
+        $this->saveRelationshipsUsing(static function (Repeater $component, HasForms $livewire, ?array $state): void {
             if (! is_array($state)) {
                 $state = [];
             }

diff --git a/packages/forms/src/Components/Select.php b/packages/forms/src/Components/Select.php
@@ -286,7 +286,7 @@ public function getCreateOptionAction(): ?Action
                     $component->getRelationship() ? $component->getRelationship()->getModel()::class : null,
                 ));
             })
-            ->action(static function (Action $action, array $arguments, Select $component, array $data, Schema $form) {
+            ->action(static function (Action $action, array $arguments, Select $component, array $data, Schema $form): void {
                 if (! $component->getCreateOptionUsing()) {
                     throw new Exception("Select field [{$component->getStatePath()}] must have a [createOptionUsing()] closure set.");
                 }
@@ -442,7 +442,7 @@ public function getEditOptionAction(): ?Action
                 );
             })
             ->fillForm($this->getEditOptionActionFormData())
-            ->action(static function (Action $action, array $arguments, Select $component, array $data, Schema $form) {
+            ->action(static function (Action $action, array $arguments, Select $component, array $data, Schema $form): void {
                 if (! $component->getUpdateOptionUsing()) {
                     throw new Exception("Select field [{$component->getStatePath()}] must have a [updateOptionUsing()] closure set.");
                 }
@@ -998,7 +998,7 @@ public function relationship(string | Closure | null $name = null, string | Clos
                 ->toArray();
         });
 
-        $this->saveRelationshipsUsing(static function (Select $component, Model $record, $state) use ($modifyQueryUsing) {
+        $this->saveRelationshipsUsing(static function (Select $component, Model $record, $state) use ($modifyQueryUsing): void {
             $relationship = $component->getRelationship();
 
             if (
@@ -1075,7 +1075,7 @@ public function relationship(string | Closure | null $name = null, string | Clos
             return $component->getSelectedRecord()?->attributesToArray();
         });
 
-        $this->updateOptionUsing(static function (array $data, Schema $form) {
+        $this->updateOptionUsing(static function (array $data, Schema $form): void {
             $form->getRecord()?->update($data);
         });
 

diff --git a/packages/infolists/src/Components/Concerns/CanFormatState.php b/packages/infolists/src/Components/Concerns/CanFormatState.php
@@ -412,7 +412,7 @@ public function formatState(mixed $state): mixed
                 $suffix = e($suffix);
             }
 
-            $state = $state . $suffix;
+            $state .= $suffix;
         }
 
         return $isHtml ? new HtmlString($state) : $state;

diff --git a/packages/notifications/src/NotificationsServiceProvider.php b/packages/notifications/src/NotificationsServiceProvider.php
@@ -36,7 +36,7 @@ public function packageBooted(): void
 
         Livewire::component('notifications', Notifications::class);
 
-        on('dehydrate', function (Component $component) {
+        on('dehydrate', function (Component $component): void {
             if (! Livewire::isLivewireRequest()) {
                 return;
             }