Laravel-Backpack / CRUD

src/app/Library/Uploaders/SingleFile.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
             return $this->getPath().$fileName;
         }
 
-        if (! $value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
+        if (!$value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
             Storage::disk($this->getDisk())->delete($previousFile);
 
             return null;
@@ -55,10 +55,10 @@ 
 
         foreach ($previousRepeatableValues as $row => $file) {
             if ($file) {
-                if (! isset($orderedFiles[$row])) {
+                if (!isset($orderedFiles[$row])) {
                     $orderedFiles[$row] = null;
                 }
-                if (! in_array($file, $orderedFiles)) {
+                if (!in_array($file, $orderedFiles)) {
                     Storage::disk($this->getDisk())->delete($file);
                 }
             }

src/app/Library/CrudPanel/Traits/Create.php

Spacing +5 added lines, -5 removed lines

@@ -124,7 +124,7 @@ 
                     // - a single dimensional array: [1,2,3]
                     // - an array of arrays: [[1][2][3]]
                     // if is as single dimensional array we can only attach.
-                    if ($relationValues === null || ! is_multidimensional_array($relationValues)) {
+                    if ($relationValues === null || !is_multidimensional_array($relationValues)) {
                         $this->attachManyRelation($item, $relation, $relationDetails, $relationValues);
                     } else {
                         $this->createManyEntries($item, $relation, $relationMethod, $relationDetails);
@@ -146,7 +146,7 @@ 
                             
                             $toDelete = array_diff($dbValues, $sentIds);
                             
-                            if (! empty($toDelete)) {
+                            if (!empty($toDelete)) {
                                 foreach ($toDelete as $id) {
                                     $relation->newPivot()->where($keyName, $id)->delete();
                                 }
@@ -187,7 +187,7 @@ 
         }
     }
 
-    private function preparePivotAttributesForSave(array $attributes, BelongsToMany|MorphToMany $relation, string|int $relatedItemKey, $pivotKeyName)
+    private function preparePivotAttributesForSave(array $attributes, BelongsToMany | MorphToMany $relation, string | int $relatedItemKey, $pivotKeyName)
     {
         $attributes[$relation->getForeignPivotKeyName()] = $relatedItemKey;
         $attributes[$relation->getRelatedPivotKeyName()] = $attributes[$relation->getRelationName()];
@@ -333,7 +333,7 @@ 
         }
         // if column is not nullable in database, and there is no column default (null),
         // we will delete the entry from the database, otherwise it will throw and ugly DB error.
-        if (! $relationColumnIsNullable && $dbColumnDefault === null) {
+        if (!$relationColumnIsNullable && $dbColumnDefault === null) {
             return $removedEntries->lazy()->each->delete();
         }
 
@@ -378,7 +378,7 @@ 
         }
 
         // use the collection of sent ids to match against database ids, delete the ones not found in the submitted ids.
-        if (! empty($relatedItemsSent)) {
+        if (!empty($relatedItemsSent)) {
             // we perform the cleanup of removed database items
             $entry->{$relationMethod}()->whereNotIn($relatedModelLocalKey, $relatedItemsSent)->lazy()->each->delete();
         }

src/macros.php

Spacing +21 added lines, -21 removed lines

@@ -19,8 +19,8 @@ 
  *      - when true: `address[street]`
  *      - when false: `[address][street]`
  */
-if (! Str::hasMacro('dotsToSquareBrackets')) {
-    Str::macro('dotsToSquareBrackets', function ($string, $ignore = [], $keyFirst = true) {
+if (!Str::hasMacro('dotsToSquareBrackets')) {
+    Str::macro('dotsToSquareBrackets', function($string, $ignore = [], $keyFirst = true) {
         $stringParts = explode('.', $string);
         $result = '';
 
@@ -34,8 +34,8 @@ 
         return $result;
     });
 }
-if (! CrudColumn::hasMacro('withFiles')) {
-    CrudColumn::macro('withFiles', function ($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
+if (!CrudColumn::hasMacro('withFiles')) {
+    CrudColumn::macro('withFiles', function($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
         $uploadDefinition = is_array($uploadDefinition) ? $uploadDefinition : [];
         /** @var CrudField|CrudColumn $this */
         RegisterUploadEvents::handle($this, $uploadDefinition, 'withFiles', $subfield, $registerUploaderEvents);
@@ -44,8 +44,8 @@ 
     });
 }
 
-if (! CrudField::hasMacro('withFiles')) {
-    CrudField::macro('withFiles', function ($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
+if (!CrudField::hasMacro('withFiles')) {
+    CrudField::macro('withFiles', function($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
         $uploadDefinition = is_array($uploadDefinition) ? $uploadDefinition : [];
         /** @var CrudField|CrudColumn $this */
         RegisterUploadEvents::handle($this, $uploadDefinition, 'withFiles', $subfield, $registerUploaderEvents);
@@ -54,8 +54,8 @@ 
     });
 }
 
-if (! CrudColumn::hasMacro('linkTo')) {
-    CrudColumn::macro('linkTo', function (string|array|Closure $routeOrConfiguration, ?array $parameters = []): static {
+if (!CrudColumn::hasMacro('linkTo')) {
+    CrudColumn::macro('linkTo', function(string | array | Closure $routeOrConfiguration, ?array $parameters = []): static {
         $wrapper = $this->attributes['wrapper'] ?? [];
 
         // parse the function input to get the actual route and parameters we'll be working with
@@ -68,7 +68,7 @@ 
 
         // if the route is a closure, we'll just call it
         if ($route instanceof Closure) {
-            $wrapper['href'] = function ($crud, $column, $entry, $related_key) use ($route) {
+            $wrapper['href'] = function($crud, $column, $entry, $related_key) use ($route) {
                 return $route($entry, $related_key, $column, $crud);
             };
             $this->wrapper($wrapper);
@@ -77,13 +77,13 @@ 
         }
 
         // if the route doesn't exist, we'll throw an exception
-        if (! $routeInstance = Route::getRoutes()->getByName($route)) {
+        if (!$routeInstance = Route::getRoutes()->getByName($route)) {
             throw new \Exception("Route [{$route}] not found while building the link for column [{$this->attributes['name']}].");
         }
 
         // calculate the parameters we'll be using for the route() call
         // (eg. if there's only one parameter and user didn't provide it, we'll assume it's the entry's related key)
-        $parameters = (function () use ($parameters, $routeInstance, $route) {
+        $parameters = (function() use ($parameters, $routeInstance, $route) {
             $expectedParameters = $routeInstance->parameterNames();
 
             if (count($expectedParameters) === 0) {
@@ -94,7 +94,7 @@ 
             if (count($autoInferredParameter) > 1) {
                 throw new \Exception("Route [{$route}] expects parameters [".implode(', ', $expectedParameters)."]. Insufficient parameters provided in column: [{$this->attributes['name']}].");
             }
-            $autoInferredParameter = current($autoInferredParameter) ? [current($autoInferredParameter) => function ($entry, $related_key, $column, $crud) {
+            $autoInferredParameter = current($autoInferredParameter) ? [current($autoInferredParameter) => function($entry, $related_key, $column, $crud) {
                 $entity = $crud->isAttributeInRelationString($column) ? Str::before($column['entity'], '.') : $column['entity'];
 
                 return $related_key ?? $entry->{$entity}?->getKey();
@@ -104,7 +104,7 @@ 
         })();
 
         // set up the wrapper href attribute
-        $wrapper['href'] = function ($crud, $column, $entry, $related_key) use ($route, $parameters) {
+        $wrapper['href'] = function($crud, $column, $entry, $related_key) use ($route, $parameters) {
             // if the parameter is callable, we'll call it
             $parameters = collect($parameters)->map(fn ($item) => is_callable($item) ? $item($entry, $related_key, $column, $crud) : $item)->toArray();
 
@@ -121,17 +121,17 @@ 
     });
 }
 
-if (! CrudColumn::hasMacro('linkToShow')) {
-    CrudColumn::macro('linkToShow', function (): static {
+if (!CrudColumn::hasMacro('linkToShow')) {
+    CrudColumn::macro('linkToShow', function(): static {
         $name = $this->attributes['name'];
         $entity = $this->attributes['entity'] ?? null;
         $route = "$entity.show";
 
-        if (! $entity) {
+        if (!$entity) {
             throw new \Exception("Entity not found while building the link for column [{$name}].");
         }
 
-        if (! Route::getRoutes()->getByName($route)) {
+        if (!Route::getRoutes()->getByName($route)) {
             throw new \Exception("Route '{$route}' not found while building the link for column [{$name}].");
         }
 
@@ -142,8 +142,8 @@ 
     });
 }
 
-if (! CrudColumn::hasMacro('linkTarget')) {
-    CrudColumn::macro('linkTarget', function (string $target = '_self'): static {
+if (!CrudColumn::hasMacro('linkTarget')) {
+    CrudColumn::macro('linkTarget', function(string $target = '_self'): static {
         $this->wrapper([
             ...$this->attributes['wrapper'] ?? [],
             'target' => $target,
@@ -159,8 +159,8 @@ 
  *
  * It will go to the given CrudController and get the setupRoutes() method on it.
  */
-if (! Route::hasMacro('crud')) {
-    Route::macro('crud', function ($name, $controller) {
+if (!Route::hasMacro('crud')) {
+    Route::macro('crud', function($name, $controller) {
         // put together the route name prefix,
         // as passed to the Route::group() statements
         $routeName = '';

src/app/Console/Commands/Themes/InstallsTheme.php

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
         }
 
         // Display general error in case it failed
-        if (! $this->isInstalled()) {
+        if (!$this->isInstalled()) {
             $this->errorProgressBlock();
             $this->note('For further information please check the log file.');
             $this->note('You can also follow the manual installation process documented on GitHub.');
@@ -84,7 +84,7 @@ 
         $this->progressBlock('Publish theme config file');
 
         // manually include the provider in the run-time
-        if (! class_exists(self::$addon['provider'])) {
+        if (!class_exists(self::$addon['provider'])) {
             include self::$addon['provider_path'] ?? self::$addon['path'].'/src/AddonServiceProvider.php';
             app()->register(self::$addon['provider']);
         }

src/app/Library/CrudPanel/CrudPanel.php

Spacing +17 added lines, -17 removed lines

@@ -109,11 +109,11 @@ 
      */
     public function setModel($model_namespace)
     {
-        if (! class_exists($model_namespace)) {
+        if (!class_exists($model_namespace)) {
             throw new Exception('The model does not exist.', 500);
         }
 
-        if (! method_exists($model_namespace, 'hasCrudTrait')) {
+        if (!method_exists($model_namespace, 'hasCrudTrait')) {
             throw new Exception('Please use CrudTrait on the model.', 500);
         }
 
@@ -206,7 +206,7 @@ 
 
         $complete_route = $route.'.index';
 
-        if (! \Route::has($complete_route)) {
+        if (!\Route::has($complete_route)) {
             throw new Exception('There are no routes for this route name.', 404);
         }
 
@@ -304,7 +304,7 @@ 
      */
     public function getFirstOfItsTypeInArray($type, $array)
     {
-        return Arr::first($array, function ($item) use ($type) {
+        return Arr::first($array, function($item) use ($type) {
             return $item['type'] == $type;
         });
     }
@@ -324,8 +324,8 @@ 
      */
     public function sync($type, $fields, $attributes)
     {
-        if (! empty($this->{$type})) {
-            $this->{$type} = array_map(function ($field) use ($fields, $attributes) {
+        if (!empty($this->{$type})) {
+            $this->{$type} = array_map(function($field) use ($fields, $attributes) {
                 if (in_array($field['name'], (array) $fields)) {
                     $field = array_merge($field, $attributes);
                 }
@@ -355,15 +355,15 @@ 
     {
         $relationArray = explode('.', $relationString);
 
-        if (! isset($length)) {
+        if (!isset($length)) {
             $length = count($relationArray);
         }
 
-        if (! isset($model)) {
+        if (!isset($model)) {
             $model = $this->model;
         }
 
-        $result = array_reduce(array_splice($relationArray, 0, $length), function ($obj, $method) {
+        $result = array_reduce(array_splice($relationArray, 0, $length), function($obj, $method) {
             try {
                 $result = $obj->$method();
 
@@ -399,7 +399,7 @@ 
             if (is_array($entries)) {
                 //if attribute does not exist in main array we have more than one entry OR the attribute
                 //is an accessor that is not in $appends property of model.
-                if (! isset($entries[$attribute])) {
+                if (!isset($entries[$attribute])) {
                     //we first check if we don't have the attribute because it's an accessor that is not in appends.
                     if ($model_instance->hasGetMutator($attribute) && isset($entries[$modelKey])) {
                         $entry_in_database = $model_instance->find($entries[$modelKey]);
@@ -438,21 +438,21 @@ 
      */
     public function parseTranslatableAttributes($model, $attribute, $value)
     {
-        if (! method_exists($model, 'isTranslatableAttribute')) {
+        if (!method_exists($model, 'isTranslatableAttribute')) {
             return $value;
         }
 
-        if (! $model->isTranslatableAttribute($attribute)) {
+        if (!$model->isTranslatableAttribute($attribute)) {
             return $value;
         }
 
-        if (! is_array($value)) {
+        if (!is_array($value)) {
             $decodedAttribute = json_decode($value, true) ?? ($value !== null ? [$value] : []);
         } else {
             $decodedAttribute = $value;
         }
 
-        if (is_array($decodedAttribute) && ! empty($decodedAttribute)) {
+        if (is_array($decodedAttribute) && !empty($decodedAttribute)) {
             if (isset($decodedAttribute[app()->getLocale()])) {
                 return $decodedAttribute[app()->getLocale()];
             } else {
@@ -470,7 +470,7 @@ 
         if (method_exists($model, 'translationEnabled') && $model->translationEnabled()) {
             $locale = $this->getRequest()->input('_locale', app()->getLocale());
             if (in_array($locale, array_keys($model->getAvailableLocales()))) {
-                $model->setLocale(! is_bool($useFallbackLocale) ? $useFallbackLocale : $locale);
+                $model->setLocale(!is_bool($useFallbackLocale) ? $useFallbackLocale : $locale);
                 $model->useFallbackLocale = (bool) $useFallbackLocale;
             }
         }
@@ -493,7 +493,7 @@ 
         $relation = $model->{$firstRelationName};
 
         $results = [];
-        if (! is_null($relation)) {
+        if (!is_null($relation)) {
             if ($relation instanceof Collection) {
                 $currentResults = $relation->all();
             } elseif (is_array($relation)) {
@@ -506,7 +506,7 @@ 
 
             array_shift($relationArray);
 
-            if (! empty($relationArray)) {
+            if (!empty($relationArray)) {
                 foreach ($currentResults as $currentResult) {
                     $results = array_merge_recursive($results, $this->getRelatedEntries($currentResult, implode('.', $relationArray)));
                 }

tests/config/database/migrations/2024_02_15_125654_create_uploaders_table.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
      */
     public function up(): void
     {
-        Schema::create('uploaders', function (Blueprint $table) {
+        Schema::create('uploaders', function(Blueprint $table) {
             $table->bigIncrements('id')->unsigned();
             $table->string('upload')->nullable();
             $table->string('image')->nullable();

src/app/Library/Uploaders/Support/RegisterUploadEvents.php

Spacing +15 added lines, -15 removed lines

@@ -14,18 +14,18 @@ 
     private string $crudObjectType;
 
     public function __construct(
-        private readonly CrudField|CrudColumn $crudObject,
+        private readonly CrudField | CrudColumn $crudObject,
         private readonly array $uploaderConfiguration,
         private readonly string $macro
         ) {
         $this->crudObjectType = is_a($crudObject, CrudField::class) ? 'field' : (is_a($crudObject, CrudColumn::class) ? 'column' : null);
 
-        if (! $this->crudObjectType) {
+        if (!$this->crudObjectType) {
             abort(500, 'Upload handlers only work for CrudField and CrudColumn classes.');
         }
     }
 
-    public static function handle(CrudField|CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
+    public static function handle(CrudField | CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
     {
         $instance = new self($crudObject, $uploaderConfiguration, $macro);
 
@@ -35,9 +35,9 @@ 
     /*******************************
      * Private methods - implementation
      *******************************/
-    private function registerEvents(array|null $subfield = [], ?bool $registerModelEvents = true): void
+    private function registerEvents(array | null $subfield = [], ?bool $registerModelEvents = true): void
     {
-        if (! empty($subfield)) {
+        if (!empty($subfield)) {
             $this->registerSubfieldEvent($subfield, $registerModelEvents);
 
             return;
@@ -82,7 +82,7 @@ 
         }
 
         $subfields = collect($this->crudObject->getAttributes()['subfields']);
-        $subfields = $subfields->map(function ($item) use ($subfield, $uploader) {
+        $subfields = $subfields->map(function($item) use ($subfield, $uploader) {
             if ($item['name'] === $subfield['name']) {
                 $item['upload'] = true;
                 $item['disk'] = $uploader->getDisk();
@@ -112,7 +112,7 @@ 
         }
 
         if ($this->crudObjectType === 'field') {
-            $model::saving(function ($entry) use ($uploader) {
+            $model::saving(function($entry) use ($uploader) {
                 $entry = $uploader->storeUploadedFiles($entry);
             });
         }
@@ -125,7 +125,7 @@ 
             // is not called in pivot models when loading the relations.
             $retrieveModel = $this->getModelForRetrieveEvent($model, $uploader);
 
-            $retrieveModel::retrieved(function ($entry) use ($uploader) {
+            $retrieveModel::retrieved(function($entry) use ($uploader) {
                 if ($entry->translationEnabled()) {
                     $locale = request('_locale', app()->getLocale());
                     if (in_array($locale, array_keys($entry->getAvailableLocales()))) {
@@ -136,14 +136,14 @@ 
             });
         }
 
-        $model::deleting(function ($entry) use ($uploader) {
+        $model::deleting(function($entry) use ($uploader) {
             $uploader->deleteUploadedFiles($entry);
         });
 
         // if the uploader is a relationship and handles repeatable files, we will also register the deleting event on the
         // parent model. that way we can control the deletion of the files when the parent model is deleted.
         if ($uploader->isRelationship() && $uploader->handleRepeatableFiles) {
-            app('crud')->model::deleting(function ($entry) use ($uploader) {
+            app('crud')->model::deleting(function($entry) use ($uploader) {
                 $uploader->deleteUploadedFiles($entry);
             });
         }
@@ -165,18 +165,18 @@ 
     {
         $hasCustomUploader = isset($uploaderConfiguration['uploader']);
 
-        if ($hasCustomUploader && ! is_a($uploaderConfiguration['uploader'], UploaderInterface::class, true)) {
+        if ($hasCustomUploader && !is_a($uploaderConfiguration['uploader'], UploaderInterface::class, true)) {
             throw new Exception('Invalid uploader class provided for '.$this->crudObjectType.' type: '.$crudObject['type']);
         }
 
         if ($hasCustomUploader) {
-            return $uploaderConfiguration['uploader']::for($crudObject, $uploaderConfiguration);
+            return $uploaderConfiguration['uploader']::for ($crudObject, $uploaderConfiguration);
         }
 
         $uploader = app('UploadersRepository')->hasUploadFor($crudObject['type'], $this->macro);
 
         if ($uploader) {
-            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for($crudObject, $uploaderConfiguration);
+            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for ($crudObject, $uploaderConfiguration);
         }
 
         throw new Exception('Undefined upload type for '.$this->crudObjectType.' type: '.$crudObject['type']);
@@ -197,7 +197,7 @@ 
 
     private function getSubfieldModel(array $subfield, UploaderInterface $uploader)
     {
-        if (! $uploader->isRelationship()) {
+        if (!$uploader->isRelationship()) {
             return $subfield['baseModel'] ?? get_class(app('crud')->getModel());
         }
 
@@ -210,7 +210,7 @@ 
 
     private function getModelForRetrieveEvent(string $model, UploaderInterface $uploader)
     {
-        if (! $uploader->isRelationship()) {
+        if (!$uploader->isRelationship()) {
             return $model;
         }
 

src/app/Library/Uploaders/SingleBase64Image.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
         $value = $value ?? CRUD::getRequest()->get($this->getName());
         $previousImage = $this->getPreviousFiles($entry);
 
-        if (! $value && $previousImage) {
+        if (!$value && $previousImage) {
             Storage::disk($this->getDisk())->delete($previousImage);
 
             return null;
@@ -68,6 +68,6 @@ 
 
     public function shouldKeepPreviousValueUnchanged(Model $entry, $entryValue): bool
     {
-        return $entry->exists && is_string($entryValue) && ! Str::startsWith($entryValue, 'data:image');
+        return $entry->exists && is_string($entryValue) && !Str::startsWith($entryValue, 'data:image');
     }
 }

src/app/Library/Uploaders/Uploader.php

Spacing +10 added lines, -10 removed lines

@@ -25,7 +25,7 @@ 
 
     private bool $deleteWhenEntryIsDeleted = true;
 
-    private bool|string $attachedToFakeField = false;
+    private bool | string $attachedToFakeField = false;
 
     /**
      * Cloud disks have the ability to generate temporary URLs to files, should we do it?
@@ -62,7 +62,7 @@ 
     /*******************************
      * Static methods
      *******************************/
-    public static function for(array $crudObject, array $definition): UploaderInterface
+    public static function for (array $crudObject, array $definition): UploaderInterface
     {
         return new static($crudObject, $definition);
     }
@@ -102,7 +102,7 @@ 
 
     public function deleteUploadedFiles(Model $entry): void
     {
-        if (! in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
+        if (!in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
             $this->performFileDeletion($entry);
 
             return;
@@ -177,7 +177,7 @@ 
 
     public function getPreviousFiles(Model $entry): mixed
     {
-        if (! $this->attachedToFakeField) {
+        if (!$this->attachedToFakeField) {
             return $this->getOriginalValue($entry);
         }
         $value = $this->getOriginalValue($entry, $this->attachedToFakeField);
@@ -196,7 +196,7 @@ 
         return $this->attachedToFakeField !== false;
     }
 
-    public function getFakeAttribute(): bool|string
+    public function getFakeAttribute(): bool | string
     {
         return $this->attachedToFakeField;
     }
@@ -223,7 +223,7 @@ 
         return $this;
     }
 
-    public function fake(bool|string $isFake): self
+    public function fake(bool | string $isFake): self
     {
         $this->attachedToFakeField = $isFake;
 
@@ -254,7 +254,7 @@ 
         }
 
         if ($this->handleMultipleFiles) {
-            if (! isset($entry->getCasts()[$this->getName()]) && is_string($value)) {
+            if (!isset($entry->getCasts()[$this->getName()]) && is_string($value)) {
                 $entry->{$this->getAttributeName()} = json_decode($value, true);
             }
 
@@ -268,7 +268,7 @@ 
 
     protected function deleteFiles(Model $entry)
     {
-        if (! $this->shouldDeleteFiles()) {
+        if (!$this->shouldDeleteFiles()) {
             return;
         }
 
@@ -303,7 +303,7 @@ 
 
     private function performFileDeletion(Model $entry)
     {
-        if (! $this->handleRepeatableFiles && $this->deleteWhenEntryIsDeleted) {
+        if (!$this->handleRepeatableFiles && $this->deleteWhenEntryIsDeleted) {
             $this->deleteFiles($entry);
 
             return;
@@ -332,7 +332,7 @@ 
 
         $previousValue = $entry->getOriginal($field);
 
-        if (! $previousValue) {
+        if (!$previousValue) {
             return $previousValue;
         }