NOTE: See comments #3/4 for the actual bug report here...
--- original report ---
I can't find documentation for form_node although it is used for adding configuration fields to nodes. It is used especially in hook_form_node_form_alter, see :
http://api.drupal.org/api/search/7/form_node
So is form_node or hook_form_node_form_alter documentation needs to be written ? Where is it ?
I searched the doc :
http://api.drupal.org/api/search/7/hook_form_node_form_alter
I also found a single implementation of hook_form_node_form_alter in the node module.
Comment | File | Size | Author |
---|---|---|---|
#5 | drupal-small_doc_header_fix_1415794-4.patch | 551 bytes | b2f |
Comments
Comment #0.0
b2f CreditAttribution: b2f commentedsmall update
Comment #1
b2f CreditAttribution: b2f commentedActually, 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.
Comment #2
jhodgdonThere is documentation for hook_form_FORM_alter(). That is probably/possibly what you were looking for?
Also, issues about documentation on api.drupal.org belong in the Drupal Core project issue queue.
Comment #3
b2f CreditAttribution: b2f commentedI 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 :
It's not a problem for me, but is this lacking documentation ?
Comment #4
jhodgdonThanks! That documentation is wrong. I am opening this issue back up so that it can get fixed.
http://api.drupal.org/api/drupal/core--modules--node--tests--node_access...
The problem is that it references hook_form_node_form_alter(), which does not exist. It should reference hook_form_BASE_FORM_ID_alter() instead.
Good novice project...
Comment #4.0
jhodgdonsmall update
Comment #5
b2f CreditAttribution: b2f commentedI made my first git patch, attached here. I followed these guidelines : http://drupal.org/node/1054616.
Is it correct ?
Comment #6
jhodgdonLooks good to me, thanks!
For future reference, when you upload a patch, set the status to "needs review", so the test bot can test it. You might also be interested in reading this section of documentation:
http://drupal.org/novice
Comment #7
webchickCommitted and pushed to 7.x. Thanks!
Comment #8.0
(not verified) CreditAttribution: commentedreference actual problem