Laravel-Backpack / CRUD

src/helpers.php

Spacing +14 added lines, -14 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (! function_exists('backpack_url')) {
+if (!function_exists('backpack_url')) {
     /**
      * Appends the configured backpack prefix and returns
      * the URL using the standard Laravel helpers.
@@ -11,13 +11,13 @@ 
      */
     function backpack_url($path = null, $parameters = [], $secure = null)
     {
-        $path = ! $path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
+        $path = !$path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
 
         return url(config('backpack.base.route_prefix', 'admin').$path, $parameters, $secure);
     }
 }
 
-if (! function_exists('backpack_authentication_column')) {
+if (!function_exists('backpack_authentication_column')) {
     /**
      * Return the username column name.
      * The Laravel default (and Backpack default) is 'email'.
@@ -30,7 +30,7 @@ 
     }
 }
 
-if (! function_exists('backpack_users_have_email')) {
+if (!function_exists('backpack_users_have_email')) {
     /**
      * Check if the email column is present on the user table.
      *
@@ -45,7 +45,7 @@ 
     }
 }
 
-if (! function_exists('backpack_avatar_url')) {
+if (!function_exists('backpack_avatar_url')) {
     /**
      * Returns the avatar URL of a user.
      *
@@ -78,7 +78,7 @@ 
     }
 }
 
-if (! function_exists('backpack_middleware')) {
+if (!function_exists('backpack_middleware')) {
     /**
      * Return the key of the middleware used across Backpack.
      * That middleware checks if the visitor is an admin.
@@ -93,7 +93,7 @@ 
     }
 }
 
-if (! function_exists('backpack_guard_name')) {
+if (!function_exists('backpack_guard_name')) {
     /*
      * Returns the name of the guard defined
      * by the application config
@@ -104,7 +104,7 @@ 
     }
 }
 
-if (! function_exists('backpack_auth')) {
+if (!function_exists('backpack_auth')) {
     /*
      * Returns the user instance if it exists
      * of the currently authenticated admin
@@ -116,7 +116,7 @@ 
     }
 }
 
-if (! function_exists('backpack_user')) {
+if (!function_exists('backpack_user')) {
     /*
      * Returns back a user instance without
      * the admin guard, however allows you
@@ -128,7 +128,7 @@ 
     }
 }
 
-if (! function_exists('mb_ucfirst')) {
+if (!function_exists('mb_ucfirst')) {
     /**
      * Capitalize the first letter of a string,
      * even if that string is multi-byte (non-latin alphabet).
@@ -150,7 +150,7 @@ 
     }
 }
 
-if (! function_exists('backpack_view')) {
+if (!function_exists('backpack_view')) {
     /**
      * Returns a new displayable view based on the configured backpack view namespace.
      * If that view doesn't exist, it will load the one from the original theme.
@@ -170,7 +170,7 @@ 
 
         $returnView = $theme.$view;
 
-        if (! view()->exists($returnView)) {
+        if (!view()->exists($returnView)) {
             $returnView = $originalTheme.$view;
         }
 
@@ -178,7 +178,7 @@ 
     }
 }
 
-if (! function_exists('square_brackets_to_dots')) {
+if (!function_exists('square_brackets_to_dots')) {
     /**
      * Turns a string from bracket-type array to dot-notation array.
      * Ex: array[0][property] turns into array.0.property.
@@ -195,7 +195,7 @@ 
     }
 }
 
-if (! function_exists('is_countable')) {
+if (!function_exists('is_countable')) {
     /**
      * We need this because is_countable was only introduced in PHP 7.3,
      * and in PHP 7.2 you should check if count() argument is really countable.

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

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
 
         foreach ($fields as &$field) {
             // set the value
-            if (! isset($field['value'])) {
+            if (!isset($field['value'])) {
                 if (isset($field['subfields'])) {
                     $field['value'] = [];
                     foreach ($field['subfields'] as $subfield) {
@@ -65,7 +65,7 @@ 
         }
 
         // always have a hidden input for the entry id
-        if (! array_key_exists('id', $fields)) {
+        if (!array_key_exists('id', $fields)) {
             $fields['id'] = [
                 'name'  => $entry->getKeyName(),
                 'value' => $entry->getKey(),
@@ -91,7 +91,7 @@ 
 
             $relation_array = explode('.', $relational_entity);
 
-            $relatedModel = $relatedModel = array_reduce(array_splice($relation_array, 0, -1), function ($obj, $method) {
+            $relatedModel = $relatedModel = array_reduce(array_splice($relation_array, 0, -1), function($obj, $method) {
                 return $obj->{$method} ? $obj->{$method} : $obj;
             }, $model);
 

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

Spacing +10 added lines, -10 removed lines

@@ -69,7 +69,7 @@ 
      */
     public function overwriteFieldNameFromDotNotationToArray($field)
     {
-        if (! is_array($field['name']) && strpos($field['name'], '.') !== false) {
+        if (!is_array($field['name']) && strpos($field['name'], '.') !== false) {
             $entity_array = explode('.', $field['name']);
             $name_string = '';
 
@@ -96,7 +96,7 @@ 
             return ['name' => $field];
         }
 
-        if (is_array($field) && ! isset($field['name'])) {
+        if (is_array($field) && !isset($field['name'])) {
             abort(500, 'All fields must have their name defined');
         }
 
@@ -154,13 +154,13 @@ 
     protected function makeSureFieldHasRelationshipData($field)
     {
         // only do this if "entity" is defined on the field
-        if (! isset($field['entity'])) {
+        if (!isset($field['entity'])) {
             return $field;
         }
 
         $extraFieldAttributes = $this->inferFieldAttributesFromRelationship($field);
 
-        if (! empty($extraFieldAttributes)) {
+        if (!empty($extraFieldAttributes)) {
             $field = array_merge($field, $extraFieldAttributes);
         } else {
             abort(500, 'Unable to process relationship data: '.$field['name']);
@@ -172,7 +172,7 @@ 
     protected function overwriteFieldNameFromEntity($field)
     {
         // if the entity doesn't have a dot, it means we don't need to overwrite the name
-        if (! Str::contains($field['entity'], '.')) {
+        if (!Str::contains($field['entity'], '.')) {
             return $field;
         }
 
@@ -184,7 +184,7 @@ 
         if (count(explode('.', $field['entity'])) == count(explode('.', $this->getOnlyRelationEntity($field)))) {
             $field['name'] = implode('.', array_slice(explode('.', $field['entity']), 0, -1));
             $relation = $this->getRelationInstance($field);
-            if (! empty($field['name'])) {
+            if (!empty($field['name'])) {
                 $field['name'] .= '.';
             }
             $field['name'] .= $relation->getForeignKeyName();
@@ -197,7 +197,7 @@ 
     {
         // if there's a model defined, but no attribute
         // guess an attribute using the indentifiableAttribute functionality in CrudTrait
-        if (isset($field['model']) && ! isset($field['attribute'])) {
+        if (isset($field['model']) && !isset($field['attribute'])) {
             $field['attribute'] = call_user_func([(new $field['model']), 'identifiableAttribute']);
         }
 
@@ -213,7 +213,7 @@ 
      */
     protected function makeSureFieldHasLabel($field)
     {
-        if (! isset($field['label'])) {
+        if (!isset($field['label'])) {
             $name = is_array($field['name']) ? $field['name'][0] : $field['name'];
             $name = str_replace('_id', '', $name);
             $field['label'] = mb_ucfirst(str_replace('_', ' ', $name));
@@ -231,7 +231,7 @@ 
      */
     protected function makeSureFieldHasType($field)
     {
-        if (! isset($field['type'])) {
+        if (!isset($field['type'])) {
             $field['type'] = isset($field['relation_type']) ? $this->inferFieldTypeFromFieldRelation($field) : $this->inferFieldTypeFromDbColumnType($field['name']);
         }
 
@@ -248,7 +248,7 @@ 
     {
         // if a tab was mentioned, we should enable it
         if (isset($field['tab'])) {
-            if (! $this->tabsEnabled()) {
+            if (!$this->tabsEnabled()) {
                 $this->enableTabs();
             }
         }

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

Spacing +6 added lines, -6 removed lines

@@ -114,7 +114,7 @@ 
      */
     public function makeFirstColumn()
     {
-        if (! $this->columns()) {
+        if (!$this->columns()) {
             return false;
         }
 
@@ -159,7 +159,7 @@ 
      */
     public function removeColumns($columns)
     {
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             foreach ($columns as $columnKey) {
                 $this->removeColumn($columnKey);
             }
@@ -252,7 +252,7 @@ 
     {
         $columns = $this->columns();
 
-        return collect($columns)->pluck('entity')->reject(function ($value, $key) {
+        return collect($columns)->pluck('entity')->reject(function($value, $key) {
             return $value == null;
         })->toArray();
     }
@@ -329,7 +329,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;
         });
 
@@ -345,7 +345,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;
         });
     }
@@ -378,7 +378,7 @@ 
 
         // check if it's a method on the model,
         // that means it's a relationship
-        if (! $columnExistsInDb && method_exists($this->model, $column['name'])) {
+        if (!$columnExistsInDb && method_exists($this->model, $column['name'])) {
             $relatedModel = $this->model->{$column['name']}()->getRelated();
             $column['entity'] = $column['name'];
             $column['model'] = get_class($relatedModel);

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

Spacing +9 added lines, -10 removed lines

@@ -21,7 +21,7 @@ 
 
         // make sure the column has a priority in terms of visibility
         // if no priority has been defined, use the order in the array plus one
-        if (! array_key_exists('priority', $column)) {
+        if (!array_key_exists('priority', $column)) {
             $position_in_columns_array = (int) array_search($column['key'], array_keys($this->columns()));
             $allColumns[$column['key']]['priority'] = $position_in_columns_array + 1;
         }
@@ -42,7 +42,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);
         }
 
@@ -58,7 +58,7 @@ 
      */
     protected function makeSureColumnHasLabel($column)
     {
-        if (! isset($column['label'])) {
+        if (!isset($column['label'])) {
             $column['label'] = mb_ucfirst($this->makeLabel($column['name']));
         }
 
@@ -75,11 +75,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';
         }
 
@@ -96,7 +96,7 @@ 
      */
     protected function makeSureColumnHasKey($column)
     {
-        if (! isset($column['key'])) {
+        if (!isset($column['key'])) {
             $column['key'] = str_replace('.', '__', $column['name']);
         }
 
@@ -114,7 +114,7 @@ 
      */
     protected function makeSureColumnHasWrapper($column)
     {
-        if (! isset($column['wrapper'])) {
+        if (!isset($column['wrapper'])) {
             $column['wrapper'] = [];
         }
 
@@ -132,7 +132,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']);
         }
 
@@ -153,8 +153,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);

src/app/Console/Commands/AddCustomRouteContent.php

Spacing +4 added lines, -4 removed lines

@@ -72,7 +72,7 @@ 
 
             $process = new Process($command, null, null, null, 300, null);
 
-            $process->run(function ($type, $buffer) {
+            $process->run(function($type, $buffer) {
                 if (Process::ERR === $type) {
                     $this->line($buffer);
                 } else {
@@ -81,7 +81,7 @@ 
             });
 
             // executes after the command finishes
-            if (! $process->isSuccessful()) {
+            if (!$process->isSuccessful()) {
                 throw new ProcessFailedException($process);
             }
 
@@ -100,7 +100,7 @@ 
 
         // otherwise, in case the last line HAS been modified
         // return the last line that has an ending in it
-        $possible_end_lines = array_filter($file_lines, function ($k) {
+        $possible_end_lines = array_filter($file_lines, function($k) {
             return strpos($k, '});') === 0;
         });
 
@@ -121,7 +121,7 @@ 
      */
     private function getLastLineNumberThatContains($needle, $haystack)
     {
-        $matchingLines = array_filter($haystack, function ($k) use ($needle) {
+        $matchingLines = array_filter($haystack, function($k) use ($needle) {
             return strpos($k, $needle) !== false;
         });
 

src/app/Console/Commands/AddSidebarContent.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
      */
     private function getLastLineNumberThatContains($needle, $haystack)
     {
-        $matchingLines = array_filter($haystack, function ($k) use ($needle) {
+        $matchingLines = array_filter($haystack, function($k) use ($needle) {
             return strpos($k, $needle) !== false;
         });
 

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

Spacing +10 added lines, -10 removed lines

@@ -73,7 +73,7 @@ 
         $this->enableFilters();
 
         // check if another filter with the same name exists
-        if (! isset($options['name'])) {
+        if (!isset($options['name'])) {
             abort(500, 'All your filters need names.');
         } else {
             // make sure we check against the converted camel name before adding.
@@ -123,7 +123,7 @@ 
     {
         $unappliedFilters = $this->filters()->where('applied', false);
         if ($unappliedFilters->count()) {
-            $unappliedFilters->each(function ($filter) {
+            $unappliedFilters->each(function($filter) {
                 $filter->apply();
             });
         }
@@ -173,7 +173,7 @@ 
     {
         $filter = $this->filters()->firstWhere('name', $name);
 
-        if (! $filter) {
+        if (!$filter) {
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.');
         }
 
@@ -188,7 +188,7 @@ 
 
     public function replaceFilter($name, $newFilter)
     {
-        $newFilters = $this->filters()->map(function ($filter, $key) use ($name, $newFilter) {
+        $newFilters = $this->filters()->map(function($filter, $key) use ($name, $newFilter) {
             if ($filter->name != $name) {
                 return $filter;
             }
@@ -201,7 +201,7 @@ 
 
     public function removeFilter($name)
     {
-        $strippedCollection = $this->filters()->reject(function ($filter) use ($name) {
+        $strippedCollection = $this->filters()->reject(function($filter) use ($name) {
             return $filter->name == $name;
         });
 
@@ -244,7 +244,7 @@ 
      */
     public function makeFirstFilter()
     {
-        if (! $this->filters()) {
+        if (!$this->filters()) {
             return false;
         }
 
@@ -274,15 +274,15 @@ 
         $destinationFilter = $this->firstFilterWhere('name', $destination);
         $destinationKey = $this->getFilterKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
-        $newFilters = $this->filters()->filter(function ($value, $key) use ($target) {
+        $newFilters = $this->filters()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetFilter) {
+        if (!$targetFilter) {
             return;
         }
 
-        if (! $destinationFilter) {
+        if (!$destinationFilter) {
             return;
         }
 
@@ -290,7 +290,7 @@ 
         $lastSlice = $newFilters->slice($newDestinationKey, null);
 
         $newFilters = $firstSlice->push($targetFilter);
-        $lastSlice->each(function ($item, $key) use ($newFilters) {
+        $lastSlice->each(function($item, $key) use ($newFilters) {
             $newFilters->push($item);
         });
 

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

Spacing +6 added lines, -8 removed lines

@@ -45,8 +45,8 @@ 
         // otherwise assume the arguments are actually the configuration array
         $config = [];
 
-        if (! is_array($arg)) {
-            if (! class_exists($arg)) {
+        if (!is_array($arg)) {
+            if (!class_exists($arg)) {
                 return response()->json(['error' => 'Class: '.$arg.' does not exists'], 500);
             }
             $config['model'] = $arg;
@@ -76,8 +76,7 @@ 
 
         if ($search_string === false) {
             return ($config['paginate'] !== false) ?
-            $config['query']->paginate($config['paginate']) :
-            $config['query']->get();
+            $config['query']->paginate($config['paginate']) : $config['query']->get();
         }
 
         $textColumnTypes = ['string', 'json_string', 'text', 'longText', 'json_array'];
@@ -88,8 +87,8 @@ 
         // .... 'query' => function($model) { return $model->where('active', 1); }
         // So it reads: SELECT ... WHERE active = 1 AND (XXX = x OR YYY = y) and not SELECT ... WHERE active = 1 AND XXX = x OR YYY = y;
 
-        if (! empty($config['query']->getQuery()->wheres)) {
-            $config['query'] = $config['query']->where(function ($query) use ($model_instance, $config, $search_string, $textColumnTypes) {
+        if (!empty($config['query']->getQuery()->wheres)) {
+            $config['query'] = $config['query']->where(function($query) use ($model_instance, $config, $search_string, $textColumnTypes) {
                 foreach ((array) $config['searchable_attributes'] as $k => $searchColumn) {
                     $operation = ($k == 0) ? 'where' : 'orWhere';
                     $columnType = $model_instance->getColumnType($searchColumn);
@@ -118,7 +117,6 @@ 
 
         // return the results with or without pagination
         return ($config['paginate'] !== false) ?
-                    $config['query']->paginate($config['paginate']) :
-                    $config['query']->get();
+                    $config['query']->paginate($config['paginate']) : $config['query']->get();
     }
 }