Change 271345 had a related patch set uploaded (by Bene):
Allow injection of entity views

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

From my point of view, WikibaseView is a component providing widgets for a specific set of WikibaseDataModel concepts. Since the widgets for items and properties use all other widgets, we provide an EntityViewFactory wiring them together. Wikibase\View\EntityViewFactory could look like the following for all I (aka WikibaseView) care about:

EntityViewFactory
  __construct()
  newItemView( ...allTheServicesAndValues ): ItemView
  newPropertyView( ...allTheServicesAndValues ): PropertyView

Also, WikibaseView should define an EntityView interface:

EntityView
  getTitleHtml( Entity ): HtmlString
  getHtml( Entity ): HtmlString
  getPlaceholders(): mixed[]

With that, WikibaseView seems done to me.

WikibaseRepo (namely EntityParserOutputGenerator) on the other hand needs something where it can just put in an entity type and a few other things and get a view. That factory thing should be defined in WikibaseRepo, instantiated in WikibaseRepo.php based on config, and passed to EntityParserOutputGenerator. It would use Wikibase\View\EntityViewFactory.

In T127191#2039646, @adrianheine wrote:

From my point of view, WikibaseView is a component providing widgets for a specific set of WikibaseDataModel concepts. Since the widgets for items and properties use all other widgets, we provide an EntityViewFactory wiring them together. Wikibase\View\EntityViewFactory could look like the following for all I (aka WikibaseView) care about:

EntityViewFactory
  __construct()
  newItemView( ...allTheServicesAndValues ): ItemView
  newPropertyView( ...allTheServicesAndValues ): PropertyView

I'd also add factory methods for the StatementSectionsView and EntityTermsView because extensions of Wikibase defining new entity types want to reuse those views.

Also, WikibaseView should define an EntityView interface:

EntityView
  getTitleHtml( Entity ): HtmlString
  getHtml( Entity ): HtmlString
  getPlaceholders(): mixed[]

This is almost what EntityView currently does. We can however refactor this and define a new public interface of EntityView which doesn't contain the protected abstract functions for main and side HTML.

With that, WikibaseView seems done to me.

WikibaseRepo (namely EntityParserOutputGenerator) on the other hand needs something where it can just put in an entity type and a few other things and get a view. That factory thing should be defined in WikibaseRepo, instantiated in WikibaseRepo.php based on config, and passed to EntityParserOutputGenerator. It would use Wikibase\View\EntityViewFactory.

I agree on that.

In T127191#2054679, @Bene wrote:

I'd also add factory methods for the StatementSectionsView and EntityTermsView because extensions of Wikibase defining new entity types want to reuse those views.

Why is a factory method necessary for them? They can just be instantiated in the factory method creating the view.

@adrianheine mainly because they need access to the TemplateFactory which isn't available outside of Wikibase.

But TemplateFactory is constructed in WikibaseRepo via TemplateFactory::getDefaultInstance() and then injected into EntityViewFactory. The same could be done for a MediaInfoViewFactory.

TemplateFactory doesn't seem to serve a purpose here because we cannot inject other templates from another component/extension. It might be nice to make that class modular to be able to support other template files as well but for now it isn't useful for any other extensions.

In T127191#2079460, @Bene wrote:

TemplateFactory doesn't seem to serve a purpose here because we cannot inject other templates from another component/extension. It might be nice to make that class modular to be able to support other template files as well but for now it isn't useful for any other extensions.

Sure, but what does that have to do with the point here?

Change 271345 merged by jenkins-bot:
Move entity type dispatching out of WikibaseView

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

Change 274393 had a related patch set uploaded (by Bene):
Dispatch entity views based on callbacks

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

Change 274393 merged by jenkins-bot:
Dispatch entity views based on callbacks

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

Change 274731 had a related patch set uploaded (by Bene):
Add view-factory-callback to the entity type definitions

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

Change 274731 merged by jenkins-bot:
Add view-factory-callback to the entity type definitions

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