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

Comments

troky’s picture

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

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 commented

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

merlinofchaos’s picture

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 commented
Status: Closed (fixed) » Needs review

I don't see this is fixed in cvs.

eclipsegc’s picture

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 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.