Page MenuHomePhabricator

Wikibase\MediaInfo\DataModel\MediaInfo must implement method `clear`
Closed, ResolvedPublic

Description

Outdated

MediaInfo is currently on wikibase/data-model 7.3.0, but if/when we upgrade, we're going to have to implement function clear, which is now part of interface EntityDocument (or interface Clearable, actually)

See https://github.com/wmde/WikibaseDataModel/commit/ec1d98abc8f6fd3e255c93689954cd43316ce46a

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 460903 had a related patch set uploaded (by Cparle; owner: Cparle):
[mediawiki/extensions/WikibaseMediaInfo@master] Implemented clear() method in MediaInfo class

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

Change 460903 merged by jenkins-bot:
[mediawiki/extensions/WikibaseMediaInfo@master] Implemented clear() method in MediaInfo class

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

Addshore updated the task description. (Show Details)
Addshore added a subscriber: Addshore.

So, the situation here changed and datamodel changed to avoid the breaking change.
MediaInfo now uses 9.x, but the MediaInfo entity does have a clear method, (that may not be used?), and also the entity does not implement ClearableEntity.

Either:

  • The clear method should be removed if it is not used.
  • MediaInfo entity should implement the ClearableEntity interface.
Addshore added a subscriber: Cparle.

Poke to @Cparle
This task is probably easily fixable or closable :)

Change 520453 had a related patch set uploaded (by Cparle; owner: Cparle):
[mediawiki/extensions/WikibaseMediaInfo@master] Remove clear() method as it's not used

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

Change 520453 merged by jenkins-bot:
[mediawiki/extensions/WikibaseMediaInfo@master] Remove clear() method as it's not used

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

Change 520734 had a related patch set uploaded (by Cparle; owner: Cparle):
[mediawiki/extensions/WikibaseMediaInfo@master] Use regex to replace commas in quantity value

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