diff flys-client/src/main/java/de/intevation/flys/client/server/auth/UserClient.java @ 3695:5e4d91304484

Be more defensive about fetchimg users from session. flys-client/trunk@5394 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author Sascha L. Teichmann <sascha.teichmann@intevation.de>
date Fri, 07 Sep 2012 15:58:10 +0000
parents 2c306a3dacef
children a050cb5f303f
line wrap: on
line diff
--- a/flys-client/src/main/java/de/intevation/flys/client/server/auth/UserClient.java	Fri Sep 07 13:15:28 2012 +0000
+++ b/flys-client/src/main/java/de/intevation/flys/client/server/auth/UserClient.java	Fri Sep 07 15:58:10 2012 +0000
@@ -31,11 +31,12 @@
     }
 
     public boolean userExists(User user) throws ConnectionException {
-        Element data = this.findUser(user);
-
         if (user == null) {
             return false;
         }
+
+        Element data = this.findUser(user);
+
         String XPATH_USERACCOUNT = "/art:user/art:account/@art:name";
 
         String account = XMLUtils.xpathString(
@@ -127,11 +128,19 @@
         action.appendChild(account);
         document.appendChild(action);
 
-        logger.debug("Find user request xml: " + XMLUtils.toString(document));
+        boolean debug = logger.isDebugEnabled();
+
+        if (debug) {
+            logger.debug("Find user request xml: " +
+                XMLUtils.toString(document));
+        }
 
         Document resp = client.findUser(document);
 
-        logger.debug("Find user request response xml: " + XMLUtils.toString(resp));
+        if (debug) {
+            logger.debug("Find user request response xml: " +
+                XMLUtils.toString(resp));
+        }
 
         return resp.getDocumentElement();
     }

http://dive4elements.wald.intevation.org