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.
When having made changes to custom body - Message body - in Form handler page - email handler
then the page doesn't show the entered data, when i open the page again, unless I change between standard and back to custom body.
and second in the body once there is a display ENTER OTHER and once there is an numberlist display.
what is that.
Comment | File | Size | Author |
---|---|---|---|
#16 | Screen Shot 2016-10-27 at 10.13.38 AM.png | 163.5 KB | jrockowitz |
#12 | email_handler_custom-2818359-12.patch | 753 bytes | jrockowitz |
#3 | Screen Shot 2016-10-25 at 1.41.07 PM.png | 78.26 KB | cilefen |
#3 | Screen Shot 2016-10-25 at 1.39.17 PM.png | 60.1 KB | cilefen |
YAML Email handler 3.png | 60.73 KB | lomale@bluewin.ch |
Comments
Comment #2
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedSomeone else has also reported this issue but I can't duplicate.
Can you please look at the browser's JavaScript console and see if there is an error being logged?
Comment #3
cilefen CreditAttribution: cilefen as a volunteer commentedThere are no errors in the JavaScript console.
This is the second sub-issue about the numbered listing:
Comment #4
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedWow, I just can't duplicate this issue.
Can you please provide the steps (and browser info) required to reproduce this issue?
If possible use the Contact form or a template as the example.
Comment #5
cilefen CreditAttribution: cilefen as a volunteer commentedIt seems to have something to do with having multiple languages installed. And in fact, on other yamlform admin UIs I am missing textarea elements and there are console errors:
It begins with a 404 on: admin/structure/yamlform/manage/sample_application/lang/en.js?t=G87E
Comment #6
cilefen CreditAttribution: cilefen as a volunteer commentedWhat I found in #5 may or may not be a separate issue. It's that lang/en.js which is evidently part of ckeditor, is loading on the wrong path. I am still trying to figure out exactly how to reproduce this issue. To be fair, I can't make the email body work properly at all at the moment.
Comment #7
cilefen CreditAttribution: cilefen as a volunteer commentedI can get this on a clean Drupal 8.2.1 install, this module on 8.x-1.x, without multilingual modules installed. It's Chrome 54.0.2840.71 but I don't think that matters because something is wrong in the references to JavaScript files. I think #5 may be the same problem however with this issue itself, the email body, there are no console errors.
Comment #8
cilefen CreditAttribution: cilefen as a volunteer commentedYou are probably wondering if any CKEditor fields are working on this site, and yes, they are.
Also, Drupal is installed on a subdirectory path.
Comment #9
cilefen CreditAttribution: cilefen as a volunteer commentedThe problem on a given YAML form settings UI is happening with JS aggregation on but not when aggregation is off. The email body weirdness happens regardless.
Comment #10
cilefen CreditAttribution: cilefen as a volunteer commentedI just now tested on simplytest.me on Drupal 8.2.1 with 8.x-1.x of this module and the email body issue is partly reproducible in the sense that you have to switch away from "Custom body..." and back again to edit the body.
In better news, I can now confirm that the problems on the settings page have to do with having downloaded the third party libraries for this module.
So let's keep this issue to its original purpose and I opened #2822080: Current third-party libraries break settings forms for the other thing.
Comment #11
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedSteps to reproduce this issue...
Additional notes...
The problem is that modal dialogs dynamically load the content's JS and CSS libraries and this can cause issues with the initialization of JS behaviors. I have seen this problem with the CKEditor and CodeMirror. (http://cgit.drupalcode.org/yamlform/tree/modules/yamlform_ui/src/YamlFor...). The fix is going to be figuring out which libraries need to be preloaded for the dialog's #states to be properly initialized. Hopefully, I just need to preload the #states library.
Comment #12
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedFor now, I am just going to copy the fix from the YamlFormUiEntityForm. If we continue to run into this issue, I will come up with a more reusable solution.
Comment #14
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedSorry, it took so long to figure out the problem.
Since all the tests passed and I am familiar with this issue, I committed the patch, so please download or checkout the latest dev release to test.
Comment #15
cilefen CreditAttribution: cilefen as a volunteer commentedThis is not fixed for me on 8.x-1.x.
Comment #16
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedI am seeing the below error message using the YAML Form 8.x-1.x branch on Simplytest.me (https://simplytest.me/project/yamlform/8.x-1.x). Simplytest.me does not have the private file system enabled (/admin/config/media/file-system).
Comment #17
cilefen CreditAttribution: cilefen as a volunteer commentedI think you are referencing a different issue.
Comment #18
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedDuh! I am working on too many tickets at the same time.
Comment #19
jrockowitz CreditAttribution: jrockowitz as a volunteer commented