diff --git a/app/fax_queue/fax_queue.php b/app/fax_queue/fax_queue.php index 49a7e5c316..b83fee27bf 100644 --- a/app/fax_queue/fax_queue.php +++ b/app/fax_queue/fax_queue.php @@ -35,6 +35,20 @@ exit; } +//set defaults + $database = database::new(); + $domain_uuid = $_SESSION['domain_uuid'] ?? ''; + $user_uuid = $_SESSION['user_uuid'] ?? ''; + $settings = new settings(['database' => $database, 'domain_uuid' => $domain_uuid, 'user_uuid' => $user_uuid]); + +//set default permissions + $permission = []; + $permission['fax_queue_add'] = permission_exists('fax_queue_add'); + $permission['fax_queue_delete'] = permission_exists('fax_queue_delete'); + $permission['fax_queue_domain'] = permission_exists('fax_queue_domain'); + $permission['fax_queue_all'] = permission_exists('fax_queue_all'); + $permission['fax_queue_edit'] = permission_exists('fax_queue_edit'); + //add multi-lingual support $language = new text; $text = $language->get(); @@ -59,19 +73,19 @@ switch ($action) { case 'copy': - if (permission_exists('fax_queue_add')) { + if ($permission['fax_queue_add']) { $obj = new fax_queue; $obj->copy($fax_queue); } break; //case 'toggle': - // if (permission_exists('fax_queue_edit')) { + // if ($permission['fax_queue_edit']) { // $obj = new fax_queue; // $obj->toggle($fax_queue); // } // break; case 'delete': - if (permission_exists('fax_queue_delete')) { + if ($permission['fax_queue_delete']) { $obj = new fax_queue; $obj->delete($fax_queue); } @@ -79,17 +93,12 @@ } //redirect the user - header('Location: fax_queue.php'.(!empty($search) ? '?search='.urlencode($search) : null)); + header('Location: fax_queue.php'.(!empty($search) ? '?search='.urlencode($search) : '')); exit; } //set the time zone - if (isset($_SESSION['domain']['time_zone']['name'])) { - $time_zone = $_SESSION['domain']['time_zone']['name']; - } - else { - $time_zone = date_default_timezone_get(); - } + $time_zone = $settings->get('domain', 'time_zone', date_default_timezone_get()); //get order and order by $order_by = $_GET["order_by"] ?? null; @@ -99,11 +108,11 @@ $sql = "select count(fax_queue_uuid) "; $sql .= "from v_fax_queue as q "; $sql .= "LEFT JOIN v_users AS u ON q.insert_user = u.user_uuid "; - if (!empty($_GET['show']) && $_GET['show'] == "all" && permission_exists('fax_queue_all')) { + if (!empty($_GET['show']) && $_GET['show'] == "all" && $permission['fax_queue_all']) { // show faxes for all domains $sql .= "WHERE true "; } - elseif (permission_exists('fax_queue_domain')) { + elseif ($permission['fax_queue_domain']) { // show faxes for one domain $sql .= "WHERE q.domain_uuid = :domain_uuid "; $parameters['domain_uuid'] = $domain_uuid; @@ -113,7 +122,7 @@ $sql .= "WHERE q.domain_uuid = :domain_uuid "; $sql .= "AND u.user_uuid = :user_uuid "; $parameters['domain_uuid'] = $domain_uuid; - $parameters['user_uuid'] = $_SESSION['user_uuid']; + $parameters['user_uuid'] = $user_uuid; } if (isset($search)) { @@ -135,14 +144,13 @@ $sql .= "AND q.fax_status = :fax_status "; $parameters['fax_status'] = $_GET["fax_status"]; } - $database = new database; $num_rows = $database->select($sql, $parameters ?? null, 'column'); unset($sql, $parameters); //prepare to page the results - $rows_per_page = (!empty($_SESSION['domain']['paging']['numeric'])) ? $_SESSION['domain']['paging']['numeric'] : 50; + $rows_per_page = $settings->get('domain', 'paging', 50); $param = !empty($search) ? "&search=".$search : null; - $param = (!empty($_GET['show']) && $_GET['show'] == 'all' && permission_exists('fax_queue_all')) ? "&show=all" : null; + $param = (!empty($_GET['show']) && $_GET['show'] == 'all' && $permission['fax_queue_all']) ? "&show=all" : null; $page = !empty($_GET['page']) && is_numeric($_GET['page']) ? $_GET['page'] : 0; list($paging_controls, $rows_per_page) = paging($num_rows, $param, $rows_per_page); list($paging_controls_mini, $rows_per_page) = paging($num_rows, $param, $rows_per_page, true); @@ -179,11 +187,11 @@ $sql .= "LEFT JOIN v_users AS u ON q.insert_user = u.user_uuid "; $sql .= "JOIN v_domains AS d ON q.domain_uuid = d.domain_uuid "; - if (!empty($_GET['show']) && $_GET['show'] == "all" && permission_exists('fax_queue_all')) { + if (!empty($_GET['show']) && $_GET['show'] == "all" && $permission['fax_queue_all']) { // show faxes for all domains $sql .= "WHERE true "; } - elseif (permission_exists('fax_queue_domain')) { + elseif ($permission['fax_queue_domain']) { // show faxes for one domain $sql .= "WHERE q.domain_uuid = :domain_uuid "; $parameters['domain_uuid'] = $domain_uuid; @@ -193,7 +201,7 @@ $sql .= "WHERE q.domain_uuid = :domain_uuid "; $sql .= "AND u.user_uuid = :user_uuid "; $parameters['domain_uuid'] = $domain_uuid; - $parameters['user_uuid'] = $_SESSION['user_uuid']; + $parameters['user_uuid'] = $user_uuid; } if (isset($search)) { @@ -234,24 +242,25 @@ echo "