From cc038bd53352521b69ce683520b04b008d4ac21c Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Tue, 3 Apr 2012 12:19:38 +0000
Subject: [PATCH] do not hide tab after users changes in roles

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1308843 13f79535-47bb-0310-9956-ffa450edef68
---
 .../src/main/webapp/js/redback/roles.js           | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/redback/roles.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/redback/roles.js
index 6ea9234d3..aeadf2c10 100644
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/redback/roles.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/redback/roles.js
@@ -165,12 +165,8 @@ define("redback.roles",["jquery","order!utils","i18n","jquery.validate","order!k
     }
 
     updateModifiedRoles=function(){
-
-
       var modifiedRoles = getModifiedRoles();
       $.log("modifiedRoles:"+modifiedRoles);
-
-
       openDialogConfirm(function(){
                           for(i=0;i<modifiedRoles.length;i++){
                             var modifiedRole=modifiedRoles[i];
@@ -225,7 +221,6 @@ define("redback.roles",["jquery","order!utils","i18n","jquery.validate","order!k
           mainContent.find("#roles-view-tabs #roles-view-tabs-a-roles-grid").tab("show");
           activateRolesGridTab();
           removeMediumSpinnerImg();
-
         }
       }
     );
@@ -244,24 +239,28 @@ define("redback.roles",["jquery","order!utils","i18n","jquery.validate","order!k
       for (var i = 0; i < removed.length; i++) {
         $.log("add user:"+removed[i].username());
         currentRole.users.push(removed[i]);
-        //role.modified(true);
         role.usersModified(true);
       }
       selectedOtherUsers([]);
+      $("#role-collapse" ).removeClass("in");
+      $("#role-users-collapse" ).addClass("in");
       activateRoleUsersEditTab();
     }
 
     removeUser=function(){
+      $.log("removeUser");
       var added = currentRole.users.removeAll(selectedUsers());
       for (var i = 0; i < added.length; i++) {
         currentRole.otherUsers.push(added[i]);
         currentRole.removedUsers.push(added[i]);
-        //role.modified(true);
         role.usersModified(true);
       }
       selectedUsers([]);
-      activateRoleUsersEditTab()
+      $("#role-collapse" ).removeClass("in");
+      $("#role-users-collapse" ).addClass("in");
+      activateRoleUsersEditTab();
     }
+
     saveRoleDescription=function(){
       currentRole.updateDescription();
     }
-- 
2.39.5