From 1f838d43bc3a818a3f5033ff093ce37f24d9f876 Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Mon, 13 Feb 2012 08:59:51 +0000
Subject: [PATCH] role screen bulk add confirm dialog before saving all

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1243444 13f79535-47bb-0310-9956-ffa450edef68
---
 .../apache/archiva/i18n/default.properties    |  4 +++
 .../src/main/webapp/js/redback/roles.js       | 35 +++++++++++++------
 2 files changed, 28 insertions(+), 11 deletions(-)

diff --git a/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties b/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties
index 9b03ca57d..1dc5f5032 100644
--- a/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties
+++ b/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties
@@ -148,3 +148,7 @@ proxy-connectors.remoterepo.settings.networkproxy=Network Proxy
 proxy.connector.bulk.save.confirm=Are you sure to update {0} Proxy Connector(s)
 proxyconnector.delete.confirm=Are you sure to delete the Proxy Connector
 
+
+#roles
+role.bulk.save.confirm=Are you sure to update {0} Role(s)
+
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 51fcce823..e20d219f2 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
@@ -164,19 +164,32 @@ $(function() {
     }
 
     updateModifiedRoles=function(){
+
+
       var modifiedRoles = getModifiedRoles();
       $.log("modifiedRoles:"+modifiedRoles);
-      for(i=0;i<modifiedRoles.length;i++){
-        var modifiedRole=modifiedRoles[i];
-        if (modifiedRole.modified()){
-          modifiedRole.updateDescription();
-          modifiedRole.modified(false);
-        }
-        if (modifiedRole.usersModified()){
-          modifiedRole.updateUsers();
-          modifiedRole.usersModified(false);
-        }
-      }
+
+
+      openDialogConfirm(function(){
+                          for(i=0;i<modifiedRoles.length;i++){
+                            var modifiedRole=modifiedRoles[i];
+                            if (modifiedRole.modified()){
+                              modifiedRole.updateDescription();
+                              modifiedRole.modified(false);
+                            }
+                            if (modifiedRole.usersModified()){
+                              modifiedRole.updateUsers();
+                              modifiedRole.usersModified(false);
+                            }
+                          }
+                          closeDialogConfirm();
+                        },
+                        $.i18n.prop('ok'),
+                        $.i18n.prop('cancel'),
+                        $.i18n.prop('bulk.save.confirm.title'),
+                        $.i18n.prop('role.bulk.save.confirm',modifiedRoles.length));
+
+
     }
 
     updateRole=function(modifiedRole){
-- 
2.39.5