Page MenuHomePhabricator

Add datatype to snaks in the external JSON
Closed, ResolvedPublic

Description

The external JSON should contain a reference to the datatype of the snak for PropertyValue-Snaks.


Version: unspecified
Severity: normal

Details

Reference
bz54320

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:00 AM
bzimport set Reference to bz54320.
bzimport added a subscriber: Unknown Object (MLST).

Change 96258 had a related patch set uploaded by Daniel Kinzler:
Allow service injection into serializers.

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

Patch Ib2137ad adds support for the datatype field to SnakSerializer, which should be sufficient to consider this fixed. That however doesn't cause the datatype to show up in all JSON output, since the necessary lookup service isn't yet injected everywhere. For the specific use cases, I filed:

  • bug 57212
  • bug 57213
  • bug 57231

Change 96258 merged by jenkins-bot:
(bug 54320) Include datatype in snak serialization.

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