Opened 5 years ago

Closed 3 years ago

Last modified 3 years ago

#8821 closed enhancement (duplicate)

FlexibleReporterNotifcationPlugin not working

Reported by: anonymous Owned by: rjollos
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@… 3 years ago. working with Trac 1.0.*
t8821.patch (3.6 KB) - added by rjollos 3 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 3 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 3 years ago by o.rossmueller@… working with Trac 1.0.*

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

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

comment:3 Changed 3 years ago by rjollos

  • Owner changed from CStrac to rjollos
  • Status changed from new to accepted

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

Changed 3 years ago by rjollos

comment:4 Changed 3 years ago by rjollos

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 3 years ago by rjollos

  • Resolution set to duplicate
  • Status changed from accepted to closed

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

comment:6 Changed 3 years ago by rjollos

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 3 years ago by rjollos

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

Add Comment

Modify Ticket

as closed The owner will remain rjollos.
The resolution will be deleted. Next status will be 'reopened'.

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

Note: See TracTickets for help on using tickets.