avatar Martin Kong Pro Mar 24. 2010. 4:30 am
AC is configured with /public as document root. Now I'm tweaking it to serve clean URL. I have done:

1) .htaccess in the public directory (btw, +FollowSymlinks should be added to the online document)

Options -Indexes +FollowSymLinks
<IfModule mod_rewrite.c>
  RewriteEngine On

  RewriteCond %{REQUEST_FILENAME} -f [OR]
  RewriteCond %{REQUEST_FILENAME} -d
  RewriteRule ^(.+) - [PT,L]

  RewriteRule ^projects_icons/(.*)$ projects_icons/$1 [L]
  RewriteRule ^avatars/(.*)$ avatars/$1 [L]
  RewriteRule ^logos/(.*)$ logos/$1 [L]
  RewriteRule ^thumb.php$ thumb.php [L]
  RewriteRule ^captcha.php$ captcha.php [L]
  RewriteRule ^$ index.php [L]
  RewriteRule ^(.*) index.php?path_info=$1 [L]
</IfModule>


2) made necessary changes to config/config.php

define('ROOT_URL', 'http://ac.domain.com');
define('URL_BASE', ROOT_URL . '/');
define('ASSETS_URL', ROOT_URL . '/assets');
//define('PATH_INFO_THROUGH_QUERY_STRING', true);


Everything seems ok, at least I can log in. So far there is one problem, the brand/logo.gif is broken. It is being referenced as http://ac.domain.com/public/brand/logo.gif. Seems to be a problem with function.brand.php.

I want to avoid changing function.brand.php directly. What's the proper fix for this?
avatar Ilija Studen Staff Mar 24. 2010. 9:21 am
As far as I remember, this should work properly. Did you follow these instructions?
avatar Martin Kong Pro Mar 24. 2010. 12:09 pm
Yes, I did everything by the book. I did the "/public Folder as Document Root" first and it worked fine. Then I did the clean URL configuration change. Now the brand/logo.gif is not referenced correctly.

Actually, if you like at function.brand.php, you can easily spot the error:

case 'logo':
  return URL_BASE == ROOT_URL . '/' ? ROOT_URL . '/public/brand/logo.gif' : ROOT_URL . '/brand/logo.gif';


As you can see, if URL_BASE == ROOT_URL . '/', then ROOT_URL . '/public/brand/logo.gif' is returned. Which is wrong.
avatar Martin Kong Pro Mar 29. 2010. 4:31 am
Should I submit an official request about this?
avatar Ilija Studen Staff Mar 29. 2010. 10:23 am
No need. We'll check it out and update documentation and files accordingly when we get to it.
avatar Martin Kong Pro Jun 27. 2010. 2:02 am
Has this been fixed in 2.3.1?
avatar Ilija Studen Staff Jun 28. 2010. 5:24 am
Hello Martin,

Sorry, fix for this issue was not included in activeCollab 2.3.1. We'll have it covered in the next bug fix release.
avatar Martin Kong Pro Aug 28. 2010. 3:47 am
This doesn't seem to have been fixed in 2.3.2 yet. Please confirm when this will be fixed. Thanks!
avatar Ilija Studen Staff Aug 29. 2010. 3:30 pm
Hello,

Sorry, this has not been covered in activeCollab 2.3.2.

v2.3.2 was pushed a bit sooner than usual because of a couple of important issues that had to be covered ASAP. That's why some of the least important bug fixes that were planned for v2.3.2 were not included in it.