From: Richard Millet Date: Wed, 3 Nov 2010 21:51:55 +0000 (+0000) Subject: CSPACE-3150: Removed debugging print statements. X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=2aa21c8fb035d34156b014086d36c75947ba933d;p=tmp%2Fjakarta-migration.git CSPACE-3150: Removed debugging print statements. --- 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()); }