← Back to team overview

dhis2-devs team mailing list archive

Re: Sync Anonymous Event Data along with coordinates code fix

 

Hi Morten

That would be great. Please let us know when you commit the changes. We ll
take a merge from the trunk.

Regards
Vanya

On Wed, Jun 15, 2016 at 9:01 AM, Morten Olav Hansen <morten@xxxxxxxxx>
wrote:

> Hi
>
> We usually turn off most of that Jackson auto discovery magic, I see we
> haven't done that properly in Jackson*<tracker object classes>, I will add
> that now (so it follows JacksonUtils, RenderService etc)
>
> --
> Morten Olav Hansen
> Senior Engineer, DHIS 2
> University of Oslo
> http://www.dhis2.org
>
> On Tue, Jun 14, 2016 at 8:41 PM, Anilkumar Kodi <anilkumk@xxxxxxxxxxxxxxxx
> > wrote:
>
>> Hi Morten,
>>
>> We are doing Anonymous Event data push. As part of that we are also
>> sending coordinates.
>>
>> Coordinate object has 4 public properties which are listed below
>>
>> @JsonProperty( required = true )
>> @JacksonXmlProperty( isAttribute = true )
>> public Double getLatitude()
>>
>> @JsonProperty( required = true )
>> @JacksonXmlProperty( isAttribute = true )
>> public Double getLongitude()
>>
>> public boolean isValid()
>>
>> public String getCoordinateString()
>>
>>
>> We don't have @JsonIgnore/@JsonProperty on the Other 2 properties.
>>
>> Because of this we are getting Error while Deserializing.
>>
>> Moreover, the JSON Mapper for the class <JacksonEventService>  has configuration FAIL_ON_UNKNOWN_PROPERTIES  set to *true*
>>
>> We have fixed this by adding @JsonIgnore on those properties.
>>
>> We Would like to know if you see any downstream issues with this minor change.
>>
>>
>>
>> Thanks,
>>
>> Anil Kumar & Srikanth
>>
>>
>>
>>
>>
>>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~dhis2-devs
> Post to     : dhis2-devs@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~dhis2-devs
> More help   : https://help.launchpad.net/ListHelp
>
>


-- 
With Regards
ThoughtWorks Technologies
Hyderabad

--Stay Hungry Stay Foolish!!

Follow ups

References