Page MenuHomePhabricator

CU 2.0: Namespace Service Name
Closed, ResolvedPublic

Description

To avoid name conflicts, the service name should be prefixed with the extension's name.

https://www.mediawiki.org/wiki/Dependency_Injection#Quick_Start

Details

Related Gerrit Patches:
mediawiki/extensions/CheckUser : masterNamespace Services

Event Timeline

dbarratt created this task.Tue, Dec 3, 4:16 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptTue, Dec 3, 4:16 AM

Change 554230 had a related patch set uploaded (by Dbarratt; owner: Dbarratt):
[mediawiki/extensions/CheckUser@master] Namespace Services

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

dbarratt updated the task description. (Show Details)Tue, Dec 3, 4:23 AM
dbarratt moved this task from In Progress to Review on the Anti-Harassment (The Letter Song) board.
dbarratt claimed this task.Tue, Dec 3, 4:26 AM

Change 554230 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Namespace Services

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

dom_walden added a subscriber: dom_walden.

I guess this is fine. I don't really know...

dbarratt closed this task as Resolved.Tue, Dec 3, 3:35 PM

I guess this is fine. I don't really know...

The service isn't being used yet, so there isn't even a regression to test. :)