No server side validation of required grid components

There is a bug, but specifically the server only validates that the grid contains at least one answer, rather than all the questions are answered.

The correct fix is to implement webform_validate_grid(). See webform_validate_date() as an example.

To reproduce this for testing purposes, you can comment out the following lines from webform_expand_grid:

      // Add HTML5 required attribute, if needed.
      if ($element['#required']) {
        $element[$key]['#attributes']['required'] = 'required';
      }

Committed to 7.x-4.x.

Hi,

I've been testing a webform using Grid components and general Textfields nested underneath. On updating from 7.x-4.13 -> 7.x-4.14 now my non-required Textfields are treated as required and I'm thinking it's related to this update?

Question 1: Grid required
---Comments: Textfield Not-required

Is this what it's meant to do? If so I'll have to take my Textfield out of the Grid but thought that was the way to format things nicely...

Cheers
Andrea

I agree. Nested components within a required grid should not be required unless they themselves are required. Also, if the entire grid is set as being an error, then all nested components are shown in error highlighting, even if they are not required. Consequently, we have to set each individual question as an error.

Committed to 7.x-4.x.

Both patches need up-port to D8.

Closing to clear out the old Webform 8.x-4.x branch. See #2827845: [roadmap] YAML Form 8.x-1.x to Webform 8.x-5.x.