[wp-trac] [WordPress Trac] #21663: Use PDO or mysqli for MySQL queries when available

WordPress Trac noreply at wordpress.org
Thu Jan 30 14:51:16 UTC 2014


#21663: Use PDO or mysqli for MySQL queries when available
--------------------------------------------------+------------------
 Reporter:  scottconnerly                         |       Owner:
     Type:  task (blessed)                        |      Status:  new
 Priority:  normal                                |   Milestone:  3.9
Component:  Database                              |     Version:  3.5
 Severity:  normal                                |  Resolution:
 Keywords:  dev-feedback has-patch needs-testing  |     Focuses:
--------------------------------------------------+------------------

Comment (by pento):

 Replying to [comment:167 markoheijnen]:
 > It needs a lot of attention to get how we want it and currently that is
 the major issue. We only want to do ugly hot fixes instead of making the
 code great.

 There are two major issues:
 * We need to move away from ext/mysql as soon as possible, due to it being
 deprecated.
 * We need to modernise WPDB.

 This ticket is about moving away from ext/mysql as quickly and with as
 little pain as possible, and the easiest way to do that is with as few
 changes to the codebase as possible, which is the aim of the current
 patch.

 I understand that you would prefer to have the correctly engineered
 solution, that would be my first preference as well. But the more pressing
 need is to have 3.9 be able to use mysqli, which we won’t be able to do if
 we implement it as driver solution.

 Replying to [comment:170 knutsp]:
 > Not bad. WordPress still not requiring 5.0.6 is bad, if WordPress really
 needs this to be better and consistent regarding charset. I’m ok with just
 requiring 5.0.x in general, but wonder how bad it would be to raise the
 requirement just a bit.

 We do upgrade the minimum requirements as we need to, usually when the
 usage levels are negligible. I don’t have the usage breakdown handy, nacin
 will know the answer to that.

 That said, however, for the sake of a few lines of code, I’m not inclined
 to upgrade the minimum MySQL requirements from 5.0.0 to 5.0.6. Perhaps
 when we decide to make use of views (added in MySQL 5.0.3), we’ll upgrade
 it.

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


More information about the wp-trac mailing list