diff options
| author |   Maciej Wejs <maciej.wejs@nokia.com> | 2018-07-11 14:17:24 +0200 |
|---|---|---|
| committer | Maciej Wejs <maciej.wejs@nokia.com> | 2018-07-11 14:52:48 +0200 |
| commit | 0e905b62dc538003ef54c671e08385ad4d9089d5 (patch) | |
| tree | 7ed168e8be540d84db709b6651d75b0004ec5aa1 /auth/auth-core | |
| parent | d37b5467a3b8b375b603579d2888a4443a8b06a7 (diff) | |
Sonar issues fixes in Validator
Change-Id: I95e702f0fad598c004bc009d7dbfb42bfe42280f
Issue-ID: AAF-391
Signed-off-by: Maciej Wejs <maciej.wejs@nokia.com>
Diffstat (limited to 'auth/auth-core')
| -rw-r--r-- | auth/auth-core/src/main/java/org/onap/aaf/auth/validation/Validator.java | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/auth/auth-core/src/main/java/org/onap/aaf/auth/validation/Validator.java b/auth/auth-core/src/main/java/org/onap/aaf/auth/validation/Validator.java index de20e476..8302e771 100644 --- a/auth/auth-core/src/main/java/org/onap/aaf/auth/validation/Validator.java +++ b/auth/auth-core/src/main/java/org/onap/aaf/auth/validation/Validator.java @@ -43,9 +43,9 @@ public class Validator { public static final Pattern ID_CHARS = Pattern.compile("[\\w.-]+@[\\w.-]+"); public static final Pattern NAME_CHARS = Pattern.compile("[\\w.-]+"); public static final Pattern DESC_CHAR = Pattern.compile("["+ESSENTIAL+"\\x20]+"); - public static List<String> nsKeywords; - protected final Pattern actionChars; - protected final Pattern instChars; + protected static List<String> nsKeywords; + private final Pattern actionChars; + private final Pattern instChars; private StringBuilder msgs; static { @@ -110,7 +110,7 @@ public class Validator { if(res==null) { msgs.append("Result object is blank"); } else if(res.notOK()) { - msgs.append(res.getClass().getSimpleName() + " is not OK"); + msgs.append(res.getClass().getSimpleName()).append(" is not OK"); } return this; } @@ -130,10 +130,8 @@ public class Validator { } protected Validator description(String type, String description) { - if(description!=null) { - if(noMatch(description, DESC_CHAR)) { - msg(type + " Description is invalid."); - } + if (description != null && noMatch(description, DESC_CHAR)) { + msg(type + " Description is invalid."); } return this; } @@ -151,7 +149,7 @@ public class Validator { } else if(ns==null) { msg("Perm NS is null"); } else if(nob(type,NAME_CHARS)) { - msg("Perm Type [" + (ns+(type.length()==0?"":'.'))+type + "] is invalid."); + msg("Perm Type [" + (ns+(type.length()==0?"":'.')) + type + "] is invalid."); } return this; } @@ -208,4 +206,5 @@ public class Validator { return this; } + } |