Page MenuHomePhabricator

Remove entityViewInit's dependency on entityLoaded
Closed, ResolvedPublic


entityViewInit depends on the hook system and some module firing the entityLoaded hook.
The entityLoaded module does have dependencies which are not modeled in ResourceLoader and consequently expressing it as a dependency can result in a broken dependency chain (if the entityLoaded module's preconditions are not satisfied).

Event Timeline

Change 592416 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] entityViewInit: don't express entityLoaded module dep

Pablo-WMDE changed the task status from Open to Stalled.May 4 2020, 8:08 AM

Hoo man wrote

This can be merged no earlier than June 3 (so that it wont be deployed earlier than 30 days after Ib676c5820 is live on Wikidata)... for good measure I suggest waiting another week, merging this on/after June 10 only.

Change 592416 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] entityViewInit: don't express entityLoaded module dep