| Comment | File | Size | Author |
|---|---|---|---|
| #10 | resend_submission_s-2839897-9.patch | 1.39 KB | jrockowitz |
| #10 | webform-resend-after.png | 166.28 KB | jrockowitz |
| #10 | webform-resend-before.png | 160.63 KB | jrockowitz |
| #10 | Screen Shot 2017-01-27 at 8.50.57 AM.png | 81.77 KB | jrockowitz |
| #7 | Contact Submission 1 d8webform.png | 174.94 KB | zhouhana |
Comments
Comment #2
jrockowitz commentedI can't duplicate the issue using the 'Test: Handler: Email advanced' (/webform/form/test-handler-email-advanced)
Can you please document the steps needed to reproduce this issue?
Comment #3
vanyamtv commentedComment #4
jrockowitz commentedCan you upload your example form so that I can duplicate the issue?
Are you able to duplicate the problem using 'Test: Handler: Email advanced'?
You will need to add
$settings['extension_discovery_scan_tests'] = TRUE;to settings.php and enable to the webform_test.module to access this form.Comment #5
zhouhana commentedI just ran into the same issue. When I try to resend the e-mail, the letter "ö" is suddenly " & o u m l ; " and Drupal complains that "Entity 'ouml' not defined". See attached image.
Comment #6
jrockowitz commentedI need to be able to duplicate the issue so that I can fix it.
@zhouhana Can you please create a working example with hardcoded values that duplicates the problem, export it, and upload it here?
Comment #7
zhouhana commentedI'm not sure how to do that, but all you need to do is
1. Install a fresh copy of Drupal 8 with Webform enabled
2. Edit the body message of one of the e-mail handlers of the example Contact form to read, for example, "ö", instead of "[webform_submission:values:message:value]".
3. Use the test function to send it.
4. Go to Results and try to resend it.
See attached image of the result I get after completing these steps.
To make the patch fool proof, I guess it's a good idea to test with several HTML entities: https://dev.w3.org/html5/html-author/charref
Comment #8
jrockowitz commented@zhouhana That helps a lot. I was not hard coding the special character in the actual email body. I was entering it via the message body, which works fine.
Comment #10
jrockowitz commentedThe sent email looks fine.
One immediate issue is the 'Resend' form is displaying the HTML source, which might be causing some confusion.
The best solution is to just enable the HTML editor on the 'Resend' form, which creates a much cleaner and predictable UX.
If all the tests pass, I commit the attached patch and wait for more feedback.
Comment #12
jrockowitz commentedI committed the patch and I am going to mark this as fixed.
Please review using the latest dev release and reopen this ticket if the issue still persists.
Comment #13
zhouhana commentedWorks great here as well. Thank you.