Laravel-Backpack / CRUD

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

Doc Comments +2 added lines, -4 removed lines

@@ -326,8 +326,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
      */
@@ -358,7 +356,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)
     {
@@ -369,7 +367,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  mixed $field
+     * @param  string $field
      * @return string Namespaced version of the field type name. Ex: 'text', 'custom.view.path.text'
      */
     public function getFieldTypeWithNamespace($field)

Spacing +10 added lines, -10 removed lines

@@ -55,7 +55,7 @@ 
      */
     public function afterField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, false);
         });
     }
@@ -67,7 +67,7 @@ 
      */
     public function beforeField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, true);
         });
     }
@@ -79,7 +79,7 @@ 
      */
     public function removeField($name)
     {
-        $this->transformFields(function ($fields) use ($name) {
+        $this->transformFields(function($fields) use ($name) {
             array_forget($fields, $name);
 
             return $fields;
@@ -93,7 +93,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);
             }
@@ -106,7 +106,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']);
             }
@@ -180,7 +180,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) {
@@ -209,7 +209,7 @@ 
      */
     public function orderFields($order)
     {
-        $this->transformFields(function ($fields) use ($order) {
+        $this->transformFields(function($fields) use ($order) {
             return $this->applyOrderToFields($fields, $order);
         });
     }
@@ -236,7 +236,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;
         });
 
@@ -299,7 +299,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);
 
@@ -340,7 +340,7 @@ 
      */
     public function fieldTypeNotLoaded($field)
     {
-        return ! in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
+        return !in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
     }
 
     /**

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

Spacing +4 added lines, -4 removed lines

@@ -57,7 +57,7 @@ 
      */
     public function getInlineCreateModal()
     {
-        if (! request()->has('entity')) {
+        if (!request()->has('entity')) {
             abort(400, 'No "entity" inside the request.');
         }
 
@@ -80,17 +80,17 @@ 
      */
     public function inlineRefreshOptions()
     {
-        if (! request()->has('field')) {
+        if (!request()->has('field')) {
             abort(400, 'No "field" inside the request.');
         }
 
         $field = $this->crud->fields()[request()->get('field')];
         $options = [];
 
-        if (! empty($field)) {
+        if (!empty($field)) {
             $relatedModelInstance = new $field['model']();
 
-            if (! isset($field['options'])) {
+            if (!isset($field['options'])) {
                 $options = $field['model']::all()->pluck($field['attribute'], $relatedModelInstance->getKeyName());
             } else {
                 $options = call_user_func($field['options'], $field['model']::query()->pluck($field['attribute'], $relatedModelInstance->getKeyName()));

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

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
     /**
      * Infer a field type, judging from the database column type.
      *
-     * @param string $field Field name.
+     * @param string $fieldName Field name.
      *
      * @return string Field type.
      */

Spacing +7 added lines, -7 removed lines

@@ -12,12 +12,12 @@ 
      */
     public function setFromDb()
     {
-        if (! $this->driverIsMongoDb()) {
+        if (!$this->driverIsMongoDb()) {
             $this->setDoctrineTypesMapping();
             $this->getDbColumnTypes();
         }
 
-        array_map(function ($field) {
+        array_map(function($field) {
             $new_field = [
                 'name'       => $field,
                 'label'      => $this->makeLabel($field),
@@ -29,11 +29,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),
@@ -105,7 +105,7 @@ 
 
         $dbColumnTypes = $this->getDbColumnTypes();
 
-        if (! isset($dbColumnTypes[$fieldName])) {
+        if (!isset($dbColumnTypes[$fieldName])) {
             return 'text';
         }
 
@@ -163,7 +163,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);
             }
         }
@@ -222,7 +222,7 @@ 
             // 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/Library/CrudPanel/Traits/Relationships.php

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
     /**
      * Gets the relation data from the method in the model.
      *
-     * @param ReflectionMethod $method
+     * @param \ReflectionMethod $method
      * @return array
      */
     public function inferFieldAttributesFromRelationship($method)

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
         }
 
         if (in_array($fieldName, array_column($relations, 'name'))) {
-            return array_filter($relations, function ($arr) use ($fieldName) {
+            return array_filter($relations, function($arr) use ($fieldName) {
                 if (isset($arr['name'])) {
                     return $arr['name'] == $fieldName;
                 }
@@ -72,7 +72,7 @@ 
      */
     public function inferFieldAttributesFromRelationship($method)
     {
-        if (! method_exists($this->model, $method)) {
+        if (!method_exists($this->model, $method)) {
             return false;
         }
 

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

Spacing +6 added lines, -6 removed lines

@@ -39,7 +39,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');
         }
 
@@ -92,7 +92,7 @@ 
     protected function makeSureFieldHasRelationshipData($field)
     {
         // only do this if "entity" is defined on the field
-        if (! isset($field['entity'])) {
+        if (!isset($field['entity'])) {
             return $field;
         }
 
@@ -111,7 +111,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($field['entity']) && ! isset($field['model'])) {
+        if (isset($field['entity']) && !isset($field['model'])) {
             $field['model'] = $this->getRelationModel($field['entity']);
         }
 
@@ -127,7 +127,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));
@@ -145,7 +145,7 @@ 
      */
     protected function makeSureFieldHasType($field)
     {
-        if (! isset($field['type'])) {
+        if (!isset($field['type'])) {
             $field['type'] = $this->inferFieldTypeFromDbColumnType($field['name']);
         }
 
@@ -162,7 +162,7 @@ 
     {
         // if a tab was mentioned, we should enable it
         if (isset($field['tab'])) {
-            if (! $this->tabsEnabled()) {
+            if (!$this->tabsEnabled()) {
                 $this->enableTabs();
             }
         }