From 2c7e4c1dd4c4a8ec7da61b940e8ebee30ab56ce2 Mon Sep 17 00:00:00 2001
From: Ingo Schommer <ingo@silverstripe.com>
Date: Sat, 21 Nov 2009 03:21:10 +0000
Subject: [PATCH] BUGFIX Fixed 'add member' popup in SecurityAdmin

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@92851 467b73ca-7a2a-4603-9d3b-597d59a354a9
---
 javascript/SecurityAdmin.js | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/javascript/SecurityAdmin.js b/javascript/SecurityAdmin.js
index 4e89a82c..bbceb813 100755
--- a/javascript/SecurityAdmin.js
+++ b/javascript/SecurityAdmin.js
@@ -17,13 +17,20 @@
 			}
 		);
 	});
-}(jQuery));
-/**
- * CAUTION: Assumes that a MemberTableField-instance is present as an editing form
- */
-function action_addmember_right() {
-	var memberTableFields = document.getElementsBySelector('#Form_EditForm div.MemberTableField');
-	var tables = document.getElementsBySelector('#Form_EditForm div.MemberTableField table');
-	var addLinks = document.getElementsBySelector('#Form_EditForm div.MemberTableField a.addlink');
-	memberTableFields[0].openPopup(null,addLinks[0].href,tables[0]);
-}
\ No newline at end of file
+	
+	$.concrete('ss', function($){
+		$('#Form_EditForm .Actions #action_addmember').concrete({
+			onclick: function(e) {
+				// CAUTION: Assumes that a MemberTableField-instance is present as an editing form
+				var t = $('#Form_EditForm_Members');
+				t[0].openPopup(
+					null,
+					$('base').attr('href') + t.find('a.addlink').attr('href'),
+					t.find('table')[0]
+				);
+				return false;
+			}
+		});
+	});
+	
+}(jQuery));
\ No newline at end of file