[drupal-devel] [bug] Putting a .theme file into themes/ makes all themes not working

[Dries]

Issue status update for http://drupal.org/node/16914

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

- That should probably be an error message, rather than a status
(success) message.
- The coding style needs work.
- For consistency the theme name should be wrapped in em's.
- I'd prefer not to scan ./themes for .theme files and to ignore any
.theme files places directly in ./themes.
Won't commit as is.


Dries



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

February 7, 2005 - 09:13 : chx

Try putting a .theme file into themes/ without any subdir. Enable it
(maybe just saving theme settings is enough), Nothing will work after
this. To restore your site: remove the offending theme, issue an SQL:
DELETE FROM {system} WHERE type='theme' and save theme settings again.


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

February 7, 2005 - 09:14 : chx

effects HEAD, too.


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

February 7, 2005 - 14:33 : chx

Attachment: http://drupal.org/files/issues/themesfix.patch (542 bytes)

Here is a patch which fixes the problem. However, I do not have the
feeling that this is "the" solution. Feel free to come up with
something better.


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

February 7, 2005 - 14:35 : chx

Oops forgot to set to CVS, 'cos the fix is against HEAD. Not that 4.5.2
does not need a patch...


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

February 7, 2005 - 23:15 : Anonymous

While this patch would allow a theme to be placed directly in the
"themes" directory, it disables the ability for that theme to use a
"style.css" file, which is inconsistent. I would prefer a patch that
enforces the criteria that themes MUST be placed in a SUBDIRECTORY of
"themes" -- themes placed directly in the "themes" folder should be
ignored... maybe we could send the admin a drupal_set_message that
would tell them about the problem when visiting admin/themes too.


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

February 23, 2005 - 01:07 : chx

Attachment: http://drupal.org/files/issues/themefix.patch (693 bytes)

Anonymous is right. I have made a better patch. Please commit, this is
very important IMHO.


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

February 23, 2005 - 01:30 : chx

Attachment: http://drupal.org/files/issues/themefix_1.patch (712 bytes)

Message was wrong.


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

March 2, 2005 - 01:26 : chx

May I know why this is not getting commited? This is a very nasty issue,
the poor site admin who tries a few themes and misplaces one is totally
baffled why his themes do not work after this.