aboutsummaryrefslogtreecommitdiffstats
path: root/includes/ActorMigrationBase.php
Commit message (Collapse)AuthorAgeFilesLines
* Improve docs of various ::getQueryInfo functionsUmherirrender2022-08-131-7/+8
| | | | | | Explict types helps phan to detect errors with keys like ef82d95 Change-Id: Iee4d964ecc58273eb9c73356ed34e13f29fbd275
* Use User::isRegistered for readability instead of ::getId falsy checkUmherirrender2022-04-291-1/+1
| | | | Change-Id: I42aab149559e3e899cde6c77af76c66936ed0ef0
* Schema migration for revison_actor_temp table removalTim Starling2021-07-141-42/+87
| | | | | | | | | | | | | | | | | | | | | | | | | Introduce a new schema migration stage in which rev_actor is used directly and the revision_actor_temp table is no longer needed. This becomes the new "new" stage whereas the previous situation is now termed SCHEMA_COMPAT_TEMP. Introduce migrateRevisionActorTemp which copies data from revision_actor_temp to rev_actor. The code is similar to migrateImageCommentTemp.php except that it doesn't delete from the old table. Partial revert of c29909e59fd89d9fc6. That change removed direct references to $wgActorTableSchemaMigrationStage and made queries involving revision_actor_temp be unconditional. Such changes need to be reverted to make the use of revision_actor_temp be conditional again. In ActorMigrationTest, I compacted provideGetJoin() and provideGetWhere(), removing most of the duplication between expected values. I gave all the stages a short name, and mostly used the name in providers. Bug: T275246 Change-Id: I7498107dd6433ab7de5bf2e7b3fe2aa5e10e345d
* Remove hard-deprecated ActorMigrationBase methods.Petr Pchelko2021-06-041-44/+0
| | | | Change-Id: I0ab4864dfd6ebf3a3bd1c0fb80a24cdb097cadfc
* Split a base class out of ActorMigrationTim Starling2021-05-031-0/+513
AbuseFilter needs a class which is almost identical to core's ActorMigration, so I split out the generic facilities into ActorMigrationBase, leaving the bits specific to core in ActorMigration. I changed the way fields and tables are specified so that it's easier to override in subclasses. Proper injection of ActorStoreFactory in ActorMigrationTest::testGetWhere() was necessary to avoid a test failure due to a teardown issue. Bug: T278917 Change-Id: I3a6486532f2ef360d1fd01b3a609de71d412f739