Laravel-Backpack / CRUD

tests/Unit/Http/CrudControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
         $app['router']->get('admin/users', "$controller@index")->name('admin.users.index');
 
-        $app->singleton('crud', function ($app) {
+        $app->singleton('crud', function($app) {
             return new CrudPanel($app);
         });
 

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

Spacing +9 added lines, -10 removed lines

@@ -49,7 +49,7 @@ 
             $column = ['name' => $column];
         }
 
-        if (is_array($column) && ! isset($column['name'])) {
+        if (is_array($column) && !isset($column['name'])) {
             $column['name'] = 'anonymous_column_'.Str::random(5);
         }
 
@@ -65,7 +65,7 @@ 
      */
     protected function makeSureColumnHasLabel($column)
     {
-        if (! isset($column['label'])) {
+        if (!isset($column['label'])) {
             $column['label'] = mb_ucfirst($this->makeLabel($column['name']));
         }
 
@@ -82,11 +82,11 @@ 
     {
         // if it's got a method on the model with the same name
         // then it should be a relationship
-        if (! isset($column['type']) && method_exists($this->model, $column['name'])) {
+        if (!isset($column['type']) && method_exists($this->model, $column['name'])) {
             $column['type'] = 'relationship';
         }
 
-        if (! isset($column['type'])) {
+        if (!isset($column['type'])) {
             $column['type'] = 'text';
         }
 
@@ -103,7 +103,7 @@ 
      */
     protected function makeSureColumnHasKey($column)
     {
-        if (! isset($column['key'])) {
+        if (!isset($column['key'])) {
             $column['key'] = str_replace('.', '__', $column['name']);
         }
 
@@ -121,7 +121,7 @@ 
      */
     protected function makeSureColumnHasWrapper($column)
     {
-        if (! isset($column['wrapper'])) {
+        if (!isset($column['wrapper'])) {
             $column['wrapper'] = [];
         }
 
@@ -139,7 +139,7 @@ 
     {
         // if this is a relation type field and no corresponding model was specified,
         // get it from the relation method defined in the main model
-        if (isset($column['entity']) && ! isset($column['model'])) {
+        if (isset($column['entity']) && !isset($column['model'])) {
             $column['model'] = $this->getRelationModel($column['entity']);
         }
 
@@ -160,8 +160,7 @@ 
         $columnsArray = $this->columns();
 
         if (array_key_exists($targetColumnName, $columnsArray)) {
-            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($columnsArray)) :
-                array_search($targetColumnName, array_keys($columnsArray)) + 1;
+            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($columnsArray)) : array_search($targetColumnName, array_keys($columnsArray)) + 1;
 
             $element = array_pop($columnsArray);
             $beginningPart = array_slice($columnsArray, 0, $targetColumnPosition, true);
@@ -184,7 +183,7 @@ 
     {
         static $cache = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return true;
         }
 

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

Spacing +7 added lines, -7 removed lines

@@ -16,7 +16,7 @@ 
             $this->getDbColumnTypes();
         }
 
-        array_map(function ($field) {
+        array_map(function($field) {
             $new_field = [
                 'name'       => $field,
                 'label'      => $this->makeLabel($field),
@@ -28,11 +28,11 @@ 
                 'autoset'    => true,
             ];
 
-            if (! isset($this->fields()[$field])) {
+            if (!isset($this->fields()[$field])) {
                 $this->addField($new_field);
             }
 
-            if (! in_array($field, $this->model->getHidden()) && ! in_array($field, $this->columns())) {
+            if (!in_array($field, $this->model->getHidden()) && !in_array($field, $this->columns())) {
                 $this->addColumn([
                     'name'    => $field,
                     'label'   => $this->makeLabel($field),
@@ -110,7 +110,7 @@ 
 
         $dbColumnTypes = $this->getDbColumnTypes();
 
-        if (! isset($dbColumnTypes[$fieldName])) {
+        if (!isset($dbColumnTypes[$fieldName])) {
             return 'text';
         }
 
@@ -168,7 +168,7 @@ 
         $types = ['enum' => 'string'];
         $platform = $this->getSchema()->getConnection()->getDoctrineConnection()->getDatabasePlatform();
         foreach ($types as $type_key => $type_value) {
-            if (! $platform->hasDoctrineTypeMappingFor($type_key)) {
+            if (!$platform->hasDoctrineTypeMappingFor($type_key)) {
                 $platform->registerDoctrineTypeMapping($type_key, $type_value);
             }
         }
@@ -221,13 +221,13 @@ 
     {
         $fillable = $this->model->getFillable();
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             $columns = $fillable;
         } else {
             // Automatically-set columns should be both in the database, and in the $fillable variable on the Eloquent Model
             $columns = $this->model->getConnection()->getSchemaBuilder()->getColumnListing($this->model->getTable());
 
-            if (! empty($fillable)) {
+            if (!empty($fillable)) {
                 $columns = array_intersect($columns, $fillable);
             }
         }

src/app/Http/Controllers/Operations/ListOperation.php

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
     {
         $this->crud->allowAccess('list');
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
         });
     }
@@ -114,8 +114,8 @@ 
         // if there was an order set, this will be the last one (after all others were applied)
         $orderBy = $this->crud->query->getQuery()->orders;
         $hasOrderByPrimaryKey = false;
-        collect($orderBy)->each(function ($item, $key) use ($hasOrderByPrimaryKey) {
-            if (! isset($item['column'])) {
+        collect($orderBy)->each(function($item, $key) use ($hasOrderByPrimaryKey) {
+            if (!isset($item['column'])) {
                 return false;
             }
 
@@ -125,7 +125,7 @@ 
                 return false;
             }
         });
-        if (! $hasOrderByPrimaryKey) {
+        if (!$hasOrderByPrimaryKey) {
             $this->crud->orderByWithPrefix($this->crud->model->getKeyName(), 'DESC');
         }
 

src/app/Models/Traits/HasRelationshipFields.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
         $table = $instance->getTableWithPrefix();
 
         // MongoDB columns are alway nullable
-        if (! in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
+        if (!in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
             return true;
         }
 
@@ -82,7 +82,7 @@ 
             // check for NOT NULL
             $notNull = $column->getNotnull();
             // return the value of nullable (aka the inverse of NOT NULL)
-            return ! $notNull;
+            return !$notNull;
         } catch (\Exception $e) {
             return true;
         }

tests/Unit/CrudPanel/CrudPanelReadTest.php

Spacing +1 added lines, -1 removed lines

@@ -429,7 +429,7 @@
         $this->assertCount(2, $this->crudPanel->getOperationSetting('pageLengthMenu'));
         $this->assertTrue(in_array(-1, $this->crudPanel->getOperationSetting('pageLengthMenu')[0]));
 
-        $this->crudPanel->setPageLengthMenu([-1 , 1]);
+        $this->crudPanel->setPageLengthMenu([-1, 1]);
         $this->assertCount(2, $this->crudPanel->getOperationSetting('pageLengthMenu'));
         $this->assertTrue(in_array(-1, $this->crudPanel->getOperationSetting('pageLengthMenu')[0]));
 

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

Spacing +18 added lines, -18 removed lines

@@ -38,7 +38,7 @@ 
     {
         $id = $this->getCurrentEntryId();
 
-        if (! $id) {
+        if (!$id) {
             return false;
         }
 
@@ -54,7 +54,7 @@ 
      */
     public function getEntry($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             $this->entry = $this->model->findOrFail($id);
             $this->entry = $this->entry->withFakes();
         }
@@ -184,9 +184,9 @@ 
      */
     public function setDefaultPageLength($value)
     {
-        if($value !== 0) {
+        if ($value !== 0) {
             $this->setOperationSetting('defaultPageLength', $value);
-        }else{
+        } else {
             abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
         }
     }
@@ -217,7 +217,7 @@ 
             if ($position !== false) {
                 array_unshift($values, $this->getDefaultPageLength());
                 array_unshift($labels, $labels[$position]);
-            }else{
+            } else {
                 // if it's not in array we add it as the first element
                 array_unshift($values, $this->getDefaultPageLength());
                 array_unshift($labels, $this->getDefaultPageLength());
@@ -239,27 +239,27 @@ 
      */
     public function setPageLengthMenu($menu)
     {
-        if(is_array($menu)) {
+        if (is_array($menu)) {
             // start checking $menu integrity
-            if(count($menu) !== count($menu, COUNT_RECURSIVE)) {
+            if (count($menu) !== count($menu, COUNT_RECURSIVE)) {
                 // developer defined as setPageLengthMenu([[50, 100, 300]]) or setPageLengthMenu([[50, 100, 300],['f','h','t']])
                 // we will apply the same labels as the values to the menu if developer didn't
-                if(!in_array(0, $menu[0])) {
-                    if (! isset($menu[1]) || ! is_array($menu[1])) {
+                if (!in_array(0, $menu[0])) {
+                    if (!isset($menu[1]) || !is_array($menu[1])) {
                         $menu[1] = $menu[0];
                     }
-                }else{
+                } else {
                     abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
                 }
-            }else{
+            } else {
                 // developer defined setPageLengthMenu([10 => 'f', 100 => 'h', 300 => 't']) OR setPageLengthMenu([50, 100, 300])
                 $menu = $this->buildPageLengthMenuFromArray($menu);
             }
-        }else{
+        } else {
             // developer added only a single value setPageLengthMenu(10)
-            if($menu !== 0) {
+            if ($menu !== 0) {
                 $menu = [[$menu], [$menu]];
-            }else{
+            } else {
                 abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
             }
         }
@@ -279,20 +279,20 @@ 
     private function buildPageLengthMenuFromArray($menu) {
         // check if the values of the array are strings, in case developer defined:
         // setPageLengthMenu([0 => 'f', 100 => 'h', 300 => 't'])
-        if(count(array_filter(array_values($menu), 'is_string')) > 0) {
+        if (count(array_filter(array_values($menu), 'is_string')) > 0) {
             $values = array_keys($menu);
             $labels = array_values($menu);
 
-            if(!in_array(0, $values)) {
+            if (!in_array(0, $values)) {
                 return [$values, $labels];
             }
 
             abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
 
-        }else{
+        } else {
             // developer defined length as setPageLengthMenu([50, 100, 300])
             // we will use the same values as labels
-            if(!in_array(0, $menu)) {
+            if (!in_array(0, $menu)) {
                 return [$menu, $menu];
             }