From 081443ece10b944e2c0461ed738e752e90e1599d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?=
 <ogonzalez@igalia.com>
Date: Wed, 23 Jun 2010 18:34:02 +0200
Subject: [PATCH] ItEr60S04ValidacionEProbasFuncionaisItEr59S04: [Bug #519] Fix
 bug.

Changing the set instance was interfering with delete-orphan. The
initial persistent set must be kept in order to delete-orphan keep on
working. If a new set is assigned hibernate loses track of the
removals done and doesn't delete the orphans. Instead of assigning to
new set, the advanceMeasurements set is reset to the new contents.
---
 .../business/advance/entities/DirectAdvanceAssignment.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/DirectAdvanceAssignment.java b/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/DirectAdvanceAssignment.java
index 5c400e950..3a6c81021 100644
--- a/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/DirectAdvanceAssignment.java
+++ b/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/DirectAdvanceAssignment.java
@@ -98,8 +98,8 @@ public class DirectAdvanceAssignment extends AdvanceAssignment {
 
     public void setAdvanceMeasurements(
             SortedSet<AdvanceMeasurement> advanceMeasurements) {
-        this.advanceMeasurements = new TreeSet<AdvanceMeasurement>(
-                advanceMeasurements);
+        this.advanceMeasurements.clear();
+        this.advanceMeasurements.addAll(advanceMeasurements);
     }
 
     public AdvanceMeasurement getLastAdvanceMeasurement() {