Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
use dedicated, negated check instead of manually negating positive check

Co-authored-by: Arved Solth <[email protected]>
  • Loading branch information
gerlingsm and solth authored Sep 2, 2024
1 parent 802d1d3 commit dee50b0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Kitodo/src/main/java/org/kitodo/production/ldap/LdapUser.java
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ public static String toHexString(byte[] bytes) {

@Override
public Attributes getAttributes(String name) throws NamingException {
if (!StringUtils.isBlank(name)) {
if (StringUtils.isNotBlank(name)) {
throw new NameNotFoundException();
}
return (Attributes) this.attributes.clone();
Expand All @@ -292,7 +292,7 @@ public Attributes getAttributes(Name name) throws NamingException {

@Override
public Attributes getAttributes(String name, String[] ids) throws NamingException {
if (!StringUtils.isBlank(name)) {
if (StringUtils.isNotBlank(name)) {
throw new NameNotFoundException();
}

Expand Down

0 comments on commit dee50b0

Please sign in to comment.