Opened 13 years ago

Closed 10 years ago

Last modified 10 years ago

#8821 closed enhancement (duplicate)

FlexibleReporterNotifcationPlugin not working

Reported by: anonymous Owned by: Ryan J Ollos
Priority: high Component: FlexibleReporterNotificationPlugin
Severity: blocker Keywords:
Cc: Trac Release: 0.11



I have downloaded FlexibleReporterNotifcationPlugin plugin and made changes in trac.ini file as required but it is not working as expected.Do I need to do any other changes or this plugin need any changes? My requirement is restrict the notification email to reporter only opening and closing a ticket, please help me, i need this.

Attachments (2) (2.4 KB) - added by o.rossmueller@… 10 years ago. working with Trac 1.0.*
t8821.patch (3.6 KB) - added by Ryan J Ollos 10 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 10 years ago by anonymous

I am having the same issue, I am attempting to walk through the python code but have been unable to find the fix.

Can someone please direct me to where I can learn about testing and debuggind Trac Python code?

Changed 10 years ago by o.rossmueller@…

Attachment: added working with Trac 1.0.*

comment:2 Changed 10 years ago by o.rossmueller@…

I got the plugin working with Trac 1.0.*, see the attached.

comment:3 Changed 10 years ago by Ryan J Ollos

Owner: changed from CStrac to Ryan J Ollos
Status: newaccepted

Would you kindly attach that as a patch file (trac:TracDev/SubmittingPatches)?

Changed 10 years ago by Ryan J Ollos

Attachment: t8821.patch added

comment:4 Changed 10 years ago by Ryan J Ollos

There are two major issues with the patch t8821.patch (a diff of against r11964):

  • We should not call open_environment in the plugin. The Environment object is bound to self.
  • The patch is not compatible with Trac < 1.0.

comment:5 Changed 10 years ago by Ryan J Ollos

Resolution: duplicate
Status: acceptedclosed

Duplicate of #7554. A fix will be committed against that ticket.

comment:6 Changed 10 years ago by Ryan J Ollos

In 13701:

Complete rework of plugin, using ideas from #11210 in the patch by chris-at. Fixes #7554, Refs #8821, #11210.

Note that only one instance of the reporter in to_recipients is removed. So for instance, if the reporter is also the owner and always_notify_owner is True, then the reporter will still receive a notification.

This has been lightly tested, so please report back if you find any issues.

comment:7 Changed 10 years ago by Ryan J Ollos

I used some of the ideas in to prepare the fix for #7554. Please test it out and report back if you can.

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

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

Note: See TracTickets for help on using tickets.