base_path used for logo source when no default/custom logo is defined [#57333]

...as per the title. Please, review attached patch. Thanks

Comment	File	Size	Author
	theme.inc.no_logo.patch	609 bytes	markus_petrux

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Steven CreditAttribution: Steven commented 4 April 2006 at 23:30

Status:

Needs review

» Fixed

Committed to HEAD, thanks.

Comment #2

dlr CreditAttribution: dlr commented 14 April 2006 at 07:51

I'am not sure this is quite a good idea. Adding drupal base path to every logo path can cause trouble : i have defined a file upload directory that is not in my drupal path (for exemple /var/www/files and for drupal /var/www/html/drupal/). So with the patch '$logo' is /drupal/var/www/files and it does'nt work.