#3780 closed defect (fixed)
Notification may not be sent if owner/reporter is the updater
Reported by: | Owned by: | Emmanuel Blot | |
---|---|---|---|
Priority: | normal | Milestone: | 0.10.3 |
Component: | ticket system | Version: | 0.10rc1 |
Severity: | normal | Keywords: | notification email |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
Whether this is a defect or a feature is probably up to you, but this was confusing to me.
When configuring notification parameter always_notify_updater
as false
, Trac not only does not notify the updater, but if the updater is the owner or reporter, it doesn't send the notification to that recipient even if it was told to "always" send it to them (e.g. always_notify_reporter
).
If you actually desire that kind of bahavior (I could see why), I think the name of the parameter is confusing. Maybe just notify_updater
would be more appropriate.
Attachments (0)
Change History (6)
comment:1 by , 18 years ago
Keywords: | notification email added |
---|
comment:2 by , 18 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 18 years ago
Milestone: | → 0.10.3 |
---|
comment:4 by , 18 years ago
Milestone: | 0.10.5 → 0.10.3 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
So I guess this one was fixed, but see also #3093…
comment:5 by , 17 years ago
Yeah, Trac is definitely flip-flopping on this issue, because it seems that people can't agree on the priority of the notification directives (since they aren't mutually exclusive). I just upgraded from 0.10.2 to 0.10.5dev and now the behavior is different. In 0.10.0 release we patched by hand to make it work the way that 0.10.2 does — in there the patch is integrated. In 0.10.5dev it is effectively reverted. In researching this I found this ticket with associated r4299 and the original change r3275.
Probably there needs to be a stronger stance on this from the Trac team on what the position is, and ultimately holding off until the per-user settings or more detailed settings. Hopefully this will prevent flip-flopping within minor bug-fix releases of Trac.
comment:6 by , 17 years ago
Just a note there is a plugin at http://trac-hacks.org/wiki/NeverNotifyUpdaterPlugin that is related to this. With this patch you can make it so that it will not ever notify the person who made the change but will notify everyone else.
Should be fixed in [4299]