Laravel-Backpack / CRUD

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

Spacing +6 added lines, -6 removed lines

@@ -30,25 +30,25 @@
         $this->crud->allowAccess('show');
         $this->crud->setOperationSetting('setFromDb', true);
 
-        $this->crud->operation('show', function () {
+        $this->crud->operation('show', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
 
-            if (! method_exists($this, 'setupShowOperation')) {
+            if (!method_exists($this, 'setupShowOperation')) {
                 $this->autoSetupShowOperation();
             }
         });
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->addButton('line', 'show', 'view', 'crud::buttons.show', 'beginning');
         });
 
-        $this->crud->operation(['create', 'update'], function () {
+        $this->crud->operation(['create', 'update'], function() {
             $this->crud->addSaveAction([
                 'name' => 'save_and_preview',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('show');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     $itemId = $itemId ?: $request->input('id');
                     $redirectUrl = $crud->route.'/'.$itemId.'/show';
                     if ($request->has('_locale')) {

src/resources/views/crud/fields/checklist_dependency.blade.php

Indentation +24 added lines, -24 removed lines

@@ -33,35 +33,35 @@
         $old_primary_dependency = old_empty_or_null($primary_dependency['name'], false) ?? false;
         $old_secondary_dependency = old_empty_or_null($secondary_dependency['name'], false) ?? false;
 
-      //for update form, get initial state of the entity
-      if (isset($id) && $id) {
+        //for update form, get initial state of the entity
+        if (isset($id) && $id) {
 
         //get entity with relations for primary dependency
-          $entity_dependencies = $entity_model->with($primary_dependency['entity'])
-          ->with($primary_dependency['entity'].'.'.$primary_dependency['entity_secondary'])
-          ->find($id);
+            $entity_dependencies = $entity_model->with($primary_dependency['entity'])
+            ->with($primary_dependency['entity'].'.'.$primary_dependency['entity_secondary'])
+            ->find($id);
 
-          $secondaries_from_primary = [];
+            $secondaries_from_primary = [];
 
-          //convert relation in array
-          $primary_array = $entity_dependencies->{$primary_dependency['entity']}->toArray();
+            //convert relation in array
+            $primary_array = $entity_dependencies->{$primary_dependency['entity']}->toArray();
 
-          $secondary_ids = [];
-          //create secondary dependency from primary relation, used to check what checkbox must be checked from second checklist
-          if ($old_primary_dependency) {
-              foreach ($old_primary_dependency as $primary_item) {
-                  foreach ($dependencyArray[$primary_item] as $second_item) {
-                      $secondary_ids[$second_item] = $second_item;
-                  }
-              }
-          } else { //create dependencies from relation if not from validate error
-              foreach ($primary_array as $primary_item) {
-                  foreach ($primary_item[$secondary_dependency['entity']] as $second_item) {
-                      $secondary_ids[$second_item['id']] = $second_item['id'];
-                  }
-              }
-          }
-      }
+            $secondary_ids = [];
+            //create secondary dependency from primary relation, used to check what checkbox must be checked from second checklist
+            if ($old_primary_dependency) {
+                foreach ($old_primary_dependency as $primary_item) {
+                    foreach ($dependencyArray[$primary_item] as $second_item) {
+                        $secondary_ids[$second_item] = $second_item;
+                    }
+                }
+            } else { //create dependencies from relation if not from validate error
+                foreach ($primary_array as $primary_item) {
+                    foreach ($primary_item[$secondary_dependency['entity']] as $second_item) {
+                        $secondary_ids[$second_item['id']] = $second_item['id'];
+                    }
+                }
+            }
+        }
 
         //json encode of dependency matrix
         $dependencyJson = json_encode($dependencyArray);

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

Spacing +13 added lines, -13 removed lines

@@ -117,7 +117,7 @@ 
      */
     public function afterField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, false);
         });
     }
@@ -129,7 +129,7 @@ 
      */
     public function beforeField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, true);
         });
     }
@@ -141,7 +141,7 @@ 
      */
     public function makeFirstField()
     {
-        if (! $this->fields()) {
+        if (!$this->fields()) {
             return false;
         }
 
@@ -156,7 +156,7 @@ 
      */
     public function removeField($name)
     {
-        $this->transformFields(function ($fields) use ($name) {
+        $this->transformFields(function($fields) use ($name) {
             Arr::forget($fields, $name);
 
             return $fields;
@@ -170,7 +170,7 @@ 
      */
     public function removeFields($array_of_names)
     {
-        if (! empty($array_of_names)) {
+        if (!empty($array_of_names)) {
             foreach ($array_of_names as $name) {
                 $this->removeField($name);
             }
@@ -183,7 +183,7 @@ 
     public function removeAllFields()
     {
         $current_fields = $this->getCleanStateFields();
-        if (! empty($current_fields)) {
+        if (!empty($current_fields)) {
             foreach ($current_fields as $field) {
                 $this->removeField($field['name']);
             }
@@ -280,7 +280,7 @@ 
                 $jsonCastables = ['array', 'object', 'json'];
                 $fieldCasting = $casted_attributes[$field['name']];
 
-                if (in_array($fieldCasting, $jsonCastables) && isset($input[$field['name']]) && ! empty($input[$field['name']]) && ! is_array($input[$field['name']])) {
+                if (in_array($fieldCasting, $jsonCastables) && isset($input[$field['name']]) && !empty($input[$field['name']]) && !is_array($input[$field['name']])) {
                     try {
                         $input[$field['name']] = json_decode($input[$field['name']]);
                     } catch (\Exception $e) {
@@ -309,7 +309,7 @@ 
      */
     public function orderFields($order)
     {
-        $this->transformFields(function ($fields) use ($order) {
+        $this->transformFields(function($fields) use ($order) {
             return $this->applyOrderToFields($fields, $order);
         });
     }
@@ -335,7 +335,7 @@ 
     public function hasUploadFields()
     {
         $fields = $this->getCleanStateFields();
-        $upload_fields = Arr::where($fields, function ($value, $key) {
+        $upload_fields = Arr::where($fields, function($value, $key) {
             return isset($value['upload']) && $value['upload'] == true;
         });
 
@@ -398,7 +398,7 @@ 
         $alreadyLoaded = $this->getLoadedFieldTypes();
         $type = $this->getFieldTypeWithNamespace($field);
 
-        if (! in_array($type, $this->getLoadedFieldTypes(), true)) {
+        if (!in_array($type, $this->getLoadedFieldTypes(), true)) {
             $alreadyLoaded[] = $type;
             $this->setLoadedFieldTypes($alreadyLoaded);
 
@@ -439,7 +439,7 @@ 
      */
     public function fieldTypeNotLoaded($field)
     {
-        return ! in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
+        return !in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
     }
 
     /**
@@ -479,7 +479,7 @@ 
      */
     public function hasFieldWhere($attribute, $value)
     {
-        $match = Arr::first($this->getCleanStateFields(), function ($field, $fieldKey) use ($attribute, $value) {
+        $match = Arr::first($this->getCleanStateFields(), function($field, $fieldKey) use ($attribute, $value) {
             return isset($field[$attribute]) && $field[$attribute] == $value;
         });
 
@@ -495,7 +495,7 @@ 
      */
     public function firstFieldWhere($attribute, $value)
     {
-        return Arr::first($this->getCleanStateFields(), function ($field, $fieldKey) use ($attribute, $value) {
+        return Arr::first($this->getCleanStateFields(), function($field, $fieldKey) use ($attribute, $value) {
             return isset($field[$attribute]) && $field[$attribute] == $value;
         });
     }

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +2 added lines, -2 removed lines

@@ -593,7 +593,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',
@@ -727,7 +727,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',

tests/config/database/migrations/2020_03_31_114745_create_pivotable_relations_tables.php

Spacing +15 added lines, -15 removed lines

@@ -13,14 +13,14 @@ 
      */
     public function up()
     {
-        Schema::create('comments', function (Blueprint $table) {
+        Schema::create('comments', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('text')->nullable();
             $table->morphs('commentable');
             $table->timestamps();
         });
 
-        Schema::create('recommendables', function (Blueprint $table) {
+        Schema::create('recommendables', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('text')->nullable();
             $table->morphs('recommendable');
@@ -28,7 +28,7 @@ 
             $table->timestamps();
         });
 
-        Schema::create('billables', function (Blueprint $table) {
+        Schema::create('billables', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('text')->nullable();
             $table->morphs('billable');
@@ -36,7 +36,7 @@ 
             $table->timestamps();
         });
 
-        Schema::create('articles_user', function (Blueprint $table) {
+        Schema::create('articles_user', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('article_id')->unsigned();
             $table->integer('user_id')->unsigned();
@@ -46,26 +46,26 @@ 
             $table->nullableTimestamps();
         });
 
-        Schema::create('recommends', function (Blueprint $table) {
+        Schema::create('recommends', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('title')->nullable();
             $table->timestamps();
         });
 
-        Schema::create('bills', function (Blueprint $table) {
+        Schema::create('bills', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('title')->nullable();
             $table->timestamps();
         });
 
-        Schema::create('stars', function (Blueprint $table) {
+        Schema::create('stars', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('starable_type');
             $table->bigInteger('starable_id');
             $table->string('title')->nullable();
         });
 
-        Schema::create('universes', function (Blueprint $table) {
+        Schema::create('universes', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id');
             $table->string('title')->nullable();
@@ -73,13 +73,13 @@ 
             $table->date('end_date')->nullable();
         });
 
-        Schema::create('planets', function (Blueprint $table) {
+        Schema::create('planets', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id')->nullable();
             $table->string('title')->nullable();
         });
 
-        Schema::create('transactions', function (Blueprint $table) {
+        Schema::create('transactions', function(Blueprint $table) {
             $table->bigIncrements('id')->unique();
             $table->bigInteger('user_id');
             $table->string('label')->nullable();
@@ -87,29 +87,29 @@ 
             $table->string('type');
         });
 
-        Schema::create('planets_non_nullable', function (Blueprint $table) {
+        Schema::create('planets_non_nullable', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id');
             $table->string('title')->nullable();
         });
 
-        Schema::create('comets', function (Blueprint $table) {
+        Schema::create('comets', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id')->default(0);
         });
 
-        Schema::create('bangs', function (Blueprint $table) {
+        Schema::create('bangs', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('name');
         });
 
-        Schema::create('account_details_bang', function (Blueprint $table) {
+        Schema::create('account_details_bang', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('account_details_id');
             $table->bigInteger('bang_id');
         });
 
-        Schema::create('account_details_bangs_pivot', function (Blueprint $table) {
+        Schema::create('account_details_bangs_pivot', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('account_details_id');
             $table->bigInteger('bang_id');

tests/Unit/CrudPanel/CrudPanelReadTest.php

Spacing +2 added lines, -2 removed lines

@@ -284,7 +284,7 @@ 
 
     public function testEnableDetailsRow()
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             $this->expectException(\Backpack\CRUD\app\Exceptions\BackpackProRequiredException::class);
         }
 
@@ -319,7 +319,7 @@ 
 
     public function testEnableExportButtons()
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             $this->expectException(\Backpack\CRUD\app\Exceptions\BackpackProRequiredException::class);
         }
 

tests/Unit/CrudPanel/CrudPanelFieldsTest.php

Spacing +1 added lines, -1 removed lines

@@ -604,7 +604,7 @@
     {
         $this->crudPanel->setOperationSetting(
             'strippedRequest',
-            static function (Request $request) {
+            static function(Request $request) {
                 return $request->toArray();
             },
             'update'

src/config/backpack/base.php

Spacing +4 added lines, -4 removed lines

@@ -70,13 +70,13 @@ 
     ],
 
     // CSS files that are loaded in all pages, using Laravel's mix() helper
-    'mix_styles' => [ // file_path => manifest_directory_path
+    'mix_styles' => [// file_path => manifest_directory_path
         // 'css/app.css' => '',
     ],
 
     // CSS files that are loaded in all pages, using Laravel's @vite() helper
     // Please note that support for Vite was added in Laravel 9.19. Earlier versions are not able to use this feature.
-    'vite_styles' => [ // resource file_path
+    'vite_styles' => [// resource file_path
         // 'resources/css/app.css' => '',
     ],
 
@@ -152,12 +152,12 @@ 
     ],
 
     // JS files that are loaded in all pages, using Laravel's mix() helper
-    'mix_scripts' => [ // file_path => manifest_directory_path
+    'mix_scripts' => [// file_path => manifest_directory_path
         // 'js/app.js' => '',
     ],
 
     // JS files that are loaded in all pages, using Laravel's @vite() helper
-    'vite_scripts' => [ // resource file_path
+    'vite_scripts' => [// resource file_path
         // 'resources/js/app.js',
     ],
 

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.
@@ -97,7 +97,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.
      *
@@ -112,7 +112,7 @@ 
     }
 }
 
-if (! function_exists('backpack_avatar_url')) {
+if (!function_exists('backpack_avatar_url')) {
     /**
      * Returns the avatar URL of a user.
      *
@@ -123,7 +123,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;
@@ -134,7 +134,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.
@@ -148,7 +148,7 @@ 
     }
 }
 
-if (! function_exists('backpack_guard_name')) {
+if (!function_exists('backpack_guard_name')) {
     /*
      * Returns the name of the guard defined
      * by the application config
@@ -159,7 +159,7 @@ 
     }
 }
 
-if (! function_exists('backpack_auth')) {
+if (!function_exists('backpack_auth')) {
     /*
      * Returns the user instance if it exists
      * of the currently authenticated admin
@@ -171,7 +171,7 @@ 
     }
 }
 
-if (! function_exists('backpack_user')) {
+if (!function_exists('backpack_user')) {
     /*
      * Returns back a user instance without
      * the admin guard, however allows you
@@ -183,7 +183,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).
@@ -204,7 +204,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.
@@ -223,7 +223,7 @@ 
 
         $returnView = $theme.$view;
 
-        if (! view()->exists($returnView)) {
+        if (!view()->exists($returnView)) {
             $returnView = $originalTheme.$view;
         }
 
@@ -231,7 +231,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.
@@ -247,7 +247,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:
@@ -279,7 +279,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.
      *
@@ -298,7 +298,7 @@ 
     }
 }
 
-if (! function_exists('backpack_pro')) {
+if (!function_exists('backpack_pro')) {
     /**
      * Check if the backpack/pro package is installed.
      *
@@ -306,7 +306,7 @@ 
      */
     function backpack_pro()
     {
-        if (! \Composer\InstalledVersions::isInstalled('backpack/pro')) {
+        if (!\Composer\InstalledVersions::isInstalled('backpack/pro')) {
             return false;
         }