Merge pull request #5280 from Laravel-Backpack/make-sure-no-white-spaces-on-crud-comma-separated-values

Fix comma separated field names breaking when a space was added

Author: pxpm

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

@@ -47,13 +47,15 @@ protected function makeSureColumnHasPriority($column)
     protected function makeSureColumnHasName($column)
     {
         if (is_string($column)) {
-            $column = ['name' => $column];
+            return ['name' => Str::replace(' ', '', $column)];
         }
 
         if (is_array($column) && ! isset($column['name'])) {
             $column['name'] = 'anonymous_column_'.Str::random(5);
         }
 
+        $column['name'] = Str::replace(' ', '', $column['name']);
+
         return $column;
     }
 

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

@@ -112,13 +112,15 @@ protected function makeSureFieldHasName($field)
         }
 
         if (is_string($field)) {
-            return ['name' => $field];
+            return ['name' => Str::replace(' ', '', $field)];
         }
 
         if (is_array($field) && ! isset($field['name'])) {
             abort(500, 'All fields must have their name defined');
         }
 
+        $field['name'] = Str::replace(' ', '', $field['name']);
+
         return $field;
     }
 
@@ -272,6 +274,8 @@ protected function makeSureSubfieldsHaveNecessaryAttributes($field)
                 $subfield = ['name' => $subfield];
             }
 
+            $subfield['name'] = Str::replace(' ', '', $subfield['name']);
+
             $subfield['parentFieldName'] = $field['name'];
 
             if (! isset($field['model'])) {