[whups] E-Mail Notification on Ticket Chagnes
Chuck Hagenbuch
chuck at horde.org
Tue Jul 20 19:51:02 PDT 2004
Quoting Mike Bydalek <mbydalek at mobilemini.com>:
> This makes sense...but by deleting the notify*() methods and leaving just
> the notifyTicket() one, are you going to use this to to differentiate
> between New Ticket, New Comment, New Attachment, etc.? I'm just curious as
> to which way you are thinking of going.
It'll all be done in Ticket.php (as part of commit(), with a possible
exception
for new tickets, but probably createTicket() will just be moved into
Ticket.php) when it's done. All relevant info will be in one email.
> But since you requested a specific, one is in either _formatComments
> (Driver.php) or permissionsFilter (Whups.php) - depending on how you think
> it should be addressed. Basically _formatComments tried to call
> $comments['comment_text'] when it's not even set by the permissionsFilter.
> So, for now, I just added a couple lines in the permissionsFilter to create
> this key.
>
> And, to avoid the ever-so-famous "patch?" - I attached what I did. I
> thought the permissionsFilter was more appropriate since in order to
> accomplish this in the formatComments, it basically required the same code.
Committed for now, thanks.
-chuck
--
"Regard my poor demoralized mule!" - Juan Valdez
More information about the whups
mailing list