Edgewall Software
Modify

Opened 20 years ago

Closed 20 years ago

#544 closed defect (fixed)

Email notifications are broken by [747]

Reported by: mrowe@… Owned by: Jonas Borgström
Priority: normal Milestone:
Component: general Version: 0.7.1
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Some of the changes made in [747] break email notifications. Attempting to create or update a ticket results in a traceback similar to the following:

Traceback (most recent call last):
  File "/usr/lib/python2.3/site-packages/trac/core.py", line 492, in cgi_start
    real_cgi_start()
  File "/usr/lib/python2.3/site-packages/trac/core.py", line 487, in real_cgi_start
    dispatch_request(path_info, args, req, env, database)
  File "/usr/lib/python2.3/site-packages/trac/core.py", line 397, in dispatch_request
    module.run()
  File "/usr/lib/python2.3/site-packages/trac/Module.py", line 45, in run
    self.render()
  File "/usr/lib/python2.3/site-packages/trac/Ticket.py", line 387, in render
    self.save_changes (id)
  File "/usr/lib/python2.3/site-packages/trac/Ticket.py", line 322, in save_changes
    tn.notify(ticket, newticket=0, modtime=time.time())
  File "/usr/lib/python2.3/site-packages/trac/Notify.py", line 223, in notify
    NotifyEmail.notify(self, ticket['id'], subject)
  File "/usr/lib/python2.3/site-packages/trac/Notify.py", line 127, in notify
    Notify.notify(self, resid)
  File "/usr/lib/python2.3/site-packages/trac/Notify.py", line 74, in notify
    rcpts = self.get_recipients(resid)
  File "/usr/lib/python2.3/site-packages/trac/Notify.py", line 270, in get_recipients
    cursor.execute('SELECT reporter,cc FROM ticket WHERE id=%s', ticket['id'])
TypeError: string indices must be integers

This appears to be due to using ticket['id'] instead of either self.ticket['id'] or just plain ticket.

Attachments (0)

Change History (2)

comment:1 by Jonas Borgström, 20 years ago

Status: newassigned

comment:2 by Jonas Borgström, 20 years ago

Resolution: fixed
Status: assignedclosed

Should be fixed in [761].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.