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 <larshe...@gmail.com> 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 <msing...@thoughtworks.com > > 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
_______________________________________________ Mailing list: https://launchpad.net/~dhis2-devs Post to : dhis2-devs@lists.launchpad.net Unsubscribe : https://launchpad.net/~dhis2-devs More help : https://help.launchpad.net/ListHelp
