[drupal-devel] [bug] Logo uploads are broken

[Dries]

Issue status update for 
http://drupal.org/node/26822
Post a follow up: 
http://drupal.org/project/comments/add/26822

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     bug reports
 Priority:     normal
 Assigned to:  drumm
 Reported by:  drumm
 Updated by:   Dries
 Status:       patch

I tested this patch and noticed a couple things:


1. The bluemarine theme did not pick up my custom logo.  It took me 5
minutes to figure out.


2. When I upload my logo, the 'custom logo' path did not change. 
Combined with (1) this makes for a pretty bad experience.


When I switched theme, this patch seems to fix an important bug.  Just
wondering if you could look into (1) and (2) before this gets
committed?  Clearly, there is room for improvement here.  If you don't
have time for this, I'll commit your patch as is.




Dries



Previous comments:
------------------------------------------------------------------------

1121507221 : drumm

Attachment: http://drupal.org/files/issues/system.module_4.diff (1.95 KB)

Custom logo uploading broke. System_settings_save() was doing a
drupal_goto() before the code to save the logo got executed.




------------------------------------------------------------------------

1121507946 : drumm

I inspected the code for 4.6 and this appears to be a problem there too.