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

Event Timeline

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

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

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...

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. :)