Apply fixes from StyleCI

StyleCIBot · StyleCIBot · commit 0d782550fe03 · 2025-02-04T22:02:57.000Z
diff --git a/src/Actions/ActionEvent.php b/src/Actions/ActionEvent.php
@@ -65,12 +65,12 @@ public function target()
         $queryWithTrashed = static fn ($query) => $query->withTrashed();
 
         return $this->morphTo('target', 'target_type', 'target_id')
-                    ->constrain(
-                        collect(Nova::$resources)
-                            ->filter(static fn ($resource) => $resource::softDeletes())
-                            ->mapWithKeys(static fn ($resource) => [$resource::$model => $queryWithTrashed])
-                            ->all()
-                    )->when(true, static fn ($query) => $query->hasMacro('withTrashed') ? $queryWithTrashed($query) : $query);
+            ->constrain(
+                collect(Nova::$resources)
+                    ->filter(static fn ($resource) => $resource::softDeletes())
+                    ->mapWithKeys(static fn ($resource) => [$resource::$model => $queryWithTrashed])
+                    ->all()
+            )->when(true, static fn ($query) => $query->hasMacro('withTrashed') ? $queryWithTrashed($query) : $query);
     }
 
     /**
@@ -344,9 +344,9 @@ public static function prune(Collection $models, int $limit = 25): void
                 ->whereNotIn('id', static function ($query) use ($model, $limit) {
                     $query->select('id')->fromSub(
                         static::select('id')->orderBy('id', 'desc')
-                                ->where('actionable_id', $model['actionable_id'])
-                                ->where('actionable_type', $model['actionable_type'])
-                                ->limit($limit)->toBase(),
+                            ->where('actionable_id', $model['actionable_id'])
+                            ->where('actionable_type', $model['actionable_type'])
+                            ->limit($limit)->toBase(),
                         'action_events_temp'
                     );
                 })->delete();
@@ -359,7 +359,7 @@ public static function prune(Collection $models, int $limit = 25): void
     public static function markBatchAsRunning(string $batchId): int
     {
         return static::where('batch_id', $batchId)
-                    ->whereNotIn('status', ['finished', 'failed'])->update([
+            ->whereNotIn('status', ['finished', 'failed'])->update([
                         'status' => 'running',
                     ]);
     }
@@ -370,7 +370,7 @@ public static function markBatchAsRunning(string $batchId): int
     public static function markBatchAsFinished(string $batchId): int
     {
         return static::where('batch_id', $batchId)
-                    ->whereNotIn('status', ['finished', 'failed'])->update([
+            ->whereNotIn('status', ['finished', 'failed'])->update([
                         'status' => 'finished',
                     ]);
     }
@@ -393,7 +393,7 @@ public static function markAsFinished(string $batchId, $model): int
     public static function markBatchAsFailed(string $batchId, Throwable|string|null $e = null): int
     {
         return static::where('batch_id', $batchId)
-                    ->whereNotIn('status', ['finished', 'failed'])->update([
+            ->whereNotIn('status', ['finished', 'failed'])->update([
                         'status' => 'failed',
                         'exception' => $e ? (string) $e : '',
                     ]);
@@ -417,9 +417,9 @@ public static function markAsFailed(string $batchId, $model, Throwable|string|nu
     public static function updateStatus(string $batchId, $model, string $status, Throwable|string|null $e = null): int
     {
         return static::where('batch_id', $batchId)
-                        ->where('model_type', $model->getMorphClass())
-                        ->where('model_id', $model->getKey())
-                        ->update(['status' => $status, 'exception' => (string) $e]);
+            ->where('model_type', $model->getMorphClass())
+            ->where('model_id', $model->getKey())
+            ->update(['status' => $status, 'exception' => (string) $e]);
     }
 
     /**
diff --git a/src/Actions/CallQueuedAction.php b/src/Actions/CallQueuedAction.php
@@ -58,7 +58,7 @@ public function handle()
             }
 
             return $action->withActionBatchId($this->actionBatchId)
-                        ->{$this->method}($this->fields, $this->models);
+                ->{$this->method}($this->fields, $this->models);
         });
     }
 
diff --git a/src/Auth/Actions/LoginViewResponse.php b/src/Auth/Actions/LoginViewResponse.php
@@ -19,6 +19,7 @@ public function toResponse($request)
         $initialYear = config('app.initialyear', date('Y'));
         $currentYear = date('Y');
         $years = $initialYear <= $currentYear ? range($initialYear, $currentYear) : [];
+
         return Inertia::render('Nova.Login', [
             'username' => Nova::fortify()->username,
             'email' => Nova::fortify()->email,
diff --git a/src/Auth/Actions/LogoutResponse.php b/src/Auth/Actions/LogoutResponse.php
@@ -4,7 +4,6 @@
 
 use Illuminate\Http\JsonResponse;
 use Laravel\Fortify\Contracts\LogoutResponse as Responsable;
-use Laravel\Nova\Nova;
 
 class LogoutResponse implements Responsable
 {
diff --git a/src/Concerns/HandlesRoutes.php b/src/Concerns/HandlesRoutes.php
@@ -46,8 +46,8 @@ public static function url(?string $url = null): string
     public static function router(?array $middleware = null, ?string $prefix = null): RouteRegistrar
     {
         return Route::domain(config('nova.domain', null))
-                    ->prefix(static::url($prefix))
-                    ->middleware($middleware ?? config('nova.middleware', []));
+            ->prefix(static::url($prefix))
+            ->middleware($middleware ?? config('nova.middleware', []));
     }
 
     /**
diff --git a/src/Fields/DetachesPivotModels.php b/src/Fields/DetachesPivotModels.php
@@ -26,11 +26,11 @@ protected function detachmentCallback(): callable
                 $pivotFields = $resource->resolvePivotFields($request, $request->resource);
 
                 $pivotFields->whereInstanceOf(Deletable::class)
-                        ->filter->isPrunable()
-                        ->each(static function ($field) use ($request, $pivot) {
-                            /** @var \Laravel\Nova\Fields\Field&\Laravel\Nova\Contracts\Deletable $field */
-                            DeleteField::forRequest($request, $field, $pivot)->save();
-                        });
+                    ->filter->isPrunable()
+                    ->each(static function ($field) use ($request, $pivot) {
+                        /** @var \Laravel\Nova\Fields\Field&\Laravel\Nova\Contracts\Deletable $field */
+                        DeleteField::forRequest($request, $field, $pivot)->save();
+                    });
 
                 $pivot->delete();
             }
diff --git a/src/Fields/HasOne.php b/src/Fields/HasOne.php
@@ -120,16 +120,16 @@ public function __construct($name, ?string $attribute = null, ?string $resource
 
             if ($this->ofManyRelationship === false && $request->viaRelationship === $this->attribute && $request->viaResourceId) {
                 $parent = $parentResource::newModel()
-                            ->with($this->attribute)
-                            ->find($request->viaResourceId);
+                    ->with($this->attribute)
+                    ->find($request->viaResourceId);
 
                 return optional($parent->{$this->attribute})->exists === true;
             }
 
             return false;
         })->showOnCreating(static fn ($request) => ! in_array($request->relationshipType, ['hasOne', 'morphOne']))
-        ->showOnUpdating(static fn ($request) => ! in_array($request->relationshipType, ['hasOne', 'morphOne']))
-        ->nullable();
+            ->showOnUpdating(static fn ($request) => ! in_array($request->relationshipType, ['hasOne', 'morphOne']))
+            ->nullable();
     }
 
     /**
diff --git a/src/Fields/ID.php b/src/Fields/ID.php
@@ -65,8 +65,8 @@ public static function forResource(Resource $resource): ?static
         /** @phpstan-ignore argument.templateType */
         $field = transform(
             $resource->availableFieldsOnIndexOrDetail(app(NovaRequest::class))
-                    ->whereInstanceOf(self::class)
-                    ->first(),
+                ->whereInstanceOf(self::class)
+                ->first(),
             static fn ($field) => tap($field)->resolve($model),
             static fn () => ! is_null($model) && $model->exists ? static::forModel($model) : null,
         );
diff --git a/src/Fields/MorphTo.php b/src/Fields/MorphTo.php
@@ -533,9 +533,9 @@ protected function filterableAttribute(NovaRequest $request)
     protected function defaultFilterableCallback()
     {
         $morphToTypes = collect($this->morphToTypes)
-                            ->pluck('type')
-                            ->mapWithKeys(static fn ($type) => [$type => $type::newModel()->getMorphClass()])
-                            ->all();
+            ->pluck('type')
+            ->mapWithKeys(static fn ($type) => [$type => $type::newModel()->getMorphClass()])
+            ->all();
 
         return static function (NovaRequest $request, $query, $value, $attribute) use ($morphToTypes) {
             $query->whereHasMorph(
diff --git a/src/Fields/MorphToMany.php b/src/Fields/MorphToMany.php
@@ -229,9 +229,9 @@ public function singularLabel(Stringable|string $singularLabel)
     public function asPanel(): Panel
     {
         return Panel::make($this->name, [$this])
-                    ->withMeta([
-                        'prefixComponent' => true,
-                    ])->withComponent('relationship-panel');
+            ->withMeta([
+                'prefixComponent' => true,
+            ])->withComponent('relationship-panel');
     }
 
     /**
diff --git a/src/Fields/Number.php b/src/Fields/Number.php
@@ -145,6 +145,6 @@ public function jsonSerialize(): array
             'max' => $this->max,
             'step' => $this->step,
         ])->reject(static fn ($value) => is_null($value) || (empty($value) && $value !== 0))
-        ->all());
+            ->all());
     }
 }
diff --git a/src/Fields/Select.php b/src/Fields/Select.php
@@ -66,8 +66,8 @@ public function displayUsingLabels()
             }
 
             return collect($this->serializeOptions(false))
-                    ->where('value', $value)
-                    ->first()['label'] ?? $value;
+                ->where('value', $value)
+                ->first()['label'] ?? $value;
         });
 
         return $this;
diff --git a/src/Http/Controllers/AttachableController.php b/src/Http/Controllers/AttachableController.php
@@ -14,13 +14,13 @@ class AttachableController extends Controller
     public function __invoke(NovaRequest $request): array
     {
         $field = $request->newResource()
-                    ->availableFields($request)
-                    ->filterForManyToManyRelations()
-                    ->filter(static function ($field) use ($request) {
-                        return $field->resourceName === $request->field && // @phpstan-ignore property.notFound
-                            $field->component === $request->component &&
-                            $field->attribute === $request->viaRelationship;
-                    })->first();
+            ->availableFields($request)
+            ->filterForManyToManyRelations()
+            ->filter(static function ($field) use ($request) {
+                return $field->resourceName === $request->field && // @phpstan-ignore property.notFound
+                    $field->component === $request->component &&
+                    $field->attribute === $request->viaRelationship;
+            })->first();
 
         abort_if(is_null($field), 404);
 
@@ -92,8 +92,8 @@ protected function getAttachableQueryResolver(NovaRequest $request, PivotableFie
                 $relation = $relatedModel->{$field->manyToManyRelationship}();
 
                 return $relation->applyDefaultPivotQuery($query)
-                        ->select($relation->getRelatedPivotKeyName())
-                        ->whereColumn($relation->getQualifiedRelatedKeyName(), $relation->getQualifiedRelatedPivotKeyName());
+                    ->select($relation->getRelatedPivotKeyName())
+                    ->whereColumn($relation->getQualifiedRelatedKeyName(), $relation->getQualifiedRelatedPivotKeyName());
             });
         };
     }
diff --git a/src/Http/Controllers/CreationPivotFieldController.php b/src/Http/Controllers/CreationPivotFieldController.php
@@ -31,7 +31,7 @@ public function sync(ResourceCreateOrAttachRequest $request): JsonResponse
                 return $request->query('field') === $field->attribute &&
                         $request->query('component') === $field->dependentComponentKey();
             })->applyDependsOn($request)
-            ->first()
+                ->first()
         );
     }
 }
diff --git a/src/Http/Controllers/UpdatePivotFieldController.php b/src/Http/Controllers/UpdatePivotFieldController.php
@@ -31,7 +31,7 @@ public function sync(ResourceUpdateOrUpdateAttachedRequest $request): JsonRespon
                 return $request->query('field') === $field->attribute &&
                         $request->query('component') === $field->dependentComponentKey();
             })->applyDependsOn($request)
-            ->first()
+                ->first()
         );
     }
 }
diff --git a/src/Http/Middleware/HandleInertiaRequests.php b/src/Http/Middleware/HandleInertiaRequests.php
@@ -4,7 +4,6 @@
 
 use Closure;
 use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Cache;
 use Illuminate\Support\Facades\Config;
 use Inertia\Inertia;
 use Inertia\Middleware;
diff --git a/src/Http/Requests/LensRequest.php b/src/Http/Requests/LensRequest.php
@@ -114,9 +114,9 @@ public function toResources(Collection $models): Collection
                 $payload['actions'] = collect(
                     $hasId === true ? array_values($lensResource->actions($this)) : []
                 )->filter(static fn ($action) => $action->shownOnIndex() || $action->shownOnTableRow())
-                ->filter->authorizedToSee($this)
-                ->filter->authorizedToRun($this, $model)
-                ->values();
+                    ->filter->authorizedToSee($this)
+                    ->filter->authorizedToRun($this, $model)
+                    ->values();
 
                 return $payload;
             });
diff --git a/src/Jobs/DeletesFields.php b/src/Jobs/DeletesFields.php
@@ -29,10 +29,10 @@ protected function deleteFields(NovaRequest $request, $model, bool $skipSoftDele
         }
 
         $request->newResourceWith($model)
-                    ->deletableFields($request)
-                    ->filter->isPrunable()
-                    ->each(static function ($field) use ($request, $model) {
-                        DeleteField::forRequest($request, $field, $model);
-                    });
+            ->deletableFields($request)
+            ->filter->isPrunable()
+            ->each(static function ($field) use ($request, $model) {
+                DeleteField::forRequest($request, $field, $model);
+            });
     }
 }
diff --git a/src/Menu/Breadcrumb.php b/src/Menu/Breadcrumb.php
@@ -42,13 +42,13 @@ public static function resource(Resource|string $resourceClass): static
                     'title' => $resourceClass->title(),
                 ])
             )->path('/resources/'.$resourceClass::uriKey().'/'.$resourceClass->getKey())
-            ->canSee(static fn ($request) => $resourceClass->authorizedToView($request));
+                ->canSee(static fn ($request) => $resourceClass->authorizedToView($request));
         }
 
         return static::make(
             Nova::__($resourceClass::label())
         )->path('/resources/'.$resourceClass::uriKey())
-        ->canSee(static fn ($request) => $resourceClass::availableForNavigation($request) && $resourceClass::authorizedToViewAny($request));
+            ->canSee(static fn ($request) => $resourceClass::availableForNavigation($request) && $resourceClass::authorizedToViewAny($request));
     }
 
     /**
diff --git a/src/Menu/Menu.php b/src/Menu/Menu.php
@@ -96,8 +96,8 @@ public function jsonSerialize(): array
         $request = app(NovaRequest::class);
 
         return $this->items->flatten()
-                ->reject(static fn ($item) => method_exists($item, 'authorizedToSee') && ! $item->authorizedToSee($request))
-                ->values()
-                ->jsonSerialize();
+            ->reject(static fn ($item) => method_exists($item, 'authorizedToSee') && ! $item->authorizedToSee($request))
+            ->values()
+            ->jsonSerialize();
     }
 }
diff --git a/src/Menu/MenuSection.php b/src/Menu/MenuSection.php
@@ -82,7 +82,7 @@ public static function dashboard(string $dashboard)
             return static::make(
                 $dashboard->label()
             )->path('/dashboards/'.$dashboard->uriKey())
-            ->canSee(static fn ($request) => $dashboard->authorizedToSee($request));
+                ->canSee(static fn ($request) => $dashboard->authorizedToSee($request));
         });
     }
 
@@ -97,7 +97,7 @@ public static function resource(string $resourceClass)
         return static::make(
             $resourceClass::label()
         )->path('/resources/'.$resourceClass::uriKey())
-        ->canSee(static fn ($request) => $resourceClass::availableForNavigation($request) && $resourceClass::authorizedToViewAny($request));
+            ->canSee(static fn ($request) => $resourceClass::availableForNavigation($request) && $resourceClass::authorizedToViewAny($request));
     }
 
     /**
diff --git a/src/Metrics/Trend.php b/src/Metrics/Trend.php
@@ -392,12 +392,12 @@ protected function aggregate(
         $wrappedColumn = $query->getQuery()->getGrammar()->wrap($column);
 
         $results = $query
-                ->select(DB::raw("{$expression} as date_result, {$function}({$wrappedColumn}) as aggregate"))
-                ->tap(fn ($query) => $this->applyFilterQuery($request, $query))
-                ->whereBetween($dateColumn, $this->formatQueryDateBetween([$startingDate, $endingDate]))
-                ->groupBy(DB::raw($expression))
-                ->orderBy('date_result')
-                ->get();
+            ->select(DB::raw("{$expression} as date_result, {$function}({$wrappedColumn}) as aggregate"))
+            ->tap(fn ($query) => $this->applyFilterQuery($request, $query))
+            ->whereBetween($dateColumn, $this->formatQueryDateBetween([$startingDate, $endingDate]))
+            ->groupBy(DB::raw($expression))
+            ->orderBy('date_result')
+            ->get();
 
         $possibleDateKeys = array_keys($possibleDateResults);
 
@@ -410,7 +410,7 @@ protected function aggregate(
                     $request->twelveHourTime === 'true'
                 ) => round($result->aggregate ?? 0, $this->roundingPrecision, $this->roundingMode),
             ])->reject(static fn ($value, $key) => ! in_array($key, $possibleDateKeys))
-            ->all()
+                ->all()
         );
 
         return $this->result(Arr::last($results))->trend(
diff --git a/src/Nova.php b/src/Nova.php
@@ -8,16 +8,13 @@
 use Closure;
 use Illuminate\Console\Command;
 use Illuminate\Contracts\Auth\Access\Gate as GateContract;
-use Illuminate\Http\Client\Response as ClientResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Support\Facades\Blade;
-use Illuminate\Support\Facades\Cache;
 use Illuminate\Support\Facades\File;
 use Illuminate\Support\Facades\Gate;
 use Illuminate\Support\Facades\Hash;
-use Illuminate\Support\Facades\Http;
 use Illuminate\Support\Str;
 use Illuminate\Validation\Rules\Password;
 use Laravel\Nova\Actions\ActionResource;
diff --git a/src/PerformsQueries.php b/src/PerformsQueries.php
@@ -57,13 +57,13 @@ protected static function applySearch(Builder $query, string $search): Builder
 
         /** @phpstan-ignore nullCoalesce.expr */
         $searchColumns = collect(static::searchableColumns() ?? [])
-                            ->transform(static function ($column) use ($modelKeyName) {
-                                if ($column === $modelKeyName) {
-                                    return new PrimaryKey($column, static::maxPrimaryKeySize());
-                                }
+            ->transform(static function ($column) use ($modelKeyName) {
+                if ($column === $modelKeyName) {
+                    return new PrimaryKey($column, static::maxPrimaryKeySize());
+                }
 
-                                return $column;
-                            })->all();
+                return $column;
+            })->all();
 
         return static::initializeSearch($query, $search, $searchColumns);
     }
diff --git a/src/ResolvesActions.php b/src/ResolvesActions.php
diff --git a/src/ResolvesFilters.php b/src/ResolvesFilters.php
diff --git a/src/Rules/NotExactlyAttached.php b/src/Rules/NotExactlyAttached.php
diff --git a/src/Rules/Relatable.php b/src/Rules/Relatable.php
diff --git a/src/Tools/Dashboard.php b/src/Tools/Dashboard.php
diff --git a/src/Util.php b/src/Util.php

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ public function handle()`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`return $action->withActionBatchId($this->actionBatchId)`
`61`		`- ->{$this->method}($this->fields, $this->models);`
	`61`	`+ ->{$this->method}($this->fields, $this->models);`
`62`	`62`	`});`
`63`	`63`	`}`
`64`	`64`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,6 @@`
`4`	`4`
`5`	`5`	`use Illuminate\Http\JsonResponse;`
`6`	`6`	`use Laravel\Fortify\Contracts\LogoutResponse as Responsable;`
`7`		`-use Laravel\Nova\Nova;`
`8`	`7`
`9`	`8`	`class LogoutResponse implements Responsable`
`10`	`9`	`{`
Original file line number	Diff line number	Diff line change
`@@ -46,8 +46,8 @@ public static function url(?string $url = null): string`
`46`	`46`	`public static function router(?array $middleware = null, ?string $prefix = null): RouteRegistrar`
`47`	`47`	`{`
`48`	`48`	`return Route::domain(config('nova.domain', null))`
`49`		`- ->prefix(static::url($prefix))`
`50`		`- ->middleware($middleware ?? config('nova.middleware', []));`
	`49`	`+ ->prefix(static::url($prefix))`
	`50`	`+ ->middleware($middleware ?? config('nova.middleware', []));`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -229,9 +229,9 @@ public function singularLabel(Stringable\|string $singularLabel)`
`229`	`229`	`public function asPanel(): Panel`
`230`	`230`	`{`
`231`	`231`	`return Panel::make($this->name, [$this])`
`232`		`- ->withMeta([`
`233`		`- 'prefixComponent' => true,`
`234`		`- ])->withComponent('relationship-panel');`
	`232`	`+ ->withMeta([`
	`233`	`+ 'prefixComponent' => true,`
	`234`	`+ ])->withComponent('relationship-panel');`
`235`	`235`	`}`
`236`	`236`
`237`	`237`	`/**`