Loading Bootstrap from theme directory [#1785988]

When I was trying to install this theme, I copied the bootstrap library from twitter.github.com/bootstrap to the theme directory
So that sites/all/themes/bh_bootstrap/bootstrap/css/bootstrap.css is reachable from the webserver.

When I enabled the theme, I received the message:
"The Bootstrap framework could not be found. In order to use the BH Bootstrap theme, you must download the framework from the Bootstrap project page, and extract it to sites/all/libraries/bootstrap (if the 7.x-1.x version of the Libraries module is installed) or to sites/all/themes/bh_bootstrap/bootstrap."

I traced the error in template.php line 517, Where I found a lot of confusing code, If after line 512 in dpm($bootstrap_theme_path) it returns with: /var/www/local/local/sites/all/themes/bh_bootstrap/bootstrap

Because my drupal installation is in /var/www/local reachable at localhost/local, Their is a additional local here.

Additional Info, RewriteBase / is commented in .htaccess

Comment	File	Size	Author
#2	bh_bootstrap-bootstrap_library_path-1785988-2.patch	1.23 KB	khaled.zaidan

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

bedlam

Lyon

CreditAttribution: bedlam commented 16 September 2012 at 19:54

Assigned:	Unassigned	» bedlam
Priority:	Normal	» Major

Good catch. Kludgy code :)

Should be fixed in dev. Please try the most recent dev version and let me know if your issue is resolved.

Comment #2

khaled.zaidan CreditAttribution: khaled.zaidan commented 18 September 2012 at 11:50

Status:

Active

» Needs review

File	Size
bh_bootstrap-bootstrap_library_path-1785988-2.patch	1.23 KB

Hi,

I just ran into the same issue, it is still there also in the dev version :S

I think that we simply do not need the $base_path variable, since it is already included in DRUPAL_ROOT. Instead we only need to add '/'.

Here's a small patch (against dev) to fix the issue :)

Comment #3

bedlam

Lyon

CreditAttribution: bedlam commented 18 September 2012 at 16:16

We'll try again :)

Should be fixed in dev. Please test that it works for you (it ought to, now that I understand the issue & somebody actually wrote the patch for me...)

Many thanks for the patch, khaled.zaidan.

If it works without further issues I'll make a new stable release incorporating this fix.