diff --git a/include/ajax.tickets.php b/include/ajax.tickets.php index 34c46325ed84d304c7a8c031bb8a5b686ea337a1..b0089b1dc8fbff5659f0e77fef47fda4996f34ce 100644 --- a/include/ajax.tickets.php +++ b/include/ajax.tickets.php @@ -191,7 +191,7 @@ class TicketsAjaxAPI extends AjaxController { function acquireLock($tid) { global $cfg,$thisstaff; - if(!$tid or !is_numeric($tid) or !$thisstaff or !$cfg or !$cfg->getLockTime()) + if(!$tid || !is_numeric($tid) || !$thisstaff || !$cfg || !$cfg->getLockTime()) return 0; if(!($ticket = Ticket::lookup($tid)) || !$ticket->checkStaffAccess($thisstaff)) @@ -218,10 +218,10 @@ class TicketsAjaxAPI extends AjaxController { function renewLock($tid, $id) { global $thisstaff; - if(!$id or !is_numeric($id) or !$thisstaff) + if(!$tid || !is_numeric($tid) || !is_numeric($id) || !$thisstaff) return $this->json_encode(array('id'=>0, 'retry'=>true)); - $lock= TicketLock::lookup($id); + $lock= TicketLock::lookup($id, $tid); if(!$lock || !$lock->getStaffId() || $lock->isExpired()) //Said lock doesn't exist or is is expired return self::acquireLock($tid); //acquire the lock