Mike, I'll let you handle this patch. (I swear I remember us deciding *not* to do this in the DAO, but to do it in the evaluator instead. But from the commit on Christian's ticket, it's in the DAO.)
-Darius On Tue, Aug 16, 2011 at 6:24 AM, Bailly Rurangirwa <[email protected]>wrote: > The method "getPatientsHavingObs" in HibernateCohortQueryDAO.java, which > all > the ObsCohortQuery classes use, was not using DateUtil class to check the > end Date. > > Using DateUtil.getEndOfDayIfTimeExcluded(onOrBefore) when setting time on > the query should fix this in all ObsCohortQuery classes. > I attached a patch for this on the ticket( > https://tickets.openmrs.org/browse/REPORT-166 REPORT-166 ). > > -- > View this message in context: > http://openmrs-mailing-list-archives.1560443.n2.nabble.com/Reporting-module-end-date-is-not-included-when-calculating-values-for-reports-tp6675961p6691283.html > Sent from the Developers mailing list archive at Nabble.com. > > _________________________________________ > > To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to > [email protected] with "SIGNOFF openmrs-devel-l" in the body > (not the subject) of your e-mail. > > [mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l] > _________________________________________ To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to [email protected] with "SIGNOFF openmrs-devel-l" in the body (not the subject) of your e-mail. [mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l]
