Laravel-Backpack / CRUD

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

Spacing +11 added lines, -11 removed lines

@@ -46,10 +46,10 @@ 
     private function getValidationRulesFromFieldsAndSubfields($fields)
     {
         $rules = collect($fields)
-            ->filter(function ($value, $key) {
+            ->filter(function($value, $key) {
                 // only keep fields where 'validationRules' OR there are subfields
                 return array_key_exists('validationRules', $value) || array_key_exists('subfields', $value);
-            })->map(function ($item, $key) {
+            })->map(function($item, $key) {
                 $validationRules = [];
                 // only keep the rules, not the entire field definition
                 if (isset($item['validationRules'])) {
@@ -57,7 +57,7 @@ 
                 }
                 // add validation rules for subfields
                 if (array_key_exists('subfields', $item)) {
-                    $subfieldsWithValidation = array_filter($item['subfields'], function ($subfield) {
+                    $subfieldsWithValidation = array_filter($item['subfields'], function($subfield) {
                         return array_key_exists('validationRules', $subfield);
                     });
 
@@ -82,10 +82,10 @@ 
     {
         $messages = [];
         collect($fields)
-            ->filter(function ($value, $key) {
+            ->filter(function($value, $key) {
                 // only keep fields where 'validationMessages' OR there are subfields
                 return array_key_exists('validationMessages', $value) || array_key_exists('subfields', $value);
-            })->each(function ($item, $key) use (&$messages) {
+            })->each(function($item, $key) use (&$messages) {
                 if (isset($item['validationMessages'])) {
                     foreach ($item['validationMessages'] as $rule => $message) {
                         $messages[$key.'.'.$rule] = $message;
@@ -93,7 +93,7 @@ 
                 }
                 // add messages from subfields
                 if (array_key_exists('subfields', $item)) {
-                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function ($subfield) {
+                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function($subfield) {
                         return array_key_exists('validationRules', $subfield);
                     });
 
@@ -129,7 +129,7 @@ 
      */
     public function setValidation($classOrRulesArray = false, $messages = [])
     {
-        if (! $classOrRulesArray) {
+        if (!$classOrRulesArray) {
             $this->setValidationFromFields();
         } elseif (is_array($classOrRulesArray)) {
             $this->setValidationFromArray($classOrRulesArray, $messages);
@@ -210,7 +210,7 @@ 
             return $this->checkRequestValidity($extendedRules, $extendedMessages, $formRequest);
         }
 
-        return ! empty($rules) ? $this->checkRequestValidity($rules, $messages) : $this->getRequest();
+        return !empty($rules) ? $this->checkRequestValidity($rules, $messages) : $this->getRequest();
     }
 
     /**
@@ -225,7 +225,7 @@ 
     {
         $extendedRules = [];
         $requestRules = $this->getRequestRulesAsArray($request);
-        $rules = array_map(function ($ruleDefinition) {
+        $rules = array_map(function($ruleDefinition) {
             return is_array($ruleDefinition) ? $ruleDefinition : explode('|', $ruleDefinition);
         }, $rules);
 
@@ -322,7 +322,7 @@ 
      */
     public function isRequired($inputKey)
     {
-        if (! $this->hasOperationSetting('requiredFields')) {
+        if (!$this->hasOperationSetting('requiredFields')) {
             return false;
         }
 
@@ -343,7 +343,7 @@ 
     {
         [$rules, $messages] = $this->getValidationRulesAndMessagesFromField($field, $parent);
 
-        if (! empty($rules)) {
+        if (!empty($rules)) {
             $this->setValidation($rules, $messages);
         }
     }

tests/Unit/CrudPanel/CrudPanelValidationTest.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
             'password' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Unit\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -76,7 +76,7 @@ 
             'email'    => '',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Unit\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -116,7 +116,7 @@ 
             'password' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Unit\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 

tests/BaseTest.php

Spacing +1 added lines, -1 removed lines

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

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/resources/views/crud/reorder.blade.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
     <?php
     function tree_element($entry, $key, $all_entries, $crud)
     {
-        if (! isset($entry->tree_element_shown)) {
+        if (!isset($entry->tree_element_shown)) {
             // mark the element as shown
             $all_entries[$key]->tree_element_shown = true;
             $entry->tree_element_shown = true;
@@ -71,7 +71,7 @@ 
                 <ol class="sortable mt-0">
                     <?php
                     $all_entries = collect($entries->all())->sortBy('lft')->keyBy($crud->getModel()->getKeyName());
-    $root_entries = $all_entries->filter(function ($item) {
+    $root_entries = $all_entries->filter(function($item) {
         return $item->parent_id == 0;
     });
     foreach ($root_entries as $key => $entry) {

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