Laravel-Backpack / CRUD

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

Spacing +10 added lines, -10 removed lines

@@ -10,7 +10,7 @@ 
     /**
      * Set an operation as having access using the Settings API.
      */
-    public function allowAccess(array|string $operation): bool
+    public function allowAccess(array | string $operation): bool
     {
         foreach ((array) $operation as $op) {
             $this->set($op.'.access', true);
@@ -22,13 +22,13 @@ 
     /**
      * Disable the access to a certain operation, or the current one.
      */
-    public function denyAccess(array|string $operation): bool
+    public function denyAccess(array | string $operation): bool
     {
         foreach ((array) $operation as $op) {
             $this->set($op.'.access', false);
         }
 
-        return ! $this->hasAccessToAny($operation);
+        return !$this->hasAccessToAny($operation);
     }
 
     /**
@@ -52,7 +52,7 @@ 
     /**
      * Check if any operations are allowed for a Crud Panel. Return false if not.
      */
-    public function hasAccessToAny(array|string $operation_array, ?Model $entry = null): bool
+    public function hasAccessToAny(array | string $operation_array, ?Model $entry = null): bool
     {
         foreach ((array) $operation_array as $key => $operation) {
             if ($this->hasAccess($operation, $entry) == true) {
@@ -66,10 +66,10 @@ 
     /**
      * Check if all operations are allowed for a Crud Panel. Return false if not.
      */
-    public function hasAccessToAll(array|string $operation_array, ?Model $entry = null): bool
+    public function hasAccessToAll(array | string $operation_array, ?Model $entry = null): bool
     {
         foreach ((array) $operation_array as $key => $operation) {
-            if (! $this->hasAccess($operation, $entry)) {
+            if (!$this->hasAccess($operation, $entry)) {
                 return false;
             }
         }
@@ -84,7 +84,7 @@ 
      */
     public function hasAccessOrFail(string $operation, ?Model $entry = null): bool
     {
-        if (! $this->hasAccess($operation, $entry)) {
+        if (!$this->hasAccess($operation, $entry)) {
             throw new AccessDeniedException(trans('backpack::crud.unauthorized_access', ['access' => $operation]));
         }
 
@@ -95,7 +95,7 @@ 
      * Get an operation's access condition, if set. A condition
      * can be anything, but usually a boolean or a callable.
      */
-    public function getAccessCondition(string $operation): bool|callable|null
+    public function getAccessCondition(string $operation): bool | callable | null
     {
         return $this->get($operation.'.access');
     }
@@ -103,7 +103,7 @@ 
     /**
      * Set the condition under which an operation is allowed for a Crud Panel.
      */
-    public function setAccessCondition(array|string $operation, bool|callable|null $condition): void
+    public function setAccessCondition(array | string $operation, bool | callable | null $condition): void
     {
         foreach ((array) $operation as $op) {
             $this->set($op.'.access', $condition);
@@ -131,7 +131,7 @@ 
      * Allow access only to operations in the array.
      * By denying access to all other operations.
      */
-    public function allowAccessOnlyTo(array|string $operation): void
+    public function allowAccessOnlyTo(array | string $operation): void
     {
         $this->denyAllAccess();
         $this->allowAccess($operation);

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($autoInferedParameter) > 1) {
                 throw new \Exception("Route [{$route}] expects parameters [".implode(', ', $expectedParameters)."]. Insuficient parameters provided in column: [{$this->attributes['name']}].");
             }
-            $autoInferedParameter = current($autoInferedParameter) ? [current($autoInferedParameter) => function ($entry, $related_key, $column, $crud) {
+            $autoInferedParameter = current($autoInferedParameter) ? [current($autoInferedParameter) => 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();
 
@@ -117,17 +117,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}].");
         }
 
@@ -138,8 +138,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,
@@ -155,8 +155,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/routes/backpack/custom.php

Spacing +1 added lines, -1 removed lines

@@ -15,5 +15,5 @@
         (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

src/routes/backpack/base.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         'middleware' => config('backpack.base.web_middleware', 'web'),
         'prefix' => config('backpack.base.route_prefix'),
     ],
-    function () {
+    function() {
         // if not otherwise configured, setup the auth routes
         if (config('backpack.base.setup_auth_routes')) {
             // Authentication Routes...

src/routes/backpack/testing.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     (array) config('backpack.base.middleware_key', 'admin'),
     'prefix' => config('backpack.base.route_prefix', 'admin'),
 ],
-    function () {
+    function() {
         Route::crud('users', 'Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController');
     }
 );

tests/BaseTestClass.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
             (array) config('backpack.base.middleware_key', 'admin'),
             'prefix' => config('backpack.base.route_prefix', 'admin'),
         ],
-            function () {
+            function() {
                 Route::get('articles/{id}/show/{detail}', ['as' => 'article.show.detail', 'action' => 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController@detail']);
                 Route::crud('users', 'Backpack\CRUD\Tests\config\Http\Controllers\UserCrudController');
                 Route::crud('articles', 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController');

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +2 added lines, -2 removed lines

@@ -605,7 +605,7 @@ 
 
     public function testCreateBelongsToFake()
     {
-        $belongsToField = [   // select_grouped
+        $belongsToField = [// select_grouped
             'label' => 'Select_grouped',
             'type' => 'select_grouped', //https://github.com/Laravel-Backpack/CRUD/issues/502
             'name' => 'bang_relation_field',
@@ -741,7 +741,7 @@ 
                 'nickname' => 'i_have_has_one',
                 'profile_picture' => 'ohh my picture 1.jpg',
                 'article' => 1,
-                'addresses' => [ // HasOne is tested in other test function
+                'addresses' => [// HasOne is tested in other test function
                     [
                         'city' => 2,
                         'street' => 'test',

tests/Unit/CrudPanel/CrudPanelValidationTest.php

Spacing +4 added lines, -4 removed lines

@@ -45,7 +45,7 @@ 
             'password' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -85,7 +85,7 @@ 
             'email' => '',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -125,7 +125,7 @@ 
             'password' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -247,7 +247,7 @@ 
             'name' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 

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

Spacing +15 added lines, -15 removed lines

@@ -13,7 +13,7 @@ 
 {
     public bool $handleRepeatableFiles = false;
 
-    public null|string $repeatableContainerName = null;
+    public null | string $repeatableContainerName = null;
 
     /*******************************
      * Setters - fluently configure the uploader
@@ -30,7 +30,7 @@ 
     /*******************************
      * Getters
      *******************************/
-    public function getRepeatableContainerName(): null|string
+    public function getRepeatableContainerName(): null | string
     {
         return $this->repeatableContainerName;
     }
@@ -76,7 +76,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;
@@ -96,7 +96,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);
             }
@@ -111,7 +111,7 @@ 
 
     private function retrieveRepeatableRelationFiles(Model $entry)
     {
-        switch($this->getRepeatableRelationType()) {
+        switch ($this->getRepeatableRelationType()) {
             case 'BelongsToMany':
             case 'MorphToMany':
                 $pivotClass = app('crud')->getModel()->{$this->getUploaderSubfield()['baseEntity']}()->getPivotClass();
@@ -157,12 +157,12 @@ 
 
         $repeatableValues = collect($entry->{$this->getName()});
         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;
                 }
 
@@ -185,7 +185,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) {
@@ -207,7 +207,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;
         });
@@ -219,18 +219,18 @@ 
     {
         $previousValues = json_decode($entry->getOriginal($uploader->getRepeatableContainerName()), true);
 
-        if (! empty($previousValues)) {
+        if (!empty($previousValues)) {
             $previousValues = array_column($previousValues, $uploader->getName());
         }
 
         return $previousValues ?? [];
     }
 
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $upload)
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $upload)
     {
         $uploadedValues = $item[$upload->getName()] ?? null;
         if (is_array($uploadedValues)) {
-            return array_map(function ($value) use ($upload) {
+            return array_map(function($value) use ($upload) {
                 return Str::after($value, $upload->getPath());
             }, $uploadedValues);
         }
@@ -242,19 +242,19 @@ 
     {
         if (in_array($this->getRepeatableRelationType(), ['BelongsToMany', 'MorphToMany'])) {
             $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;
             }
 
             $files = $connectedPivot->getOriginal()['pivot_'.$this->getAttributeName()];
 
-            if (! $files) {
+            if (!$files) {
                 return;
             }