Custom Query (332 matches)

Hello,

I realize that there are other tickets on this subject, but this is (in my opinion) slightly different. The similarity is that notifications only get sent out when adding a ticket via the web interface. Adding via email2trac works, but no notification is sent. When running the test with the given mail, I get the following:

# cat email2trac-0.13/msg.txt | email2trac --project=pprovost
TD: Failure sending notification on creation of ticket #18: 'str' object is not callable

This is with Trac 0.11 and email2trac 0.13

When sending a ticket update via mail to a project that has no components, email2trac will manage to update the ticket in the database, but the email notification isn't sent.

It will fail with:

email2trac: Traceback (most recent call last):
email2trac:   File "/usr/local/bin/email2trac", line 1137, in <module>     tktparser.parse(sys.stdin)
email2trac:   File "/usr/local/bin/email2trac", line 723, in parse     if self.ticket_update(m):
email2trac:   File "/usr/local/bin/email2trac", line 602, in ticket_update     self.notify(tkt, False, when)
email2trac:   File "/usr/local/bin/email2trac", line 857, in notify     if tkt['component'] == 'Spam':
email2trac:   File "/var/lib/python-support/python2.5/trac/ticket/model.py", line 104, in __getitem__     return self.values[name]
email2trac: KeyError: 'component' 

Given a non-spam email, i.e. the X-Spam-Score header is present, but has no value, the script will crash with:

Traceback (most recent call last):
  File "/usr/bin/email2trac", line 1137, in ?     tktparser.parse(sys.stdin)
  File "/usr/bin/email2trac", line 710, in parse     component =  self.spam(m)
  File "/usr/bin/email2trac", line 245, in spam     number = spam_l[0].count('*')
IndexError: list index out of range

Patch as follows:

--- email2trac.orig     2008-05-29 15:02:15.000000000 -0700
+++ email2trac  2008-05-29 15:04:09.000000000 -0700
@@ -242,7 +242,10 @@
                spam = False
                if message.has_key('X-Spam-Score'):
                        spam_l = string.split(message['X-Spam-Score'])
-                       number = spam_l[0].count('*')
+                       if len(spam_l) == 0:
+                               number = 0
+                       else:
+                               number = spam_l[0].count('*')
 
                        if number >= self.SPAM_LEVEL:
                                spam = True