dhis2-devs team mailing list archive

[Lars Helge Øverland <larshelge@xxxxxxxxx>]

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
>