Last modified: 2014-06-01 13:29:45 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T47288, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45288 - Need to replace mysql_*() functions for in favor of mysqli or pdo
Need to replace mysql_*() functions for in favor of mysqli or pdo
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Database (Other open bugs)
1.21.x
All All
: High critical (vote)
: 1.22.0 release
Assigned To: Nobody - You can work on this!
: newphp, platformeng
Depends on:
Blocks: 49975 58153 65998
  Show dependency treegraph
 
Reported: 2013-02-22 18:54 UTC by Chad H.
Modified: 2014-06-01 13:29 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Chad H. 2013-02-22 18:54:51 UTC
PHP 5.5 (using current master) has dropped support for the original mysql plugin. We'll have to use mysqli or pdo.
Comment 1 Krinkle 2013-02-22 21:49:19 UTC
PHP 5.5 release plans:
 http://marc.info/?l=php-internals&m=134710500026355&w=2

Looks like it is coming soon (Feb/March 2013). Lets try to get this for 1.21 so that we don't have to start including a maximum version in our PHP support ("5.3.2 or higher") for our release.
Comment 2 Chad H. 2013-02-22 22:28:37 UTC
Indeed. I'm curious if we should go the mysqli or pdo route.
Comment 3 Alex Monk 2013-03-03 15:23:22 UTC
Has anyone got any preferences? I think I have a working patch for MySQLi but I'd like to see a decision made before I upload it.
Comment 4 Marius Hoch 2013-03-03 15:28:12 UTC
I personally prefer PDO though I haven't yet used it with a project even near this size.

Using PDO could kick in handy especially in case more than one DB class starts to use it...
Comment 5 Max Semenik 2013-03-03 21:28:45 UTC
(In reply to comment #4)
> Using PDO could kick in handy especially in case more than one DB class
> starts
> to use it...

SQLite uses PDO already.
Comment 6 Andre Klapper 2013-03-21 15:07:10 UTC
This has the 1.21.0 target milestone set by Krinkle, so there are roughly two months left.

The first step here is probably to decide whether to go for PDO or for MySQLi. CC'ing Greg, plus wondering if I should add the "platformeng" keyword.

(In reply to comment #3 by Krenair)
> I think I have a working patch for MySQLi

Krenair: Could you share that patch, maybe? 
Asking as it's one potential way to get some progress here...
Comment 7 Alex Monk 2013-03-21 18:28:18 UTC
(In reply to comment #6)
> Krenair: Could you share that patch, maybe? 

Gerrit change #55106
Comment 8 Greg Grossmeier 2013-03-21 20:55:45 UTC
This probably needs a bigger discussion than the 6 people on the CC list; should've probably been brought up during our quarterly review mid-February.
Comment 9 Greg Grossmeier 2013-03-21 22:56:23 UTC
Alright, officially bumped to 1.22. We won't be updating to 5.5 at WMF during our use of 1.21 and the time needed to do this is too much right now.
Comment 10 Platonides 2013-03-21 22:59:11 UTC
We should use a DatabaseMysqli file.
Comment 11 Andre Klapper 2013-03-22 10:50:46 UTC
Should the MediaWiki 1.21 release notes then mention that there could be problems using it with PHP 5.5?
Comment 12 Greg Grossmeier 2013-03-22 15:20:48 UTC
(In reply to comment #11)
> Should the MediaWiki 1.21 release notes then mention that there could be
> problems using it with PHP 5.5?

PHP 5.5 isn't out yet (it is still in Beta) so I'd say not.
Comment 13 Tyler Romeo 2013-03-27 18:58:59 UTC
https://gerrit.wikimedia.org/r/56193
Comment 14 Tyler Romeo 2013-03-27 19:00:17 UTC
Also changed topic, since the mysql extension has long been on its way out, even without PHP 5.5.
Comment 15 Gerrit Notification Bot 2013-05-03 02:01:45 UTC
Related URL: https://gerrit.wikimedia.org/r/62130 (Gerrit Change I905d4a4550377bc849a860f0962dad710d9dc71f)
Comment 16 Gerrit Notification Bot 2013-05-03 18:10:40 UTC
Related URL: https://gerrit.wikimedia.org/r/62173 (Gerrit Change I6733fe21c4aa7443e409c5dfa7c789552b2b62b7)
Comment 17 Gerrit Notification Bot 2013-05-03 18:10:44 UTC
Related URL: https://gerrit.wikimedia.org/r/62174 (Gerrit Change I7334c4872f51ef38ac17c747e2ef4a8a0236623a)
Comment 18 Tyler Romeo 2013-05-03 21:12:51 UTC
Re-assigned to Jakub since he's the owner of the new patches. Looks like we may be able to close this soon.
Comment 19 Andre Klapper 2013-09-26 14:39:00 UTC
https://gerrit.wikimedia.org/r/#/c/62173/ and https://gerrit.wikimedia.org/r/#/c/62174/ have received -1 or -2 and hence need rework.
Comment 20 Gerrit Notification Bot 2013-10-08 21:58:13 UTC
Change 62174 abandoned by Chad:
Allow selection of MySQLi database type during installation

Reason:
Per Tim's comment, this absolutely should not be done like this.

https://gerrit.wikimedia.org/r/62174
Comment 21 Gerrit Notification Bot 2013-10-16 23:23:40 UTC
Change 62173 merged by jenkins-bot:
Add support for mysqli extension

https://gerrit.wikimedia.org/r/62173

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links