Modify

Opened 6 years ago

Last modified 22 months ago

#7998 new defect

LegacyTicket preferences do not have any effect

Reported by: Mazda86 Owned by:
Priority: normal Component: AnnouncerPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

Using Trac 0.11.7 and Python 2.4.6, and the latest 0.11 version of AnnouncerPlugin, I find that no combination of settings in trac.ini or in user Announcements preferences results in anything but maximum email.

A simple patch fixes this:

@@ -146,7 +146,7 @@
                         component.owner)
                 if notify is None:
                     notify = self.always_notify_component_owner
-                if notify:
+                if notify == 'True':
                     self._log_sub(component.owner, True,
                             'always_notify_component_owner')
                     return ('email', component.owner, True, None)
@@ -160,7 +160,7 @@
             notify = self._check_user_setting('notify_owner', ticket['owner'])
             if notify is None:
                 notify = self.always_notify_owner
-            if notify:
+            if notify == 'True':
                 owner = ticket['owner']
                 if '@' in owner:
                     name, authenticated, address = None, False, owner
@@ -174,7 +174,7 @@
             notify = self._check_user_setting('notify_reporter', ticket['reporter'])
             if notify is None:
                 notify = self.always_notify_reporter
-            if notify:
+            if notify == 'True':
                 reporter = ticket['reporter']
                 if '@' in reporter:
                     name, authenticated, address = None, False, reporter
@@ -188,7 +188,7 @@
             notify = self._check_user_setting('notify_updater', event.author)
             if notify is None:
                 notify = self.always_notify_updater
-            if notify:
+            if notify == 'True':
                 self._log_sub(event.author, True, 'always_notify_updater')
                 return ('email', event.author, True, None)

Attachments (0)

Change History (4)

comment:1 Changed 6 years ago by Mazda86

Neglected to mention the affected file:

0.11/announcerplugin/subscribers/ticket_compat.py

comment:2 Changed 6 years ago by Robert Corsaro

Thanks, I'll look when I have time (next 2 weeks, probably)

comment:3 Changed 5 years ago by Rick Macdonald

I spent a few hours trying to get the configuration working with LegacyTicketSubscriber enabled, then went searching here for bug reports and found this one. The simple patch above fixed it for me (trac 0.11.7, AnnouncerPlugin 0.11.1). Perhaps the source could be fixed? I looked at the trunk to see if the fix is there, but the code seems to have been completely rewritten. I don't see ticket_compat.py at all.

comment:4 Changed 22 months ago by Ryan J Ollos

Owner: Robert Corsaro deleted

Modify Ticket

Action
as new The ticket will remain with no owner.

Add Comment


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

 
Note: See TracTickets for help on using tickets.