This bug is 3 fold:

1) Forms added in beta12 or below can no longer be submitted after upgrading to beta13 because:
2) If a user configures a webform through the UI and doesn't click the fairly hidden "Edit submit buttons" link, the Webform still shows a submit button but pressing it does nothing except reload the page. Debugging into it, the WebformSubmissionForm::submitForm function isn't even executed
3) If a user fails to configure the Submit button label while editing the submit buttons for a form the above also happens.

Reverting to beta12 fixes the issue. I can't seem to replicate the problem on simplytest.me which leads me to believe there's some config that may have not been set on the upgrade?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

acbramley created an issue. See original summary.

jrockowitz’s picture

Can you please export and upload a broken form so that I can take a look at it.

acbramley’s picture

Sure, sorry should've thought of that! For some reason I can only reproduce it with this certain form so maybe it's something to do with one of the elements on it.

jrockowitz’s picture

I can't reproduce the issue. My best guess is that the issue is related to your theme, Do you still have problems when switching to Bartik?

jrockowitz’s picture

...doesn't click the fairly hidden "Edit submit buttons" link...

I agree.

@see #2884316: Improve customizing submit buttons UI/UX

acbramley’s picture

My best guess is that the issue is related to your theme, Do you still have problems when switching to Bartik?

good idea, unfortunately I get the same thing in Bartik.

jrockowitz’s picture

...but if you upload your form to a simpletest.me test site is the submit button okay.

Are you using customized form buttons? What happens if you delete the customized form buttons?

acbramley’s picture

Status: Active » Closed (cannot reproduce)

Yeah it works fine on the simplytest.me site...

Customised form buttons as in twig overrides? That wouldn't matter when switched to Bartik right?

In any case it must be something funky with our set up.

jrockowitz’s picture

For the record, I think this change #2878193: Allow actions (aka submit buttons) to be placed anywhere on a webform is causing your regression.