Index: trunk/admin/users/group_select.php =================================================================== diff -u -r13 -r338 --- trunk/admin/users/group_select.php (.../group_select.php) (revision 13) +++ trunk/admin/users/group_select.php (.../group_select.php) (revision 338) @@ -134,13 +134,13 @@ $order = $objConfig->Get("GroupSelect_SortOrder"); -$SelectorType="checkbox"; -if($_GET["Selector"]) - $SelectorType=$_GET["Selector"]; +$SelectorType = GetVar('Selector'); +if(!$SelectorType) $SelectorType = 'checkbox'; +$source = GetVar('source'); -if(strlen($_GET["source"])>0) +if($source) { - switch($_GET["source"]) + switch($source) { case "addcategory_permissions": $SelectorType = "radio"; @@ -209,23 +209,19 @@ $objListView->ColumnHeaders->SetSort($objConfig->Get("GroupSelect_SortField"),$order); - - $objListView->PrintToolBar = FALSE; $objListView->CurrentPageVar = "Page_Grouplist"; $objListView->PerPageVar = "Perpage_Grouplist"; $objListView->CheckboxName = "itemlist[]"; -$objListView->TotalItemCount = $itemcount; -$objListView->SelectorType=$SelectorType; +$objListView->TotalItemCount = $objGroups->QueryItemCount; +$objListView->SelectorType = $SelectorType; - - +$title = 'Select Group'; int_header($objListToolBar,NULL,$title); -if(strlen($_GET["values"])) -{ - $current_value = explode(",",$_GET["values"]); -} +$values = GetVar('values'); +if($values) $current_value = explode(',', $values); + ?>