Edgewall Software
Modify

Opened 8 years ago

Closed 7 years ago

Last modified 6 years ago

#3780 closed defect (fixed)

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

Reported by: Dan <trac@…> Owned by: eblot
Priority: normal Milestone: 0.10.3
Component: ticket system Version: 0.10rc1
Severity: normal Keywords: notification email
Cc:
Release Notes:
API 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 Changed 8 years ago by eblot

  • Keywords notification email added

comment:2 Changed 8 years ago by eblot

  • Owner changed from jonas to eblot
  • Status changed from new to assigned

comment:3 Changed 8 years ago by eblot

  • Milestone set to 0.10.3

Should be fixed in [4299]

comment:4 Changed 7 years ago by cboos

  • Milestone changed from 0.10.5 to 0.10.3
  • Resolution set to fixed
  • Status changed from assigned to closed

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

comment:5 Changed 7 years ago by JasonWinnebeck

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 Changed 6 years ago by bobbysmith007

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.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed The owner will remain eblot.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from eblot to the specified user.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.