Mercurial > dive4elements > river
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(); }