Fix Remove Role command 83/58983/1
authorDavid Suarez <david.suarez.fuentes@ericsson.com>
Wed, 14 Jun 2017 23:12:20 +0000 (01:12 +0200)
committerDavid Suarez <david.suarez.fuentes@ericsson.com>
Wed, 14 Jun 2017 23:12:20 +0000 (01:12 +0200)
The Remove Role command was actually removing the user with roleId
instead of the role.

Change-Id: I8564c35d658a6367dbf3e16eab23f46803119721
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
aaa-cli/src/main/java/org/opendaylight/aaa/cli/dmstore/RemoveRole.java

index 8195342dba30f0d85b626d5d0995cef4e2631bfa..47a5b720a8fa3d2bfe9363fdf03a1991d8226a86 100644 (file)
@@ -25,7 +25,7 @@ public class RemoveRole extends AaaCliAbstractCommand {
 
     @Option(name = "-name", aliases = {
             "--roleName" }, description = "The role name", required = true, multiValued = false)
-    private String roleName = "";
+    private final String roleName = "";
 
     public RemoveRole(final IIDMStore identityStore) {
         super(identityStore);
@@ -40,7 +40,7 @@ public class RemoveRole extends AaaCliAbstractCommand {
         if (roleId == null) {
             return "Role does not exist";
         }
-        if (identityStore.deleteUser(roleId) == null) {
+        if (identityStore.deleteRole(roleId) == null) {
             return "Failed to delete role " + roleName;
         }
         return "Role " + roleName + "has been deleted.";