Laravel-Backpack / CRUD

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

Spacing +11 added lines, -11 removed lines

@@ -65,7 +65,7 @@ 
         $model = $this->getRelationModel($entity, -1);
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.');
 
-        if (! method_exists($model, $lastSegmentAfterDot)) {
+        if (!method_exists($model, $lastSegmentAfterDot)) {
             return (string) Str::of($field['entity'])->beforeLast('.');
         }
 
@@ -86,7 +86,7 @@ 
 
         return collect($this->getCleanStateFields())
             ->whereIn('relation_type', $relation_types)
-            ->filter(function ($item) use ($nested) {
+            ->filter(function($item) use ($nested) {
                 if ($nested) {
                     return true;
                 }
@@ -130,7 +130,7 @@ 
      */
     private function getRelationFieldsWithoutRelationType($relations, $fields = [])
     {
-        if (! is_array($relations)) {
+        if (!is_array($relations)) {
             $relations = [$relations];
         }
 
@@ -139,8 +139,8 @@ 
         }
 
         foreach ($relations as $relation) {
-            $fields = array_filter($fields, function ($field) use ($relation) {
-                if (! isset($field['relation_type'])) {
+            $fields = array_filter($fields, function($field) use ($relation) {
+                if (!isset($field['relation_type'])) {
                     return false;
                 }
 
@@ -174,7 +174,7 @@ 
                     $fields = array_merge($field['subfields'], $fields);
                 }
             }
-            $fields = array_filter($fields, function ($field) {
+            $fields = array_filter($fields, function($field) {
                 return isset($field['relation_type']) && $field['relation_type'] === 'BelongsTo';
             });
         }
@@ -243,8 +243,8 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
-            return isset($value['pivot']) && ! $value['pivot'];
+        return Arr::where($all_relation_fields, function($value, $key) {
+            return isset($value['pivot']) && !$value['pivot'];
         });
     }
 
@@ -257,7 +257,7 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
+        return Arr::where($all_relation_fields, function($value, $key) {
             return isset($value['pivot']) && $value['pivot'];
         });
     }
@@ -338,7 +338,7 @@ 
         }
 
         // relationships are always public methods.
-        if (! $methodReflection->isPublic()) {
+        if (!$methodReflection->isPublic()) {
             return false;
         }
 
@@ -368,7 +368,7 @@ 
      */
     private function isAttributeInRelationString($field)
     {
-        if (! str_contains($field['entity'], '.')) {
+        if (!str_contains($field['entity'], '.')) {
             return false;
         }
 

src/ViewNamespaces.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     public static function addFor(string $domain, $viewNamespaces)
     {
         foreach ((array) $viewNamespaces as $viewNamespace) {
-            if (! in_array($viewNamespace, self::getForDomain($domain))) {
+            if (!in_array($viewNamespace, self::getForDomain($domain))) {
                 self::$viewNamespaces[$domain][] = $viewNamespace;
             }
         }
@@ -83,7 +83,7 @@ 
      */
     public static function getViewPathsFor(string $domain, string $viewName)
     {
-        return array_map(function ($item) use ($viewName) {
+        return array_map(function($item) use ($viewName) {
             return $item.'.'.$viewName;
         }, self::getFor($domain));
     }

tests/Unit/CrudPanel/CrudPanelTitlesAndHeadingsTest.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);
@@ -51,7 +51,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);
@@ -66,7 +66,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);

tests/Unit/CrudPanel/CrudPanelMacroTest.php

Indentation +3 added lines, -3 removed lines

@@ -25,8 +25,8 @@
         } catch (\Throwable $e) {
         }
         $this->assertEquals(
-          new \Symfony\Component\HttpKernel\Exception\HttpException(500, 'Cannot register \'setModel\' macro. \'setModel()\' already exists on Backpack\CRUD\app\Library\CrudPanel\CrudPanel'),
-          $e
-      );
+            new \Symfony\Component\HttpKernel\Exception\HttpException(500, 'Cannot register \'setModel\' macro. \'setModel()\' already exists on Backpack\CRUD\app\Library\CrudPanel\CrudPanel'),
+            $e
+        );
     }
 }

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 {
     public function testItCanRegisterMacro()
     {
-        $this->crudPanel::macro('validMacro', function () {
+        $this->crudPanel::macro('validMacro', function() {
             return true;
         });
 
@@ -19,7 +19,7 @@ 
     public function testThrowsErrorIfMacroExists()
     {
         try {
-            $this->crudPanel::macro('setModel', function () {
+            $this->crudPanel::macro('setModel', function() {
                 return true;
             });
         } catch (\Throwable $e) {

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +2 added lines, -2 removed lines

@@ -605,7 +605,7 @@ 
 
     public function testCreateBelongsToFake()
     {
-        $belongsToField = [   // select_grouped
+        $belongsToField = [// select_grouped
             'label'                      => 'Select_grouped',
             'type'                       => 'select_grouped', //https://github.com/Laravel-Backpack/CRUD/issues/502
             'name'                       => 'bang_relation_field',
@@ -741,7 +741,7 @@ 
                 'nickname'        => 'i_have_has_one',
                 'profile_picture' => 'ohh my picture 1.jpg',
                 'article'         => 1,
-                'addresses'       => [ // HasOne is tested in other test function
+                'addresses'       => [// HasOne is tested in other test function
                     [
                         'city'   => 2,
                         'street' => 'test',

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

src/app/Console/Commands/PublishAssets.php

Spacing +2 added lines, -2 removed lines

@@ -41,12 +41,12 @@
     private function runConsoleCommand($command)
     {
         $process = new Process($command, null, null, null, 60, null);
-        $process->run(function ($type, $buffer) {
+        $process->run(function($type, $buffer) {
             $this->line($buffer);
         });
 
         // executes after the command finishes
-        if (! $process->isSuccessful()) {
+        if (!$process->isSuccessful()) {
             throw new ProcessFailedException($process);
         }
     }

src/helpers.php

Spacing +20 added lines, -20 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.
@@ -10,13 +10,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'.
@@ -29,7 +29,7 @@ 
     }
 }
 
-if (! function_exists('backpack_email_column')) {
+if (!function_exists('backpack_email_column')) {
     /**
      * Return the email column name.
      * The Laravel default (and Backpack default) is 'email'.
@@ -42,7 +42,7 @@ 
     }
 }
 
-if (! function_exists('backpack_form_input')) {
+if (!function_exists('backpack_form_input')) {
     /**
      * Parse the submitted input in request('form') to an usable array.
      * Joins the multiple[] fields in a single key and transform the dot notation fields into arrayed ones.
@@ -99,7 +99,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.
      *
@@ -114,7 +114,7 @@ 
     }
 }
 
-if (! function_exists('backpack_avatar_url')) {
+if (!function_exists('backpack_avatar_url')) {
     /**
      * Returns the avatar URL of a user.
      *
@@ -125,7 +125,7 @@ 
     {
         switch (config('backpack.base.avatar_type')) {
             case 'gravatar':
-                if (backpack_users_have_email() && ! empty($user->email)) {
+                if (backpack_users_have_email() && !empty($user->email)) {
                     return Gravatar::fallback(config('backpack.base.gravatar_fallback'))->get($user->email);
                 }
                 break;
@@ -136,7 +136,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.
@@ -150,7 +150,7 @@ 
     }
 }
 
-if (! function_exists('backpack_guard_name')) {
+if (!function_exists('backpack_guard_name')) {
     /*
      * Returns the name of the guard defined
      * by the application config
@@ -161,7 +161,7 @@ 
     }
 }
 
-if (! function_exists('backpack_auth')) {
+if (!function_exists('backpack_auth')) {
     /*
      * Returns the user instance if it exists
      * of the currently authenticated admin
@@ -173,7 +173,7 @@ 
     }
 }
 
-if (! function_exists('backpack_user')) {
+if (!function_exists('backpack_user')) {
     /*
      * Returns back a user instance without
      * the admin guard, however allows you
@@ -185,7 +185,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).
@@ -206,7 +206,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.
@@ -225,7 +225,7 @@ 
 
         $returnView = $theme.$view;
 
-        if (! view()->exists($returnView)) {
+        if (!view()->exists($returnView)) {
             $returnView = $originalTheme.$view;
         }
 
@@ -233,7 +233,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.
@@ -249,7 +249,7 @@ 
     }
 }
 
-if (! function_exists('old_empty_or_null')) {
+if (!function_exists('old_empty_or_null')) {
     /**
      * This method is an alternative to Laravel's old() helper, which mistakenly
      * returns NULL it two cases:
@@ -281,7 +281,7 @@ 
     }
 }
 
-if (! function_exists('is_multidimensional_array')) {
+if (!function_exists('is_multidimensional_array')) {
     /**
      * If any of the items inside a given array is an array, the array is considered multidimensional.
      *
@@ -300,7 +300,7 @@ 
     }
 }
 
-if (! function_exists('backpack_pro')) {
+if (!function_exists('backpack_pro')) {
     /**
      * Check if the backpack/pro package is installed.
      *
@@ -311,7 +311,7 @@ 
         if (app()->runningUnitTests()) {
             return true;
         }
-        if (! \Composer\InstalledVersions::isInstalled('backpack/pro')) {
+        if (!\Composer\InstalledVersions::isInstalled('backpack/pro')) {
             return false;
         }
 

tests/Unit/CrudPanel/CrudPanelOperationsTest.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
     public function testItCanConfigureOperations()
     {
-        $this->crudPanel->operation(['create', 'update'], function () {
+        $this->crudPanel->operation(['create', 'update'], function() {
             $this->crudPanel->addField(['name' => 'test', 'type' => 'text']);
         });
         $this->crudPanel->applyConfigurationFromSettings('create');