Right now you can't put FAPE inside a dialog and expect validation errors to work properly. In fact, you can't have ajax submission at all, because the entity_save is in the page handler instead of the form submit callback.

I moved it to the submit callback and it seems to work. Is there something I'm overlooking? If not, I'll submit a patch.

Thanks.

CommentFileSizeAuthor
#2 fape-2657272-2-ajax_fix.patch713 bytescaptainack

Comments

captainack created an issue. See original summary.

captainack’s picture

captainack commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new fape-2657272-2-ajax_fix.patch713 bytes

Status: Needs review » Needs work

The last submitted patch, 2: fape-2657272-2-ajax_fix.patch, failed testing.

captainack’s picture

captainack commented
Status: Needs work » Needs review

weird... there's no test but it's trying to run one?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

@captainack: It's a known bug in testbot.

captainack’s picture

captainack commented
Related issues: +#2716935: Give FAPE's ajax submission the same race condition immunity as non-ajax submission