diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/GenericResourceAllocation.java b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/GenericResourceAllocation.java index f5dff3046..acb1a4793 100644 --- a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/GenericResourceAllocation.java +++ b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/GenericResourceAllocation.java @@ -130,7 +130,7 @@ public class GenericResourceAllocation extends ResourceAllocation { } Integer workableHours = SameWorkHoursEveryDay .getDefaultWorkingDay().getWorkableHours(day); - return resourcesPerDay.getAmount() * workableHours; + return resourcesPerDay.asHoursGivenResourceWorkingDayOf(workableHours); } private boolean isTaskFreeDay(LocalDate day) { diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourcePerDayUnit.java b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourcePerDayUnit.java index 1fc372c81..fdc0f62f0 100644 --- a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourcePerDayUnit.java +++ b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourcePerDayUnit.java @@ -8,7 +8,6 @@ public class ResourcePerDayUnit { public static ResourcePerDayUnit amount(int amount) { return new ResourcePerDayUnit(amount); - } private ResourcePerDayUnit(int amount) { @@ -19,4 +18,9 @@ public class ResourcePerDayUnit { public int getAmount() { return amount; } + + public int asHoursGivenResourceWorkingDayOf( + Integer resourceWorkingDayHours) { + return getAmount() * resourceWorkingDayHours; + } } diff --git a/navalplanner-business/src/test/java/org/navalplanner/business/test/planner/entities/ResourcePerDayUnitTest.java b/navalplanner-business/src/test/java/org/navalplanner/business/test/planner/entities/ResourcePerDayUnitTest.java index 848329084..09451fffd 100644 --- a/navalplanner-business/src/test/java/org/navalplanner/business/test/planner/entities/ResourcePerDayUnitTest.java +++ b/navalplanner-business/src/test/java/org/navalplanner/business/test/planner/entities/ResourcePerDayUnitTest.java @@ -19,4 +19,10 @@ public class ResourcePerDayUnitTest { assertThat(units.getAmount(), equalTo(2)); } + @Test + public void canBeConvertedToHoursGivenTheWorkingDayHours() { + ResourcePerDayUnit units = ResourcePerDayUnit.amount(2); + assertThat(units.asHoursGivenResourceWorkingDayOf(8), equalTo(16)); + } + }