From c880377cb16e396d043a00e375d8f0e33e9b3532 Mon Sep 17 00:00:00 2001
From: Jared Hancock <jared@osticket.com>
Date: Sun, 2 Aug 2015 20:07:53 -0500
Subject: [PATCH] Fix incorrect usage of department members for alerts
---
include/class.task.php | 2 +-
include/class.ticket.php | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/include/class.task.php b/include/class.task.php
index 348e69393..841e0dad8 100644
--- a/include/class.task.php
+++ b/include/class.task.php
@@ -703,7 +703,7 @@ class Task extends TaskModel implements RestrictedAccess, Threadable {
}
} elseif ($cfg->alertDeptMembersONTaskTransfer() && !$this->isAssigned()) {
// Only alerts dept members if the task is NOT assigned.
- if ($members = $dept->getMembersForAlerts())
+ if ($members = $dept->getMembersForAlerts()->all())
$recipients = array_merge($recipients, $members);
}
diff --git a/include/class.ticket.php b/include/class.ticket.php
index 6a6af2ce1..8d5b6cc35 100644
--- a/include/class.ticket.php
+++ b/include/class.ticket.php
@@ -1215,7 +1215,7 @@ implements RestrictedAccess, Threadable {
// Only alerts dept members if the ticket is NOT assigned.
if ($cfg->alertDeptMembersONNewTicket() && !$this->isAssigned()) {
- if ($members = $dept->getMembersForAlerts())
+ if ($members = $dept->getMembersForAlerts()->all())
$recipients = array_merge($recipients, $members);
}
@@ -1640,7 +1640,7 @@ implements RestrictedAccess, Threadable {
}
elseif ($cfg->alertDeptMembersONOverdueTicket() && !$this->isAssigned()) {
// Only alerts dept members if the ticket is NOT assigned.
- if ($members = $dept->getMembersForAlerts())
+ if ($members = $dept->getMembersForAlerts()->all())
$recipients = array_merge($recipients, $members);
}
// Always alert dept manager??
@@ -1888,7 +1888,7 @@ implements RestrictedAccess, Threadable {
}
elseif ($cfg->alertDeptMembersONTransfer() && !$this->isAssigned()) {
// Only alerts dept members if the ticket is NOT assigned.
- if ($members = $dept->getMembersForAlerts())
+ if ($members = $dept->getMembersForAlerts()->all())
$recipients = array_merge($recipients, $members);
}
--
GitLab