Laravel-Backpack / CRUD

tests/Feature/UploadersValidationTest.php

Spacing +12 added lines, -12 removed lines

@@ -62,7 +62,7 @@ 
 
         $this->assertDatabaseHas('uploaders', [
             'upload' => 'avatar1.jpg',
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
         ]);
 
         $this->assertTrue(Storage::disk('uploaders')->exists('avatar1.jpg'));
@@ -97,8 +97,8 @@ 
 
         $response = $this->put($this->testBaseUrl.'/1', [
             'upload' => $this->getUploadedFile('avatar4.jpg'),
-            'upload_multiple' => $this->getUploadedFiles(['avatar5.jpg',  'avatar6.jpg']),
-            'clear_upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
+            'upload_multiple' => $this->getUploadedFiles(['avatar5.jpg', 'avatar6.jpg']),
+            'clear_upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
             'id' => 1,
         ]);
 
@@ -110,7 +110,7 @@ 
 
         $this->assertDatabaseHas('uploaders', [
             'upload' => 'avatar4.jpg',
-            'upload_multiple' => json_encode(['avatar5.jpg',  'avatar6.jpg']),
+            'upload_multiple' => json_encode(['avatar5.jpg', 'avatar6.jpg']),
         ]);
 
         $files = Storage::disk('uploaders')->allFiles();
@@ -142,7 +142,7 @@ 
         self::initUploaderWithFiles();
 
         $response = $this->put($this->testBaseUrl.'/1', [
-            'upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
+            'upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
             'id' => 1,
         ]);
 
@@ -154,7 +154,7 @@ 
 
         $this->assertDatabaseHas('uploaders', [
             'upload' => 'avatar1.jpg',
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
         ]);
 
         $files = Storage::disk('uploaders')->allFiles();
@@ -171,7 +171,7 @@ 
         self::initUploaderWithFiles();
 
         $response = $this->put($this->testBaseUrl.'/1', [
-            'upload_multiple' => $this->getUploadedFiles(['avatar4.jpg',  'avatar5.jpg']),
+            'upload_multiple' => $this->getUploadedFiles(['avatar4.jpg', 'avatar5.jpg']),
             'clear_upload_multiple' => ['avatar2.jpg'],
             'id' => 1,
         ]);
@@ -184,7 +184,7 @@ 
 
         $this->assertDatabaseHas('uploaders', [
             'upload' => 'avatar1.jpg',
-            'upload_multiple' => json_encode(['avatar3.jpg', 'avatar4.jpg',  'avatar5.jpg']),
+            'upload_multiple' => json_encode(['avatar3.jpg', 'avatar4.jpg', 'avatar5.jpg']),
         ]);
 
         $files = Storage::disk('uploaders')->allFiles();
@@ -341,7 +341,7 @@ 
         $response = $this->put($this->testBaseUrl.'/1', [
             'upload' => null,
             'upload_multiple' => null,
-            'clear_upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
+            'clear_upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
             'id' => 1,
         ]);
 
@@ -364,7 +364,7 @@ 
 
         $response = $this->put($this->testBaseUrl.'/1', [
             'upload_multiple' => $this->getUploadedFiles(['avatar2.jpg']),
-            'clear_upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
+            'clear_upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
             'id' => 1,
         ]);
 
@@ -386,7 +386,7 @@ 
 
         Uploader::create([
             'upload' => 'avatar1.jpg',
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
         ]);
     }
 
@@ -398,7 +398,7 @@ 
 
         Uploader::create([
             'upload' => 'avatar1.jpg',
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
         ]);
     }
 

tests/config/Uploads/HasUploadedFiles.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
     protected function getUploadedFiles(array $fileNames, string $mime = 'image/jpg')
     {
-        return array_map(function ($fileName) use ($mime) {
+        return array_map(function($fileName) use ($mime) {
             return new UploadedFile(__DIR__.'/assets/'.$fileName, $fileName, $mime, null, true);
         }, $fileNames);
     }

src/app/Library/Database/DatabaseSchema.php

Spacing +6 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
     {
         $connection = $connection ?: config('database.default');
 
-        self::$schema[$connection] = LazyCollection::make(self::getCreateSchema($connection)->getTables())->mapWithKeys(function ($table, $key) use ($connection) {
+        self::$schema[$connection] = LazyCollection::make(self::getCreateSchema($connection)->getTables())->mapWithKeys(function($table, $key) use ($connection) {
             $tableName = is_array($table) ? $table['name'] : $table->getName();
 
             if ($existingTable = self::$schema[$connection][$tableName] ?? false) {
@@ -65,7 +65,7 @@ 
      */
     private static function generateDatabaseSchema(string $connection, string $table)
     {
-        if (! isset(self::$schema[$connection][$table])) {
+        if (!isset(self::$schema[$connection][$table])) {
             self::$schema[$connection][$table] = self::mapTable($connection, $table);
         }
     }
@@ -74,13 +74,12 @@ 
     {
         try {
             $table = method_exists(self::getCreateSchema($connection), 'getTable') ?
-                        self::getCreateSchema($connection)->getTable($tableName) :
-                        self::getCreateSchema($connection)->getColumns($tableName);
+                        self::getCreateSchema($connection)->getTable($tableName) : self::getCreateSchema($connection)->getColumns($tableName);
         } catch (\Exception $e) {
             return new Table($tableName, []);
         }
 
-        if (! is_array($table)) {
+        if (!is_array($table)) {
             return $table;
         }
 
@@ -91,7 +90,7 @@ 
         $schemaManager = self::getSchemaManager($connection);
         $indexes = $schemaManager->getIndexes($tableName);
 
-        $indexes = array_map(function ($index) {
+        $indexes = array_map(function($index) {
             return $index['columns'];
         }, $indexes);
 
@@ -109,7 +108,7 @@ 
 
         $indexes = self::$schema[$connection][$table]->getIndexes();
 
-        $indexes = \Illuminate\Support\Arr::flatten(array_map(function ($index) {
+        $indexes = \Illuminate\Support\Arr::flatten(array_map(function($index) {
             return is_string($index) ? $index : $index->getColumns();
         }, $indexes));
 

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

Spacing +13 added lines, -13 removed lines

@@ -34,7 +34,7 @@ 
      */
     public function markAsHandled(string $objectName): void
     {
-        if (! in_array($objectName, $this->handledUploaders)) {
+        if (!in_array($objectName, $this->handledUploaders)) {
             $this->handledUploaders[] = $objectName;
         }
     }
@@ -60,7 +60,7 @@ 
      */
     public function getUploadFor(string $objectType, string $group): string
     {
-        if (! $this->hasUploadFor($objectType, $group)) {
+        if (!$this->hasUploadFor($objectType, $group)) {
             throw new \Exception('There is no uploader defined for the given field type.');
         }
 
@@ -82,7 +82,7 @@ 
     {
         // ensure all uploaders implement the UploaderInterface
         foreach ($uploaders as $uploader) {
-            if (! is_a($uploader, UploaderInterface::class, true)) {
+            if (!is_a($uploader, UploaderInterface::class, true)) {
                 throw new \Exception('The uploader class must implement the UploaderInterface.');
             }
         }
@@ -102,7 +102,7 @@ 
      */
     public function registerRepeatableUploader(string $uploadName, UploaderInterface $uploader): void
     {
-        if (! array_key_exists($uploadName, $this->repeatableUploaders) || ! in_array($uploader, $this->repeatableUploaders[$uploadName])) {
+        if (!array_key_exists($uploadName, $this->repeatableUploaders) || !in_array($uploader, $this->repeatableUploaders[$uploadName])) {
             $this->repeatableUploaders[$uploadName][] = $uploader;
         }
     }
@@ -136,7 +136,7 @@ 
      */
     public function getRegisteredUploadNames(string $uploadName): array
     {
-        return array_map(function ($uploader) {
+        return array_map(function($uploader) {
             return $uploader->getName();
         }, $this->getRepeatableUploadersFor($uploadName));
     }
@@ -167,11 +167,11 @@ 
 
             $uploaders = $this->getRepeatableUploadersFor($repeatableContainerName);
 
-            $uploader = Arr::first($uploaders, function ($uploader) use ($requestInputName) {
+            $uploader = Arr::first($uploaders, function($uploader) use ($requestInputName) {
                 return $uploader->getName() === $requestInputName;
             });
 
-            if (! $uploader) {
+            if (!$uploader) {
                 abort(500, 'Could not find the field in the repeatable uploaders.');
             }
 
@@ -182,16 +182,16 @@ 
             abort(500, 'Could not find the field in the CRUD fields.');
         }
 
-        if (! $uploaderMacro = $this->getUploadCrudObjectMacroType($crudObject)) {
+        if (!$uploaderMacro = $this->getUploadCrudObjectMacroType($crudObject)) {
             abort(500, 'There is no uploader defined for the given field type.');
         }
 
-        if (! $this->isValidUploadField($crudObject, $uploaderMacro)) {
+        if (!$this->isValidUploadField($crudObject, $uploaderMacro)) {
             abort(500, 'Invalid field for upload.');
         }
 
         $uploaderConfiguration = $crudObject[$uploaderMacro] ?? [];
-        $uploaderConfiguration = ! is_array($uploaderConfiguration) ? [] : $uploaderConfiguration;
+        $uploaderConfiguration = !is_array($uploaderConfiguration) ? [] : $uploaderConfiguration;
         $uploaderClass = $this->getUploadFor($crudObject['type'], $uploaderMacro);
 
         return new $uploaderClass(['name' => $requestInputName], $uploaderConfiguration);
@@ -200,7 +200,7 @@ 
     /**
      * Get the upload field macro type for the given object.
      */
-    private function getUploadCrudObjectMacroType(array $crudObject): string|null
+    private function getUploadCrudObjectMacroType(array $crudObject): string | null
     {
         $uploadersGroups = $this->getUploadersGroupsNames();
 
@@ -217,11 +217,11 @@ 
     {
         if (Str::contains($crudObject['name'], '#')) {
             $container = Str::before($crudObject['name'], '#');
-            $field = array_filter(CRUD::fields()[$container]['subfields'] ?? [], function ($item) use ($crudObject, $uploaderMacro) {
+            $field = array_filter(CRUD::fields()[$container]['subfields'] ?? [], function($item) use ($crudObject, $uploaderMacro) {
                 return $item['name'] === $crudObject['name'] && in_array($item['type'], $this->getAjaxUploadTypes($uploaderMacro));
             });
 
-            return ! empty($field);
+            return !empty($field);
         }
 
         return in_array($crudObject['type'], $this->getAjaxUploadTypes($uploaderMacro));

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/Traits/Read.php

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
     {
         $entry = $this->getCurrentEntry();
 
-        if (! $entry) {
+        if (!$entry) {
             return false;
         }
 
@@ -68,7 +68,7 @@ 
      */
     public function getEntry($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             $this->entry = $this->getModelWithCrudPanelQuery()->findOrFail($id);
             $this->entry = $this->entry->withFakes();
         }
@@ -76,7 +76,7 @@ 
         return $this->entry;
     }
 
-    private function shouldUseFallbackLocale(): bool|string
+    private function shouldUseFallbackLocale(): bool | string
     {
         $fallbackRequestValue = $this->getRequest()->get('_fallback_locale');
 
@@ -92,7 +92,7 @@ 
      */
     public function getEntryWithLocale($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             $this->entry = $this->getEntry($id);
         }
 
@@ -174,7 +174,7 @@ 
      */
     public function enableDetailsRow()
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             throw new BackpackProRequiredException('Details row');
         }
 
@@ -273,7 +273,7 @@ 
                 // we will apply the same labels as the values to the menu if developer didn't
                 $this->abortIfInvalidPageLength($menu[0]);
 
-                if (! isset($menu[1]) || ! is_array($menu[1])) {
+                if (!isset($menu[1]) || !is_array($menu[1])) {
                     $menu[1] = $menu[0];
                 }
             } else {
@@ -361,7 +361,7 @@ 
      */
     public function enableExportButtons()
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             throw new BackpackProRequiredException('Export buttons');
         }