[wp-trac] [WordPress Trac] #10693: Bug in dbDelta

WordPress Trac wp-trac at lists.automattic.com
Sat Nov 14 13:57:49 UTC 2009


#10693: Bug in dbDelta
--------------------------+-------------------------------------------------
 Reporter:  arena         |       Owner:  ryan        
     Type:  defect (bug)  |      Status:  new         
 Priority:  normal        |   Milestone:  2.9         
Component:  Database      |     Version:  2.8.5       
 Severity:  major         |    Keywords:  dev-feedback
--------------------------+-------------------------------------------------
Changes (by arena):

  * keywords:  needs-patch => dev-feedback
  * version:  2.8.4 => 2.8.5


Comment:

 Hi ryan,
 I investigated a bit on that ticket, to notice the following :

 * it is not a problem of column sequence. (Log_debug_20091114a.txt)

 * columns name should be separated by at least a space from the other
 declarations (type, etc...). (some of my column names were ended by a tab,
 space is better)

 * during my tests i changed following lines in dbDelta
 (Log_debug_20091114b.txt)

 {{{
                                         if($validfield) {
 $cfields[strtolower($fieldname)] = trim($fld, ", \n");
                                         }
 }}}

 into

 {{{
                                         if($validfield) {
 $cfields[trim(strtolower($fieldname))] = trim($fld, ", \n");
                                         }
 }}}

 * HOWEVER it seems that the multi index tables are not handled properly.
 (Log_debug_20091114c.txt & Log_debug_20091114d.txt)

 i attach the logs of my different tests for future use if needed

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10693#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list