Modify ↓
Opened 11 years ago
Closed 9 years ago
#11533 closed defect (wontfix)
Users removed in batch modification are not notified
| Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | notification | Version: | |
| Severity: | normal | Keywords: | batch-modify |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
As noted in comment:19:ticket:2311, changing the ticket owner or removing a user from the CC field does not send a notification to the "outgoing" user, in contrast with standard ticket edits. My initial thought is that batch modification and standard edits should have the same behavior.
Attachments (1)
Change History (11)
comment:1 by , 10 years ago
| Milestone: | next-stable-1.0.x → 1.0.3 |
|---|
comment:2 by , 10 years ago
| Owner: | set to |
|---|---|
| Status: | new → assigned |
comment:3 by , 10 years ago
| Milestone: | 1.0.3 → 1.0.4 |
|---|
follow-up: 6 comment:4 by , 10 years ago
by , 10 years ago
| Attachment: | T11533_notify_previous_owner_or_cc.diff added |
|---|
comment:5 by , 10 years ago
| Milestone: | 1.0.4 → 1.0.5 |
|---|
comment:6 by , 10 years ago
comment:8 by , 10 years ago
| Milestone: | 1.0.6 → 1.0.7 |
|---|
comment:9 by , 9 years ago
| Milestone: | 1.0.7 → 1.2 |
|---|
comment:10 by , 9 years ago
| Milestone: | 1.2 |
|---|---|
| Resolution: | → wontfix |
| Status: | assigned → closed |
Confirmed that on 1.2dev the outgoing owner is notified when a ticket is reassigned, and users removed from the CC list are notified.
Closing this as wontfix for 1.0-stable.
Note:
See TracTickets
for help on using tickets.
In 1.1.3 the old owner (and users removed from CC) are not notified anymore even in normal (non-batch) updates.
get_ticket_notification_recipients(changed in #2311 to notify previous owner and users removed from CC) is not used anymore. Instead the TicketOwnerSubscriber and CarbonCopySubscriber are responsible for finding relevant recipients / subscriptions, but they only consider the current (new) owner and CC list.In th:ticket:6452 the old owner is considered in TicketOwnerSubscriber of th:AnnouncerPlugin. (There it's mentioned that users removed from CC are notified, but I don't think that's the case.)
Alternatively one could add
TicketRemovedOwnerSubscriberandRemovedFromCarbonCopySubscriber(similar to TicketPreviousUpdatersSubscriber of comment:35:ticket:4056) although that may be over-complicating things for no reason if there's a consensus that these additional notifications are desirable.