Currently, there are several differing naming schemes for the #process callback for RenderAPI elements (those returned from hook_elements): expand_*, form_expand_* and process_*. This patch unifies the naming to form_process_*. It does not change any functionality whatsoever.

CommentFileSizeAuthor
#5 unify-process-callbacks.patch6.74 KBkkaefer
unify-process-callbacks.patch7.08 KBkkaefer

Comments

moshe weitzman’s picture

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

I tested and code reviewed. RTBC.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

Excellent. This has bothered me too. Thanks, kkaefer :)

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Component: base system » forms system
dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

Can we reroll? Doesn't seem to apply.

kkaefer’s picture

kkaefer commented
Status: Needs work » Needs review
StatusFileSize
new unify-process-callbacks.patch6.74 KB

Updated. Ran some of the tests, nothing failed.

damien tournoud’s picture

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

Applied, grepped thru the core to verify that no traces of old functions remains. Everything looks good.

egrep -r '#process' . returns:

modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_password_confirm'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_radios'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_checkboxes'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_weight', 'form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_date'),
modules/system/system.module:    '#process' => array('form_process_ahah'),
modules/system/system.module:    '#process' => array('form_process_ahah'),

Neat.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

Nice little clean-up. I've committed this to CVS HEAD. I'm marking this 'code needs work' as it probably requires a couple of documentation updates, both in the upgrading guide as in the API docs for CVS HEAD. Please mark this closed once the documentation is in good shape. Thanks for the great work!

dries’s picture

dries commented

Nice little clean-up. I've committed this to CVS HEAD. I'm marking this 'code needs work' as it probably requires a couple of documentation updates, both in the upgrading guide as in the API docs for CVS HEAD. Please mark this closed once the documentation is in good shape. Thanks for the great work!

kkaefer’s picture

kkaefer commented
Status: Needs work » Fixed

Documented at http://drupal.org/node/224333#process-functions.

Anonymous’s picture

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

Automatically closed -- issue fixed for two weeks with no activity.

axyjo’s picture

axyjo commented

Fixed spelling mistake from 'occured' to 'occurred' in http://drupal.org/update/modules/6/7#process_functions

do’s picture

do commented

In the current Drupal7.0-beta2 the function "form_process_ahah" is gone. What is its replacement?

The documentation http://drupal.org/update/modules/6/7 about updateing modules still references it.

Thanks,
udo