| Maintenance_bot | |
| Jul 7 2022, 6:28 AM |
| F35737281: image.png | |
| Nov 10 2022, 2:53 AM |
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 | None | T313519 Migrate usage of Database::select to SelectQueryBuilder in extensions maintained by Growth team | ||
| Resolved | MPGuy2824 | T348835 Migrate code to use new query builder functions | ||
| Resolved | Scardenasmolinar | T312480 Migrate usage of Database::select to SelectQueryBuilder in PageTriage | ||
| · · · |
Looks like the type of refactoring asked for in this ticket is to replace $dbr->select( style code with $dbr->newSelectQueryBuilder()->select( style code. The first example takes up to 6 parameters, mostly arrays, and doesn't use method chaining. The second style uses method chaining.
More info at https://www.mediawiki.org/wiki/Manual:Database_access#Wrapper_function%3A_select()
Getting around 23 search results.
PageTriage doesn't have many tests. Mass refactoring SQL functions could be a bit dangerous. Will need to be careful.
PageTriage doesn't have many tests. Mass refactoring SQL functions could be a bit dangerous. Will need to be careful.
Agreed. We can start converting the IDatabase::select() functions in the tests themselves and then move on to other select statements, with the condition that a unit test is added.
Change 911959 had a related patch set uploaded (by Scardenasmolinar; author: Scardenasmolinar):
[mediawiki/extensions/PageTriage@master] [WIP] Change Database::select to new query builder
Some of the select queries I couldn't migrate because it would mean a lot of reworking. We could tackle those in a follow-up ticket.
Test wiki created on Patch demo by SCardenas (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/0bf752f8c0/w
Change 929784 had a related patch set uploaded (by Jsn.sherman; author: Scardenasmolinar):
[mediawiki/extensions/PageTriage@master] WIP Demonstrate test issue for Change Database::select to new query builder
Change 911959 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Change Database::select to new query builder
Test wiki on Patch demo by SCardenas (WMF) using patch(es) linked to this task was deleted:
Change 929784 abandoned by Jsn.sherman:
[mediawiki/extensions/PageTriage@master] WIP Demonstrate test issue for Change Database::select to new query builder
Reason:
Change 971219 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):
[mediawiki/extensions/PageTriage@master] Migrate to SQB for ArticleCompile
Change 971219 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Migrate to SQB for ArticleCompile