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

[Bug #1325] Fix issue remove criteria from configuration unit

Browse source 
FEA: ItEr76S04BugFixing
This commit is contained in:
Manuel Rego Casasnovas 2012-01-04 11:29:23 +01:00
parent d7e2625b18
commit 031a23335d
1 changed files with 4 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
libreplan-webapp/src/main/java/org/libreplan/web/resources/machine/MachineConfigurationController.java
View file

@ -28,7 +28,6 @@ import java.util.List;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.libreplan.business.common.exceptions.InstanceNotFoundException;
import org.libreplan.business.resources.entities.Criterion;
import org.libreplan.business.resources.entities.CriterionWithItsType;
import org.libreplan.business.resources.entities.MachineWorkerAssignment;
@ -50,7 +49,6 @@ import org.zkoss.zul.Listbox;
import org.zkoss.zul.Listitem;
import org.zkoss.zul.Row;
import org.zkoss.zul.Rows;
import org.zkoss.zul.Textbox;
import org.zkoss.zul.api.Bandbox;
@ -197,15 +195,10 @@ public class MachineConfigurationController extends GenericForwardComposer {
public void deleteRequiredCriterion(Criterion criterion, Rows component) {
String unitString = ((Textbox) component.getParent()
.getPreviousSibling()).getValue();
try {
MachineWorkersConfigurationUnit unit = machineModel
.getConfigurationUnitById(Long.valueOf(unitString));
unit.removeRequiredCriterion(criterion);
} catch (InstanceNotFoundException e) {
LOG.error("Configuration unit not found", e);
}
MachineWorkersConfigurationUnit unit = (MachineWorkersConfigurationUnit) ((Row) component
.getParent().getParent().getParent().getParent().getParent())
.getValue();
unit.removeRequiredCriterion(criterion);
Util.reloadBindings(component.getParent().getParent());
}