Empty list in the pop-up

Roulion - July 5, 2009 - 20:34
Project:Inline and link Drupal objects (Linodef)
Version:6.x-1.x-dev
Component:Buttons API
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

When i upgraded to the very last dev version (luly 2nd), i noticed the pop-up list my BUEditor should open was empty. Here is the code for the button

<?php
: // Button for Linodef tags.
// Enter a fieldname, content type or vocabulary ID.
// E.g. 'field_example', 'story' or '10'.
$drupal_element = 'joueurs';
// Set the button type to 'list', 'popup' or 'autocomplete'.
$button_type = 'popup';
// Add tag options, separated by comma.
// E.g. 'nolink' or 'nolink,translation=de'.
$options = '';

// Do not modify.
if (module_hook(linodef_buttons_bueditor, buttons)) {
    return
linodef_buttons_bueditor_buttons($drupal_element, $button_type, $options);
}
else {
    return
t('Module !module not enabled!', array('!module' => 'Linodef Buttons - BUEditor'));
}
?>

where 'joueurs' is a node type... What is wrong?

#1

Roulion - July 5, 2009 - 20:39

the list option works correctly but the popup and autocomplete one don't (empty popup and autocomplete never find any node even if the title is good)

#2

Roi Danton - July 5, 2009 - 22:02
Component:Code (general)» Buttons API
Status:active» fixed

Thank you for the detailed report! Themehook in Buttons API had wrong arguments. I haven't recognized it since my theme cache wasn't flushed during testing. Fix committed.

Either download the dev release (after it gets updated by drupal.org what could take a half day) or apply the patch provided with this post or download this version of buttonsapi.module and overwrite the old file. Then empty your cache (e.g. using Devel module) so the hook theme function gets renewed.

AttachmentSize
linodef_buttonsapi.module_corrected_hooktheme_args.patch 1.16 KB

#3

System Message - July 19, 2009 - 22:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.