EntityDifferFactory provides access to the correct EntityDiffer for a given entity type.
Currently, the differ is baked into Entity::getDiff. Since it has been deprecated for a long time, we should drop Entity::getDiff completely, and move its functionality into ItemContent::getDiff and friends.
Use case: EntityDifferFactory is needed by the proposed EntityDiff API module.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | Feature | None | T66468 improve diff visualization for Wikibase entities | ||
| Open | None | T106306 [Story] API module for retrieving diffs between entity revisions (wbentitydiff) | |||
| Invalid | None | T106308 [Task] Create an EntityDifferFactory |
I think it'd have been better if this was phrased as "create means to construct an EntityDiffer", which does not already decide on a specific approach.
As far as I can tell, there is no need for this at all. What would such a factory class do? Just have the entity differ be constructed in the appropriate DI system (ie the WikibaseRepo class). Furthermore, even though Entity::getDiff was deprecated, it did have a replacement. Which is what you should use now, as Entity::getDiff has been removed already.