6.0.0-beta1
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
11/9/25
H
istory
A
ttachments
C
omment
W
atch
Download
Comment on [#1568] Observer Mechanism Implementation
*
Your Email Address
*
Spam protection
Enter the letters below:
. ..__ . .. \ / \ /[__)| || >< \/ [__)|__||___/ \
Comment
> 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. > > > > > >
Attachment
Watch this ticket
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers