Laravel-Backpack / CRUD

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

Doc Comments -1 removed lines

@@ -9,7 +9,6 @@
     /**
      * Define which routes are needed for this operation.
      *
-     * @param string $name       Name of the current entity (singular). Used as first URL segment.
      * @param string $routeName  Prefix of the route name.
      * @param string $controller Name of the current CrudController.
      */

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
     {
         $this->crud->allowAccess('update');
 
-        $this->crud->operation('update', function () {
+        $this->crud->operation('update', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
 
             if ($this->crud->getModel()->translationEnabled()) {
@@ -53,7 +53,7 @@ 
             }
         });
 
-        $this->crud->operation(['list', 'show'], function () {
+        $this->crud->operation(['list', 'show'], function() {
             $this->crud->addButton('line', 'update', 'view', 'crud::buttons.update', 'end');
         });
     }

src/resources/views/crud/columns/array_count.blade.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
     $suffix = isset($column['suffix']) ? $column['suffix'] : 'items';
 
     // the value should be an array wether or not attribute casting is used
-    if (! is_array($array)) {
+    if (!is_array($array)) {
         $array = json_decode($array, true);
     }
     if ($array && count($array)) {

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

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@ 
 <?php
     $max = isset($field['max']) && (int) $field['max'] > 0 ? $field['max'] : -1;
     $min = isset($field['min']) && (int) $field['min'] > 0 ? $field['min'] : -1;
-    $item_name = strtolower(isset($field['entity_singular']) && ! empty($field['entity_singular']) ? $field['entity_singular'] : $field['label']);
+    $item_name = strtolower(isset($field['entity_singular']) && !empty($field['entity_singular']) ? $field['entity_singular'] : $field['label']);
 
     $items = old(square_brackets_to_dots($field['name'])) ?? $field['value'] ?? $field['default'] ?? '';
 
@@ -15,7 +15,7 @@ 
         } else {
             $items = '[]';
         }
-    } elseif (is_string($items) && ! is_array(json_decode($items))) {
+    } elseif (is_string($items) && !is_array(json_decode($items))) {
         $items = '[]';
     }
 ?>

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

Spacing +2 added lines, -2 removed lines

@@ -9,10 +9,10 @@
 
     $field_language = isset($field['date_picker_options']['language']) ? $field['date_picker_options']['language'] : \App::getLocale();
 
-    if (! isset($field['attributes']['style'])) {
+    if (!isset($field['attributes']['style'])) {
         $field['attributes']['style'] = 'background-color: white!important;';
     }
-    if (! isset($field['attributes']['readonly'])) {
+    if (!isset($field['attributes']['readonly'])) {
         $field['attributes']['readonly'] = 'readonly';
     }
 ?>

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

Spacing +2 added lines, -2 removed lines

@@ -3,11 +3,11 @@
 <?php
     // if the column has been cast to Carbon or Date (using attribute casting)
     // get the value as a date string
-    if (! function_exists('formatDate')) {
+    if (!function_exists('formatDate')) {
         function formatDate($entry, $dateFieldName)
         {
             $formattedDate = null;
-            if (isset($entry) && ! empty($entry->{$dateFieldName})) {
+            if (isset($entry) && !empty($entry->{$dateFieldName})) {
                 $dateField = $entry->{$dateFieldName};
                 if ($dateField instanceof \Carbon\CarbonInterface) {
                     $formattedDate = $dateField->format('Y-m-d H:i:s');

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

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         </div>
         <div class="col-sm-9">
             <!-- external link input -->
-              <div class="page_or_link_value page_or_link_external_link <?php if (! isset($entry) || $entry->type != 'external_link') {
+              <div class="page_or_link_value page_or_link_external_link <?php if (!isset($entry) || $entry->type != 'external_link') {
     echo 'd-none';
 } ?>">
                 <input
@@ -56,7 +56,7 @@ 
                     >
               </div>
               <!-- internal link input -->
-              <div class="page_or_link_value page_or_link_internal_link <?php if (! isset($entry) || $entry->type != 'internal_link') {
+              <div class="page_or_link_value page_or_link_internal_link <?php if (!isset($entry) || $entry->type != 'internal_link') {
     echo 'd-none';
 } ?>">
                 <input

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

Doc Comments +2 added lines, -9 removed lines

@@ -22,7 +22,6 @@ 
      * Add a field to the create/update form or both.
      *
      * @param string|array $field The new field.
-     * @param string       $form  The CRUD form. Can be 'create', 'update' or 'both'. Default is 'both'.
      *
      * @return self
      */
@@ -70,7 +69,6 @@ 
      * Add multiple fields to the create/update form or both.
      *
      * @param array  $fields The new fields.
-     * @param string $form   The CRUD form. Can be 'create', 'update' or 'both'. Default is 'both'.
      */
     public function addFields($fields)
     {
@@ -85,7 +83,6 @@ 
      * Move the most recently added field after the given target field.
      *
      * @param string $targetFieldName The target field name.
-     * @param string $form            The CRUD form. Can be 'create', 'update' or 'both'. Default is 'both'.
      */
     public function afterField($targetFieldName)
     {
@@ -98,7 +95,6 @@ 
      * Move the most recently added field before the given target field.
      *
      * @param string $targetFieldName The target field name.
-     * @param string $form            The CRUD form. Can be 'create', 'update' or 'both'. Default is 'both'.
      */
     public function beforeField($targetFieldName)
     {
@@ -141,7 +137,6 @@ 
      * Remove a certain field from the create/update/both forms by its name.
      *
      * @param string $name Field name (as defined with the addField() procedure)
-     * @param string $form update/create/both
      */
     public function removeField($name)
     {
@@ -330,8 +325,6 @@ 
      * Check if the create/update form has upload fields.
      * Upload fields are the ones that have "upload" => true defined on them.
      *
-     * @param string   $form create/update/both - defaults to 'both'
-     * @param bool|int $id   id of the entity - defaults to false
      *
      * @return bool
      */
@@ -362,7 +355,7 @@ 
     /**
      * Set an array of field type names as already loaded for the current operation.
      *
-     * @param array $fieldTypes
+     * @param string[] $fieldTypes
      */
     public function setLoadedFieldTypes($fieldTypes)
     {
@@ -373,7 +366,7 @@ 
      * Get a namespaced version of the field type name.
      * Appends the 'view_namespace' attribute of the field to the `type', using dot notation.
      *
-     * @param  array $field Field array
+     * @param  string $field Field array
      * @return string Namespaced version of the field type name. Ex: 'text', 'custom.view.path.text'
      */
     public function getFieldTypeWithNamespace($field)

Spacing +14 added lines, -14 removed lines

@@ -38,23 +38,23 @@ 
 
         // 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($newField['entity']) && ! isset($newField['model'])) {
+        if (isset($newField['entity']) && !isset($newField['model'])) {
             $newField['model'] = $this->getRelationModel($newField['entity']);
         }
 
         // if the label is missing, we should set it
-        if (! isset($newField['label'])) {
+        if (!isset($newField['label'])) {
             $newField['label'] = mb_ucfirst(str_replace('_', ' ', $newField['name']));
         }
 
         // if the field type is missing, we should set it
-        if (! isset($newField['type'])) {
+        if (!isset($newField['type'])) {
             $newField['type'] = $this->getFieldTypeFromDbColumnType($newField['name']);
         }
 
         // if a tab was mentioned, we should enable it
         if (isset($newField['tab'])) {
-            if (! $this->tabsEnabled()) {
+            if (!$this->tabsEnabled()) {
                 $this->enableTabs();
             }
         }
@@ -90,7 +90,7 @@ 
      */
     public function afterField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, false);
         });
     }
@@ -103,7 +103,7 @@ 
      */
     public function beforeField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, true);
         });
     }
@@ -146,7 +146,7 @@ 
      */
     public function removeField($name)
     {
-        $this->transformFields(function ($fields) use ($name) {
+        $this->transformFields(function($fields) use ($name) {
             array_forget($fields, $name);
 
             return $fields;
@@ -160,7 +160,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);
             }
@@ -173,7 +173,7 @@ 
     public function removeAllFields()
     {
         $current_fields = $this->getCurrentFields();
-        if (! empty($current_fields)) {
+        if (!empty($current_fields)) {
             foreach ($current_fields as $field) {
                 $this->removeField($field['name']);
             }
@@ -247,7 +247,7 @@ 
                 $jsonCastables = ['array', 'object', 'json'];
                 $fieldCasting = $casted_attributes[$field['name']];
 
-                if (in_array($fieldCasting, $jsonCastables) && isset($data[$field['name']]) && ! empty($data[$field['name']]) && ! is_array($data[$field['name']])) {
+                if (in_array($fieldCasting, $jsonCastables) && isset($data[$field['name']]) && !empty($data[$field['name']]) && !is_array($data[$field['name']])) {
                     try {
                         $data[$field['name']] = json_decode($data[$field['name']]);
                     } catch (\Exception $e) {
@@ -276,7 +276,7 @@ 
      */
     public function orderFields($order)
     {
-        $this->transformFields(function ($fields) use ($order) {
+        $this->transformFields(function($fields) use ($order) {
             return $this->applyOrderToFields($fields, $order);
         });
     }
@@ -339,7 +339,7 @@ 
     public function hasUploadFields()
     {
         $fields = $this->getFields();
-        $upload_fields = array_where($fields, function ($value, $key) {
+        $upload_fields = array_where($fields, function($value, $key) {
             return isset($value['upload']) && $value['upload'] == true;
         });
 
@@ -399,7 +399,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);
 
@@ -440,7 +440,7 @@ 
      */
     public function fieldTypeNotLoaded($field)
     {
-        return ! in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
+        return !in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
     }
 
     /**

src/app/Models/Traits/InheritsRelationsFromParentModel.php

Spacing +3 added lines, -3 removed lines

@@ -17,14 +17,14 @@ 
 
     public static function bootHasParent()
     {
-        static::creating(function ($model) {
+        static::creating(function($model) {
             if ($model->parentHasHasChildrenTrait()) {
                 $model->forceFill(
                     [$model->getInheritanceColumn() => $model->classToAlias(get_class($model))]
                 );
             }
         });
-        static::addGlobalScope(function ($query) {
+        static::addGlobalScope(function($query) {
             $instance = new static();
             if ($instance->parentHasHasChildrenTrait()) {
                 $query->where($instance->getTable().'.'.$instance->getInheritanceColumn(), $instance->classToAlias(get_class($instance)));
@@ -39,7 +39,7 @@ 
 
     public function getTable()
     {
-        if (! isset($this->table)) {
+        if (!isset($this->table)) {
             return str_replace('\\', '', snake_case(str_plural(class_basename($this->getParentClass()))));
         }