LegacyTicket preferences do not have any effect

[Mazda86]

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)

[Mazda86]

Neglected to mention the affected file:

0.11/announcerplugin/subscribers/ticket_compat.py

[Robert Corsaro]

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

[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.