[Tickets #2332] Provide a notification that a task has been added to your tasklist

bugs@bugs.horde.org bugs at bugs.horde.org
Sun Nov 13 03:20:32 PST 2005


DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=2332
-----------------------------------------------------------------------
 Ticket             | 2332
 Updated By         | Jan Schneider <jan at horde.org>
 Summary            | Provide a notification that a task has been added to your tasklist
 Queue              | Nag
 Version            | HEAD
-State              | Accepted
+State              | Feedback
 Priority           | 1. Low
 Type               | Enhancement
 Owners             | 
-----------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2005-11-13 03:20) wrote:

>> How about splitting up add() to Driver::add() and Driver_foo::_add()?
>> History logging and notificiation would be done in add() and the
>> actual adding in _add().
>
> If I understand you right, that would mean leaving the backend 
> specific code in Driver_foo::add() and drawing the common code into 
> Driver::add()?

Not quite, the backend specific code goes into Driver_foo::_add(), because
you will call these methods from the parent class, not the parent class'
method from the inherited class.

> I really only wanted to get notifications working, but if it makes 
> sense to make those changes, I can certainly try :)

It makes a lot of sense, because you would make notifications work with
*all* driver at once.




More information about the bugs mailing list