gerritbot added a comment.
Change 443104 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Make EntityReferenceExtractors configurable per entity type
https://gerrit.wikimedia.org/r/443104TASK DETAILhttps://phabricator.wikimedia.org/T198295EMAIL
gerritbot added a comment.
Change 443451 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Clean up around EntityParserOutputDataUpdater related code
https://gerrit.wikimedia.org/r/443451TASK DETAILhttps://phabricator.wikimedia.org/T198295EMAIL
gerritbot added a comment.
Change 443451 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Clean up around EntityParserOutputDataUpdater related code
https://gerrit.wikimedia.org/r/443451TASK DETAILhttps://phabricator.wikimedia.org/T198295EMAIL
gerritbot added a comment.
Change 442864 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Refactor ParserOutputDataUpdater
https://gerrit.wikimedia.org/r/442864TASK DETAILhttps://phabricator.wikimedia.org/T198295EMAIL
gerritbot added a comment.
Change 443104 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Make EntityReferenceExtractors configurable per entity type
https://gerrit.wikimedia.org/r/443104TASK DETAILhttps://phabricator.wikimedia.org/T198295EMAIL
gerritbot added a comment.
Change 442864 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Refactor ParserOutputDataUpdater
https://gerrit.wikimedia.org/r/442864TASK DETAILhttps://phabricator.wikimedia.org/T198295EMAIL
Jakob_WMDE added a comment.
More concrete plan:
rename EntityParserOutputDataUpdater to EntityParserOutputDataUpdaterCollection (naming suggestions welcome)
it should only be used to register EntityParserOutputDataUpdaters, pass them the entity they should operate on, and call updateParserOutput
Jakob_WMDE added a comment.
Random notes from an initial investigation
EntityParserOutputDataUpdater should probably not know about StatementDataUpdater and SiteLinkDataUpdater.
ReferencedEntitiesDataUpdater
split into 3 classes: one actual ReferencedEntitiesDataUpdater, one service that