[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-08-06 Thread Siegfried Gevatter
** Changed in: zeitgeist Milestone: 0.4.1 = 0.5.0 -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Zeitgeist Framework:

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-07-12 Thread Seif Lotfy
** Changed in: zeitgeist Assignee: (unassigned) = Seif Lotfy (seif) ** Changed in: zeitgeist Status: Fix Committed = Fix Released -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because you are a member of Zeitgeist

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-07-01 Thread Mikkel Kamstrup Erlandsen
** Changed in: zeitgeist Status: Triaged = Fix Committed -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Zeitgeist

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-26 Thread Seif Lotfy
On second thought i think a delete hook must be after.. since its the only way of knowing something was successfully deleted -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because you are a member of Zeitgeist Framework Team, which

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-26 Thread Seif Lotfy
BTW what should the delete_event_hook return? the whole event or the id -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-26 Thread Seif Lotfy
Does the extension need to know the sender of the delete ? ** Branch linked: lp:~zeitgeist/zeitgeist/delete-hooks -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because you are a member of Zeitgeist Framework Team, which is

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-26 Thread Seif Lotfy
Ok here is my frist draft of a delete_event_hook that takes in the ids of deleted events and it hooks in AFTER the events are deleted. Take a look the test case is in there too -- delete_event_hook for extensions https://bugs.launchpad.net/bugs/595919 You received this bug notification because

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-18 Thread Markus Korn
I agree, we can add this delete hook to extensions, but I'm not sure what this hook should be able to do. Is it to just notify an extension that a certain event is deleted, or is about to be deleted? Or can this hook also block the removal of an event? For notification purposes we have the

[Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-18 Thread Mikkel Kamstrup Erlandsen
I proposed one event_id because the insert hook also only receives one event. The idea was simply as a notification mechanism, I am not sure it's a good idea to allow extensions to bar events from deletion, although I can't fully see the ramifications of this... We can't use the current

Re: [Zeitgeist] [Bug 595919] Re: delete_event_hook for extensions

2010-06-18 Thread Siegfried Gevatter
2010/6/18 Seif Lotfy s...@lotfy.com: Maybe a possibility is to add to hooks delete_event_hook and delete_event_hook_after +1, so that if another extension decides a particular thing shouldn't be deleted FTS's hook won't be called. -- delete_event_hook for extensions