content_type.inc fapi 2 cleanup bug [#83316]

Seem this function node_type_form_submit got missed in the fapi form change clean up

Changing $op = isset($_POST['op']) ? $_POST['op'] : '';
To $op = isset($form_values['op']) ? $form_values['op'] : '';

So that the op information gets retrieved from the $form_values rather than $_POST to work with drupal_execute.

Tested and all seems to still work appropriately.

Comment	File	Size	Author
	content_type.inc.patch	664 bytes	nickl

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

eaton CreditAttribution: eaton commented 10 September 2006 at 00:46

Status:

Needs review

» Reviewed & tested by the community

This is a very simple change, and it is good. Yay.

There are a number of forms scattered around the system that peek at $_POST unecessarily; most of them do it to extract the current op. The seriesof patches that went in befor ethe freeze made that ugly trick unecessary, and made it possible for these forms to work with programmatic submission.