node_access_test_form_node_form_alter() has wrong documentation header

small update

Actually, I found the functionality I was looking for with hook_field_attach_form. Maybe this issue is not relevant because hook_form_node_form_alter is obsolete.

I am a bit confused, I came across "i18n_node_form_node_form_alter" so I thought it was about hook_form_alter but in fact it's hook_form_BASE_FORM_ID_alter... Anyway, look inside node/tests/node_access_test.module line 162 you will find :

/**
 * Implements hook_form_node_form_alter().
 */
function node_access_test_form_node_form_alter(&$form, $form_state) {
  // Only show this checkbox for NodeAccessBaseTableTestCase.
  if (variable_get('node_access_test_private')) {
    $form['private'] = array(
      '#type' => 'checkbox',
      '#title' => t('Private'),
      '#description' => t('Check here if this content should be set private and only shown to privileged users.'),
      '#default_value' => isset($form['#node']->private) ? $form['#node']->private : FALSE,
    );
  }
}

It's not a problem for me, but is this lacking documentation ?

I made my first git patch, attached here. I followed these guidelines : http://drupal.org/node/1054616.
Is it correct ?