From c4cb0e8aa13308645e25b4062affb32090f92870 Mon Sep 17 00:00:00 2001 From: Manuel Rego Casasnovas Date: Mon, 16 Nov 2009 11:44:34 +0100 Subject: [PATCH] ItEr34S13CUAsignacionRecursosEspecificosAPlanificacionItEr33S15: Fixed bug when there are more than one resource. --- .../allocation/AdvancedAllocationController.java | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/allocation/AdvancedAllocationController.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/allocation/AdvancedAllocationController.java index 4c2bab22e..e7e8a9dfa 100644 --- a/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/allocation/AdvancedAllocationController.java +++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/allocation/AdvancedAllocationController.java @@ -741,7 +741,8 @@ class Row { appendListener(assignmentFunctionsCombo); hbox.appendChild(assignmentFunctionsCombo); - hbox.appendChild(getAssignmentFunctionsConfigureButton()); + hbox + .appendChild(getAssignmentFunctionsConfigureButton(assignmentFunctionsCombo)); return hbox; } @@ -797,7 +798,6 @@ class Row { .getAssignmentFunction(); Combobox combobox = new Combobox(); - combobox.setId("assigment_functions_combo"); Comboitem comboitem = new Comboitem(_("None")); comboitem.setValue(null); @@ -819,17 +819,16 @@ class Row { return combobox; } - private Button getAssignmentFunctionsConfigureButton() { + private Button getAssignmentFunctionsConfigureButton( + final Combobox assignmentFunctionsCombo) { final Button button = new Button(_("Configure")); button.addEventListener(Events.ON_CLICK, new EventListener() { @Override public void onEvent(Event event) throws Exception { - Combobox combobox = (Combobox) button - .getFellow("assigment_functions_combo"); - Class assignmentFunction = (Class) combobox.getSelectedItem() - .getValue(); + Class assignmentFunction = (Class) assignmentFunctionsCombo + .getSelectedItem().getValue(); if (assignmentFunction == null) { Messagebox .show(