Page MenuHomePhabricator

Move WikibaseRepo ItemRedirectCreationInteractor to service container
Closed, ResolvedPublic

Description

This service takes a Context, so this will be a trickier migration. I suspect we’ll want to add a context argument to EntityRedirectCreationInteractor::createRedirect() so that we can remove it from the constructor.

Event Timeline

Change 681939 had a related patch set uploaded (by Itamar Givon; author: Itamar Givon):

[mediawiki/extensions/Wikibase@master] Pass IContextSource directly into createRedirect()

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

Change 681948 had a related patch set uploaded (by Itamar Givon; author: Itamar Givon):

[mediawiki/extensions/WikibaseLexeme@master] Pass $context into createRedirect method instead of parent constructor

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

Change 681995 had a related patch set uploaded (by Itamar Givon; author: Itamar Givon):

[mediawiki/extensions/Wikibase@master] Move WikibaseRepo ItemRedirectCreationInteractor to service container

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

Change 682121 had a related patch set uploaded (by Itamar Givon; author: Itamar Givon):

[mediawiki/extensions/Wikibase@master] Restore typehinting after removal of compatibility code

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

Change 681939 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Pass IContextSource directly into createRedirect()...

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

Change 681948 merged by jenkins-bot:

[mediawiki/extensions/WikibaseLexeme@master] Pass $context into createRedirect method instead of parent constructor

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

Change 681995 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Move WikibaseRepo ItemRedirectCreationInteractor to service container

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

Change 682121 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Restore typehinting after removal of compatibility code

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