X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=admin%2Fsecurity_roles.php;h=2905f30fab624688209defa1cb69180b0f74b67c;hb=8a2ac7536a58c7860cbebba4f786f51f761ac2bf;hp=bce6661f2e2cc88a414f68f4238f33c38ee44b10;hpb=74eabb5769f01203484f13b9736c8a56011b48f4;p=fa-stable.git

diff --git a/admin/security_roles.php b/admin/security_roles.php
index bce6661f..2905f30f 100644
--- a/admin/security_roles.php
+++ b/admin/security_roles.php
@@ -85,15 +85,15 @@ 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[] = $a;
+				$areas[] = (int)$a;
 			}
-			if (substr($p,0,7) == 'Section')
-				$sections[] = substr($p, 7);
+			if (substr($p,0,7) == 'Section' && $val == 1)
+				$sections[] = (int)substr($p, 7);
 		}
 //		$areas = sort_areas($areas);
 
@@ -173,7 +173,7 @@ if (!isset($_POST['role']) || get_post('clone') || list_updated('role')) {
 
 start_form();
 
-start_table("class='tablestyle_noborder'");
+start_table(TABLESTYLE_NOBORDER);
 start_row();
 security_roles_list_cells(_("Role:"). "&nbsp;", 'role', null, true, true, check_value('show_inactive'));
 $new_role = get_post('role')=='';
@@ -191,13 +191,13 @@ if (find_submit('_Section')) {
 }
 //-----------------------------------------------------------------------------------------------
 div_start('details');
-start_table($table_style2);
+start_table(TABLESTYLE2);
 	text_row(_("Role name:"), 'name', null, 20, 22);
 	text_row(_("Role description:"), 'description', null, 50, 52);
 	record_status_list_row(_("Current status:"), 'inactive');
 end_table(1);
 
-	start_table("$table_style width=40%");
+	start_table(TABLESTYLE, "width='40%'");
 
 	$k = $j = 0; //row colour counter
 	$ext = $sec = $m = -1;