From 347c0214388bae18616a9003adf84d7be509fa86 Mon Sep 17 00:00:00 2001
From: Phil Davis <phil.davis@inf.org>
Date: Sun, 20 Mar 2016 16:06:02 +0545
Subject: Reengineer Form_Button setAttribute

As per what was done for https://github.com/pfsense/pfsense/pull/2765 -
do it to the rest of them.
Seems to work OK.
---
 src/usr/local/www/system_groupmanager.php | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

(limited to 'src/usr/local/www/system_groupmanager.php')

diff --git a/src/usr/local/www/system_groupmanager.php b/src/usr/local/www/system_groupmanager.php
index aaf814c..5c2eb3a 100644
--- a/src/usr/local/www/system_groupmanager.php
+++ b/src/usr/local/www/system_groupmanager.php
@@ -452,14 +452,14 @@ if ($pconfig['gid'] != 1998) { // all users group
 		'Move to "Members"',
 		null,
 		'fa-angle-double-right'
-	))->removeClass('btn-primary')->addClass('btn-info btn-sm');
+	))->setAttribute('type','button')->removeClass('btn-primary')->addClass('btn-info btn-sm');
 
 	$group->add(new Form_Button(
 		'movetodisabled',
 		'Move to "Not members',
 		null,
 		'fa-angle-double-left'
-	))->removeClass('btn-primary')->addClass('btn-info btn-sm');
+	))->setAttribute('type','button')->removeClass('btn-primary')->addClass('btn-info btn-sm');
 
 	$group->setHelp('Hold down CTRL (pc)/COMMAND (mac) key to select multiple items');
 	$section->add($group);
@@ -484,11 +484,6 @@ print $form;
 //<![CDATA[
 events.push(function() {
 
-	// Make buttons plain buttons, not submit
-	$("#movetodisabled").prop('type','button');
-	$("#movetoenabled").prop('type','button');
-
-
 	// On click . .
 	$("#movetodisabled").click(function() {
 		moveOptions($('[name="members[]"] option'), $('[name="notmembers[]"]'));
-- 
cgit v1.1