FormAPI 3 eliminated support for hook_submit() and nodeapi op submit, since all those tasks can be performed by adding a submit handler to the form itself.

Ah, but then we forgot to put in the four lines that properly trigger those submit handlers during node form submission. Whoops!

This patch fixes that oversight.

CommentFileSizeAuthor
#3 node_form_fix_0.patch941 byteseaton
node_form_fix.patch789 byteseaton

Comments

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community

umm yes.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

eaton’s picture

eaton commented
StatusFileSize
new node_form_fix_0.patch941 bytes

This one might have had a bit TOO speedy response. *grin* One line still needs to go in -- the one where the altered data from the assorted handlers is actually passed on to node_save()

Gurpartap Singh’s picture

Gurpartap Singh commented
Status: Fixed » Needs review
dries’s picture

dries commented
Status: Needs review » Fixed

Committed to CVS HEAD.

eaton’s picture

eaton commented

Thanks, Dries!

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)