X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=admin%2Fsecurity_roles.php;h=4b438091f237794d6cfce657ea247aed74b7f2a1;hb=406fee129771496f26ef91c0a25d487c7a77c054;hp=9ffe8d7bc17f09cb3158f4c2a8217bdfff9189e4;hpb=17b390efcf904072b02ec866b2a427490471a260;p=fa-stable.git

diff --git a/admin/security_roles.php b/admin/security_roles.php
index 9ffe8d7b..4b438091 100644
--- a/admin/security_roles.php
+++ b/admin/security_roles.php
@@ -85,14 +85,14 @@ if (get_post('addupdate'))
 		$sections = array();
 		$areas = array();
 		foreach($_POST as $p =>$val) {
-			if (substr($p,0,4) == 'Area') {
+			if (substr($p,0,4) == 'Area' && $val == 1) {
 				$a = substr($p, 4);
 				if (($a&~0xffff) && (($a&0xff00)<(99<<8))) {
 					$sections[] = $a&~0xff;	// add extended section for plugins
 				}
 				$areas[] = (int)$a;
 			}
-			if (substr($p,0,7) == 'Section')
+			if (substr($p,0,7) == 'Section' && $val == 1)
 				$sections[] = (int)substr($p, 7);
 		}
 //		$areas = sort_areas($areas);
@@ -147,9 +147,6 @@ if (!isset($_POST['role']) || get_post('clone') || list_updated('role')) {
 		$row = get_security_role($id);
 		$_POST['description'] = $row['description'];
 		$_POST['name'] = $row['role'];
-//	if ($row['inactive']
-//		$_POST['inactive'] = 1;
-	
 		$_POST['inactive'] = $row['inactive'];
 		$access = $row['areas'];
 		$sections = $row['sections'];
@@ -214,8 +211,6 @@ end_table(1);
 			$ext = $newext; 
 			$sec = $newsec;
 			$m = $parms[0] & ~0xff;
-//			if(!isset($security_sections[$m]))
-//			 display_error(sprintf("Bad section %X:", $m));
 			label_row($security_sections[$m].':', 
 				checkbox( null, 'Section'.$m, null, true, 
 					_("On/off set of features")),