maintenance_page not registered correctly in drupal_common_theme()

Fix looks good, but we don't define arrays that way. :) (Except for 'arguments' which I guess is to make it look more like a function signature).

Changed the patch to put each key on its own line and committed. Thanks!

This is broken. Right now the theme system does not accept having a template and a specified file containing a preprocess function in different directories. The 'path' is used for both.

There are 3 possible solutions:

1) move theme.maintenance.inc into system as system.maintenance.inc
2) move the template to includes
3) split 'path' so that we can have a template path and a file path.

Solution 2 is clearly the weakest.

Solution 1 is probably the easiest.
Solution 3 could solve a few other issues, but I think would lead to a larger re-org, because we would almost certainly move all core templates into one directory (which would still leave module's templates out in the cold but what can you do?)

BTW, if you want to see the bug, set the Stark theme as your maintenance theme in settings.php. You'll get a blank page because it tries to look in "includes" for the maintenance-page.tpl.php file.

This extremely easy patch that sets the admin theme on /batch fails because of this http://drupal.org/node/563634

The maintenance-page is in system.module and the theme function is in theme.inc. How can we solve this?

This seems to solve this on my local machine.

Wrong patch. That was weird.

Of course a patch extension is needed. Ouch.

+  arg(0) == 'batch' 

this change should not be in the patch

+
+
+

please use only 1 empty line between functions

This fixes the problem for me.

Committed to HEAD! Thanks! I will be so happy for those phantom errors about this file being missing to go away. :)