Laravel-Backpack / CRUD

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/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;
         }
 

src/app/Library/Uploaders/Support/Traits/HandleRepeatableUploads.php

Spacing +23 added lines, -23 removed lines

@@ -18,7 +18,7 @@ 
 {
     public bool $handleRepeatableFiles = false;
 
-    public null|string $repeatableContainerName = null;
+    public null | string $repeatableContainerName = null;
 
     /*******************************
      * Setters - fluently configure the uploader
@@ -35,7 +35,7 @@ 
     /*******************************
      * Getters
      *******************************/
-    public function getRepeatableContainerName(): null|string
+    public function getRepeatableContainerName(): null | string
     {
         return $this->repeatableContainerName;
     }
@@ -162,7 +162,7 @@ 
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $uploader) {
             $uploadedValues = $uploader->uploadRepeatableFiles($values->pluck($uploader->getAttributeName())->toArray(), $this->getPreviousRepeatableValues($entry, $uploader));
 
-            $values = $values->map(function ($item, $key) use ($uploadedValues, $uploader) {
+            $values = $values->map(function($item, $key) use ($uploadedValues, $uploader) {
                 $item[$uploader->getAttributeName()] = $uploadedValues[$key] ?? null;
 
                 return $item;
@@ -193,7 +193,7 @@ 
 
         $values = $entry->{$this->getRepeatableContainerName()};
         $values = is_string($values) ? json_decode($values, true) : $values;
-        $values = array_map(function ($item) use ($repeatableUploaders) {
+        $values = array_map(function($item) use ($repeatableUploaders) {
             foreach ($repeatableUploaders as $upload) {
                 $item[$upload->getAttributeName()] = $this->getValuesWithPathStripped($item, $upload);
             }
@@ -208,7 +208,7 @@ 
 
     private function retrieveRepeatableRelationFiles(Model $entry)
     {
-        switch($this->getRepeatableRelationType()) {
+        switch ($this->getRepeatableRelationType()) {
             case 'BelongsToMany':
             case 'MorphToMany':
                 $pivotClass = app('crud')->getModel()->{$this->getUploaderSubfield()['baseEntity']}()->getPivotClass();
@@ -261,12 +261,12 @@ 
         $repeatableValues ??= collect($entry->{$this->getRepeatableContainerName()});
 
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $upload) {
-            if (! $upload->shouldDeleteFiles()) {
+            if (!$upload->shouldDeleteFiles()) {
                 continue;
             }
             $values = $repeatableValues->pluck($upload->getName())->toArray();
             foreach ($values as $value) {
-                if (! $value) {
+                if (!$value) {
                     continue;
                 }
 
@@ -289,7 +289,7 @@ 
     /**
      * Given two multidimensional arrays/collections, merge them recursively.
      */
-    protected function mergeValuesRecursive(array|Collection $array1, array|Collection $array2): array|Collection
+    protected function mergeValuesRecursive(array | Collection $array1, array | Collection $array2): array | Collection
     {
         $merged = $array1;
         foreach ($array2 as $key => &$value) {
@@ -311,7 +311,7 @@ 
     {
         $items = CRUD::getRequest()->input('_order_'.$this->getRepeatableContainerName()) ?? [];
 
-        array_walk($items, function (&$key, $value) {
+        array_walk($items, function(&$key, $value) {
             $requestValue = $key[$this->getName()] ?? null;
             $key = $this->handleMultipleFiles ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
         });
@@ -323,22 +323,22 @@ 
     {
         $previousValues = $entry->getOriginal($uploader->getRepeatableContainerName());
 
-        if (! is_array($previousValues)) {
+        if (!is_array($previousValues)) {
             $previousValues = json_decode($previousValues, true);
         }
 
-        if (! empty($previousValues)) {
+        if (!empty($previousValues)) {
             $previousValues = array_column($previousValues, $uploader->getName());
         }
 
         return $previousValues ?? [];
     }
 
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $uploader)
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $uploader)
     {
         $uploadedValues = $item[$uploader->getName()] ?? null;
         if (is_array($uploadedValues)) {
-            return array_map(function ($value) use ($uploader) {
+            return array_map(function($value) use ($uploader) {
                 return $uploader->getValueWithoutPath($value);
             }, $uploadedValues);
         }
@@ -348,8 +348,8 @@ 
 
     private function deleteRelationshipFiles(Model $entry): void
     {
-        if (! is_a($entry, Pivot::class, true) &&
-            ! $entry->relationLoaded($this->getRepeatableContainerName()) &&
+        if (!is_a($entry, Pivot::class, true) &&
+            !$entry->relationLoaded($this->getRepeatableContainerName()) &&
             method_exists($entry, $this->getRepeatableContainerName())
         ) {
             $entry->loadMissing($this->getRepeatableContainerName());
@@ -376,8 +376,8 @@ 
             $relatedEntries = $entry->{$this->getRepeatableContainerName()} ?? [];
         }
 
-        if (! is_a($relatedEntries ?? '', Collection::class, true)) {
-            $relatedEntries = ! empty($relatedEntries) ? [$relatedEntries] : [$entry];
+        if (!is_a($relatedEntries ?? '', Collection::class, true)) {
+            $relatedEntries = !empty($relatedEntries) ? [$relatedEntries] : [$entry];
         }
 
         foreach ($relatedEntries as $relatedEntry) {
@@ -385,9 +385,9 @@ 
         }
     }
 
-    protected function deletePivotFiles(Pivot|Model $entry)
+    protected function deletePivotFiles(Pivot | Model $entry)
     {
-        if (! is_a($entry, Pivot::class, true)) {
+        if (!is_a($entry, Pivot::class, true)) {
             $pivots = $entry->{$this->getRepeatableContainerName()};
             foreach ($pivots as $pivot) {
                 $this->deletePivotModelFiles($pivot);
@@ -397,24 +397,24 @@ 
         }
 
         $pivotAttributes = $entry->getAttributes();
-        $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function ($item) use ($pivotAttributes) {
+        $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function($item) use ($pivotAttributes) {
             $itemPivotAttributes = $item->pivot->only(array_keys($pivotAttributes));
 
             return $itemPivotAttributes === $pivotAttributes;
         })->first();
 
-        if (! $connectedPivot) {
+        if (!$connectedPivot) {
             return;
         }
 
         $this->deletePivotModelFiles($connectedPivot);
     }
 
-    private function deletePivotModelFiles(Pivot|Model $entry)
+    private function deletePivotModelFiles(Pivot | Model $entry)
     {
         $files = $entry->getOriginal()['pivot_'.$this->getAttributeName()];
 
-        if (! $files) {
+        if (!$files) {
             return;
         }
 

src/app/Console/Commands/PublishHeaderMetas.php

Spacing +3 added lines, -3 removed lines

@@ -60,7 +60,7 @@ 
         ];
 
         foreach ($filesToPublish as $destination => $stub) {
-            if (! is_string($destination)) {
+            if (!is_string($destination)) {
                 $destination = $stub;
                 $stub = null;
             }
@@ -71,11 +71,11 @@ 
                 continue;
             }
 
-            if (! File::isDirectory(dirname($destination))) {
+            if (!File::isDirectory(dirname($destination))) {
                 File::makeDirectory(dirname($destination), 0755, true);
             }
 
-            if (! $stub) {
+            if (!$stub) {
                 File::copy(__DIR__.'/../../../public/'.basename($destination), $destination);
                 $this->info("File {$destination} published.");
                 continue;

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

Spacing +7 added lines, -7 removed lines

@@ -74,7 +74,7 @@ 
     /**
      * Add a column at the end of the CRUD object's "columns" array and return it.
      */
-    public function addAndReturnColumn(array|string $column): CrudColumn
+    public function addAndReturnColumn(array | string $column): CrudColumn
     {
         $column = $this->prepareAttributesAndAddColumn($column);
 
@@ -122,7 +122,7 @@ 
      */
     public function makeFirstColumn()
     {
-        if (! $this->columns()) {
+        if (!$this->columns()) {
             return false;
         }
 
@@ -166,7 +166,7 @@ 
      */
     public function removeColumns($columns)
     {
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             foreach ($columns as $columnKey) {
                 $this->removeColumn($columnKey);
             }
@@ -331,7 +331,7 @@ 
      */
     public function hasColumnWhere($attribute, $value)
     {
-        $match = Arr::first($this->columns(), function ($column, $columnKey) use ($attribute, $value) {
+        $match = Arr::first($this->columns(), function($column, $columnKey) use ($attribute, $value) {
             return isset($column[$attribute]) && $column[$attribute] == $value;
         });
 
@@ -347,7 +347,7 @@ 
      */
     public function firstColumnWhere($attribute, $value)
     {
-        return Arr::first($this->columns(), function ($column, $columnKey) use ($attribute, $value) {
+        return Arr::first($this->columns(), function($column, $columnKey) use ($attribute, $value) {
             return isset($column[$attribute]) && $column[$attribute] == $value;
         });
     }
@@ -396,8 +396,8 @@ 
      */
     public function countColumnsWithoutActions()
     {
-        return collect($this->columns())->filter(function ($column, $key) {
-            return ! isset($column['hasActions']) || $column['hasActions'] == false;
+        return collect($this->columns())->filter(function($column, $key) {
+            return !isset($column['hasActions']) || $column['hasActions'] == false;
         })->count();
     }
 

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

Spacing +8 added lines, -8 removed lines

@@ -54,7 +54,7 @@ 
     {
         $entry = $this->getCurrentEntry();
 
-        if (! $entry) {
+        if (!$entry) {
             return false;
         }
 
@@ -69,7 +69,7 @@ 
      */
     public function getEntry($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             if ($this->getOperationSetting('eagerLoadRelationships')) {
                 $this->eagerLoadRelationshipFields();
             }
@@ -80,7 +80,7 @@ 
         return $this->entry;
     }
 
-    private function shouldUseFallbackLocale(): bool|string
+    private function shouldUseFallbackLocale(): bool | string
     {
         $fallbackRequestValue = $this->getRequest()->get('_fallback_locale');
 
@@ -96,7 +96,7 @@ 
      */
     public function getEntryWithLocale($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             $this->entry = $this->getEntry($id);
         }
 
@@ -147,7 +147,7 @@ 
         foreach ($crudObjects as $crudObjectName => $attributes) {
             $relationString = isset($attributes['entity']) && $attributes['entity'] !== false ? $attributes['entity'] : '';
 
-            if (! $relationString) {
+            if (!$relationString) {
                 continue;
             }
 
@@ -213,7 +213,7 @@ 
      */
     public function enableDetailsRow()
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             throw new BackpackProRequiredException('Details row');
         }
 
@@ -312,7 +312,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 {
@@ -400,7 +400,7 @@ 
      */
     public function enableExportButtons()
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             throw new BackpackProRequiredException('Export buttons');
         }
 

src/routes/backpack/custom.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         (array) config('backpack.base.middleware_key', 'admin')
     ),
     'namespace' => 'App\Http\Controllers\Admin',
-], function () { // custom admin routes
+], function() { // custom admin routes
 }); // this should be the absolute last line of this file
 
 /**

tests/Unit/CrudPanel/CrudPanelButtonsTest.php

Spacing +3 added lines, -3 removed lines

@@ -276,7 +276,7 @@ 
     public function testAddButtonFluently()
     {
         $button1 = CrudButton::name('lineTest')->to('line')->view('crud::buttons.test')->type('view');
-        $button2 = CrudButton::add('modelFunction')->model_function(function () {
+        $button2 = CrudButton::add('modelFunction')->model_function(function() {
             return 'test';
         })->section('top')->makeFirst();
         $this->assertEquals($button1->toArray(), $this->crudPanel->buttons()->last()->toArray());
@@ -290,7 +290,7 @@ 
 
         $this->expectException(\Symfony\Component\HttpKernel\Exception\HttpException::class);
 
-        $this->crudPanel->modifyButton('unknownButton', function ($button) {
+        $this->crudPanel->modifyButton('unknownButton', function($button) {
             $button->name = 'newName';
         });
     }
@@ -342,7 +342,7 @@ 
 
     private function getButtonByName($name)
     {
-        return $this->crudPanel->buttons()->first(function ($value) use ($name) {
+        return $this->crudPanel->buttons()->first(function($value) use ($name) {
             return $value->name == $name;
         });
     }

tests/Unit/CrudPanel/CrudPanelAccessTest.php

Spacing +5 added lines, -5 removed lines

@@ -105,7 +105,7 @@ 
 
     public function testItCanUseAClosureToResolveAccess()
     {
-        $this->crudPanel->setAccessCondition('list', function () {
+        $this->crudPanel->setAccessCondition('list', function() {
             return true;
         });
 
@@ -116,7 +116,7 @@ 
 
     public function testItCanUseAClosureToResolveAccessForMultipleOperations()
     {
-        $this->crudPanel->setAccessCondition(['list', 'create'], function () {
+        $this->crudPanel->setAccessCondition(['list', 'create'], function() {
             return true;
         });
 
@@ -127,7 +127,7 @@ 
 
     public function testItCanCheckIfAnOperationHasAccessConditions()
     {
-        $this->crudPanel->setAccessCondition(['list', 'create'], function () {
+        $this->crudPanel->setAccessCondition(['list', 'create'], function() {
             return true;
         });
 
@@ -137,7 +137,7 @@ 
 
     public function testItCanCheckAccessToAll()
     {
-        $this->crudPanel->allowAccess(['list', 'create'], function () {
+        $this->crudPanel->allowAccess(['list', 'create'], function() {
             return true;
         });
 
@@ -147,7 +147,7 @@ 
 
     public function testItCanAllowAccessToSomeSpecificOperationWhileDenyingOthers()
     {
-        $this->crudPanel->allowAccess(['list', 'create'], function () {
+        $this->crudPanel->allowAccess(['list', 'create'], function() {
             return true;
         });