[whups] E-Mail Notification on Ticket Chagnes

Mike Bydalek mbydalek at mobilemini.com
Tue Jul 20 11:22:37 PDT 2004

> Quoting Mike Bydalek <mbydalek at mobilemini.com>:
> > I noticed that the notify*() functions seemed to all combine into
> > notifyTicket().  Is there no more reason to send state 
> change, comment
> > update, etc. notifications to either both the client and/or "tech"?
> http://bugs.horde.org/details.php?id=46
> Not done yet.

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.

> > Then again, I can still see some risidual affects of the
> code change
> > that I've had to clean up (one big one being the comments being put
> > into the new ticket e-mail).  Let me know what your thoughts are on 
> > this so I can go all out for patch submission, or just need 
> to hack it
> > up enough for internal usage.
> Not really sure what you think are "residual effects". Try to
> be a bit more specific and a bit less "are you really being dumb?"

Oof - I'd think you'd give me a little more credit than that.  Based on the
answer for the first part dictated the answer for the 2nd because I know
that you weren't quite done yet. So what's the point of pointing out what's
broke if you know it already is (as seen in the bugs link)?

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.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: Whups.php.patch
Type: application/octet-stream
Size: 1093 bytes
Desc: not available
Url : http://lists.horde.org/archives/whups/attachments/20040720/8ab91e20/Whups.php.obj

More information about the whups mailing list