diff options
| author |   anushadasari <danush10@in.ibm.com> | 2019-11-20 16:43:57 +0530 |
|---|---|---|
| committer | anushadasari <danush10@in.ibm.com> | 2019-11-20 16:44:06 +0530 |
| commit | 3e7ef00ea13529ad35e4ff91c551237840d59366 (patch) | |
| tree | c563c1e2bd8ed9ba3d6b4d40a1ccdc4120495ad4 | |
| parent | bc025fbc60e7c0bef38dd0717e50a8b483393069 (diff) | |
Move "*" string literal on left side of string
It is preferable to place string literals on the left-hand side of an equals() or equalsIgnoreCase() method call.
Issue-ID: OPTFRA-647
Change-Id: Ibd04d10da4e655be44310ec0459f1fdeab791441
Signed-off-by: anushadasari <danush10@in.ibm.com>
| -rwxr-xr-x | cmso-service/src/main/java/org/onap/optf/cmso/aaf/AafPerm.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cmso-service/src/main/java/org/onap/optf/cmso/aaf/AafPerm.java b/cmso-service/src/main/java/org/onap/optf/cmso/aaf/AafPerm.java index d324f18..b8e7f5b 100755 --- a/cmso-service/src/main/java/org/onap/optf/cmso/aaf/AafPerm.java +++ b/cmso-service/src/main/java/org/onap/optf/cmso/aaf/AafPerm.java @@ -90,9 +90,9 @@ public class AafPerm { */
public boolean ok(AafPerm userPerm) {
if (type.equals(userPerm.getType())) {
- if (userPerm.getInstance().equals("*") || instance.equals("*") || userPerm.getInstance().equals(instance)) {
+ if ("*".equals(userPerm.getInstance()) || "*".equals(instance) || userPerm.getInstance().equals(instance)) {
for (String userAction : userPerm.getActions()) {
- if (userAction.equals("*") || actions.contains("*") || actions.contains(userAction)) {
+ if ("*".equals(userAction) || actions.contains("*") || actions.contains(userAction)) {
return true;
}
}
|