In boost_admin_generate_htaccess() the variable $drupal_subdir is used in the line:

$string .= " RewriteCond %{REQUEST_URI} (^$drupal_subdir(admin|$cache_dir|misc|modules|sites|system|openid|themes|node/add))|(/(comment/reply|edit
|user|user/(login|password|register))$) [OR]\n";

At the start of this function a trailing slash is removed from $drupal_subdir, therefore this way it would match e.g. drupaladmin and drupalmisc

With the attached patch it generates a working RewtiteCond like
(^/drupal/(admin|cache|misc|modules|sites|system|openid|themes|node/add))

Changing all uses of $drupal_subdir to include a trailing slash might be a slightly cleaner implementation though...

Comments

mikeytown2’s picture

Status: Active » Needs review
StatusFileSize
new9.01 KB

I decided to add in the slashes to all the variables. Also the solution to the issue you raised is to use $base_path

helmo’s picture

I applied your patch and the result looks OK, thanks.

mikeytown2’s picture

Status: Needs review » Reviewed & tested by the community
mikeytown2’s picture

Status: Reviewed & tested by the community » Fixed

committed

Status: Fixed » Closed (fixed)

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