form_execute_handlers() takes some pains to return a boolean value. This is undocumented, and none of the callers I can find make use of it.

CommentFileSizeAuthor
#1 core-form_execute_handlers-1943800-1.patch762 bytesmtift
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mtift’s picture

Status: Active » Needs review
FileSize
762 bytes

Is this what you had in mind?

joachim’s picture

Status: Needs review » Reviewed & tested by the community

Yup, pretty much!

xjm’s picture

Status: Reviewed & tested by the community » Needs work

The last submitted patch, core-form_execute_handlers-1943800-1.patch, failed testing.

xjm’s picture

Status: Needs work » Needs review
Issue tags: +Novice, +Needs backport to D7
xjm’s picture

Status: Needs review » Reviewed & tested by the community
xjm’s picture

Issue tags: +Quick fix
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8x, thanks!

This is an API change even if the return value is undocumented, so not moving to 7.x for backport.

xjm’s picture

Issue tags: -Needs backport to D7

Automatically closed -- issue fixed for 2 weeks with no activity.