API page: http://api.drupal.org/api/drupal/developer%21hooks%21core.php/function/h...

For D6

hook_form_alter(&$form, &$form_state, $form_id)
&$form_state - second arg must not be a link

Right way here:
hook_form_alter(&$form, $form_state, $form_id)

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Postponed (maintainer needs more info)

Ummm... I don't think that this issue report is correct. I believe that you can do &form_state. Why do you think that is wrong?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Project: Drupal core » Documentation
Version: 6.x-dev »
Component: documentation » API documentation files

Actually, the hook docs in Drupal 6.x are in the Documentation git repository, so moving to that project.

ilya1st’s picture

ilya1st commented

php 5.3 gives warning if &$form_state is there on d6

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Postponed (maintainer needs more info) » Active

Hm, I wonder why that is. But you are right: I looked through Drupal 6 core, and every one of the hook_form_alter() implementations there does not have & on $form_state. So let's have a patch to fix the documentation of the hook.

Thanks for taking the time to report this and follow-up with my question!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

We're not fixing Drupal 6 documentation any more.