dhis2-devs team mailing list archive

[Mansi Singhal <msinghal@xxxxxxxxxxxxxxxx>]

Hey Lars,

I checked it again clearing cache. Issue still remains the same. Can you
please check it once again on your side.

On Mon, Sep 15, 2014 at 5:35 PM, Lars Helge Øverland <larshelge@xxxxxxxxx>
wrote:

> Hi Mansi,
>
> that's strange. Actually in rev 16720 the important change is that
> analytics uses Period.getDaysInPeriod(), which adds one day to the result
> (since period end date is inclusive). See line 331 in
> DefaultAnalyticsService. Are you sure its not a cache issue? Let me double
> check on my side.
>
> Lars
>
>
>
> On Mon, Sep 15, 2014 at 11:57 AM, Mansi Singhal <msinghal@xxxxxxxxxxxxxxxx
> > wrote:
>
>> Hey Lars ,
>>
>> I have taken revision number 16720 from trunk in our branch. What i
>> observed is Number of days in a week are still taken as 6. DateUtilsTest
>> class also assumes number of days as 6.
>>
>> We were assuming number of days in a week as 7.
>>
>> Is it a bug or Do I need to take changes from some other revision too.
>>
>> --
>> Regards,
>> Mansi Singhal
>>
>
>


-- 
Regards,
Mansi Singhal

+91 9900246052