Skip to content
Snippets Groups Projects
Commit 71bb0003 authored by Peter Rotich's avatar Peter Rotich
Browse files

Change getDepartments to take in a "criteria" hashtable

parent 566a1605
No related branches found
No related tags found
No related merge requests found
......@@ -308,12 +308,15 @@ class Dept {
return ($cfg && $cfg->getDefaultDeptId() && ($name=Dept::getNameById($cfg->getDefaultDeptId())))?$name:null;
}
function getDepartments( $publiconly=false) {
function getDepartments( $criteria=null) {
$depts=array();
$sql ='SELECT dept_id, dept_name FROM '.DEPT_TABLE;
if($publiconly)
$sql.=' WHERE ispublic=1';
$sql='SELECT dept_id, dept_name FROM '.DEPT_TABLE.' WHERE 1';
if($criteria['publiconly'])
$sql.=' AND ispublic=1';
if(($manager=$criteria['manager']))
$sql.=' AND manager_id='.db_input(is_object($manager)?$manager->getId():$manager);
if(($res=db_query($sql)) && db_num_rows($res)) {
while(list($id, $name)=db_fetch_row($res))
......@@ -324,7 +327,7 @@ class Dept {
}
function getPublicDepartments() {
return self::getDepartments(true);
return self::getDepartments(array('publiconly'=>true));
}
function create($vars, &$errors) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment