RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[Bug #1319] Change division method of EffortDuration

Browse source 
Previous method was rounding to zero instead of return a fraction.

FEA: ItEr75S04BugFixing
This commit is contained in:
Manuel Rego Casasnovas 2011-12-29 08:22:45 +01:00
parent 7ab234b049
commit e3136719f7
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
libreplan-webapp/src/main/java/org/libreplan/web/planner/TaskElementAdapter.java
View file

@ -565,7 +565,10 @@ public class TaskElementAdapter {
return getBeginDate();
}
}
BigDecimal percentage = new BigDecimal(assignedEffort.divideBy(effort));
BigDecimal percentage = new BigDecimal(assignedEffort
.divivedBy(effort).doubleValue()).setScale(2,
RoundingMode.HALF_UP);
result = calculateLimitDateByPercentage(percentage);
}
@ -584,8 +587,6 @@ public class TaskElementAdapter {
.getSumChargedEffort() != null ? orderElement
.getSumChargedEffort().getTotalChargedEffort()
: EffortDuration.zero();
BigDecimal assignedHours = totalChargedEffort
.toHoursAsDecimalWithScale(2);
EffortDuration estimatedEffort = taskElement.getSumOfAssignedEffort();
@ -595,7 +596,9 @@ public class TaskElementAdapter {
return BigDecimal.ZERO;
}
}
return new BigDecimal(totalChargedEffort.divideBy(estimatedEffort));
return new BigDecimal(totalChargedEffort.divivedBy(
estimatedEffort).doubleValue()).setScale(2,
RoundingMode.HALF_UP);
}
@Override