Merge pull request #3779 from Laravel-Backpack/required-repeatable-fields

fix required repeatable fields

Author: tabacitu

src/public/packages/backpack/crud/css/form.css

@@ -4,7 +4,7 @@
 *
 */
 
-.form-group.required label:not(:empty):not(.form-check-label)::after {
+.form-group.required > label:not(:empty):not(.form-check-label)::after {
     content: ' *';
     color: #ff0000;
 }

src/resources/views/crud/fields/inc/wrapper_start.blade.php

@@ -8,11 +8,11 @@
     }
 	// if the field is required in the FormRequest, it should have an asterisk
 	$required = (isset($action) && $crud->isRequired($field['name'], $action)) ? ' required' : '';
-
+	
 	// if the developer has intentionally set the required attribute on the field
 	// forget whatever is in the FormRequest, do what the developer wants
-	$required = (isset($field['showAsterisk'])) ? ($field['showAsterisk'] ? ' required' : '') : $required;
-
+	$required = isset($field['showAsterisk']) ? ($field['showAsterisk'] ? ' required' : '') : $required;
+	
 	$field['wrapper']['class'] = $field['wrapper']['class'] ?? "form-group col-sm-12";
 	$field['wrapper']['class'] = $field['wrapper']['class'].$required;
 	$field['wrapper']['element'] = $field['wrapper']['element'] ?? 'div';

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

@@ -53,7 +53,6 @@
               $subfield = $crud->makeSureFieldHasNecessaryAttributes($subfield);
               $fieldViewNamespace = $subfield['view_namespace'] ?? 'crud::fields';
               $fieldViewPath = $fieldViewNamespace.'.'.$subfield['type'];
-              $subfield['showAsterisk'] = false;
           @endphp
 
           @include($fieldViewPath, ['field' => $subfield])

src/resources/views/crud/form_content.blade.php

@@ -22,6 +22,17 @@
 
     <!-- CRUD FORM CONTENT - crud_fields_styles stack -->
     @stack('crud_fields_styles')
+
+    {{-- Temporary fix on 4.1 --}}
+    <style>
+      .form-group.required label:not(:empty):not(.form-check-label)::after {
+        content: '';
+      }
+      .form-group.required > label:not(:empty):not(.form-check-label)::after {
+        content: ' *';
+        color: #ff0000;
+      }
+    </style>
 @endsection
 
 @section('after_scripts')