Support BuiltMap<String, Object> like JSON

[davidmorgan]

JSON protocols can have a Map<String, Object> where Object is any JSON primitive. Support this somehow with StandardJsonPlugin.

[zoechi]

Also List<Object>

Do you have short-term plans to provide something like the mentioned JSON primitive?
For a current project this would be required so that built_value could be used.

[davidmorgan]

I will try to look at it soon -- but can't promise anything. If you like you can try it yourself. New serializable types and serializers can be added as you like, e.g.:

class JsonObject {
  final Object object;

  JsonObject(this.object);

  ... utility methods here, e.g. "asInt", "asDouble", "asList"...
  ... also hashCode, operator== if you want those to work...
}

class JsonObjectSerializer implements PrimitiveSerializer<JsonObject> {
  @override
  final Iterable<Type> types = new BuiltList<Type>([JsonObject]);
  @override
  final String wireName = 'JsonObject';

  @override
  Object serialize(Serializers serializers, JsonObject object,
      {FullType specifiedType: FullType.unspecified}) {
    return object;
  }

  @override
  int deserialize(Serializers serializers, Object serialized,
      {FullType specifiedType: FullType.unspecified}) {
    return new JsonObject(serialized);
  }
}

// Add JsonObjectSerializer to Serializers to use it.
Serializers mySerializers = (serializers.toBuilder()
    ..add(new JsonObjectSerializer())).build();
mySerializers.serialize(...)

[zoechi]

Thanks for the info!
Of course no promise - it's the open source world :)
I'll have a look, but first I need to better understand the whole builder stuff - never used anything like that.

[zoechi]

JsonObject get keyValues;
works fine so far.

I also tried
MapJsonObject get keyValues; and ListJsonObject get keyValues;, but that fails when deserializing.
Is this supposed to work?

[davidmorgan]

It should work better to use

BuiltMap<String, JsonObject>

instead of MapJsonObject, and

BuiltList

instead of ListJsonObject -- so I don't think these should be needed. Possibly I missed something?

[zoechi]

Didn't think of that. Thanks for the pointer 👍