Laravel-Backpack / CRUD

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

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

Spacing +3 added lines, -3 removed lines

@@ -26,9 +26,9 @@ 
         $itemKeys = $this->model->query()->select($primaryKey)->get()->pluck($primaryKey);
 
         // filter the items that are not in the database and map the request
-        $reorderItems = collect($request)->filter(function ($item) use ($itemKeys) {
+        $reorderItems = collect($request)->filter(function($item) use ($itemKeys) {
             return $item['item_id'] !== '' && $item['item_id'] !== null && $itemKeys->contains($item['item_id']);
-        })->map(function ($item) use ($primaryKey) {
+        })->map(function($item) use ($primaryKey) {
             $item[$primaryKey] = (int) $item['item_id'];
             $item['parent_id'] = empty($item['parent_id']) ? null : (int) $item['parent_id'];
             $item['depth'] = empty($item['depth']) ? null : (int) $item['depth'];
@@ -41,7 +41,7 @@ 
         })->toArray();
 
         // wrap the queries in a transaction to avoid partial updates
-        DB::transaction(function () use ($reorderItems, $primaryKey, $itemKeys) {
+        DB::transaction(function() use ($reorderItems, $primaryKey, $itemKeys) {
             // create a string of ?,?,?,? to use as bind placeholders for item keys
             $reorderItemsBindString = implode(',', array_fill(0, count($reorderItems), '?'));
 

tests/config/CrudPanel/BaseCrudPanel.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
     {
         parent::setUp();
 
-        $this->app->singleton('crud', function ($app) {
+        $this->app->singleton('crud', function($app) {
             return new CrudPanel($app);
         });
         $this->crudPanel = app('crud');
@@ -43,7 +43,7 @@ 
         $app['config']->set('database.default', 'testing');
         $app['config']->set('backpack.base.route_prefix', 'admin');
 
-        $app->bind('App\Http\Middleware\CheckIfAdmin', function () {
+        $app->bind('App\Http\Middleware\CheckIfAdmin', function() {
             return new class
             {
                 public function handle($request, $next)