Page MenuHomePhabricator

Remove wb_terms migration configuration options
Closed, ResolvedPublic

Description

These four should all be gone from Wikibase and from the production config:

  • tmpPropertyTermsMigrationStage
  • tmpItemTermsMigrationStages
  • tmpItemSearchMigrationStage
  • tmpPropertySearchMigrationStage

They’re all MIGRATION_NEW by default on master already, so we should be able to drop them from the production config immediately. As a first step, we can replace all their uses in code with a hard-coded MIGRATION_NEW, and then later refactor that code.

Should be removed from docs, defaults and code (usages)

Event Timeline

Change 631436 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] Stop reading from migration variables from settings

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

Change 631486 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] Reinstate skipped tests.

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

Change 631486 abandoned by Tobias Andersson:
[mediawiki/extensions/Wikibase@master] Reinstate skipped tests.

Reason:
merged into parent

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

Change 631714 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] Remove the migration settings as they are no-longer read by the code.

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

They are fully removed in the last patch. The mention in the docs will be removed with https://phabricator.wikimedia.org/T264287

Change 631436 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Stop reading from migration variables from settings

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

Change 631714 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Remove the migration settings as they are no-longer read by the code.

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