Edgewall Software
Modify

Opened 18 years ago

Closed 18 years ago

Last modified 17 years ago

#3780 closed defect (fixed)

Notification may not be sent if owner/reporter is the updater

Reported by: Dan <trac@…> 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 Emmanuel Blot, 18 years ago

Keywords: notification email added

comment:2 by Emmanuel Blot, 18 years ago

Owner: changed from Jonas Borgström to Emmanuel Blot
Status: newassigned

comment:3 by Emmanuel Blot, 18 years ago

Milestone: 0.10.3

Should be fixed in [4299]

comment:4 by Christian Boos, 18 years ago

Milestone: 0.10.50.10.3
Resolution: fixed
Status: assignedclosed

So I guess this one was fixed, but see also #3093

comment:5 by JasonWinnebeck, 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 bobbysmith007, 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.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Emmanuel Blot.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Emmanuel Blot to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.