Page MenuHomePhabricator
Create Task
Maniphest T279415

[EPIC] Remove calls to deprecated WikibaseRepo::getUserLanguage
Closed, ResolvedPublic
Actions

Description

Context

The UserLanguage service has been migrated to the new service container. However, it was decided in T278819 that this service should be deprecated and gradually retired out of usage, because services in the service container should be independent of a specific request context, including the user language for that request, and $wgUser will be deprecated eventually (T159299). The method itself was deprecated in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/676422.

Objective

The service accessor method is no longer called in any of the Wikibase related extensions:

  • Wikibase
  • WikibaseLexeme
  • WikibaseCirrusSearch
  • WikibaseLexemeCirrusSearch
  • WikibaseQualityConstraints
  • WikibaseMediaInfo

Code search: https://codesearch.wmcloud.org/search/?q=WikibaseRepo%3A%3AgetUserLanguage&i=nope&files=&excludeFiles=&repos=

Additional Considerations

  • In each of these extensions, an alternative way to obtain the user language should be found to replace calls to this method, either by utilizing the RequestContext or otherwise (in places where the request is not available).

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
Don’t inject user language in CheckConstraintParametersmediawiki/extensions/WikibaseQualityConstraintsmaster+85 -51
Don’t inject user language in CheckConstraintsmediawiki/extensions/WikibaseQualityConstraintsmaster+71 -43
Customize query in gerrit

Related Objects
Search...

StatusSubtypeAssignedTask
 ResolvedLucas_Werkmeister_WMDET279415 [EPIC] Remove calls to deprecated WikibaseRepo::getUserLanguage
 ResolvedItamarWMDET281365 Remove calls to WikibaseRepo::getUserLanguage from SpecialListPropertiesDispatchingFactory
 ResolvedItamarWMDET281449 Remove calls to WikibaseRepo::getUserLanguage from the ViewFactory Service wiring
 ResolvedLucas_Werkmeister_WMDET281718 Remove calls to WikibaseRepo::getUserLanguage from entity search callbacks
 ResolvedkarapayneWMDET281720 Remove calls to WikibaseRepo::getUserLanguage from the MessageParameterFormatter Service wiring
 ResolvedLucas_Werkmeister_WMDET281721 Remove calls to deprecated WikibaseRepo::getLanguageNameLookup()
 ResolvedLucas_Werkmeister_WMDET281712 Don’t inject LanguageNameLookup into ViewFactory
 ResolvedLucas_Werkmeister_WMDET281722 Stop injecting LanguageNameLookup into EntityIdHtmlLinkFormatterFactory
 ResolvedLucas_Werkmeister_WMDET281726 Stop injecting LanguageNameLookup into WikibaseValueFormatterBuilders
 ResolvedkarapayneWMDET281724 WikibaseCirrusSearch: Remove calls to WikibaseRepo::getUserLanguage from EntityFullTextQueryBuilder
 ResolvedkarapayneWMDET281725 WikibaseCirrusSearch: Remove calls to WikibaseRepo::getUserLanguage from Hooks.php
 ResolvedkarapayneWMDET281799 WikibaseLexeme: Remove calls to WikibaseRepo::getUserLanguage from formatter factory callbacks
 ResolvedMichaelT281801 WikibaseLexeme: Remove calls to WikibaseRepo::getUserLanguage from entity diff visualizer callbacks
 ResolvedkarapayneWMDET281805 WikibaseLexemeCirrusSearch: Remove calls to WikibaseRepo::getUserLanguage from LexemeFullTextQueryBuilder
 ResolvedkarapayneWMDET281807 WikibaseMediaInfo: Remove calls to WikibaseRepo::getUserLanguage from search profile callbacks
 OpenNoneT344050 [SPIKE] WikibaseClient SidebarLinkBadgeDisplay inappropriately uses user language for parser output
 ResolvedkarapayneWMDET393003 Remove repo UserLanguage service from Wikibase

Event Timeline

ItamarWMDE created this task.Apr 6 2021, 9:23 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 6 2021, 9:23 AM
ItamarWMDE mentioned this in T278819: Move WikibaseRepo UserLanguage to service container.Apr 6 2021, 9:24 AM
Addshore moved this task from Inbox to Triaged Low (0-50) on the [DEPRECATED] wdwb-tech board.Apr 6 2021, 11:53 AM
gerritbot added a comment.Apr 6 2021, 2:57 PM

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

[mediawiki/extensions/WikibaseQualityConstraints@master] Don’t inject user language in CheckConstraintParameters

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

gerritbot added a project: Patch-For-Review.Apr 6 2021, 2:57 PM

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

[mediawiki/extensions/WikibaseQualityConstraints@master] Don’t inject user language in CheckConstraints

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

Addshore added a project: Wikidata-Campsite.Apr 13 2021, 5:05 PM
Addshore moved this task from Incoming to Prioritized Wikidata Tech Backlog (prioritised from top to bottom) on the Wikidata-Campsite board.
darthmon_wmde moved this task from Incoming to Ready to pick up on the Wikibase Service Migration board.Apr 22 2021, 10:16 AM
ItamarWMDE moved this task from Ready to pick up to Doing on the Wikibase Service Migration board.Apr 27 2021, 9:45 AM
ItamarWMDE moved this task from Doing to Ready to pick up on the Wikibase Service Migration board.
Lucas_Werkmeister_WMDE closed subtask T281449: Remove calls to WikibaseRepo::getUserLanguage from the ViewFactory Service wiring as Resolved.Apr 29 2021, 2:43 PM
Lucas_Werkmeister_WMDE closed subtask T281365: Remove calls to WikibaseRepo::getUserLanguage from SpecialListPropertiesDispatchingFactory as Resolved.
Lucas_Werkmeister_WMDE mentioned this in T277464: [EPIC] Multiple Wikibase service wiring tests fail when run as unit tests.Apr 30 2021, 9:24 AM
gerritbot added a comment.Apr 30 2021, 4:39 PM

Change 677277 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] Don’t inject user language in CheckConstraintParameters

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

gerritbot added a comment.Apr 30 2021, 4:39 PM

Change 677278 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] Don’t inject user language in CheckConstraints

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

ReleaseTaggerBot added a project: MW-1.37-notes (1.37.0-wmf.4; 2021-05-04).Apr 30 2021, 5:00 PM
Maintenance_bot removed a project: Patch-For-Review.Apr 30 2021, 5:10 PM
Lucas_Werkmeister_WMDE mentioned this in T281677: Make SummaryFormatterTest a proper unit test.May 3 2021, 8:37 AM
Lucas_Werkmeister_WMDE removed a subtask: T281712: Don’t inject LanguageNameLookup into ViewFactory.May 3 2021, 2:53 PM
Lucas_Werkmeister_WMDE updated the task description. (Show Details)May 3 2021, 4:06 PM
Addshore updated the task description. (Show Details)Jun 23 2021, 9:07 AM
Addshore updated the task description. (Show Details)
Addshore triaged this task as Low priority.Jul 14 2021, 11:12 AM
Addshore removed a project: Wikidata-Campsite.Jul 20 2021, 2:56 PM
Lucas_Werkmeister_WMDE mentioned this in T201170: Allow wikibase to render entities for an arbitrary target language, instead of the user's UI language.Sep 17 2021, 9:44 AM
Lucas_Werkmeister_WMDE mentioned this in T344050: [SPIKE] WikibaseClient SidebarLinkBadgeDisplay inappropriately uses user language for parser output.Aug 11 2023, 9:30 AM
Lucas_Werkmeister_WMDE subscribed.Aug 11 2023, 2:05 PM
Lucas_Werkmeister_WMDE closed subtask T281801: WikibaseLexeme: Remove calls to WikibaseRepo::getUserLanguage from entity diff visualizer callbacks as Resolved.Aug 11 2023, 2:08 PM
Lucas_Werkmeister_WMDE closed subtask T281721: Remove calls to deprecated WikibaseRepo::getLanguageNameLookup() as Resolved.Sep 20 2023, 1:18 PM
Lucas_Werkmeister_WMDE mentioned this in T377442: [GENERAL] Stop relying on Language object in Wikibase.Oct 17 2024, 12:33 PM
ItamarWMDE raised the priority of this task from Low to Needs Triage.Dec 6 2024, 11:29 AM
ItamarWMDE added a project: wmde-wikidata-tech (Wikidata Omega Triage).
ItamarWMDE edited projects, added wmde-wikidata-tech; removed wmde-wikidata-tech (Wikidata Omega Triage).
ItamarWMDE renamed this task from Remove calls to deprecated WikibaseRepo::getUserLanguage to [EPIC] Remove calls to deprecated WikibaseRepo::getUserLanguage.Dec 6 2024, 11:37 AM
ItamarWMDE updated the task description. (Show Details)
Restricted Application added a project: Epic. · View Herald TranscriptDec 6 2024, 11:37 AM
Maintenance_bot moved this task from Triaged Low (0-50) to Sorted Team A on the [DEPRECATED] wdwb-tech board.Dec 6 2024, 12:29 PM
ItamarWMDE moved this task from Incoming to [DOT] Prioritized Epics on the wmde-wikidata-tech board.Dec 10 2024, 11:22 AM
karapayneWMDE added a project: Wikidata-Omega.Apr 16 2025, 9:04 AM
Arian_Bozorg moved this task from Incoming to Radar/Epics/Stalled on the Wikidata-Omega board.Apr 16 2025, 9:23 AM
Arian_Bozorg edited projects, added Wikidata-Omega (Radar/Epics/Stalled); removed Wikidata-Omega.
karapayneWMDE moved this task from Incoming to Epics on the Wikidata-Omega (Radar/Epics/Stalled) board.Apr 16 2025, 9:40 AM
ArthurTaylor added a subtask: T393003: Remove repo UserLanguage service from Wikibase.Apr 30 2025, 12:21 PM
karapayneWMDE moved this task from Radar/Epics/Stalled to The Board on the Wikidata-Omega board.May 5 2025, 8:18 AM
karapayneWMDE edited projects, added Wikidata-Omega (The Board); removed Wikidata-Omega (Radar/Epics/Stalled).
karapayneWMDE moved this task from new to the board to Parent Task on the Wikidata-Omega (The Board) board.
Lucas_Werkmeister_WMDE closed subtask T281718: Remove calls to WikibaseRepo::getUserLanguage from entity search callbacks as Resolved.May 12 2025, 1:49 PM
karapayneWMDE closed subtask T281807: WikibaseMediaInfo: Remove calls to WikibaseRepo::getUserLanguage from search profile callbacks as Resolved.May 15 2025, 2:20 PM
karapayneWMDE closed subtask T281725: WikibaseCirrusSearch: Remove calls to WikibaseRepo::getUserLanguage from Hooks.php as Resolved.
karapayneWMDE closed subtask T281724: WikibaseCirrusSearch: Remove calls to WikibaseRepo::getUserLanguage from EntityFullTextQueryBuilder as Resolved.
karapayneWMDE closed subtask T281805: WikibaseLexemeCirrusSearch: Remove calls to WikibaseRepo::getUserLanguage from LexemeFullTextQueryBuilder as Resolved.
karapayneWMDE closed subtask T281799: WikibaseLexeme: Remove calls to WikibaseRepo::getUserLanguage from formatter factory callbacks as Resolved.
karapayneWMDE closed subtask T281720: Remove calls to WikibaseRepo::getUserLanguage from the MessageParameterFormatter Service wiring as Resolved.
karapayneWMDE closed subtask T393003: Remove repo UserLanguage service from Wikibase as Resolved.
karapayneWMDE moved this task from Parent Task to In Tech Verification on the Wikidata-Omega (The Board) board.May 15 2025, 2:24 PM
Lucas_Werkmeister_WMDE mentioned this in T393003: Remove repo UserLanguage service from Wikibase.May 15 2025, 2:30 PM
Lucas_Werkmeister_WMDE closed this task as Resolved.May 16 2025, 8:24 AM
Lucas_Werkmeister_WMDE claimed this task.
Lucas_Werkmeister_WMDE moved this task from In Tech Verification to Done - Moving a task here will move it to the completed tasks workboard, resolve the ticket, and assign it to whomever made the move on the Wikidata-Omega (The Board) board.
Lucas_Werkmeister_WMDE edited projects, added Wikidata-Omega (Completed Tasks); removed Wikidata-Omega (The Board).
karapayneWMDE moved this task from Newly completed tasks to 2025 Q2 on the Wikidata-Omega (Completed Tasks) board.Jun 18 2025, 6:30 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits