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

[Bug #962] Refactor fillVariables

Browse source 
FEA: ItEr73S04BugFixing
This commit is contained in:
Diego Pino Garcia 2011-04-02 21:46:11 +02:00
parent 0efbdff9ba
commit 1663f5d627
1 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
navalplanner-webapp/src/main/java/org/navalplanner/web/orders/ManageOrderElementAdvancesModel.java
View file

@ -147,7 +147,7 @@ public class ManageOrderElementAdvancesModel implements
public void refreshChangesFromOrderElement() {
List<AdvanceAssignment> listAdvanceAssignmentsCopy = new ArrayList<AdvanceAssignment>(
listAdvanceAssignments);
fillVariables();
fillAdvanceAssignmentList();
for (AdvanceAssignment advance : listAdvanceAssignmentsCopy) {
if ((!listAdvanceAssignments.contains(advance))
&& (advance instanceof DirectAdvanceAssignment)
@ -224,7 +224,7 @@ public class ManageOrderElementAdvancesModel implements
loadAdvanceTypes();
reattachmentOrderElement();
forceLoadAdvanceAssignmentsAndMeasurements();
fillVariables();
fillAdvanceAssignmentList();
}
}
@ -264,17 +264,16 @@ public class ManageOrderElementAdvancesModel implements
orderElementDAO.reattach(orderElement);
}
private void fillVariables() {
this.listAdvanceAssignments = new ArrayList<AdvanceAssignment>();
private void fillAdvanceAssignmentList() {
listAdvanceAssignments = new ArrayList<AdvanceAssignment>();
for (DirectAdvanceAssignment each : this.orderElement
for (DirectAdvanceAssignment each : orderElement
.getDirectAdvanceAssignments()) {
this.listAdvanceAssignments.add(each);
listAdvanceAssignments.add(each);
}
for (IndirectAdvanceAssignment each : orderElement
.getIndirectAdvanceAssignments()) {
this.listAdvanceAssignments.add(each);
listAdvanceAssignments.add(each);
}
}