From 8029b1b020951dc9b0434c8ada61723ee5bd901e Mon Sep 17 00:00:00 2001
From: aydreeihn <adriane@enhancesoft.com>
Date: Fri, 6 Sep 2019 11:11:09 -0500
Subject: [PATCH] Issue: Annul Closed Events
This commit fixes an issue where if you were to change a Ticket to the Resolved status and then to Closed, the dashboard statistics would show that you had closed 2 Tickets even though both events were done on the same Ticket.
We should only count one closed event per Ticket.
This was reported in Issue #5018.
---
include/class.ticket.php | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/class.ticket.php b/include/class.ticket.php
index 8890cfb9f..de9760972 100644
--- a/include/class.ticket.php
+++ b/include/class.ticket.php
@@ -1358,7 +1358,7 @@ implements RestrictedAccess, Threadable, Searchable {
$this->clearOverdue(false);
$ecb = function($t) use ($status) {
- $t->logEvent('closed', array('status' => array($status->getId(), $status->getName())));
+ $t->logEvent('closed', array('status' => array($status->getId(), $status->getName())), null, 'closed');
$t->deleteDrafts();
};
break;
--
GitLab