function ctools_jump_menu($form_state, $select, $options = array()) {

should be:

function ctools_jump_menu($form, &$form_state, $select, $options = array()) {

Patch attached.

CommentFileSizeAuthor
ctools_jump-menu.inc_.patch476 bytestroky
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

troky’s picture

troky CreditAttribution: troky commented
Status: Patch (to be ported) » Needs review
sven.lauer’s picture

sven.lauer CreditAttribution: sven.lauer commented
Status: Needs review » Reviewed & tested by the community

I ran across this bug yesterday (and posted a duplicate issue (duh!)). It renders the jump menu completely unusable (as the form builder takes the settings array to be the option array).

The patch is straightforward and the need for it well-documented: http://drupal.org/update/modules/6/7#hook_forms_signature

troky’s picture

troky CreditAttribution: troky commented

any chance to fix this soon (or commit the patch)?
thanks

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed!

Status: Fixed » Closed (fixed)

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

troky’s picture

troky CreditAttribution: troky commented
Status: Closed (fixed) » Needs review

I don't see this is fixed in cvs.

EclipseGc’s picture

EclipseGc CreditAttribution: EclipseGc commented
Status: Needs review » Reviewed & tested by the community

Patch still applies and seems to fix the problems with the jump menu.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed for real.

Status: Fixed » Closed (fixed)

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