[Tickets #1568] Observer Mechanism Implementation

bugs@bugs.horde.org bugs at bugs.horde.org
Tue May 24 05:40:40 PDT 2005


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

Ticket URL: http://bugs.horde.org/ticket/?id=1568
-----------------------------------------------------------------------
 Ticket             | 1568
 Updated By         | Jason Felice <jfelice at cronosys.com>
 Summary            | Observer Mechanism Implementation
 Queue              | Horde Framework Packages
 State              | Feedback
 Priority           | 1. Low
 Type               | Enhancement
 Owners             | 
+New Attachment     | Observer.php
-----------------------------------------------------------------------


Jason Felice <jfelice at cronosys.com> (2005-05-24 05:40) wrote:

Here's an update to my tastes, see what you think:

1) Remove integer contstants, use strings like 'delete' instead.
2) Allow an observer to return false to abort.
3) Remove validateDeletion(), instead app would use notifyObservers() with a
different "message".
4) $manager set by reference in ::singleton().

A couple of thoughts I wonder about:

* I'd prefer ObserverSite:: to Observer_Manager:: as the class name.
* What about using the Memento pattern in the ObserverSite:: and attached
Observer:: instances and not directly talk to DataTree?  That way, whups can
store the memento in the tickets table, for example.







More information about the bugs mailing list