Opened 7 years ago
Closed 4 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 7 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.