The current implementation doesn't work with the Drupal AJAX framework as the inline JS is not appended to the head region. The patch in the comment below resolves this issue by storing all settings in the Drupal.settings hash and then invoking the bxslider using those settings in the attach behavior. This solves the issue of the callbacks not working on the views admin page as well (there was special logic previously to disable the callbacks on the view admin page).

CommentFileSizeAuthor
#1 Module_doesnt_work_with_AJAX_requests-2090391-0.patch6.63 KBd.clarke
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

d.clarke’s picture

d.clarke CreditAttribution: d.clarke commented
Status: Active » Needs review
FileSize
Module_doesnt_work_with_AJAX_requests-2090391-0.patch6.63 KB

Patch to address reported issue.

Mschudders’s picture

Mschudders CreditAttribution: Mschudders commented
Assigned: Unassigned » Mschudders
Mschudders’s picture

Mschudders CreditAttribution: Mschudders commented

Thanks committed to DEV.

Not closing because I need to test it :)

d.clarke’s picture

d.clarke CreditAttribution: d.clarke commented

Hi Mschudders,

Thank you for the quick turn around on this. If you run into any issues feel free to reach out and I'll take a look.

Dale

Mschudders’s picture

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

Hi,

I've tested it and looked at the diff log, and I must say thank you very much.

I couldn't solve the problem with the callbacks passed to the JS that's why I created such a workaround but this is way nicer using jQ.extend !

Thanks
Mschudders.