Fieldset validating issue

[jraack]

Hi Guys,

I just found an issue with the fieldset. I have a form based on the fields syntax.
When i add fields under a fieldset it is not validating my input fields if they are required?

form:  
  fields:    
    -
        name: email
        outerclasses: medium-12 columns
        placeholder: E-mail adres
        type: email
        validate:
          required: true
          message: PLUGIN_LOGIN.EMAIL_VALIDATION_MESSAGE
    -
        type: fieldset
        outerclasses: row
        fields:            
        -
            name: newsletter
            outerclasses: medium-12 columns
            type: checkbox

Or do i missing something?
Issue ref Fieldset support

Best,
Joe

[bobbwal]

Hi, what's your frontmatter look like?

[jraack]

@bobbwal see example.

---
form:  
  fields:    
    -
        name: email
        outerclasses: medium-12 columns
        placeholder: E-mail address
        type: email
        validate:
          required: true
          message: PLUGIN_LOGIN.EMAIL_VALIDATION_MESSAGE
 -
        type: fieldset
        outerclasses: row
        fields:  
            -
                name: initials
                outerclasses: medium-4 columns
                placeholder: initials
                type: text
                validate:
                    required: true
            -
                name: suffix
                outerclasses: medium-4 columns
                placeholder: suffix
                type: text
                validate:
                  required: false
            -
                name: lastname
                outerclasses: medium-4 columns
                placeholder: lastname
                type: text
                validate:
                    required: true 
    -
        type: fieldset
        outerclasses: row
        fields:            
        -
            name: newsletter
            outerclasses: medium-12 columns
            type: checkbox
            label:
        -
            name: newsletter
            outerclasses: medium-12 columns
            type: checkbox
            label: 
        -
            name: big
            type: radio
            label: 
            outerclasses: medium-12 columns
            default: markdown        
            validate:
                required: false
            options:
                true: Yes
                false: No             
  buttons:
      -
          type: submit
          value: Register

  process:
      register_form: true
      display: '/registration'
      message: "Thank you"
---

[bobbwal]

Hmm, looks in order to me. Does the fieldset need a name? Also, there aren't any required fields within the fieldset here?

[jraack]

@bobbwal thanks for your reply; i have updated the frontmatter.

[bobbwal]

All I can suggest is try adding a name for the fieldsets. Sorry I can't be more help.

name: first-set
type: fieldset
fields:

[jraack]

hi @bobbwal
not a problem, i will try, thanks

[flaviocopes]

It's working fine for me too. The browser frontend validation is kicking in (Chrome)

[rhukster]

Not related, but i've fixed an issue with data.html.twig and data.txt.twig not being able to find fields inside fieldset. This will be sorted in next release. Validation works for me btw.

[jraack]

@rhukster i have updated the data.html and data.txt and the validation is working now on my side 👍
Only thing that is not working, nested fields inside fieldsets are being cleared if the forms returns an error.
If i put a field outside the fieldset, it shows my previous data.

[rhukster]

Ok this last issue is fixed in develop