In case of use theme in pair with twitter_bootstrap_ui module default bootstrap.js from theme info file also will be tried to load.
There is a patch in attachment.
Idea of patch is to remove twitter_bootstrap_ui library load from html preprocess and move it to js_alter implementation.
It make possible to exclude default js if twitter_bootstrap_ui available.

CommentFileSizeAuthor
#1 1686342-double-js-twitter.patch1.45 KBwiifm
twitter_bootstrap-double-js.patch1.6 KBIRuslan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wiifm’s picture

wiifm
Primary language English
Location Wellington, NZ
CreditAttribution: wiifm commented
Priority: Normal » Major
Status: Active » Reviewed & tested by the community
FileSize
1686342-double-js-twitter.patch1.45 KB

Patch applied cleanly when using patch -p2 < [patchfile]

Have re-rolled the patch with the correct path, so hopefully drush make will like it.

This needs to be committed ASAP, works as described as stops the duplicate request (that always returns a 404) for the bootstrap.js file.

wundo’s picture

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

Fixed, thanks!

Status: Fixed » Closed (fixed)

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

andregriffin’s picture

andregriffin CreditAttribution: andregriffin commented
Project: Twitter's Bootstrap » Bootstrap