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.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Use InsertQueryBuilder in CheckUser instead of IDatabase::insert | mediawiki/extensions/CheckUser | master | +89 -40 |
Related Objects
Related Objects
- Mentioned Here
- rMWd95a6b3167d2: rdbms: Introduce InsertQueryBuilder
Event Timeline
Comment Actions
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
Comment Actions
Change 949482 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Use InsertQueryBuilder in CheckUser instead of IDatabase::insert