Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Steps to reproduce:
1) Edit any submitted webform as Admin by visiting URL: /admin/structure/webform/manage/your_webform/submission/4/edit
2) Do the changes and save the webform.
3) Notice that the changes are not reflecting in the form.
4) Click on the edit menu again (Eventually redirecting to same page).
5) Now the data starts showing up.
Alternate of step 4:
Clear the cache and notice the data starts showing up.
The issue happens only when i am selecting confirmation type as Modal
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | 3109123-11.patch | 4.81 KB | jrockowitz |
| |||
| #10 | 3109123-10.patch | 2.44 KB | jrockowitz |
| |||
| #5 | media.io_Untitled_ Jan 28, 2020 7_43 PM.mp4 | 9.96 MB | Rohit Tiwari |
| #3 | Screenshot 2020-01-27 at 2.03.16 PM.png | 223.15 KB | Rohit Tiwari |
| Screenshot 2020-01-27 at 11.49.12 AM.png | 189.27 KB | Rohit Tiwari | |
Comments
Comment #2
Rohit Tiwari CreditAttribution: Rohit Tiwari commentedComment #3
Rohit Tiwari CreditAttribution: Rohit Tiwari commentedComment #4
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedI am not able to replicate this issue using the attached webform. Please provide the exact steps required to replicate this issue using an example webform.
Comment #5
Rohit Tiwari CreditAttribution: Rohit Tiwari commentedI have created a video of the issue.
Hope this explains well.
Issue Video
Comment #6
Rohit Tiwari CreditAttribution: Rohit Tiwari commentedComment #7
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedYep, I can replicate this when changing the contact form's confirmation to a modal.
Comment #8
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedThe attached patch starts to address this issue.
Comment #9
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedHere is a cleaner patch.
Comment #10
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedHere is the right solution.
Comment #11
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedThe attached patch adds test coverage.
Comment #13
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedI committed the patch. Please download the latest dev release to review.