Opened 13 years ago
Last modified 11 years ago
#9154 closed defect
ticket deletion error — at Version 2
Reported by: | Owned by: | Ryan J Ollos | |
---|---|---|---|
Priority: | normal | Component: | AnnouncerPlugin |
Severity: | normal | Keywords: | |
Cc: | Trac Release: | 0.12 |
Description (last modified by )
Error while deleting ticket:
Traceback (most recent call last): File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/admin/console.py", line 107, in onecmd rv = cmd.Cmd.onecmd(self, line) or 0 File "/usr/local/lib/python2.6/cmd.py", line 218, in onecmd return self.default(line) File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/admin/console.py", line 266, in default return cmd_mgr.execute_command(*args) File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/admin/api.py", line 123, in execute_command return f(*fargs) File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/ticket/admin.py", line 838, in _do_remove @self.env.with_transaction() File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/db/api.py", line 77, in transaction_wrapper fn(ldb) File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/ticket/admin.py", line 841, in do_remove ticket.delete() File "build/bdist.freebsd-8.2-STABLE-amd64/egg/trac/ticket/model.py", line 421, in delete listener.ticket_deleted(self) File "build/bdist.freebsd-8.2-STABLE-amd64/egg/announcer/subscribers.py", line 844, in ticket_deleted self.env, klass, 'ticket', page.name) NameError: global name 'page' is not defined
caused by typo in WatchSubscriber.ticket_deleted
.
Fix:
Index: subscribers.py =================================================================== --- subscribers.py.orig +++ subscribers.py @@ -841,7 +841,7 @@ def ticket_deleted(self, ticket): klass = self.__class__.__name__ SubscriptionAttribute.delete_by_class_realm_and_target( - self.env, klass, 'ticket', page.name) + self.env, klass, 'ticket', ticket.name) db = self.env.get_db_cnx() def matches(self, event):
Change History (2)
comment:1 Changed 13 years ago by
comment:2 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Owner: | changed from Robert Corsaro to Ryan J Ollos |
Status: | new → assigned |
Note: See
TracTickets for help on using
tickets.
Correct patch actually looks like: