[wp-trac] [WordPress Trac] #20263: Backticks in dbDelta cause warning and actually causes a query to alter all columns and indexes to run even if none have changed

WordPress Trac noreply at wordpress.org
Tue Mar 15 14:46:22 UTC 2016


#20263: Backticks in dbDelta cause warning and actually causes a query to alter all
columns and indexes to run even if none have changed
----------------------------------------------+----------------------------
 Reporter:  Driskell                          |       Owner:
     Type:  defect (bug)                      |      Status:  new
 Priority:  normal                            |   Milestone:  Future
Component:  Database                          |  Release
 Severity:  minor                             |     Version:  1.5
 Keywords:  has-patch early needs-unit-tests  |  Resolution:
                                              |     Focuses:
----------------------------------------------+----------------------------
Changes (by ocean90):

 * keywords:  has-patch early => has-patch early needs-unit-tests


Comment:

 We noticed the same with GlotPress which has a colum "references", see
 https://github.com/GlotPress/GlotPress-WP/pull/343.

 It seems to work with BackPress because `BP_SQL_Schema_Parser::delta()`
 supports backticks, see
 https://backpress.trac.wordpress.org/browser/trunk/includes/class.bp-sql-
 schema-parser.php?marks=118,130,144#L117.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/20263#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list