Fix content_copy_form_alter(), where $form_state argument is not passed by reference

See #649132-3: php 5.3, this should not be necessary.

Regardless of the fact that it could be a PHP 5.3 issue too, hook_form_alter() can only modify the first argument, so $form_state should not be declared as a by reference argument.

Fix committed to CVS. Thanks!

http://drupal.org/cvs?commit=297962 (CCK2)
http://drupal.org/cvs?commit=297966 (CCK3)

Nope, that is not correct. drupal_alter contains a hack that allows to have more than one by reference argument, see the huge comment at http://api.drupal.org/api/function/drupal_alter/6 and how it's called at http://api.drupal.org/api/function/drupal_prepare_form/6.

Of course, if you don't need to change $form_state, it is perfectly fine to remove the &. It's just your statement that isn't correct :)

Oops! You're right. Thanks for pointing this out. Anyway, content_copy_form_alter() does not need to modify the $form_state argument, so the patch should cause no harm.

Can someone please tell me how to apply the patch?