[wp-trac] [WordPress Trac] #21663: Use PDO or mysqli for MySQL queries when available
WordPress Trac
noreply at wordpress.org
Sun Feb 2 23:47:15 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 markoheijnen):
Replying to [comment:171 pento]:
> 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.
I disagree that this ticket is about moving away from ext/mysql. I rather
see it as hijacked to be like that. That also makes this ticket now hard
to follow since there are patches with the ticket number doing different
things. One focusses on one file with mysqli support and the other going
for the driver approach.
The problem I'm having now is that we don't focus on the best solution.
That happend in 3.8 and now also again for 3.9. With the last sentence you
mean if we go for the driver approach we will not hit 3.9?
I believe the driver approach is really stable but on the API side it can
be that it isn't something we want. The real issue is never tackled here
and that is what to do with plugins that still do raw calls to
mysql_query() for example.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/21663#comment:172>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list