Add generic copy/remove to/from collection(s)

[gpl/argeo-slc.git] / runtime / org.argeo.slc.server / src / main / java / org / argeo / slc / web / mvc / result / RemoveResultFromCollectionController.java

diff --git a/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/web/mvc/result/RemoveResultFromCollectionController.java b/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/web/mvc/result/RemoveResultFromCollectionController.java
index 3f4f52e485b41963f83d370e17c06b9eb82e95cd..a9709dc234bf2e0e0734c47c8aa45835ac8e9fc2 100644 (file)
--- a/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/web/mvc/result/RemoveResultFromCollectionController.java
+++ b/runtime/org.argeo.slc.server/src/main/java/org/argeo/slc/web/mvc/result/RemoveResultFromCollectionController.java
@@ -1,5 +1,8 @@
 package org.argeo.slc.web.mvc.result;\r
 \r
+import java.util.ArrayList;\r
+import java.util.List;\r
+\r
 import javax.servlet.http.HttpServletRequest;\r
 import javax.servlet.http.HttpServletResponse;\r
 \r
@@ -55,8 +58,9 @@ public class RemoveResultFromCollectionController extends
                        for (String attrName : attrNames) {\r
                                String attrPattern = attrPatterns[index];// safe: checked above\r
 \r
-                               for (TreeTestResult treeTestResult : sourceCollection\r
-                                               .getResults()) {\r
+                               List<TreeTestResult> results = new ArrayList<TreeTestResult>(\r
+                                               sourceCollection.getResults());\r
+                               for (TreeTestResult treeTestResult : results) {\r
                                        if (PatternMatchUtils.simpleMatch(attrPattern,\r
                                                        treeTestResult.getAttributes().get(attrName))) {\r
                                                testManagerService.removeResultFromCollection(\r
@@ -71,8 +75,9 @@ public class RemoveResultFromCollectionController extends
                                // TODO: optimize\r
                                TreeTestResultCollection sourceCollection = testResultCollectionDao\r
                                                .getTestResultCollection(collectionId);\r
-                               for (TreeTestResult treeTestResult : sourceCollection\r
-                                               .getResults()) {\r
+                               List<TreeTestResult> results = new ArrayList<TreeTestResult>(\r
+                                               sourceCollection.getResults());\r
+                               for (TreeTestResult treeTestResult : results) {\r
                                        testManagerService.removeResultFromCollection(collectionId,\r
                                                        treeTestResult.getUuid());\r
                                }\r