[drupal-devel] [feature] Nice DB error screens

[drumm]

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

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     feature requests
 Priority:     normal
 Assigned to:  drumm
 Reported by:  drumm
 Updated by:   drumm
 Status:       patch
 Attachment:   http://drupal.org/files/issues/errorhandling_0.patch (8.44 KB)

New patch with a couple suggestions from chx.



* $foo === NULL is faster than is_null($foo)
* Removed 'bootstrap' from the function name since it is not really
part of the bootstrap system.

When testing/applying updates #2 and #3 of this issue need to be
handled too.




drumm



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

Thu, 21 Jul 2005 22:43:16 +0000 : drumm

Attachment: http://drupal.org/files/issues/screenshot2.png (96.1 KB)

Screenshot.




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

Thu, 21 Jul 2005 23:21:31 +0000 : drumm

Attachment: http://drupal.org/files/issues/screenshot_10.png (88.44 KB)




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

Thu, 21 Jul 2005 23:25:57 +0000 : drumm

Attachment: http://drupal.org/files/issues/druplicon.png (3.98 KB)

New Druplicon which will need to replace misc/druplicon.png. The current
file in that location is a giant flat druplicon which isn't used
anywhere.




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

Thu, 21 Jul 2005 23:26:26 +0000 : drumm

Attachment: http://drupal.org/files/issues/maintenance.css (329 bytes)

Some CSS




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

Thu, 21 Jul 2005 23:30:34 +0000 : drumm

Attachment: http://drupal.org/files/issues/errorhandling.patch (8.47 KB)

This patch makes the database errors which occur when the database
connection information is wrong or the database isn't up much nicer and
more explanatory. To do this it adds a themeable function,
theme_maintenance_page(), and a way to get Drupal running without a
theme but the ability to execute themeable functions in the usual way.
This means we can use the same code to give upgrade.php a nicer look in
the future.




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

Fri, 22 Jul 2005 06:33:57 +0000 : kbahey

I was about to do something similar on a less ambitious scale (mainly
the connect function), because what we get now is ugly and looks
unprofessional.


+1 from me for this. Great job!




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

Fri, 22 Jul 2005 06:52:30 +0000 : gordon

+1, This looks great. Get this in and then we can do something with the
upgrade.php system and it will look just great.