From ade2fea3f870a142f2553fc6d4f79f6dfab35522 Mon Sep 17 00:00:00 2001
From: brinn <brinn>
Date: Sun, 17 Mar 2013 11:29:56 +0000
Subject: [PATCH] Remove TODO.

SVN: 28625
---
 .../generic/server/authorization/DefaultAccessController.java   | 2 --
 1 file changed, 2 deletions(-)

diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/authorization/DefaultAccessController.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/authorization/DefaultAccessController.java
index 7de02becf9e..0d9c4165fcf 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/authorization/DefaultAccessController.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/authorization/DefaultAccessController.java
@@ -157,8 +157,6 @@ public final class DefaultAccessController implements IAccessController
             final Set<RoleWithHierarchy> methodRoles = getMethodRoles(method);
             if (methodRoles.size() == 0)
             {
-                // TODO 2008-08-07, Tomasz Pylak: why this is not a programming error? What a user
-                // can do if a programmer does not put an authorization annotation for a method?
                 final String msg =
                         String.format(METHOD_ROLES_NOT_FOUND_TEMPLATE,
                                 MethodUtils.describeMethod(method));
-- 
GitLab