Separated modifying Lookup methods into LookupManager, which is only

present in a server process. This stops clients trying to write to the
directory without relying on their permissions.

3 files changed, 6 insertions, 5 deletions

diff --git a/src/main/java/com/c2kernel/entity/imports/ImportAgent.java b/src/main/java/com/c2kernel/entity/imports/ImportAgent.java
index 26e3325..2aa6533 100644
--- a/src/main/java/com/c2kernel/entity/imports/ImportAgent.java
+++ b/src/main/java/com/c2kernel/entity/imports/ImportAgent.java
@@ -36,7 +36,7 @@ public class ImportAgent extends ModuleImport implements java.io.Serializable {
         newAgent.setAgentName(name);

         newAgent.setPassword(password);

         ActiveEntity newAgentEnt = (ActiveEntity)Gateway.getCorbaServer().createEntity(newAgent);

-        Gateway.getLookup().add(newAgent);

+        Gateway.getLookupManager().add(newAgent);

         // assemble properties

         properties.add(new com.c2kernel.property.Property("Name", name, true));

         properties.add(new com.c2kernel.property.Property("Type", "Agent", false));

diff --git a/src/main/java/com/c2kernel/entity/imports/ImportItem.java b/src/main/java/com/c2kernel/entity/imports/ImportItem.java
index a27d88d..3847fbf 100644
--- a/src/main/java/com/c2kernel/entity/imports/ImportItem.java
+++ b/src/main/java/com/c2kernel/entity/imports/ImportItem.java
@@ -79,7 +79,7 @@ public class ImportItem extends ModuleImport {
 	        // create item

 	        entPath = Gateway.getNextKeyManager().generateNextEntityKey();

 	        newItem = (TraceableEntity)Gateway.getCorbaServer().createEntity(entPath);

-	        Gateway.getLookup().add(entPath);

+	        Gateway.getLookupManager().add(entPath);

         }

         

         // set the name property

@@ -181,7 +181,7 @@ public class ImportItem extends ModuleImport {
         // register domain path (before collections in case of recursive collections)

         if (!domPath.exists()) {

         	domPath.setEntity(entPath);

-        	Gateway.getLookup().add(domPath);

+        	Gateway.getLookupManager().add(domPath);

         }

     }

 }

diff --git a/src/main/java/com/c2kernel/entity/imports/ImportRole.java b/src/main/java/com/c2kernel/entity/imports/ImportRole.java
index 8313c24..5749b06 100644
--- a/src/main/java/com/c2kernel/entity/imports/ImportRole.java
+++ b/src/main/java/com/c2kernel/entity/imports/ImportRole.java
@@ -1,5 +1,6 @@
 package com.c2kernel.entity.imports;

 

+import com.c2kernel.common.CannotManageException;

 import com.c2kernel.common.ObjectAlreadyExistsException;

 import com.c2kernel.common.ObjectCannotBeUpdated;

 import com.c2kernel.process.Gateway;

@@ -12,8 +13,8 @@ public class ImportRole extends ModuleImport {
 	public ImportRole() {

 	}

 	

-	public void create(int agentId) throws ObjectAlreadyExistsException, ObjectCannotBeUpdated {

-		Gateway.getLookup().createRole(name, jobList);

+	public void create(int agentId) throws ObjectAlreadyExistsException, ObjectCannotBeUpdated, CannotManageException {

+		Gateway.getLookupManager().createRole(name, jobList);

 	}

 

 }