From 8af33ec8465e181969f05066828ea68a26c8b413 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?= Date: Tue, 18 Jan 2011 20:26:11 +0100 Subject: [PATCH] Remove parameter Use field insteand. This reduces the possibility of introducing errors by calling the method with the wrong picker. FEA: ItEr68OTS05IntroducionLimiteSobreasignacionCalendarios --- .../web/common/components/CapacityPicker.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/common/components/CapacityPicker.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/common/components/CapacityPicker.java index 6e8bc5f56..f09b62825 100644 --- a/navalplanner-webapp/src/main/java/org/navalplanner/web/common/components/CapacityPicker.java +++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/common/components/CapacityPicker.java @@ -111,16 +111,16 @@ public class CapacityPicker { public void set(Boolean value) { updateCapacity(currentCapacity.overAssignableWithoutLimit(BooleanUtils .isTrue(value))); - updateExtraEffortDisability(extraEffortPicker); + updateExtraEffortDisability(); } }); - updateExtraEffortDisability(extraEffortPicker); + updateExtraEffortDisability(); } - private void updateExtraEffortDisability( - EffortDurationPicker extraHoursPicker) { - extraHoursPicker.setDisabled(currentCapacity.isOverAssignableWithoutLimit()); + private void updateExtraEffortDisability() { + extraEffortPicker.setDisabled(currentCapacity + .isOverAssignableWithoutLimit()); } private void updateCapacity(Capacity newCapacity) { @@ -140,7 +140,7 @@ public class CapacityPicker { standardEffortPicker.setValue(capacity.getStandardEffort()); extraEffortPicker.setValue(capacity.getAllowedExtraEffort()); currentCapacity = capacity; - updateExtraEffortDisability(extraEffortPicker); + updateExtraEffortDisability(); } }