Skip to content
Snippets Groups Projects
Commit 7b8ebd86 authored by Jared Hancock's avatar Jared Hancock Committed by Peter Rotich
Browse files

Implement "Disable alerts" setting for departments

parent 371d88ce
Branches
Tags
No related merge requests found
...@@ -141,6 +141,16 @@ class Dept { ...@@ -141,6 +141,16 @@ class Dept {
return $this->getMembers(array('available'=>1)); return $this->getMembers(array('available'=>1));
} }
function getMembersForAlerts() {
if ($this->isGroupMembershipEnabled() == 2) {
// Disabled for this department
$rv = array();
}
else {
$rv = $this->getAvailableMembers();
}
return $rv;
}
function getSLAId() { function getSLAId() {
return $this->ht['sla_id']; return $this->ht['sla_id'];
......
...@@ -938,7 +938,7 @@ class Ticket { ...@@ -938,7 +938,7 @@ class Ticket {
//Only alerts dept members if the ticket is NOT assigned. //Only alerts dept members if the ticket is NOT assigned.
if($cfg->alertDeptMembersONNewTicket() && !$this->isAssigned()) { if($cfg->alertDeptMembersONNewTicket() && !$this->isAssigned()) {
if(($members=$dept->getMembers())) if(($members=$dept->getMembersForAlerts()))
$recipients=array_merge($recipients, $members); $recipients=array_merge($recipients, $members);
} }
...@@ -1211,8 +1211,8 @@ class Ticket { ...@@ -1211,8 +1211,8 @@ class Ticket {
$recipients=array_merge($recipients, $members); $recipients=array_merge($recipients, $members);
} elseif($cfg->alertDeptMembersONOverdueTicket() && !$this->isAssigned()) { } elseif($cfg->alertDeptMembersONOverdueTicket() && !$this->isAssigned()) {
//Only alerts dept members if the ticket is NOT assigned. //Only alerts dept members if the ticket is NOT assigned.
if(($members=$dept->getMembers())) if ($members = $dept->getMembersForAlerts())
$recipients=array_merge($recipients, $members); $recipients = array_merge($recipients, $members);
} }
//Always alert dept manager?? //Always alert dept manager??
if($cfg->alertDeptManagerONOverdueTicket() && $dept && ($manager=$dept->getManager())) if($cfg->alertDeptManagerONOverdueTicket() && $dept && ($manager=$dept->getManager()))
...@@ -1409,7 +1409,7 @@ class Ticket { ...@@ -1409,7 +1409,7 @@ class Ticket {
$recipients = array_merge($recipients, $members); $recipients = array_merge($recipients, $members);
} elseif($cfg->alertDeptMembersONTransfer() && !$this->isAssigned()) { } elseif($cfg->alertDeptMembersONTransfer() && !$this->isAssigned()) {
//Only alerts dept members if the ticket is NOT assigned. //Only alerts dept members if the ticket is NOT assigned.
if(($members=$dept->getMembers())) if(($members=$dept->getMembersForAlerts()))
$recipients = array_merge($recipients, $members); $recipients = array_merge($recipients, $members);
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment