Introduced static method ItemPath.fromUUIDString and made the UUID

constructor protected to better handle ItemPath and AgentPath
construction with String UUIDs, throwing the right exceptions.

5 files changed, 7 insertions, 11 deletions

diff --git a/src/main/java/com/c2kernel/lifecycle/instance/Workflow.java b/src/main/java/com/c2kernel/lifecycle/instance/Workflow.java
index 33c80a4..304c604 100644
--- a/src/main/java/com/c2kernel/lifecycle/instance/Workflow.java
+++ b/src/main/java/com/c2kernel/lifecycle/instance/Workflow.java
@@ -1,6 +1,5 @@
 package com.c2kernel.lifecycle.instance;

 import java.util.ArrayList;

-import java.util.UUID;

 

 import com.c2kernel.common.AccessRightsException;

 import com.c2kernel.common.InvalidDataException;

@@ -16,6 +15,7 @@ import com.c2kernel.graph.model.TypeNameAndConstructionInfo;
 import com.c2kernel.lifecycle.instance.predefined.PredefinedStepContainer;

 import com.c2kernel.lookup.AgentPath;

 import com.c2kernel.lookup.InvalidAgentPathException;

+import com.c2kernel.lookup.InvalidItemPathException;

 import com.c2kernel.lookup.ItemPath;

 import com.c2kernel.persistency.ClusterStorage;

 import com.c2kernel.utils.Language;

@@ -199,9 +199,9 @@ public class Workflow extends CompositeActivity implements C2KLocalObject
 		this.itemPath = itemPath;

 	}

 	

-    public void setItemUUID( String uuid )

+    public void setItemUUID( String uuid ) throws InvalidItemPathException

     {

-    	setItemPath(new ItemPath(UUID.fromString(uuid)));

+    	setItemPath(ItemPath.fromUUIDString(uuid));

     }

     

     public String getItemUUID() {

diff --git a/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddMemberToCollection.java b/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddMemberToCollection.java
index 889ee80..76b6adf 100644
--- a/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddMemberToCollection.java
+++ b/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddMemberToCollection.java
@@ -13,7 +13,6 @@ package com.c2kernel.lifecycle.instance.predefined;
 

 

 import java.util.Arrays;

-import java.util.UUID;

 

 import com.c2kernel.collection.Dependency;

 import com.c2kernel.collection.MembershipException;

@@ -65,7 +64,7 @@ public class AddMemberToCollection extends PredefinedStep
         if (Logger.doLog(3)) Logger.msg(3, "AddMemberToCollection: called by "+agent+" on "+item+" with parameters "+Arrays.toString(params));

         try {

             collName = params[0];

-            newChild = new ItemPath(UUID.fromString(params[1]));

+            newChild = ItemPath.fromUUIDString(params[1]);

             if (params.length > 2)

             	props = (CastorHashMap)Gateway.getMarshaller().unmarshall(params[2]);

             

diff --git a/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddNewSlot.java b/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddNewSlot.java
index 257c644..58b1263 100644
--- a/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddNewSlot.java
+++ b/src/main/java/com/c2kernel/lifecycle/instance/predefined/AddNewSlot.java
@@ -13,7 +13,6 @@ package com.c2kernel.lifecycle.instance.predefined;
 

 

 import java.util.Arrays;

-import java.util.UUID;

 

 import com.c2kernel.collection.Aggregation;

 import com.c2kernel.common.InvalidDataException;

@@ -67,7 +66,7 @@ public class AddNewSlot extends PredefinedStep
 

         try {

             collName = params[0];

-            if (params.length > 1  && params[1].length() > 0) descKey = new ItemPath(UUID.fromString(params[1]));

+            if (params.length > 1  && params[1].length() > 0) descKey = ItemPath.fromUUIDString(params[1]);

         } catch (Exception e) {

             throw new InvalidDataException("AddNewSlot: Invalid parameters "+Arrays.toString(params), "");

         }

diff --git a/src/main/java/com/c2kernel/lifecycle/instance/predefined/AssignItemToSlot.java b/src/main/java/com/c2kernel/lifecycle/instance/predefined/AssignItemToSlot.java
index 724e974..20ad473 100644
--- a/src/main/java/com/c2kernel/lifecycle/instance/predefined/AssignItemToSlot.java
+++ b/src/main/java/com/c2kernel/lifecycle/instance/predefined/AssignItemToSlot.java
@@ -13,7 +13,6 @@ package com.c2kernel.lifecycle.instance.predefined;
 

 

 import java.util.Arrays;

-import java.util.UUID;

 

 import com.c2kernel.collection.Aggregation;

 import com.c2kernel.collection.AggregationMember;

@@ -66,7 +65,7 @@ public class AssignItemToSlot extends PredefinedStep
         try {

             collName = params[0];

             slotNo = Integer.parseInt(params[1]);

-            childItem = new ItemPath(UUID.fromString(params[2]));

+            childItem = ItemPath.fromUUIDString(params[2]);

         } catch (Exception e) {

             throw new InvalidDataException("AssignItemToSlot: Invalid parameters "+Arrays.toString(params), "");

         }

diff --git a/src/main/java/com/c2kernel/lifecycle/instance/predefined/RemoveSlotFromCollection.java b/src/main/java/com/c2kernel/lifecycle/instance/predefined/RemoveSlotFromCollection.java
index 7caad61..26bf104 100644
--- a/src/main/java/com/c2kernel/lifecycle/instance/predefined/RemoveSlotFromCollection.java
+++ b/src/main/java/com/c2kernel/lifecycle/instance/predefined/RemoveSlotFromCollection.java
@@ -13,7 +13,6 @@ package com.c2kernel.lifecycle.instance.predefined;
 

 

 import java.util.Arrays;

-import java.util.UUID;

 

 import com.c2kernel.collection.Collection;

 import com.c2kernel.collection.CollectionMember;

@@ -65,7 +64,7 @@ public class RemoveSlotFromCollection extends PredefinedStep
         try {

             collName = params[0];

             if (params.length>1 && params[1].length()>0) slotNo = Integer.parseInt(params[1]);

-            if (params.length>2 && params[2].length()>0) currentChild = new ItemPath(UUID.fromString(params[2]));

+            if (params.length>2 && params[2].length()>0) currentChild = ItemPath.fromUUIDString(params[2]);

         } catch (Exception e) {

             throw new InvalidDataException("RemoveSlotFromCollection: Invalid parameters "+Arrays.toString(params), "");

         }