[drupal-devel] [bug] have db_prefix_tables() handle 'default' key not being set

Dries drupal-devel at drupal.org
Sun Apr 3 12:45:32 UTC 2005


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

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  danielc
 Updated by:   Dries
 Status:       patch

Any idea of how much faster the second patch is?  I don't think it makes
a real difference in which case I'd favor the first patch.


Dries



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

March 28, 2005 - 23:24 : danielc

Attachment: http://drupal.org/files/issues/database.diff (720 bytes)

The db_prefix_tables() function in includes/database.inc generates
notices if $db_prefix['default'] is not set.  This patch fixes that
problem.  This has a side benefit of making $db_prefix easier to use.


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

March 29, 2005 - 00:06 : moshe weitzman

nice ... this has been a bit a pimple for a while.


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

March 29, 2005 - 17:40 : danielc

Attachment: http://drupal.org/files/issues/database2.diff (1.17 KB)

I've come up with a new version of the patch that optimizes the speed of
the whole function.


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

March 29, 2005 - 17:56 : danielc

Change Version from "4.5.2" to "cvs".


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

March 29, 2005 - 18:01 : danielc

Pardon me, I'm testing if this goes the drupal-dev list now that the
version is "cvs."





More information about the drupal-devel mailing list