Page MenuHomePhabricator

Remove hardcoded DataAccessSettings::USE_REPOSITORY_PREFIX_BASED_FEDERATION in various service constructions
Closed, ResolvedPublic

Description

In T243841 I discovered that https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/566065/ broke the installation process.
This was due to a bit of code hardcoding DataAccessSettings::USE_REPOSITORY_PREFIX_BASED_FEDERATION in a service creation instead of using the right data access settings.
After an initial grep I also saw other code using this pattern, these should be found and fixed.

Related Objects

StatusSubtypeAssignedTask
Declineddchen
OpenNone
OpenNone
DuplicateNone
OpenFeatureNone
OpenFeatureNone
DuplicateNone
ResolvedNone
ResolvedNone
ResolvedNone
Resolved• Ramsey-WMF
ResolvedCparle
ResolvedJdforrester-WMF
ResolvedJdforrester-WMF
ResolvedNone
InvalidNone
ResolvedWMDE-leszek
ResolvedAddshore
ResolvedAddshore

Event Timeline

Change 567976 had a related patch set uploaded (by Addshore; owner: Addshore):
[mediawiki/extensions/Wikibase@master] Fix hardcoded USE_REPOSITORY_PREFIX_BASED_FEDERATION settings

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

Change 567981 had a related patch set uploaded (by Addshore; owner: Addshore):
[mediawiki/extensions/WikibaseMediaInfo@master] Fix hardcoded USE_REPOSITORY_PREFIX_BASED_FEDERATION settings

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

Change 567982 had a related patch set uploaded (by Addshore; owner: Addshore):
[mediawiki/extensions/WikibaseCirrusSearch@master] SearchEntitiesIntegrationTest use USE_ENTITY_SOURCE_BASED_FEDERATION

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

Change 567976 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Fix hardcoded USE_REPOSITORY_PREFIX_BASED_FEDERATION settings

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

Change 567981 merged by jenkins-bot:
[mediawiki/extensions/WikibaseMediaInfo@master] Fix hardcoded USE_REPOSITORY_PREFIX_BASED_FEDERATION settings

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

Change 567982 merged by jenkins-bot:
[mediawiki/extensions/WikibaseCirrusSearch@master] SearchEntitiesIntegrationTest use USE_ENTITY_SOURCE_BASED_FEDERATION

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