From: Arundathi Patil <arundpil@in.ibm.com>
Date: Wed, 12 Sep 2018 10:09:24 +0000 (+0530)
Subject: Log.java: Fixed sonar issues
X-Git-Tag: 2.1.2~45^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?p=aaf%2Fauthz.git;a=commitdiff_plain;h=9813f57650114a6515641e555a9ea84968a1b2d7

Log.java: Fixed sonar issues

Sonar Link : https://sonar.onap.org/code?id=org.onap.aaf.authz%3Aparent&selected=org.onap.aaf.authz%3Aaaf-auth-cmd%3Asrc%2Fmain%2Fjava%2Forg%2Fonap%2Faaf%2Fauth%2Fcmd%2Fmgmt%2FLog.java

Issue-ID: AAF-486
Change-Id: Iafbde3cda47c746a1330aae355b6f097a765b48a
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
---

diff --git a/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/mgmt/Log.java b/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/mgmt/Log.java
index db59df0a..237e3c9e 100644
--- a/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/mgmt/Log.java
+++ b/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/mgmt/Log.java
@@ -35,18 +35,18 @@ import org.onap.aaf.cadi.config.Config;
 import org.onap.aaf.misc.env.APIException;
 
 public class Log extends BaseCmd<Mgmt> {
-    private final static String[] options = {"add","del"};
+    private static final String[] options = {"add","del"};
 
-    public Log(Mgmt mgmt) throws APIException {
+    public Log(Mgmt mgmt) {
         super(mgmt, "log",
                 new Param(optionsToString(options),true),
                 new Param("id[,id]*",true));
     }
     
     @Override
-    public int _exec(int _idx, String ... args) throws CadiException, APIException, LocatorException {
+    public int _exec(int idxValue, String ... args) throws CadiException, APIException, LocatorException {
         int rv=409;
-        int idx = _idx;
+        int idx = idxValue;
         final int option = whichOption(options, args[idx++]);
 
         for (String name : args[idx++].split(COMMA)) {
@@ -83,7 +83,9 @@ public class Log extends BaseCmd<Mgmt> {
                             pw().println(msg + " Special Log for " + fname + " on " + client);
                             rv=200;
                         } else {
-                            if (rv==409)rv = fp.code();
+                            if (rv==409) {
+                            	rv = fp.code();
+                            };
                             error(fp);
                         }
                         return rv;
@@ -96,8 +98,8 @@ public class Log extends BaseCmd<Mgmt> {
     }
 
     @Override
-    public void detailedHelp(int _indent, StringBuilder sb) {
-            int indent = _indent;
+    public void detailedHelp(int indentValue, StringBuilder sb) {
+            int indent = indentValue;
         detailLine(sb,indent,"Clear the cache for certain tables");
         indent+=2;
         detailLine(sb,indent,"name        - name of table or 'all'");