Closed (fixed)
Project:
Examples for Developers
Version:
8.x-1.x-dev
Component:
Form API Example
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
18 Apr 2017 at 15:53 UTC
Updated:
29 Dec 2017 at 18:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
metzlerd commentedComment #3
metzlerd commentedComment #4
metzlerd commentedComment #6
metzlerd commentedFYI: #5 was really about first empty patch uploaded. This patch should be ready for review.
Sorry for any confusion caused.
Comment #7
Lal_Looks like the patch applies properly.
Comment #8
mile23After applying this patch, go to this path (with JS turned on): examples/fapi-example/modal-form
Submit a title. You'll see that there is no submit handler message. However, if you reload the patch, you see the submit handler message.
Then, if you go to the main page for this module: examples/fapi-example
Click on the 'Modal Form' link and submit a title. You'll see the submit handler message as it should be on the page. Now, if you click the Modal Form link again, you'll see the status message again in the UI modal form.
I might have just filed a duplicate issue for this here: #2883725: fapi_example modal dialog does not show result message w/ JS
Please mark it dupe if it is. Thanks.
Also, we'll need a test for this. I think it might end up being a javascript functional test.
Comment #9
mile23Comment #10
mile23Postponed on #2838276: Modal Form Example not working
Comment #11
mile23#2838276: Modal Form Example not working gives us a basic behavior, so now we can test and fix the validation phase.
Comment #12
mile23Working on this.
Comment #13
mile23Moving things around, working on the JS functional test. There are some problems with the test system: #2831506: Minimal profile disallows modal AJAX tests under JavascriptTestBase
This will fail.
Comment #15
mile23This should be it.
There are todos about #2831506: Minimal profile disallows modal AJAX tests under JavascriptTestBase
Comment #17
mile23Fix-O-Rama.
Now we have functional testing of the modal form.
This specific example should maybe be in the AJAX example, or maybe adding @see type stuff in the docblocks and human-readable parts.
Comment #18
mile23