Changed $_POST to $form_values in these files:

aggregator.module
book.module
search.module
system.module
user.module

5 files in total. Review please, thanks.

CommentFileSizeAuthor
#4 form_api_cleanup_0.patch4.76 KBJaza
form_api_cleanup.patch5.22 KBedmund.kwok

Comments

eaton’s picture

eaton commented

+1. These patches also bring aggregator management and book-hierarchies into the realm of programmatic submission. It's good stuff.

chx’s picture

chx commented

yay less superglobals!

Jaza’s picture

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

Tested for most of the affected forms, validation/submission still works as expected. Nice patch. Looks RTBC to me.

Jaza’s picture

Jaza commented
StatusFileSize
new form_api_cleanup_0.patch4.76 KB

Just fixing up an offset in user.module, and fixing the directory base for the patch. Otherwise, exactly the same. Still RTBC.

dries’s picture

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

Committed to CVS HEAD. Thanks.

Anonymous’s picture

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