diff --git a/packages/actions/database/migrations/create_exports_table.php b/packages/actions/database/migrations/create_exports_table.php index a4aab11e7cf..6a87ac32251 100644 --- 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 index 260507e858a..2f77805ee1f 100644 --- 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 index 100b7d96901..1d9c09d8387 100644 --- 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 index d7ffd4963b7..e9ecd1c39a2 100644 --- 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 index e8cfcfe2156..cc0454ecbe8 100644 --- 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 index 33b2dd16f96..20b58dfcfdc 100644 --- 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 index 6f0f04f8eee..9dbc1710c22 100644 --- 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 index d8871c63712..bb75700f307 100644 --- 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 index 37a75725b84..3b6eb3556c0 100644 --- 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 index ec1ce284888..5fc21557c7a 100644 --- 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 index 50ea98518ae..790d36c008b 100644 --- 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 index 3096d9b5eae..fdfe564d6df 100644 --- 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 index 2a85c46b3ef..46578fa7c6b 100644 --- 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 index b4dccc67cd6..5683616011d 100644 --- 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 index 00702cba061..e03640a1396 100644 --- 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 index 5682274aa55..0d7e71d9288 100644 --- 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 index f1016339780..40dbb2b5d42 100644 --- 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 index 578b23e5abb..0106e70282e 100644 --- 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 index 5ebcb788294..25a005841f0 100644 --- 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 index a13c3500238..dd46adc2f28 100644 --- 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 index 1c59878638d..929db114c2d 100644 --- 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 index f86cf5415e9..3cce3255409 100644 --- 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 index d401c7d4200..3515840ad52 100644 --- 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 index 6398481db48..397dea04679 100644 --- 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 index c7f48a273dc..1d68814089d 100644 --- 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 index 1ec883a0b41..59df0bd8a4e 100644 --- a/packages/forms/src/Components/Repeater.php +++ b/packages/forms/src/Components/Repeater.php @@ -913,7 +913,7 @@ 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; } @@ -921,13 +921,13 @@ public function relationship(string | Closure | null $name = null, ?Closure $mod $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 index b1a5fcbbd61..3b719304f44 100644 --- 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 index 71801268765..0b50083cc3d 100644 --- 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 index b8b5bfe3c16..4615509e265 100644 --- 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; } diff --git a/packages/panels/routes/web.php b/packages/panels/routes/web.php index d771ad87687..593b09e9e17 100644 --- a/packages/panels/routes/web.php +++ b/packages/panels/routes/web.php @@ -11,7 +11,7 @@ use Illuminate\Support\Facades\Route; Route::name('filament.') - ->group(function () { + ->group(function (): void { foreach (Filament::getPanels() as $panel) { /** @var Panel $panel */ $panelId = $panel->getId(); @@ -26,12 +26,12 @@ ->middleware($panel->getMiddleware()) ->name("{$panelId}." . ((filled($domain) && (count($domains) > 1)) ? "{$domain}." : '')) ->prefix($panel->getPath()) - ->group(function () use ($panel, $hasTenancy, $tenantDomain, $tenantRoutePrefix, $tenantSlugAttribute) { + ->group(function () use ($panel, $hasTenancy, $tenantDomain, $tenantRoutePrefix, $tenantSlugAttribute): void { foreach ($panel->getRoutes() as $routes) { $routes($panel); } - Route::name('auth.')->group(function () use ($panel) { + Route::name('auth.')->group(function () use ($panel): void { if ($panel->hasLogin()) { Route::get($panel->getLoginRouteSlug(), $panel->getLoginRouteAction()) ->name('login'); @@ -40,7 +40,7 @@ if ($panel->hasPasswordReset()) { Route::name('password-reset.') ->prefix($panel->getResetPasswordRoutePrefix()) - ->group(function () use ($panel) { + ->group(function () use ($panel): void { Route::get($panel->getRequestPasswordResetRouteSlug(), $panel->getRequestPasswordResetRouteAction()) ->name('request'); Route::get($panel->getResetPasswordRouteSlug(), $panel->getResetPasswordRouteAction()) @@ -73,7 +73,7 @@ if ($panel->hasEmailVerification()) { Route::name('auth.email-verification.') ->prefix($panel->getEmailVerificationRoutePrefix()) - ->group(function () use ($panel) { + ->group(function () use ($panel): void { Route::get($panel->getEmailVerificationPromptRouteSlug(), $panel->getEmailVerificationPromptRouteAction()) ->name('prompt'); Route::get($panel->getEmailVerificationRouteSlug('/{id}/{hash}'), EmailVerificationController::class) @@ -85,7 +85,7 @@ if ($panel->hasEmailChangeVerification()) { Route::name('auth.email-change-verification.') ->prefix($panel->getEmailChangeVerificationRoutePrefix()) - ->group(function () use ($panel) { + ->group(function () use ($panel): void { Route::get($panel->getEmailChangeVerificationRouteSlug('/{id}/{email}'), EmailChangeVerificationController::class) ->middleware(['signed', 'throttle:6,1']) ->name('verify'); @@ -99,7 +99,7 @@ if ($panel->hasMultiFactorAuthentication()) { Route::name('auth.multi-factor-authentication.') ->prefix($panel->getMultiFactorAuthenticationRoutePrefix()) - ->group(function () use ($panel) { + ->group(function () use ($panel): void { if ($panel->isMultiFactorAuthenticationRequired()) { Route::get($panel->getSetUpRequiredMultiFactorAuthenticationRouteSlug(), $panel->getSetUpRequiredMultiFactorAuthenticationRouteAction()) ->name('set-up-required'); diff --git a/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/DisableEmailCodeAuthenticationAction.php b/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/DisableEmailCodeAuthenticationAction.php index 29dda4d4661..c1a1b7903fc 100644 --- a/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/DisableEmailCodeAuthenticationAction.php +++ b/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/DisableEmailCodeAuthenticationAction.php @@ -22,7 +22,7 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A ->color('danger') ->icon(Heroicon::LockOpen) ->link() - ->mountUsing(function () use ($emailCodeAuthentication) { + ->mountUsing(function () use ($emailCodeAuthentication): void { /** @var HasEmailCodeAuthentication $user */ $user = Filament::auth()->user(); @@ -39,7 +39,7 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A ->belowContent(Action::make('resend') ->label(__('filament-panels::auth/multi-factor/email-code/actions/disable.modal.form.code.actions.resend.label')) ->link() - ->action(function () use ($emailCodeAuthentication) { + ->action(function () use ($emailCodeAuthentication): void { /** @var HasEmailCodeAuthentication $user */ $user = Filament::auth()->user(); @@ -63,11 +63,11 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A ]) ->modalSubmitAction(fn (Action $action) => $action ->label(__('filament-panels::auth/multi-factor/email-code/actions/disable.modal.actions.submit.label'))) - ->action(function () use ($emailCodeAuthentication) { + ->action(function () use ($emailCodeAuthentication): void { /** @var HasEmailCodeAuthentication $user */ $user = Filament::auth()->user(); - DB::transaction(function () use ($emailCodeAuthentication, $user) { + DB::transaction(function () use ($emailCodeAuthentication, $user): void { $emailCodeAuthentication->saveSecret($user, null); }); diff --git a/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/SetUpEmailCodeAuthenticationAction.php b/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/SetUpEmailCodeAuthenticationAction.php index 41860842a5b..9760f701499 100644 --- a/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/SetUpEmailCodeAuthenticationAction.php +++ b/packages/panels/src/Auth/MultiFactor/EmailCode/Actions/SetUpEmailCodeAuthenticationAction.php @@ -24,7 +24,7 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A ->color('primary') ->icon(Heroicon::LockClosed) ->link() - ->mountUsing(function (HasActions $livewire) use ($emailCodeAuthentication) { + ->mountUsing(function (HasActions $livewire) use ($emailCodeAuthentication): void { $livewire->mergeMountedActionArguments([ 'encrypted' => encrypt([ 'secret' => $secret = $emailCodeAuthentication->generateSecret(), @@ -48,7 +48,7 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A ->belowContent(Action::make('resend') ->label(__('filament-panels::auth/multi-factor/email-code/actions/set-up.modal.form.code.actions.resend.label')) ->link() - ->action(function () use ($arguments, $emailCodeAuthentication) { + ->action(function () use ($arguments, $emailCodeAuthentication): void { /** @var HasEmailCodeAuthentication $user */ $user = Filament::auth()->user(); @@ -73,7 +73,7 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A ]) ->modalSubmitAction(fn (Action $action) => $action ->label(__('filament-panels::auth/multi-factor/email-code/actions/set-up.modal.actions.submit.label'))) - ->action(function (array $arguments) use ($emailCodeAuthentication) { + ->action(function (array $arguments) use ($emailCodeAuthentication): void { /** @var Authenticatable&HasEmailCodeAuthentication $user */ $user = Filament::auth()->user(); @@ -85,7 +85,7 @@ public static function make(EmailCodeAuthentication $emailCodeAuthentication): A return; } - DB::transaction(function () use ($emailCodeAuthentication, $encrypted, $user) { + DB::transaction(function () use ($emailCodeAuthentication, $encrypted, $user): void { $emailCodeAuthentication->saveSecret($user, $encrypted['secret']); }); diff --git a/packages/panels/src/Auth/MultiFactor/EmailCode/EmailCodeAuthentication.php b/packages/panels/src/Auth/MultiFactor/EmailCode/EmailCodeAuthentication.php index 2ce65bf4924..2e667d6d935 100644 --- a/packages/panels/src/Auth/MultiFactor/EmailCode/EmailCodeAuthentication.php +++ b/packages/panels/src/Auth/MultiFactor/EmailCode/EmailCodeAuthentication.php @@ -177,7 +177,7 @@ public function getChallengeFormComponents(Authenticatable $user): array ->belowContent(Action::make('resend') ->label(__('filament-panels::auth/multi-factor/email-code/provider.login_form.code.actions.resend.label')) ->link() - ->action(function () use ($user) { + ->action(function () use ($user): void { $this->sendCode($user); Notification::make() diff --git a/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/DisableGoogleTwoFactorAuthenticationAction.php b/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/DisableGoogleTwoFactorAuthenticationAction.php index d5bb9a0b43d..fa22b6c6e60 100644 --- a/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/DisableGoogleTwoFactorAuthenticationAction.php +++ b/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/DisableGoogleTwoFactorAuthenticationAction.php @@ -74,11 +74,11 @@ public static function make(GoogleTwoFactorAuthentication $googleTwoFactorAuthen ]) ->modalSubmitAction(fn (Action $action) => $action ->label(__('filament-panels::auth/multi-factor/google-two-factor/actions/disable.modal.actions.submit.label'))) - ->action(function () use ($googleTwoFactorAuthentication, $isRecoverable) { + ->action(function () use ($googleTwoFactorAuthentication, $isRecoverable): void { /** @var HasGoogleTwoFactorAuthentication&HasGoogleTwoFactorAuthenticationRecovery $user */ $user = Filament::auth()->user(); - DB::transaction(function () use ($googleTwoFactorAuthentication, $isRecoverable, $user) { + DB::transaction(function () use ($googleTwoFactorAuthentication, $isRecoverable, $user): void { $googleTwoFactorAuthentication->saveSecret($user, null); if ($isRecoverable) { diff --git a/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/RegenerateGoogleTwoFactorAuthenticationRecoveryCodesAction.php b/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/RegenerateGoogleTwoFactorAuthenticationRecoveryCodesAction.php index 30c2b540a9f..ef5862bbbcf 100644 --- a/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/RegenerateGoogleTwoFactorAuthenticationRecoveryCodesAction.php +++ b/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/RegenerateGoogleTwoFactorAuthenticationRecoveryCodesAction.php @@ -61,7 +61,7 @@ public static function make(GoogleTwoFactorAuthentication $googleTwoFactorAuthen ->modalSubmitAction(fn (Action $action) => $action ->label(__('filament-panels::auth/multi-factor/google-two-factor/actions/regenerate-recovery-codes.modal.actions.submit.label')) ->color('danger')) - ->action(function (Action $action, HasActions $livewire) use ($googleTwoFactorAuthentication) { + ->action(function (Action $action, HasActions $livewire) use ($googleTwoFactorAuthentication): void { $recoveryCodes = $googleTwoFactorAuthentication->generateRecoveryCodes(); /** @var HasGoogleTwoFactorAuthenticationRecovery $user */ diff --git a/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/SetUpGoogleTwoFactorAuthenticationAction.php b/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/SetUpGoogleTwoFactorAuthenticationAction.php index bd5eb11070a..69785e72a39 100644 --- a/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/SetUpGoogleTwoFactorAuthenticationAction.php +++ b/packages/panels/src/Auth/MultiFactor/GoogleTwoFactor/Actions/SetUpGoogleTwoFactorAuthenticationAction.php @@ -38,7 +38,7 @@ public static function make(GoogleTwoFactorAuthentication $googleTwoFactorAuthen ->color('primary') ->icon(Heroicon::LockClosed) ->link() - ->mountUsing(function (HasActions $livewire, $action) use ($googleTwoFactorAuthentication) { + ->mountUsing(function (HasActions $livewire, $action) use ($googleTwoFactorAuthentication): void { $livewire->mergeMountedActionArguments([ 'encrypted' => encrypt([ 'secret' => $googleTwoFactorAuthentication->generateSecret(), @@ -144,7 +144,7 @@ public static function make(GoogleTwoFactorAuthentication $googleTwoFactorAuthen }) ->modalSubmitAction(fn (Action $action) => $action ->label(__('filament-panels::auth/multi-factor/google-two-factor/actions/set-up.modal.actions.submit.label'))) - ->action(function (array $arguments) use ($googleTwoFactorAuthentication) { + ->action(function (array $arguments) use ($googleTwoFactorAuthentication): void { /** @var Authenticatable&HasGoogleTwoFactorAuthentication&HasGoogleTwoFactorAuthenticationRecovery $user */ $user = Filament::auth()->user(); @@ -156,7 +156,7 @@ public static function make(GoogleTwoFactorAuthentication $googleTwoFactorAuthen return; } - DB::transaction(function () use ($googleTwoFactorAuthentication, $encrypted, $user) { + DB::transaction(function () use ($googleTwoFactorAuthentication, $encrypted, $user): void { $googleTwoFactorAuthentication->saveSecret($user, $encrypted['secret']); if ($googleTwoFactorAuthentication->isRecoverable()) { diff --git a/packages/panels/src/Auth/Pages/Login.php b/packages/panels/src/Auth/Pages/Login.php index c8da23ef9da..eb9142d2871 100644 --- a/packages/panels/src/Auth/Pages/Login.php +++ b/packages/panels/src/Auth/Pages/Login.php @@ -262,7 +262,7 @@ protected function getMultiFactorProviderFormComponent(): ?Component $enabledMultiFactorAuthenticationProviders, )) ->live() - ->afterStateUpdated(function (?string $state) use ($enabledMultiFactorAuthenticationProviders, $section, $user) { + ->afterStateUpdated(function (?string $state) use ($enabledMultiFactorAuthenticationProviders, $section, $user): void { $provider = $enabledMultiFactorAuthenticationProviders[$state] ?? null; if (! $provider) { diff --git a/packages/panels/src/Auth/Pages/PasswordReset/ResetPassword.php b/packages/panels/src/Auth/Pages/PasswordReset/ResetPassword.php index cbe57284fc5..efef88c4976 100644 --- a/packages/panels/src/Auth/Pages/PasswordReset/ResetPassword.php +++ b/packages/panels/src/Auth/Pages/PasswordReset/ResetPassword.php @@ -76,7 +76,7 @@ public function resetPassword(): ?PasswordResetResponse $status = Password::broker(Filament::getAuthPasswordBroker())->reset( $data, - function (CanResetPassword | Model | Authenticatable $user) use ($data) { + function (CanResetPassword | Model | Authenticatable $user) use ($data): void { $user->forceFill([ 'password' => Hash::make($data['password']), 'remember_token' => Str::random(60), diff --git a/packages/panels/src/FilamentServiceProvider.php b/packages/panels/src/FilamentServiceProvider.php index 46afe0e6ff9..5c219f6b602 100644 --- a/packages/panels/src/FilamentServiceProvider.php +++ b/packages/panels/src/FilamentServiceProvider.php @@ -104,7 +104,7 @@ public function packageBooted(): void SetUpPanel::class, ]); - Filament::serving(function () { + Filament::serving(function (): void { Filament::setServingStatus(); }); diff --git a/packages/panels/src/Navigation/NavigationManager.php b/packages/panels/src/Navigation/NavigationManager.php index d2ec5c155e9..9a06f751ba1 100644 --- a/packages/panels/src/Navigation/NavigationManager.php +++ b/packages/panels/src/Navigation/NavigationManager.php @@ -62,7 +62,7 @@ public function get(): array $items = $parentItems->get('') ->keyBy(fn (NavigationItem $item): string => $item->getLabel()); - $parentItems->except([''])->each(function (Collection $parentItemItems, string $parentItemLabel) use ($items) { + $parentItems->except([''])->each(function (Collection $parentItemItems, string $parentItemLabel) use ($items): void { if (! $items->has($parentItemLabel)) { return; } diff --git a/packages/panels/src/Pages/Dashboard/Actions/FilterAction.php b/packages/panels/src/Pages/Dashboard/Actions/FilterAction.php index 5b69dc91d8d..730aa809963 100644 --- a/packages/panels/src/Pages/Dashboard/Actions/FilterAction.php +++ b/packages/panels/src/Pages/Dashboard/Actions/FilterAction.php @@ -38,7 +38,7 @@ protected function setUp(): void return $livewire->filters; }); - $this->action(function (array $data, Component $livewire) { + $this->action(function (array $data, Component $livewire): void { if (! property_exists($livewire, 'filters')) { return; } diff --git a/packages/panels/src/Resources/Pages/EditRecord.php b/packages/panels/src/Resources/Pages/EditRecord.php index 0d79859fc46..1dad33bba4b 100644 --- a/packages/panels/src/Resources/Pages/EditRecord.php +++ b/packages/panels/src/Resources/Pages/EditRecord.php @@ -133,7 +133,7 @@ public function save(bool $shouldRedirect = true, bool $shouldSendSavedNotificat $this->callHook('beforeValidate'); - $data = $this->form->getState(afterValidate: function () { + $data = $this->form->getState(afterValidate: function (): void { $this->callHook('afterValidate'); $this->callHook('beforeSave'); diff --git a/packages/panels/src/Resources/Resource/Concerns/BelongsToTenant.php b/packages/panels/src/Resources/Resource/Concerns/BelongsToTenant.php index e39d8b41277..b760142dabd 100644 --- a/packages/panels/src/Resources/Resource/Concerns/BelongsToTenant.php +++ b/packages/panels/src/Resources/Resource/Concerns/BelongsToTenant.php @@ -114,7 +114,7 @@ public static function registerTenancyModelGlobalScope(Panel $panel): void return; } - $model::addGlobalScope($panel->getTenancyScopeName(), function (Builder $query) use ($panel) { + $model::addGlobalScope($panel->getTenancyScopeName(), function (Builder $query) use ($panel): void { if (Filament::getCurrentOrDefaultPanel() !== $panel) { return; } @@ -141,7 +141,7 @@ public static function observeTenancyModelCreation(Panel $panel): void return; } - $model::creating(function (Model $record) use ($panel) { + $model::creating(function (Model $record) use ($panel): void { if (Filament::getCurrentOrDefaultPanel() !== $panel) { return; } @@ -159,7 +159,7 @@ public static function observeTenancyModelCreation(Panel $panel): void } }); - $model::created(function (Model $record) use ($panel) { + $model::created(function (Model $record) use ($panel): void { if (Filament::getCurrentOrDefaultPanel() !== $panel) { return; } diff --git a/packages/panels/src/Resources/Resource/Concerns/HasGlobalSearch.php b/packages/panels/src/Resources/Resource/Concerns/HasGlobalSearch.php index 4432bb4fed7..f483093e096 100644 --- a/packages/panels/src/Resources/Resource/Concerns/HasGlobalSearch.php +++ b/packages/panels/src/Resources/Resource/Concerns/HasGlobalSearch.php @@ -168,7 +168,7 @@ protected static function applyGlobalSearchAttributeConstraints(Builder $query, ); foreach ($searchWords as $searchWord) { - $query->where(function (Builder $query) use ($searchWord) { + $query->where(function (Builder $query) use ($searchWord): void { $isFirst = true; foreach (static::getGloballySearchableAttributes() as $attributes) { diff --git a/packages/panels/src/Resources/Resource/Concerns/HasRoutes.php b/packages/panels/src/Resources/Resource/Concerns/HasRoutes.php index f3ff67a5a54..d6bf398e88a 100644 --- a/packages/panels/src/Resources/Resource/Concerns/HasRoutes.php +++ b/packages/panels/src/Resources/Resource/Concerns/HasRoutes.php @@ -73,14 +73,14 @@ public static function routes(Panel $panel, ?Closure $registerPageRoutes = null) public static function registerRoutes(Panel $panel, ?Closure $registerPageRoutes = null): void { - $registerPageRoutes ??= function () use ($panel) { + $registerPageRoutes ??= function () use ($panel): void { foreach (static::getPages() as $name => $page) { $page->registerRoute($panel)?->name($name); } }; if ($parentResource = static::getParentResourceRegistration()) { - $parentResource->getParentResource()::registerRoutes($panel, function () use ($parentResource, $registerPageRoutes) { + $parentResource->getParentResource()::registerRoutes($panel, function () use ($parentResource, $registerPageRoutes): void { Route::name($parentResource->getRouteName() . '.') ->prefix('{' . $parentResource->getParentRouteParameterName() . '}/' . $parentResource->getSlug()) ->group($registerPageRoutes); diff --git a/packages/schemas/src/Components/Concerns/EntanglesStateWithSingularRelationship.php b/packages/schemas/src/Components/Concerns/EntanglesStateWithSingularRelationship.php index af550ef3078..869adfcb39f 100644 --- a/packages/schemas/src/Components/Concerns/EntanglesStateWithSingularRelationship.php +++ b/packages/schemas/src/Components/Concerns/EntanglesStateWithSingularRelationship.php @@ -30,7 +30,7 @@ public function relationship(string $name, bool | Closure $condition = true): st $this->relationship = $name; $this->statePath($name); - $this->loadStateFromRelationshipsUsing(static function (Component | CanEntangleWithSingularRelationships $component) { + $this->loadStateFromRelationshipsUsing(static function (Component | CanEntangleWithSingularRelationships $component): void { $component->clearCachedExistingRecord(); $component->fillFromRelationship(); diff --git a/packages/spatie-laravel-media-library-plugin/src/Forms/Components/SpatieMediaLibraryFileUpload.php b/packages/spatie-laravel-media-library-plugin/src/Forms/Components/SpatieMediaLibraryFileUpload.php index c21df51911e..25b8038464c 100644 --- a/packages/spatie-laravel-media-library-plugin/src/Forms/Components/SpatieMediaLibraryFileUpload.php +++ b/packages/spatie-laravel-media-library-plugin/src/Forms/Components/SpatieMediaLibraryFileUpload.php @@ -116,7 +116,7 @@ protected function setUp(): void ]; }); - $this->saveRelationshipsUsing(static function (SpatieMediaLibraryFileUpload $component) { + $this->saveRelationshipsUsing(static function (SpatieMediaLibraryFileUpload $component): void { $component->deleteAbandonedFiles(); $component->saveUploadedFiles(); }); diff --git a/packages/spatie-laravel-settings-plugin/src/Commands/FileGenerators/SettingsPageClassGenerator.php b/packages/spatie-laravel-settings-plugin/src/Commands/FileGenerators/SettingsPageClassGenerator.php index 88e33a51227..1e5c9ae3ba6 100644 --- a/packages/spatie-laravel-settings-plugin/src/Commands/FileGenerators/SettingsPageClassGenerator.php +++ b/packages/spatie-laravel-settings-plugin/src/Commands/FileGenerators/SettingsPageClassGenerator.php @@ -215,7 +215,7 @@ public function getFormComponents(): array $isFloatType = fn (ReflectionType $type): bool => $type instanceof ReflectionNamedType && $type->getName() === 'float'; $isIntegerType = fn (ReflectionType $type): bool => $type instanceof ReflectionNamedType && $type->getName() === 'int'; - $setUpNumericComponentData = function () use (&$componentData, $componentName) { + $setUpNumericComponentData = function () use (&$componentData, $componentName): void { $componentData['numeric'] = []; if (in_array($componentName, [ diff --git a/packages/spatie-laravel-tags-plugin/src/Forms/Components/SpatieTagsInput.php b/packages/spatie-laravel-tags-plugin/src/Forms/Components/SpatieTagsInput.php index f7dc493ae1f..a4988d6f377 100644 --- a/packages/spatie-laravel-tags-plugin/src/Forms/Components/SpatieTagsInput.php +++ b/packages/spatie-laravel-tags-plugin/src/Forms/Components/SpatieTagsInput.php @@ -35,7 +35,7 @@ protected function setUp(): void $component->state($tags->pluck('name')->all()); }); - $this->saveRelationshipsUsing(static function (SpatieTagsInput $component, ?Model $record, array $state) { + $this->saveRelationshipsUsing(static function (SpatieTagsInput $component, ?Model $record, array $state): void { if (! (method_exists($record, 'syncTagsWithType') && method_exists($record, 'syncTags'))) { return; } diff --git a/packages/support/src/Colors/Color.php b/packages/support/src/Colors/Color.php index 976c2f5b698..aad083f534d 100644 --- a/packages/support/src/Colors/Color.php +++ b/packages/support/src/Colors/Color.php @@ -329,9 +329,9 @@ public static function convertToOklch(string $color): string } // Normalize RGB values to between 0 and 1 - $red = $red / 255; - $green = $green / 255; - $blue = $blue / 255; + $red /= 255; + $green /= 255; + $blue /= 255; // Linearize RGB values $red = $red <= 0.04045 ? $red / 12.92 : pow(($red + 0.055) / 1.055, 2.4); diff --git a/packages/support/src/Commands/CheckTranslationsCommand.php b/packages/support/src/Commands/CheckTranslationsCommand.php index 5b9c248750b..accd190d6df 100644 --- a/packages/support/src/Commands/CheckTranslationsCommand.php +++ b/packages/support/src/Commands/CheckTranslationsCommand.php @@ -96,7 +96,7 @@ protected function scan(string $package): void $locales = $this->argument('locales'), fn (Collection $availableLocales): Collection => $availableLocales->filter(fn (string $locale): bool => in_array($locale, $locales)) ) - ->each(function (string $locale, string $localeDir) use ($filesystem, $localeRootDirectory, $package) { + ->each(function (string $locale, string $localeDir) use ($filesystem, $localeRootDirectory, $package): void { $files = $filesystem->allFiles($localeDir); $baseFiles = $filesystem->allFiles(implode(DIRECTORY_SEPARATOR, [$localeRootDirectory, 'en'])); @@ -145,7 +145,7 @@ protected function scan(string $package): void ], ]; }) - ->tap(function (Collection $files) use ($locale, $package) { + ->tap(function (Collection $files) use ($locale, $package): void { $missingKeysCount = $files->sum(fn ($file): int => count($file['missing'])); $removedKeysCount = $files->sum(fn ($file): int => count($file['removed'])); @@ -162,7 +162,7 @@ protected function scan(string $package): void } }) ->filter(static fn ($keys): bool => count($keys['missing']) || count($keys['removed'])) - ->each(function ($keys, string $file) { + ->each(function ($keys, string $file): void { table( [$file, ''], [ diff --git a/packages/support/src/Components/ComponentManager.php b/packages/support/src/Components/ComponentManager.php index 4372cc0df6f..8102b0dc5a2 100644 --- a/packages/support/src/Components/ComponentManager.php +++ b/packages/support/src/Components/ComponentManager.php @@ -65,7 +65,7 @@ public function configureUsing(string $component, Closure $modifyUsing, ?Closure array_key_last($this->importantConfigurations[$component]) : array_key_last($this->configurations[$component]); - return function () use ($component, $configurationKey, $isImportant) { + return function () use ($component, $configurationKey, $isImportant): void { if ($isImportant) { unset($this->importantConfigurations[$component][$configurationKey]); diff --git a/packages/support/src/Facades/FilamentAsset.php b/packages/support/src/Facades/FilamentAsset.php index 114c897336e..89139dd4ea7 100644 --- a/packages/support/src/Facades/FilamentAsset.php +++ b/packages/support/src/Facades/FilamentAsset.php @@ -39,7 +39,7 @@ protected static function getFacadeAccessor(): string */ public static function register(array $assets, string $package = 'app'): void { - static::resolved(function (AssetManager $assetManager) use ($assets, $package) { + static::resolved(function (AssetManager $assetManager) use ($assets, $package): void { $assetManager->register($assets, $package); }); } @@ -49,7 +49,7 @@ public static function register(array $assets, string $package = 'app'): void */ public static function registerCssVariables(array $variables, ?string $package = null): void { - static::resolved(function (AssetManager $assetManager) use ($variables, $package) { + static::resolved(function (AssetManager $assetManager) use ($variables, $package): void { $assetManager->registerCssVariables($variables, $package); }); } @@ -59,7 +59,7 @@ public static function registerCssVariables(array $variables, ?string $package = */ public static function registerScriptData(array $data, ?string $package = null): void { - static::resolved(function (AssetManager $assetManager) use ($data, $package) { + static::resolved(function (AssetManager $assetManager) use ($data, $package): void { $assetManager->registerScriptData($data, $package); }); } diff --git a/packages/support/src/Facades/FilamentColor.php b/packages/support/src/Facades/FilamentColor.php index 3a01d2c63ae..8f7d447cfe6 100644 --- a/packages/support/src/Facades/FilamentColor.php +++ b/packages/support/src/Facades/FilamentColor.php @@ -32,7 +32,7 @@ protected static function getFacadeAccessor(): string */ public static function register(array | Closure $colors): void { - static::resolved(function (ColorManager $colorManager) use ($colors) { + static::resolved(function (ColorManager $colorManager) use ($colors): void { $colorManager->register($colors); }); } diff --git a/packages/support/src/Facades/FilamentIcon.php b/packages/support/src/Facades/FilamentIcon.php index efcff2e9e58..7067018fbf7 100644 --- a/packages/support/src/Facades/FilamentIcon.php +++ b/packages/support/src/Facades/FilamentIcon.php @@ -24,7 +24,7 @@ protected static function getFacadeAccessor(): string */ public static function register(array $icons): void { - static::resolved(function (IconManager $iconManager) use ($icons) { + static::resolved(function (IconManager $iconManager) use ($icons): void { $iconManager->register($icons); }); } diff --git a/packages/support/src/Facades/FilamentView.php b/packages/support/src/Facades/FilamentView.php index c4004e367be..2114b5cf66c 100644 --- a/packages/support/src/Facades/FilamentView.php +++ b/packages/support/src/Facades/FilamentView.php @@ -25,14 +25,14 @@ protected static function getFacadeAccessor(): string */ public static function registerRenderHook(string $name, Closure $hook, string | array | null $scopes = null): void { - static::resolved(function (ViewManager $viewManager) use ($name, $hook, $scopes) { + static::resolved(function (ViewManager $viewManager) use ($name, $hook, $scopes): void { $viewManager->registerRenderHook($name, $hook, $scopes); }); } public static function spa(bool $condition = true): void { - static::resolved(function (ViewManager $viewManager) use ($condition) { + static::resolved(function (ViewManager $viewManager) use ($condition): void { $viewManager->spa($condition); }); } @@ -42,7 +42,7 @@ public static function spa(bool $condition = true): void */ public static function spaUrlExceptions(array $exceptions): void { - static::resolved(function (ViewManager $viewManager) use ($exceptions) { + static::resolved(function (ViewManager $viewManager) use ($exceptions): void { $viewManager->spaUrlExceptions($exceptions); }); } diff --git a/packages/support/src/Partials/SupportPartials.php b/packages/support/src/Partials/SupportPartials.php index 99a7459eaab..1ea3a85aa68 100644 --- a/packages/support/src/Partials/SupportPartials.php +++ b/packages/support/src/Partials/SupportPartials.php @@ -106,7 +106,7 @@ public function dehydrate(ComponentContext $context): void $viewContext = app(ViewContext::class); - $html = $view->render(function (View $view) use ($viewContext) { + $html = $view->render(function (View $view) use ($viewContext): void { $viewContext->extractFromEnvironment($view->getFactory()); }); @@ -118,7 +118,7 @@ public function dehydrate(ComponentContext $context): void ]); } - $replaceHtml = function ($newHtml) use (&$html) { + $replaceHtml = function ($newHtml) use (&$html): void { $html = $newHtml; }; diff --git a/packages/tables/src/Columns/Concerns/CanFormatState.php b/packages/tables/src/Columns/Concerns/CanFormatState.php index 5c79e63fb20..ec30182f495 100644 --- a/packages/tables/src/Columns/Concerns/CanFormatState.php +++ b/packages/tables/src/Columns/Concerns/CanFormatState.php @@ -416,7 +416,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/tables/src/Columns/Summarizers/Concerns/CanFormatState.php b/packages/tables/src/Columns/Summarizers/Concerns/CanFormatState.php index ba76d59d986..4537db0c5a0 100644 --- a/packages/tables/src/Columns/Summarizers/Concerns/CanFormatState.php +++ b/packages/tables/src/Columns/Summarizers/Concerns/CanFormatState.php @@ -191,7 +191,7 @@ public function formatState(mixed $state): mixed $suffix = e($suffix); } - $state = $state . $suffix; + $state .= $suffix; } if (blank($state)) { diff --git a/packages/tables/src/Concerns/CanReorderRecords.php b/packages/tables/src/Concerns/CanReorderRecords.php index 7ab4ba68cb2..7fd55a53d36 100644 --- a/packages/tables/src/Concerns/CanReorderRecords.php +++ b/packages/tables/src/Concerns/CanReorderRecords.php @@ -21,7 +21,7 @@ public function reorderTable(array $order, int | string | null $draggedRecordKey $orderColumn = (string) str($this->getTable()->getReorderColumn())->afterLast('.'); - DB::transaction(function () use ($order, $orderColumn) { + DB::transaction(function () use ($order, $orderColumn): void { if ( (($relationship = $this->getTable()->getRelationship()) instanceof BelongsToMany) && in_array($orderColumn, $relationship->getPivotColumns()) diff --git a/packages/tables/src/Concerns/CanSearchRecords.php b/packages/tables/src/Concerns/CanSearchRecords.php index ad8caa0dbf5..a8f4a73b27a 100644 --- a/packages/tables/src/Concerns/CanSearchRecords.php +++ b/packages/tables/src/Concerns/CanSearchRecords.php @@ -104,7 +104,7 @@ protected function applyColumnSearchesToTableQuery(Builder $query): Builder } foreach ($this->extractTableSearchWords($search) as $searchWord) { - $query->where(function (Builder $query) use ($column, $searchWord) { + $query->where(function (Builder $query) use ($column, $searchWord): void { $isFirst = true; $column->applySearchConstraint( @@ -139,7 +139,7 @@ protected function applyGlobalSearchToTableQuery(Builder $query): Builder } if (! $this->getTable()->shouldSplitSearchTerms()) { - $query->where(function (Builder $query) use ($search) { + $query->where(function (Builder $query) use ($search): void { $isFirst = true; foreach ($this->getTable()->getColumns() as $column) { @@ -165,7 +165,7 @@ protected function applyGlobalSearchToTableQuery(Builder $query): Builder } foreach ($this->extractTableSearchWords($search) as $searchWord) { - $query->where(function (Builder $query) use ($searchWord) { + $query->where(function (Builder $query) use ($searchWord): void { $isFirst = true; foreach ($this->getTable()->getColumns() as $column) { diff --git a/packages/tables/src/Concerns/HasFilters.php b/packages/tables/src/Concerns/HasFilters.php index 7bd3a8fd031..5eb6f9adce5 100644 --- a/packages/tables/src/Concerns/HasFilters.php +++ b/packages/tables/src/Concerns/HasFilters.php @@ -157,7 +157,7 @@ protected function applyFiltersToTableQuery(Builder $query): Builder ); } - return $query->where(function (Builder $query) { + return $query->where(function (Builder $query): void { foreach ($this->getTable()->getFilters() as $filter) { $filter->apply( $query, diff --git a/packages/tables/src/Filters/QueryBuilder.php b/packages/tables/src/Filters/QueryBuilder.php index fdb87395d2b..9b5cdd2f81a 100644 --- a/packages/tables/src/Filters/QueryBuilder.php +++ b/packages/tables/src/Filters/QueryBuilder.php @@ -38,11 +38,11 @@ protected function setUp(): void ->live(onBlur: true), ]); - $this->query(function (Builder $query, array $data) { + $this->query(function (Builder $query, array $data): void { $this->applyRulesToQuery($query, $data['rules'], $this->getRuleBuilder()); }); - $this->baseQuery(function (Builder $query, array $data) { + $this->baseQuery(function (Builder $query, array $data): void { $this->applyRulesToBaseQuery($query, $data['rules'], $this->getRuleBuilder()); }); @@ -101,11 +101,11 @@ public function applyRulesToQuery(Builder $query, array $rules, RuleBuilder $rul $ruleBuilderBlockContainer = $ruleBuilder->getChildComponentContainer($ruleIndex); if ($rule['type'] === RuleBuilder::OR_BLOCK_NAME) { - $query->where(function (Builder $query) use ($rule, $ruleBuilderBlockContainer) { + $query->where(function (Builder $query) use ($rule, $ruleBuilderBlockContainer): void { $isFirst = true; foreach ($rule['data'][RuleBuilder::OR_BLOCK_GROUPS_REPEATER_NAME] as $orGroupIndex => $orGroup) { - $query->{$isFirst ? 'where' : 'orWhere'}(function (Builder $query) use ($orGroup, $orGroupIndex, $ruleBuilderBlockContainer) { + $query->{$isFirst ? 'where' : 'orWhere'}(function (Builder $query) use ($orGroup, $orGroupIndex, $ruleBuilderBlockContainer): void { $this->applyRulesToQuery( $query, $orGroup['rules'], diff --git a/packages/upgrade/src/Rector/SimpleMethodChangesRector.php b/packages/upgrade/src/Rector/SimpleMethodChangesRector.php index 73ceae732e7..7a54b459676 100644 --- a/packages/upgrade/src/Rector/SimpleMethodChangesRector.php +++ b/packages/upgrade/src/Rector/SimpleMethodChangesRector.php @@ -43,13 +43,13 @@ public function getChanges(): array Page::class, ], 'changes' => [ - 'getFooterWidgetsColumns' => function (ClassMethod $node) { + 'getFooterWidgetsColumns' => function (ClassMethod $node): void { $node->returnType = new UnionType([new Identifier('int'), new Identifier('array')]); }, - 'getHeaderWidgetsColumns' => function (ClassMethod $node) { + 'getHeaderWidgetsColumns' => function (ClassMethod $node): void { $node->returnType = new UnionType([new Identifier('int'), new Identifier('array')]); }, - 'getSubNavigationPosition' => function (ClassMethod $node) { + 'getSubNavigationPosition' => function (ClassMethod $node): void { $node->flags &= Modifiers::STATIC; }, ], @@ -59,7 +59,7 @@ public function getChanges(): array CreateRecord::class, ], 'changes' => [ - 'canCreateAnother' => function (ClassMethod $node) { + 'canCreateAnother' => function (ClassMethod $node): void { $node->flags &= ~Modifiers::STATIC; }, ], @@ -69,7 +69,7 @@ public function getChanges(): array Dashboard::class, ], 'changes' => [ - 'getColumns' => function (ClassMethod $node) { + 'getColumns' => function (ClassMethod $node): void { $node->returnType = new UnionType([new Identifier('int'), new Identifier('array')]); }, ], diff --git a/packages/upgrade/src/Rector/SimplePropertyChangesRector.php b/packages/upgrade/src/Rector/SimplePropertyChangesRector.php index 834597dbcff..143615be772 100644 --- a/packages/upgrade/src/Rector/SimplePropertyChangesRector.php +++ b/packages/upgrade/src/Rector/SimplePropertyChangesRector.php @@ -41,7 +41,7 @@ public function getChanges(): array Widget::class, ], 'changes' => [ - 'view' => function (Property $node) { + 'view' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, ], @@ -52,7 +52,7 @@ public function getChanges(): array EditProfile::class, ], 'changes' => [ - 'maxWidth' => function (Property $node) { + 'maxWidth' => function (Property $node): void { $node->type = new Name('\Filament\Support\Enums\Width | string | null'); }, ], @@ -62,7 +62,7 @@ public function getChanges(): array BasePage::class, ], 'changes' => [ - 'maxContentWidth' => function (Property $node) { + 'maxContentWidth' => function (Property $node): void { $node->type = new Name('\Filament\Support\Enums\Width | string | null'); }, ], @@ -73,13 +73,13 @@ public function getChanges(): array Page::class, ], 'changes' => [ - 'activeNavigationIcon' => function (Property $node) { + 'activeNavigationIcon' => function (Property $node): void { $node->type = new Name('string | \BackedEnum | null'); }, - 'navigationIcon' => function (Property $node) { + 'navigationIcon' => function (Property $node): void { $node->type = new Name('string | \BackedEnum | null'); }, - 'subNavigationPosition' => function (Property $node) { + 'subNavigationPosition' => function (Property $node): void { $node->type = new Name('?\Filament\Pages\Enums\SubNavigationPosition'); }, ], @@ -89,7 +89,7 @@ public function getChanges(): array RelationManager::class, ], 'changes' => [ - 'icon' => function (Property $node) { + 'icon' => function (Property $node): void { $node->type = new Name('string | \BackedEnum | null'); }, ], @@ -100,7 +100,7 @@ public function getChanges(): array StatsOverviewWidget::class, ], 'changes' => [ - 'pollingInterval' => function (Property $node) { + 'pollingInterval' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, ], @@ -110,19 +110,19 @@ public function getChanges(): array ChartWidget::class, ], 'changes' => [ - 'color' => function (Property $node) { + 'color' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, - 'heading' => function (Property $node) { + 'heading' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, - 'description' => function (Property $node) { + 'description' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, - 'maxHeight' => function (Property $node) { + 'maxHeight' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, - 'options' => function (Property $node) { + 'options' => function (Property $node): void { $node->flags &= ~Modifiers::STATIC; }, ], diff --git a/rector.php b/rector.php new file mode 100644 index 00000000000..2f77d36b863 --- /dev/null +++ b/rector.php @@ -0,0 +1,11 @@ +withPaths([ + __DIR__ . '/packages', + ]) + ->withTypeCoverageLevel(0) + ->withDeadCodeLevel(0) + ->withCodeQualityLevel(0);