diff --git a/src/main/java/hudson/plugins/script_realm/ScriptSecurityRealm.java b/src/main/java/hudson/plugins/script_realm/ScriptSecurityRealm.java index a35d5fc..dff9c84 100644 --- a/src/main/java/hudson/plugins/script_realm/ScriptSecurityRealm.java +++ b/src/main/java/hudson/plugins/script_realm/ScriptSecurityRealm.java @@ -252,6 +252,8 @@ public class ScriptSecurityRealm extends AbstractPasswordBasedSecurityRealm { fullname = unr.findNameFor(user); break; } + } + if ( fullname == null ) { LOGGER.log(Level.WARNING,"Resolver {0} not found : name not updated",new Object[]{nameResolver}); } } @@ -279,7 +281,9 @@ public class ScriptSecurityRealm extends AbstractPasswordBasedSecurityRealm { email = mar.findMailAddressFor(user); break; } - LOGGER.log(Level.WARNING,"Resolver {0} not found : e-mail not updated",new Object[]{emailResolver}); + } + if ( email == null ) { + LOGGER.log(Level.WARNING,"Resolver {0} not found : e-mail not updated",new Object[]{emailResolver}); } } if ( StringUtils.isNotBlank(email) ) { diff --git a/src/test/resources/DeleteAllUsers.groovy b/src/test/resources/DeleteAllUsers.groovy index a826c2d..1300a31 100644 --- a/src/test/resources/DeleteAllUsers.groovy +++ b/src/test/resources/DeleteAllUsers.groovy @@ -8,10 +8,10 @@ import jenkins.security.* for ( userInfo in Jenkins.instance.getPeople().users ) { User user = userInfo.getUser() - print("Deleting user "+user) + println("Deleting user "+user) user.delete() } for ( userInfo in Jenkins.instance.getPeople().users ) { - println(userInfo.getUser()+"\n") + println(userInfo.getUser()) } \ No newline at end of file