From 2aa21c8fb035d34156b014086d36c75947ba933d Mon Sep 17 00:00:00 2001 From: Richard Millet Date: Wed, 3 Nov 2010 21:51:55 +0000 Subject: [PATCH] CSPACE-3150: Removed debugging print statements. --- .../storage/PermissionDocumentHandler.java | 10 +++------- .../services/authorization/URIResourceImpl.java | 9 ++------- 2 files changed, 5 insertions(+), 14 deletions(-) diff --git a/services/authorization-mgt/service/src/main/java/org/collectionspace/services/authorization/storage/PermissionDocumentHandler.java b/services/authorization-mgt/service/src/main/java/org/collectionspace/services/authorization/storage/PermissionDocumentHandler.java index 29503a36a..9af981151 100644 --- a/services/authorization-mgt/service/src/main/java/org/collectionspace/services/authorization/storage/PermissionDocumentHandler.java +++ b/services/authorization-mgt/service/src/main/java/org/collectionspace/services/authorization/storage/PermissionDocumentHandler.java @@ -62,10 +62,6 @@ public class PermissionDocumentHandler private PermissionsList permissionsList; public CSpaceAction getAction(ActionType action) { - System.out.println("Hello, world? " + action.name()); - System.out.println("Hello, world? " + ActionType.CREATE.name()); - - try { if (ActionType.CREATE.name().equals(action.name())) { return CSpaceAction.CREATE; } else if (ActionType.READ.equals(action)) { @@ -83,9 +79,9 @@ public class PermissionDocumentHandler } else if (ActionType.STOP.equals(action)) { return CSpaceAction.STOP; } - } catch (Exception x) { - x.printStackTrace(); - } + // + // We could not find a match, so we need to throw an exception. + // throw new IllegalArgumentException("action = " + action.toString()); } diff --git a/services/authorization/service/src/main/java/org/collectionspace/services/authorization/URIResourceImpl.java b/services/authorization/service/src/main/java/org/collectionspace/services/authorization/URIResourceImpl.java index 5996bb755..04b40aae7 100644 --- a/services/authorization/service/src/main/java/org/collectionspace/services/authorization/URIResourceImpl.java +++ b/services/authorization/service/src/main/java/org/collectionspace/services/authorization/URIResourceImpl.java @@ -124,11 +124,9 @@ public class URIResourceImpl extends CSpaceResourceImpl { /* * Map a Permission ActionType to a CSpaceAction */ + //FIXME: This method is duplicated in PermissionDocumentHandler.java class. The class loader was having + //trouble with the ActionType class file. Not sure why? public static CSpaceAction getAction(ActionType action) { - System.out.println("Hello, world? " + action.name()); - System.out.println("Hello, world? " + ActionType.CREATE.name()); - - try { if (ActionType.CREATE.name().equals(action.name())) { return CSpaceAction.CREATE; } else if (ActionType.READ.equals(action)) { @@ -146,9 +144,6 @@ public class URIResourceImpl extends CSpaceResourceImpl { } else if (ActionType.STOP.equals(action)) { return CSpaceAction.STOP; } - } catch (Exception x) { - x.printStackTrace(); - } throw new IllegalArgumentException("action = " + action.toString()); } -- 2.47.3