Currently when a node type editing form is submitted, some form values (such as 'op' and 'form_token') get saved to the variables table when they shouldn't, but also the form values themselves are unset which prevents any contrib module that adds _submit handler to the form from avtually getting the form values.
Attached patch tries to catch and unet all the extraneous values that should not be set, and also copies the form values to another array for use in the unset() operations.
This is a sub-part of the issues originally discussed here: http://drupal.org/node/88633
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | cleanup_type_variables.patch | 1.33 KB | profix898 |
| cleanup_type_variables_1.diff | 1.45 KB | pwolanin |
Comments
Comment #1
Jaza commentedLooks good, same as in http://drupal.org/node/88633.
RTBC.
Comment #2
profix898 commentedAlmost the same patch as above, but I removed a few variables from the unset() array, because they are unset through 'if (isset($type->$key)) {...}' already. Otherwise patch looks good to me.
Comment #3
dries commentedCommitted to CVS HEAD. Thanks.
Comment #4
(not verified) commented