From 85aa3a5834bf381b603f63e9e769920c368fa53d Mon Sep 17 00:00:00 2001
From: rsanchez <rsanchez@curisit.net>
Date: Mon, 03 Aug 2015 09:58:35 +0000
Subject: [PATCH] #0 fix - Fixed metadata management, now based on "key" to apply modifications
---
securis/src/main/java/net/curisit/securis/services/PackResource.java | 17 +++++++++++++++--
1 files changed, 15 insertions(+), 2 deletions(-)
diff --git a/securis/src/main/java/net/curisit/securis/services/PackResource.java b/securis/src/main/java/net/curisit/securis/services/PackResource.java
index 558b5a4..2c677b3 100644
--- a/securis/src/main/java/net/curisit/securis/services/PackResource.java
+++ b/securis/src/main/java/net/curisit/securis/services/PackResource.java
@@ -2,6 +2,7 @@
import java.security.Principal;
import java.util.Date;
+import java.util.HashSet;
import java.util.List;
import java.util.Set;
@@ -236,6 +237,16 @@
pack.setLicenseType(lt);
}
+ private Set<String> getMdKeys(Set<PackMetadata> mds) {
+ Set<String> ids = new HashSet<String>();
+ if (mds != null) {
+ for (PackMetadata md : mds) {
+ ids.add(md.getKey());
+ }
+ }
+ return ids;
+ }
+
@PUT
@POST
@Path("/{packId}")
@@ -271,16 +282,18 @@
currentPack.setRenewValidPeriod(pack.getRenewValidPeriod());
Set<PackMetadata> newMD = pack.getMetadata();
+ Set<String> newMdKeys = getMdKeys(newMD);
for (PackMetadata currentMd : currentPack.getMetadata()) {
- if (newMD == null || !newMD.contains(currentMd)) {
+ if (!newMdKeys.contains(currentMd.getKey())) {
em.remove(currentMd);
}
}
if (newMD != null) {
Set<PackMetadata> oldMD = currentPack.getMetadata();
+ Set<String> oldMdKeys = getMdKeys(newMD);
for (PackMetadata md : newMD) {
- if (oldMD.contains(md)) {
+ if (oldMdKeys.contains(md.getKey())) {
em.merge(md);
} else {
md.setPack(currentPack);
--
Gitblit v1.3.2