The advanced help file describes hook_nodequeue_queue_form($queue, &$form), but the hook is actually hook_nodequeue_form($queue, &$form).

CommentFileSizeAuthor
#2 nodequeue_984552.patch5.75 KBfgm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented

Same problem with hook_nodequeue_queue_form_validate($queue, &$form_state, &$form), hook_nodequeue_queue_form_submit(&$queue, &$form_state) and hook_nodequeue_queue_form_submit_finish(&$queue, &$form_state)

These are really hook_nodequeue_form_validate($queue, &$form_state, &$form), hook_nodequeue_form_submit($queue, &$form_state, &$form) and hook_nodequeue_form_submit_finish($queue, &$form_state, &$form).

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Title: Incorrect help for smartqueue hook_nodequeue_queue_form() » Incorrect smartqueue hook_nodequeue_queue_form()
FileSize
nodequeue_984552.patch5.75 KB

Suggested patch fixes advanced_help for the module.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Title: Incorrect smartqueue hook_nodequeue_queue_form() » Incorrect help for smartqueue hook_nodequeue_queue_form()
Status: Active » Needs review

more accurate title

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Title: Incorrect smartqueue hook_nodequeue_queue_form() » Incorrect help for smartqueue hook_nodequeue_queue_form()
Status: Needs review » Fixed

Commited to 6.x and 7.x. Thanks :)

http://drupalcode.org/project/nodequeue.git/commit/af2345e
http://drupalcode.org/project/nodequeue.git/commit/a600e65

Status: Fixed » Closed (fixed)

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