Page MenuHomePhabricator

Use InsertQueryBuilder in CheckUser
Closed, ResolvedPublic

Description

The InsertQueryBuilder added in d95a6b3167d23e68cf004516e3a8a2234a8bad6a allows inserting using a builder interface. This is easier to read than the current IDatabase::insert method used in CheckUser. Ideally, CheckUser code should use the new InsertQueryBuilder to insert to the database.

Event Timeline

Change 949482 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Use InsertQueryBuilder in CheckUser instead of IDatabase::insert

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

Change 949482 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Use InsertQueryBuilder in CheckUser instead of IDatabase::insert

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

Dreamy_Jazz claimed this task.