Opened 8 years ago
Closed 5 years ago
#13263 closed defect (fixed)
Notification doesn't work
| Reported by: | Owned by: | osimons | |
|---|---|---|---|
| Priority: | normal | Component: | XmlRpcPlugin |
| Severity: | normal | Keywords: | Notification |
| Cc: | Olemis Lang | Trac Release: | 1.2 |
Description (last modified by )
When setting notify=True on the ticket.update call the INotificationSubscriber plugins don't appear to get called.
It looks like notify=True results in calling some old, outdated, Trac notification system which works off some hard-coded stuff in trac.ini, rather than the current notification system which is configurable by user preferences.
So if you're using the new user-configurable notification system all you get if you set notify=True for ticket.update is
Trac[compat] INFO: no recipient for a ticket notification
in the log, and no emails.
Attachments (0)
Change History (6)
comment:2 Changed 8 years ago by
| Description: | modified (diff) |
|---|
comment:5 Changed 7 years ago by
As a workaround, adding the old config parameters in trac.ini seems to "fix" this issue as well...
So adding these "back" in notification-section in trac.ini worked for us:
always_notify_reporter=true always_notify_owner=true
So the old to-be-deprecated code seems to be still using those flags...



What seems to work is, in ticket.py change
to
I haven't attempted to do the same for create (I've only tried it for update) and I haven't looked at whether there are any unit tests covering this area.