Laravel-Backpack / CRUD

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

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