Conflict with Admin module

muhleder - August 17, 2009 - 10:31
Project:Cufon
Version:6.x-1.0-beta3
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

Hi,

when Cufon module is enabled, the theme for the Admin module does not display. Haven't had time to do any debugging yet, so not sure if it's a fault with Cufon or Admin. The Admin module currently sets the theme from within the module, so I would guess it's something to do with that.

#1

davideads - August 17, 2009 - 20:48

Do you mind helping by debugging this a bit further? I'm a few weeks out from being able to roll a stable release of this and don't presently have the time to test it against the Admin module extensively. I would, however, love to get this fixed if it is a problem for you.

#2

hunvreus - August 18, 2009 - 06:24

The bug is coming from the line #66:
$files += file_scan_directory('./'. path_to_theme(), $pattern);
This seems to create a conflict of some sort, haven't looked further but will get it fixed sooner or later. One thing I think could cause it is the fact that the Slate/Admin theme is stored in the Admin module (not the theme folder). Will look into it later on this week.

#3

davideads - August 26, 2009 - 19:34

I wonder if path_to_theme() isn't properly resolving for you. What does dpm(path_to_theme()) tell you?

#4

davideads - August 26, 2009 - 20:25

Well, I'm wrong about that. The latest CVS version should fix this issue (see http://drupal.org/node/559830). If you could please update and help me test, I can roll a release later this week than includes the fix.

#5

davideads - August 27, 2009 - 12:45
Status:active» needs review

Fixed in 6--1-0-BETA4 -- please review!

#6

muhleder - September 23, 2009 - 13:29

Working well for us. Using it on two sites so far, one live, one in dev. Thnks.

 
 

Drupal is a registered trademark of Dries Buytaert.