[drupal-devel] [bug] Another simple patch for database.mysql.inc

chx drupal-devel at drupal.org
Sat Jul 30 17:49:32 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    database system
 Category:     bug reports
 Priority:     critical
 Assigned to:  chx
 Reported by:  kubaZygmunt
 Updated by:   chx
-Status:       patch (code needs work)
+Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/database_inc.patch (2.35 KB)

I came up with this easier to read, simpler code. I even -gasp-
commented it.


Previous comments:

Sat, 30 Jul 2005 14:03:03 +0000 : kubaZygmunt

Attachment: http://drupal.org/files/issues/database.mysql.inc_0.patch (657 bytes)

Another simple patch which removes error:
  Notice: Undefined index: dev_query in ***\includes\database.mysql.inc
on line 80


Sat, 30 Jul 2005 14:59:51 +0000 : Dries

variable_get is part of bootstrap.inc and therefore, it always exists. 
The test is probably redundant.  Please test and resubmit.  Thanks


Sat, 30 Jul 2005 17:36:29 +0000 : chx

I always forget to post this patch.

This line was part of the bootstrap patch, so this is my work -- I do
not know what got in me when I mistyped which function exists.

Why this line is necessary? Because after the bootstrap patch you can
not be sure that bootstrap.inc is there when you are running a

Also there is a need to check for $GLOBALS['conf']['dev_query']. I'll
post immediately.

More information about the drupal-devel mailing list