=== modified file 'dhis-2/dhis-services/dhis-service-patient/src/main/java/org/hisp/dhis/activityplan/DefaultActivityPlanService.java' --- dhis-2/dhis-services/dhis-service-patient/src/main/java/org/hisp/dhis/activityplan/DefaultActivityPlanService.java 2011-03-31 01:55:06 +0000 +++ dhis-2/dhis-services/dhis-service-patient/src/main/java/org/hisp/dhis/activityplan/DefaultActivityPlanService.java 2011-04-04 09:23:22 +0000 @@ -102,8 +102,12 @@ public Collection getCurrentActivitiesByProvider( OrganisationUnit organisationUnit ) { - long time = System.currentTimeMillis(); - + Date today = new Date(); + today.setHours( 0 ); + today.setMinutes( 0 ); + today.setSeconds( 0 ); + long time = today.getTime(); + List items = new ArrayList(); List programInstances = new ArrayList(); @@ -115,32 +119,23 @@ programInstances.addAll( programInstanceService.getProgramInstances( program, organisationUnit ) ); } - Calendar expiredDate = Calendar.getInstance(); - + Calendar expiredDate = Calendar.getInstance(); + for ( ProgramInstance programInstance : programInstances ) { Set programStageInstances = programInstance.getProgramStageInstances(); - Inner: for ( ProgramStageInstance programStageInstance : programStageInstances ) + for ( ProgramStageInstance programStageInstance : programStageInstances ) { if(!programStageInstance.isCompleted()){ - expiredDate.setTime( DateUtils.getDateAfterAddition( programStageInstance.getDueDate(), - programStageInstance.getProgramInstance().getProgram().getMaxDaysAllowedInputData() ) ); - if ( programStageInstance.getDueDate().getTime() < time && expiredDate.getTimeInMillis() > time ) - { - Activity activity = new Activity(); - activity.setBeneficiary( programInstance.getPatient() ); - activity.setTask( programStageInstance ); - activity.setDueDate( programStageInstance.getDueDate() ); - items.add( activity ); - } - if ( programStageInstance.getDueDate().getTime() > time && expiredDate.getTimeInMillis() > time ) - { - Activity activity = new Activity(); - activity.setBeneficiary( programInstance.getPatient() ); - activity.setTask( programStageInstance ); - activity.setDueDate( programStageInstance.getDueDate() ); - items.add( activity ); - break Inner; + expiredDate.setTime( DateUtils.getDateAfterAddition( programStageInstance.getDueDate(), programStageInstance.getProgramInstance().getProgram().getMaxDaysAllowedInputData() ) ); + + if ( programStageInstance.getDueDate().getTime() <= time && expiredDate.getTimeInMillis() > time ) + { + Activity activity = new Activity(); + activity.setBeneficiary( programInstance.getPatient() ); + activity.setTask( programStageInstance ); + activity.setDueDate( programStageInstance.getDueDate() ); + items.add( activity ); } } }