Page MenuHomePhabricator

entitysources: Clean up any remainders of the legacy back/compat config in Wikibase and extensions
Closed, ResolvedPublic


When creating T257260: entitysources: Clean up any remainders of the legacy back/compat config in the mediawiki-config repository I realized that these config options are still used in various places in code (mainly tests and default settings) which is not ideal.
These references should all be removed so that we can safely unset these variables in production.
All code should be getting these things from entity sources, including tests.

Looking at EntitySourceDefinitionsLegacyRepoSettingsParser this probably is:

And for EntitySourceDefinitionsLegacyClientSettingsParser:

The repositories old legacy thing:
The foreignRepositories old legacy thing:
The new entitysources thing:

Acceptance criteria:🏕️🌟

  • Release notes are updated (as this is user facing back compat config that is being removed), including how to migrate from old settings to new settings (possible including a code snippet that using eval.php that can output the new config for people).
  • References to the old configuration options are all removed
  • Back compat layer for the old config options are removed
  • Docs are updated

Related Objects

Event Timeline

I played around with this a bit in
Actually tackling this will probably require the removal of the current WikibaseClientEntityNamespaces and WikibaseRepoEntityNamespaces hooks which interact with the entityNamespaces setting directly to do something more fancy with either the entitySources config or perhaps this could all be done with the entity type definitions (that already has another hook).
That's it for tonight though....

Michael added a subscriber: Michael.

Given that all the subtasks are closed, I think this can move to verification.

Also, is there a checklist-task for the next major Wikibase release? That should probably get an item of making double sure that the script we created in T285474 works as expected. Quoting from that task:

I think we can close this, with verification happening in the parent tasks.

Change 711130 had a related patch set uploaded (by Matthias Mullie; author: Matthias Mullie):

[mediawiki/vagrant@master] Use 'entitySources' configuration

Change 711130 merged by jenkins-bot:

[mediawiki/vagrant@master] Use 'entitySources' configuration

Change 711538 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] docs: Final update for legacy/non-legacy entity sources

Change 711538 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] docs: Final update for legacy/non-legacy entity sources

Doing a quick grep of it seems like at some point we will also have to update some pages there.

@danshick-wmde Is this something that we should aim to do on the campsite or rather something that we could do in a doc coffee time?

I think all the current documentation on is updated now.