Laravel-Backpack / CRUD

tests/Unit/CrudPanel/CrudPanelFiltersTest.php

Spacing +8 added lines, -8 removed lines

@@ -48,7 +48,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users', 'GET', ['my_custom_filter' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('GET', 'admin/users', ['UserCrudController', 'index']))->bind($request);
         });
         $this->crudPanel->setRequest($request);
@@ -64,10 +64,10 @@ 
                     ->options(['test' => 'test'])
                     ->view('simple')
                     ->viewNamespace('crud::filters')
-                    ->ifActive(function () {
+                    ->ifActive(function() {
                         return true;
                     })
-                    ->ifInactive(function () {
+                    ->ifInactive(function() {
                         return true;
                     });
 
@@ -84,10 +84,10 @@ 
     public function testWhenActiveAndWhenInactiveAliases()
     {
         $filter = CrudFilter::name('my_filter')
-                        ->whenActive(function () {
+                        ->whenActive(function() {
                             return true;
                         })
-                        ->whenInactive(function () {
+                        ->whenInactive(function() {
                             return true;
                         });
 
@@ -98,7 +98,7 @@ 
 
     public function testWhenNotActiveAlias()
     {
-        $filter = CrudFilter::name('my_filter')->whenNotActive(function () {
+        $filter = CrudFilter::name('my_filter')->whenNotActive(function() {
             return true;
         });
         $this->assertCount(1, $this->crudPanel->filters());
@@ -107,7 +107,7 @@ 
 
     public function testIfNotActiveAlias()
     {
-        $filter = CrudFilter::name('my_filter')->ifNotActive(function () {
+        $filter = CrudFilter::name('my_filter')->ifNotActive(function() {
             return true;
         });
         $this->assertCount(1, $this->crudPanel->filters());
@@ -116,7 +116,7 @@ 
 
     public function testElseAlias()
     {
-        $filter = CrudFilter::name('my_filter')->else(function () {
+        $filter = CrudFilter::name('my_filter')->else(function() {
             return true;
         });
         $this->assertCount(1, $this->crudPanel->filters());

src/app/Library/CrudPanel/CrudPanel.php

Spacing +16 added lines, -16 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);
         }
 
@@ -205,7 +205,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);
         }
 
@@ -303,7 +303,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;
         });
     }
@@ -322,8 +322,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);
                 }
@@ -353,15 +353,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();
 
@@ -396,7 +396,7 @@ 
             if (is_array($entries)) {
                 //if attribute does not exist in main array we have more than one entry OR the attribute
                 //is an acessor 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 acessor that is not in appends.
                     if ($model_instance->hasGetMutator($attribute) && isset($entries[$modelKey])) {
                         $entry_in_database = $model_instance->find($entries[$modelKey]);
@@ -435,21 +435,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);
         } 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 {
@@ -475,7 +475,7 @@ 
         $relation = $model->{$firstRelationName};
 
         $results = [];
-        if (! is_null($relation)) {
+        if (!is_null($relation)) {
             if ($relation instanceof Collection) {
                 $currentResults = $relation->all();
             } elseif (is_array($relation)) {
@@ -488,7 +488,7 @@ 
 
             array_shift($relationArray);
 
-            if (! empty($relationArray)) {
+            if (!empty($relationArray)) {
                 foreach ($currentResults as $currentResult) {
                     $results = array_merge_recursive($results, $this->getRelatedEntries($currentResult, implode('.', $relationArray)));
                 }

tests/Unit/CrudPanel/CrudPanelSaveActionsTest.php

Spacing +6 added lines, -6 removed lines

@@ -25,10 +25,10 @@ 
         $this->singleSaveAction = [
             'name' => 'save_action_one',
             'button_text' => 'custom',
-            'redirect' => function ($crud, $request, $itemId) {
+            'redirect' => function($crud, $request, $itemId) {
                 return $crud->route;
             },
-            'visible' => function ($crud) {
+            'visible' => function($crud) {
                 return true;
             },
         ];
@@ -36,19 +36,19 @@ 
         $this->multipleSaveActions = [
             [
                 'name' => 'save_action_one',
-                'redirect' => function ($crud, $request, $itemId) {
+                'redirect' => function($crud, $request, $itemId) {
                     return $crud->route;
                 },
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return true;
                 },
             ],
             [
                 'name' => 'save_action_two',
-                'redirect' => function ($crud, $request, $itemId) {
+                'redirect' => function($crud, $request, $itemId) {
                     return $crud->route;
                 },
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return true;
                 },
             ],

tests/Unit/CrudPanel/CrudPanelButtonsTest.php

Spacing +2 added lines, -2 removed lines

@@ -285,7 +285,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());
@@ -295,7 +295,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;
         });
     }

src/app/Http/Controllers/CrudController.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
         //
         // It's done inside a middleware closure in order to have
         // the complete request inside the CrudPanel object.
-        $this->middleware(function ($request, $next) {
+        $this->middleware(function($request, $next) {
             $this->crud = app('crud');
 
             $this->crud->setRequest($request);
@@ -96,7 +96,7 @@ 
     protected function setupConfigurationForCurrentOperation()
     {
         $operationName = $this->crud->getCurrentOperation();
-        if (! $operationName) {
+        if (!$operationName) {
             return;
         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -99,7 +99,7 @@ 
      */
     public function customOrderBy($column, $columnDirection = 'asc')
     {
-        if (! isset($column['orderLogic'])) {
+        if (!isset($column['orderLogic'])) {
             return $this->query;
         }
 
@@ -189,7 +189,7 @@ 
      */
     public function getTotalQueryCount()
     {
-        if (! $this->getOperationSetting('showEntryCount')) {
+        if (!$this->getOperationSetting('showEntryCount')) {
             return 0;
         }
 
@@ -228,7 +228,7 @@ 
      */
     private function getCountFromQuery(Builder $query)
     {
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return $query->count();
         }
 
@@ -247,7 +247,7 @@ 
         $expressionColumns = [];
 
         foreach ($crudQuery->columns ?? [] as $column) {
-            if (! is_string($column) && is_a($column, 'Illuminate\Database\Query\Expression')) {
+            if (!is_string($column) && is_a($column, 'Illuminate\Database\Query\Expression')) {
                 $expressionColumns[] = $column;
             }
         }

src/app/Models/Traits/HasEnumFields.php

Spacing +1 added lines, -2 removed lines

@@ -23,8 +23,7 @@
 
         try {
             $select = app()->version() < 10 ?
-                        DB::raw('SHOW COLUMNS FROM `'.$table_prefix.$instance->getTable().'` WHERE Field = "'.$field_name.'"') :
-                        DB::raw('SHOW COLUMNS FROM `'.$table_prefix.$instance->getTable().'` WHERE Field = "'.$field_name.'"')->getValue($connection->getQueryGrammar());
+                        DB::raw('SHOW COLUMNS FROM `'.$table_prefix.$instance->getTable().'` WHERE Field = "'.$field_name.'"') : DB::raw('SHOW COLUMNS FROM `'.$table_prefix.$instance->getTable().'` WHERE Field = "'.$field_name.'"')->getValue($connection->getQueryGrammar());
 
             $type = $connection->select($select)[0]->Type;
         } catch (\Exception $e) {

src/app/Models/Traits/HasIdentifiableAttribute.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         // we get the first column from database
         // that is NOT indexed (usually primary, foreign keys)
         foreach ($columns as $columnName => $columnProperties) {
-            if (! in_array($columnName, $indexedColumns)) {
+            if (!in_array($columnName, $indexedColumns)) {
                 //check for convention "field<_id>" in case developer didn't add foreign key constraints.
                 if (strpos($columnName, '_id') !== false) {
                     continue;

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...