From: Arundathi Patil <arundpil@in.ibm.com>
Date: Wed, 5 Dec 2018 13:48:35 +0000 (+0530)
Subject: Sonar Fix: Owner.java
X-Git-Tag: 2.1.9~87^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F65%2F74265%2F1;p=aaf%2Fauthz.git

Sonar Fix: Owner.java

Fixed sonar issues/code-smells across this file.

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

diff --git a/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/ns/Owner.java b/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/ns/Owner.java
index 35dd6b96..9ca3d21f 100644
--- a/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/ns/Owner.java
+++ b/auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/ns/Owner.java
@@ -33,9 +33,9 @@ import org.onap.aaf.cadi.client.Retryable;
 import org.onap.aaf.misc.env.APIException;
 
 public class Owner extends BaseCmd<NS> {
-    private final static String[] options = {"add","del"};
+    private static final String[] options = {"add","del"};
 
-    public Owner(NS ns) throws APIException {
+    public Owner(NS ns) {
         super(ns,"owner",
                 new Param(optionsToString(options),true),
                 new Param("ns-name",true),
@@ -44,8 +44,8 @@ public class Owner extends BaseCmd<NS> {
     }
 
     @Override
-    public int _exec(int _idx, final String ... args) throws CadiException, APIException, LocatorException {
-            int idx = _idx;
+    public int _exec(int idxParam, final String ... args) throws CadiException, APIException, LocatorException {
+            int idx = idxParam;
 
         final int option = whichOption(options, args[idx++]);
         final String ns = args[idx++];
@@ -87,8 +87,8 @@ public class Owner extends BaseCmd<NS> {
     }
 
     @Override
-    public void detailedHelp(int _indent, StringBuilder sb) {
-            int indent = _indent;
+    public void detailedHelp(int indentParam, StringBuilder sb) {
+            int indent = indentParam;
         detailLine(sb,indent,"Add or Delete Responsible person to/from Namespace");
         indent+=2;
         detailLine(sb,indent,"Namespace Owners are responsible to receive Notifications and ");