Some more fapi 2 cleanup.

menu.module references $_POST variable in hook_form_alter this is available in $form['#post'].
Removed $_POST reference and redundant variable $edit and used $form['#post'] directly.

Tested and submitted node_forms still get their menus added.
Please review.

CommentFileSizeAuthor
fapi_cleanup_menu.module.patch1.38 KBnickl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jaza’s picture

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

Tested, and menu editing on node forms still works fine. This is a straightforward enough patch. Code style looks pretty good to me. I say RTBC.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Anonymous’s picture

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