Page MenuHomePhabricator

[Task] DataModelSerialization: include the property data type in Property*Snaks.
Closed, InvalidPublic

Description

The main issue here is avoiding circular dependencies between the serializer and the service interfaces.

Related Objects

StatusAssignedTask
Declineddchen
OpenNone
OpenNone
DuplicateNone
OpenNone
ResolvedAbit
OpenNone
DuplicateNone
OpenNone
OpenNone
OpenNone
DuplicateNone
InvalidLydia_Pintscher
OpenNone
OpenNone
OpenNone
OpenNone
StalledNone
ResolvedBene
OpenNone
ResolvedJanZerebecki
OpenNone
OpenNone
Resolvedthiemowmde
OpenNone
DeclinedBene
ResolvedBene
ResolvedBene
ResolvedBene
ResolvedAddshore
OpenNone
InvalidNone
Resolveddaniel
ResolvedBene
DeclinedBene

Event Timeline

daniel created this task.May 12 2015, 1:37 PM
daniel raised the priority of this task from to High.
daniel updated the task description. (Show Details)
daniel added subscribers: Bene, Wikidata-bugs, Tobi_WMDE_SW and 4 others.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 12 2015, 1:37 PM
JanZerebecki lowered the priority of this task from High to Normal.May 16 2015, 5:17 PM
JanZerebecki set Security to None.

This blocks "WikibaseDataModelSerialization with what we do with WikibaseLib" and although we have totally removed the Lib Serialization I would not consider this task complete in regards to WikibaseDataModelSerialization.

Although we have a TypedSnak object there is no real way to use this in combination with serializing a whole entity, which is basically the only place that this is needed in Wikibase. This is primarily due to the fact that TypedSnak doesnt share an interface or base class with snak..

There are currently some discussions about slotty stuff happening at:

Jonas renamed this task from DataModelSerialization: include the property data type in Property(Value)Snaks. to [Task] DataModelSerialization: include the property data type in Property(Value)Snaks. .Aug 15 2015, 12:22 PM
Addshore renamed this task from [Task] DataModelSerialization: include the property data type in Property(Value)Snaks. to [Task] DataModelSerialization: include the property data type in Property*Snaks. .Aug 15 2015, 7:07 PM
Lydia_Pintscher closed this task as Invalid.Apr 23 2017, 4:50 PM