Problem/Motivation

The time fields on later pages are being validated against the step configuration on the first page preventing movement forward in the form.

I discovered this myself in 6.1.8 but it is present also now in 6.3.x
It appears to have been introduced in the fix for #3305878

Steps to reproduce

Create a form with multiple pages
put a date time field on the second or further pages
set the step to something like 10m
I don't think it needs a default value, but appears to start out with one based on now

Proposed resolution

It appears that the step validation is the only validation that doesn't check has_access to identify if the field is on the current page being validated. I believe this will be true for wizard pages and conditional fields.

I believe the included MR is the right fix to prevent early validation. Tested this locally myself but wanted to report and share the fix.

Issue fork webform-3555424

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

ehlovader created an issue. See original summary.

ehlovader’s picture

Issue summary: View changes

Updated with additional version information

ehlovader’s picture

Status: Active » Needs review

Added merge request

jrockowitz’s picture

Change make perfect sense and aligns with existing validation logic

jrockowitz’s picture

Status: Needs review » Fixed

Change make perfect sense and aligns with existing validation logic

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.