Changeset 28161 for main/trunk

Timestamp:

2013-08-27T11:26:01+12:00 (11 years ago)

Author:

sjm84

Message:

Fixed a silly mistake that was causing non-admins to not be able to change their account settings

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/Authentication.java (modified) (4 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/Authentication.java

@@ -142 +142 @@
     public void cleanUp()
     {
-
         super.cleanUp();
 
@@ -162 +161 @@
             _derbyWrapperDoneForcedShutdown = true;
         }
-
     }
 
@@ -249 +247 @@
     }
 
-    protected void addCustomParams(String service, Element param_list, String lang)
-    {
-    }
-
-    protected void createParameter(String name, Element param_list, String lang)
-    {
-    }
-
     protected Element processGetUserInformation(Element request)
     {
@@ -332 +322 @@
         result.setAttribute(GSXML.FROM_ATT, AUTHENTICATION_SERVICE);
         result.setAttribute(GSXML.TYPE_ATT, GSXML.REQUEST_TYPE_PROCESS);
-
-        String[] userGroups = (new UserContext(request)).getGroups();
-
-        boolean found = false;
-        for (String group : userGroups)
-        {
-            if (group.equals("administrator"))
-            {
-                found = true;
-            }
-        }
-
-        if (!found)
-        {
-            GSXML.addError(this.doc, result, "This user does not have the required permissions to perform this action.");
-            return result;
-        }
 
         // Create an Authentication node put into the result