Task #36419
closed
Migrate core from mysql to mysqli
100%
Description
Without using dbal core still uses the mysql-extension instead of mysqli. But PHP officially claims mysql to be "legacy" and mysqli is shipped with PHP since quite a while now. Since PHP 5.3.0 now mysqli also supports persistent connections. So it should be "quite simple" to replace the few direct calls to mysql-functions with mysqli-equivalents. It's just a different ressource (well, an object then) that is returned, so the inner parts in TYPO3 passing around the DB-handler can (almost) stay as is.
All extensions should use TYPO3-functionality to access the database, so no change should be required on that part. It might be that some directly execute mysql_query()-calls or the like, so this is kind of a "break". But things are easy to fix - and then maybe they could switch to using the TYPO3-functions for it anyway :-)
Might this be a candidate for 6.0? Patch itself should be quite easy to do.
Updated by Philipp Gampe over 12 years ago
+1 If the API does not change and dbal works too, I see no reason not to move to modern technologies :P
Updated by Philipp Gampe over 12 years ago
Warming this up, do you want to push a patch?
Updated by Felix Kopp over 12 years ago
Am I correct that only the typo3_db sql functions must be changed +dbal?
Updated by Stefan Neufeind over 12 years ago
Last time I gave it a short look imho not too many things need updating. Imho we might think whether it makes sense in this step to also change the variable for accessing the database to be an array (to possibly hold multiple connections, which could be a pool to be used by extensions or so).
Updated by Markus Klein over 11 years ago
Since PHP 5.4 is out mysql is deprecated now. I guess it's a good time now to integrate mysgli in TYPO3 CMS 6.1.
PDO is also an option, but I don't have experience with it.
Updated by Gerrit Code Review over 11 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19280
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19280
Updated by Gerrit Code Review over 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19280
Updated by Gerrit Code Review over 11 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19280
Updated by Stefan Neufeind over 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset f0d8ed9104bc3cc2f9ef0f37abf5be498f30cb0e.
Updated by