Skip to content
Snippets Groups Projects
Commit eca531f0 authored by Jared Hancock's avatar Jared Hancock
Browse files

ticket: Fix auto-claim from canned auto-reply

parent 4f71f000
Branches
Tags
No related merge requests found
......@@ -1903,7 +1903,7 @@ class Ticket {
'cannedattachments' => $files);
$errors = array();
if(!($response=$this->postReply($info, $errors, false)))
if (!($response=$this->postReply($info, $errors, false, false)))
return null;
$this->markUnAnswered();
......@@ -1941,11 +1941,10 @@ class Ticket {
}
/* public */
function postReply($vars, &$errors, $alert = true) {
function postReply($vars, &$errors, $alert=true, $claim=true) {
global $thisstaff, $cfg;
if(!$vars['poster'] && $thisstaff)
if (!$vars['poster'] && $thisstaff)
$vars['poster'] = $thisstaff;
if(!$vars['staffId'] && $thisstaff)
......@@ -1964,10 +1963,11 @@ class Ticket {
$this->setStatus($vars['reply_status_id']);
// Claim on response bypasses the department assignment restrictions
if($thisstaff && $this->isOpen() && !$this->getStaffId()
&& $cfg->autoClaimTickets())
if ($claim && $thisstaff && $this->isOpen() && !$this->getStaffId()
&& $cfg->autoClaimTickets()
) {
$this->setStaffId($thisstaff->getId()); //direct assignment;
}
$this->onResponse($response, array('assignee' => $assignee)); //do house cleaning..
......@@ -2302,7 +2302,7 @@ class Ticket {
}
function lookupByNumber($number, $email=null) {
return self::lookup(self:: getIdByNumber($number, $email));
return self::lookup(self::getIdByNumber($number, $email));
}
static function isTicketNumberUnique($number) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment