dhis2-devs team mailing list archive

[Morten Olav Hansen <mortenoh@xxxxxxxxx>]

Hm, yes.. I see that. I guess the idea is that time dimension doesn't
really matter for aggregate values.

I don't see any harm in supporting multiple ISO formats, so if you could
provide a patch, that would be welcome (but please also allow for the old
format), maybe you could extend DateUtils to support multiple formats,
something similar to what we have done in FacilityController (ohie-module):

            private SimpleDateFormat[] simpleDateFormats = new
SimpleDateFormat[]{
                new SimpleDateFormat( "yyyy-MM-dd'T'HH:mm:ssZ" ),
                new SimpleDateFormat( "yyyy-MM-dd'T'HH:mm:ss" ),
                new SimpleDateFormat( "yyyy-MM-dd'T'HH:mm" ),
                new SimpleDateFormat( "yyyy-MM-dd'T'HH" ),
                new SimpleDateFormat( "yyyy-MM-dd" ),
                new SimpleDateFormat( "yyyy-MM" ),
                new SimpleDateFormat( "yyyy" )
            };

That would be enough at least as a interim solution. We are in the process
of supporting multiple calendars in DHIS 2, so in the future we can't
really rely on SimpleDateFormat anymore, but would have to parse this
information (possibly) using a regex (similar to what we have done in
DateUnitType).

--
Morten


On Tue, May 27, 2014 at 12:49 PM, Aravind Muralikrishnan <
aravindm@xxxxxxxxxxxxxxxx> wrote:

> Hi Morten,
>
> I was talking about the DataValue in DataValueSet used in
> DataValuesSetController. We are trying to upload a series of dataValues
> with timestamp using the *DataValuesSetController.postJsonDataValueSet *endpoint.
> When we walked through the code, we saw that the timestamp field on
> DataValue was getting formatted to just Date (ignoring the time part) in *DefaultDataValueSetService.saveDataValueSet
> (573)*.
>
> However, when we save data values through the data entry screen in DHIS,
> we could see that the time part is also getting persisted.
>
> If we see value in maintaining the same behaviour through the web api, we
> could submit a patch for the same.
>
> Please do let us know your thoughts.
>
> Thanks,
> Aravind
>
>
> On Tue, May 27, 2014 at 2:50 PM, Morten Olav Hansen <mortenoh@xxxxxxxxx>wrote:
>
>> Hi Aravind,
>>
>> I'm sorry, but could you give me some more details.. is this event
>> dataValues? and which timestamp field? is it the one on
>> trackedentitydatavalue? in the even web-api, we are just doing a simple new
>> Date() for data values
>>
>> If you are talking about programstageinstance, yes, we currently only
>> supports ISO Dates (yyyy-MM-dd), I agree that this should be improved, but
>> it should also not break any existing clients..
>>
>> --
>> Morten
>>
>>
>> On Tue, May 27, 2014 at 10:17 AM, Aravind Muralikrishnan <
>> aravindm@xxxxxxxxxxxxxxxx> wrote:
>>
>>> Hello all,
>>>
>>> We were looking at the API to save DataValues in DHIS. We saw that the
>>> timestamp field in DataValues only stores *date* value in timestamp
>>> field and *ignores the time* part completely before its saved.
>>>
>>> We have a requirement in which we would want to sync the data values at
>>> regular intervals from DHIS through the web api, and we would want the time
>>> part also to be stored in DHIS.
>>>
>>>  Is there a specific reason why we are ignoring the time part when
>>> saving DataValues through the API?
>>>
>>> Thanks,
>>> Aravind
>>>
>>> _______________________________________________
>>> 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
>>>
>>>
>>
>