Edgewall Software
Modify

Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#1954 closed enhancement (wontfix)

Generic notify class: rcpts should be attribute of class for it processing in begin_send()

Reported by: vyt@… Owned by: Jonas Borgström
Priority: normal Milestone:
Component: ticket system Version: devel
Severity: normal Keywords:
Cc: vyt@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Matthew Good)

Otherwise subclasses will need overwrite notify() instead of simple overwrite begin_send(). It needed, for example, for prepare list of recipients before sending.

I use trac from trunk [2116]

Attachments (0)

Change History (4)

comment:1 by anonymous, 19 years ago

Cc: vyt@… added

comment:2 by Matthew Good, 19 years ago

Description: modified (diff)
Resolution: wontfix
Status: newclosed

I've started on some refactoring of the notification system to allow easier extension, and due to the nature of Trac's plugin system the calling conventions for the notification classes will need to change, so this is going to be applicable.

Also, please use one of the Trac syntaxes for changesets such as [2116] or r2116 instead of "rev2116" (the description has been updated).

comment:3 by vyt@…, 19 years ago

Ok. But I use not only changeset r2116 but trunk at revision 2116 :)

comment:4 by Emmanuel Blot, 19 years ago

Sure, but 'trunk' is implicit, isn't it ? ;-)
The ticket could not refer to an official release (as the 'devel' version is specified in the ticket), and when we don't specify a 'branch', we default to the trunk subtree. - just kidding ;)

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.