Now that T243051: A query builder for MediaWiki core is done, this extension should migrate away from IDatabase::select() to SelectQueryBuilder.
It would improve readability of the code, avoids mistakes by passing the wrong order of arguments, etc.
For more information check T243051 and its documentation.
Note that query builder is a different paradigm and changes should not be one-to-one. For example, avoid using joinConds().
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T311866 Migrate Database::select usages to SelectQueryBuilder (in WMF-deployed extensions) | |||
| Resolved | Wangombe | T312337 Migrate usage of Database::select to SelectQueryBuilder in TranslationNotifications |
Change 836096 had a related patch set uploaded (by Wangombe; author: Wangombe):
[mediawiki/extensions/TranslationNotifications@master] Notify all, some or except some languages
Change 836096 merged by jenkins-bot:
[mediawiki/extensions/TranslationNotifications@master] Add option to notify all, selected or all except some languages
Change 992154 had a related patch set uploaded (by Wangombe; author: Wangombe):
[mediawiki/extensions/TranslationNotifications@master] Use Expression builder instead of raw SQL and deprecared methods
Change 992154 merged by jenkins-bot:
[mediawiki/extensions/TranslationNotifications@master] Use Expression builder instead of raw SQL and deprecated methods
Tested existing functionality on MediaWiki.org by sending a test translations to Odia translators. Nothing appears to be broken.
Change #1026654 had a related patch set uploaded (by Umherirrender; author: Umherirrender):
[mediawiki/extensions/TranslationNotifications@master] Migrate to IReadableDatabase::newSelectQueryBuilder
Change #1026654 merged by jenkins-bot:
[mediawiki/extensions/TranslationNotifications@master] Migrate to IReadableDatabase::newSelectQueryBuilder