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

Fixup merging of email recipients

This stems from a confusing similarity between the + operator for arrays
and array_merge() in php. Adding arrays will ignore items in the RHS where
keys are present in the LHS. Therefore, when adding numerically indexed
arrays together, only items on the RHS that have a key higher than the
greatest key on the LHS will be included.
parent d0367ce4
Branches
Tags
No related merge requests found
......@@ -1142,7 +1142,7 @@ class Ticket {
$recipients[] = $assignee;
} elseif(!strcasecmp(get_class($assignee), 'Team')) {
if($cfg->alertTeamMembersONAssignment() && ($members=$assignee->getMembers()))
$recipients+=$members;
$recipients = array_merge($recipients, $members);
elseif($cfg->alertTeamLeadONAssignment() && ($lead=$assignee->getTeamLead()))
$recipients[] = $lead;
}
......@@ -1398,11 +1398,11 @@ class Ticket {
if($this->getStaffId())
$recipients[]=$this->getStaff();
elseif($this->getTeamId() && ($team=$this->getTeam()) && ($members=$team->getMembers()))
$recipients+=$members;
$recipients = array_merge($recipients, $members);
} elseif($cfg->alertDeptMembersONTransfer() && !$this->isAssigned()) {
//Only alerts dept members if the ticket is NOT assigned.
if(($members=$dept->getMembers()))
$recipients+=$members;
$recipients = array_merge($recipients, $members);
}
//Always alert dept manager??
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment