diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/chart/ChartFiller.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/chart/ChartFiller.java index 239c6a195..12d45a904 100644 --- a/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/chart/ChartFiller.java +++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/chart/ChartFiller.java @@ -406,25 +406,17 @@ public abstract class ChartFiller implements IChartFiller { SortedMap> map = new TreeMap>(); for (DayAssignment dayAssignment : dayAssignments) { - LocalDate day = dayAssignment.getDay(); + final LocalDate day = dayAssignment.getDay(); + final int dayAssignmentHours = dayAssignment.getHours(); + Resource resource = dayAssignment.getResource(); if (map.get(day) == null) { - HashMap resourcesMap = new HashMap(); - resourcesMap.put(dayAssignment.getResource(), dayAssignment - .getHours()); - map.put(day, resourcesMap); - } else { - if (map.get(day).get(dayAssignment.getResource()) == null) { - map.get(day).put(dayAssignment.getResource(), - dayAssignment.getHours()); - } else { - Integer hours = map.get(day).get( - dayAssignment.getResource()); - hours += dayAssignment.getHours(); - map.get(day).put(dayAssignment.getResource(), hours); - } + map.put(day, new HashMap()); } + Map forDay = map.get(day); + Integer previousHours = forDay.get(resource); + previousHours = previousHours != null ? previousHours : 0; + forDay.put(dayAssignment.getResource(), previousHours + dayAssignmentHours); } - return map; }