Closed (fixed)
Project:
Webform
Version:
8.x-4.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
11 Jun 2014 at 19:25 UTC
Updated:
27 Jun 2014 at 20:00 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
danchadwick commentedI've implemented this change. In the case where an already-completed webform is being edited, the 'submit' button is called 'Save' rather than the standard 'Submit' or whatever the user customized it to in the webform options. While one might add yet-another customized button label, I felt that the circumstance is sufficiently rare, and often encountered by an administrator rather than the form-filler-outer, that this was unneeded. If others disagree, I will add an option.
At the same time, I noticed a code redundancy in creating the submit button which could easily be eliminated by observing that for a 1 page form, the current page == the number of pages. The patch seems big, but really is mostly code indentation change due to the eliminate of a big if clause.
I tested this with single and multi-page forms, with and without preview, with and without save-as-draft, with and without already being submitted.
Committed to 7.x-4.x and 8.x