From 0b3b3b5fef64749775701a56825413a02f61f620 Mon Sep 17 00:00:00 2001
From: Robert Anderson <randerson@lbsd.net>
Date: Fri, 8 May 2009 12:50:23 +0000
Subject: [PATCH] Added edit group menu item

---
 webgui/js/app/windows/AdminGroups.js | 36 ++++++++++++++++++++++++----
 1 file changed, 32 insertions(+), 4 deletions(-)

diff --git a/webgui/js/app/windows/AdminGroups.js b/webgui/js/app/windows/AdminGroups.js
index 0c637fc2..d950de31 100644
--- a/webgui/js/app/windows/AdminGroups.js
+++ b/webgui/js/app/windows/AdminGroups.js
@@ -18,14 +18,42 @@ function showAdminGroupWindow() {
 			// Inline toolbars
 			tbar: [
 				{
-					text:'Add/Edit',
-					tooltip:'Add or edit group',
+					text:'Add',
+					tooltip:'Add group',
 					iconCls:'add',
 					handler: function() {
-						showAdminGroupEditWindow();
+						showAdminGroupAddEditWindow();
 					}
 				}, 
 				'-', 
+				{
+					text:'Edit',
+					tooltip:'Edit group',
+					iconCls:'edit',
+					handler: function() {
+						var selectedItem = AdminGroupWindow.getComponent('gridpanel').getSelectionModel().getSelected();
+						// Check if we have selected item
+						if (selectedItem) {
+							// If so display window
+							showAdminGroupAddEditWindow(selectedItem.data.ID);
+						} else {
+							AdminGroupWindow.getEl().mask();
+
+							// Display error
+							Ext.Msg.show({
+								title: "Nothing selected",
+								msg: "No group selected",
+								icon: Ext.MessageBox.ERROR,
+								buttons: Ext.Msg.CANCEL,
+								modal: false,
+								fn: function() {
+									AdminGroupWindow.getEl().unmask();
+								}
+							});
+						}
+					}
+				},
+				'-', 
 				{
 					text:'Remove',
 					tooltip:'Remove group',
@@ -169,7 +197,7 @@ function showAdminGroupWindow() {
 
 
 // Display edit/add form
-function showAdminGroupEditWindow(id) {
+function showAdminGroupAddEditWindow(id) {
 
 	var submitAjaxConfig;
 
-- 
GitLab