| Summary | No notification message for newly created tickets |
| Queue | Whups |
| Queue Version | Git master |
| Type | Bug |
| State | Resolved |
| Priority | 2. Medium |
| Owners | jan (at) horde (dot) org |
| Requester | heinz (at) htl-steyr (dot) ac (dot) at |
| Created | 06/24/2013 (4520 days ago) |
| Due | |
| Updated | 07/15/2013 (4499 days ago) |
| Assigned | 07/02/2013 (4512 days ago) |
| Resolved | 07/15/2013 (4499 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | No |
commit aeb89e9d03e021ffbc9da1d972fb7fa5d5d9fe0c
Author: Jan Schneider <jan@horde.org>
Date: Mon Jul 15 21:16:30 2013 +0200
[jan] Fix counting of responsible users (
Bug #12406).whups/docs/CHANGES | 1 +
whups/lib/Ticket.php | 10 +++++++---
whups/lib/Whups.php | 9 ++++-----
whups/package.xml | 2 ++
whups/templates/ticket/watchers.inc | 6 +++---
5 files changed, 17 insertions(+), 11 deletions(-)
http://git.horde.org/horde-git/-/commit/aeb89e9d03e021ffbc9da1d972fb7fa5d5d9fe0c
State ⇒ Assigned
Priority ⇒ 2. Medium
Patch ⇒ No
Milestone ⇒
Queue ⇒ Whups
Summary ⇒ No notification message for newly created tickets
Type ⇒ Bug
State ⇒ Unconfirmed
Code in whups/lib/Ticket.php, function notify():
if (empty($listeners)) {
if ($conf['mail']['incl_resp'] ||
!count(current($whups_driver->getOwners($this->_id)))) {
/* Include all responsible. */
$listeners = $whups_driver->getListeners(
$this->_id, true, true, true);
....
Problem:
$whups_driver->getOwners($this->_id) returns an empty array for new
ticket (that's OK)
current (..) returns false on empty arrays
count (..) returns 1 (because false as argument) and therefore 1 owner
is assumed and no notification is sent!
The current()-call should be omitted!
Same construct in function Whups_Ticket::delete