changeset 129:43102fecb14b

Fix some typos. Don't expect ids of removed items back from storage, but use the list we got ourselves.
author Ralph Meijer <ralphm@ik.nu>
date Sun, 24 Apr 2005 17:24:35 +0000
parents b27a66637a10
children 3d77f3808bfa
files idavoll/generic_backend.py
diffstat 1 files changed, 8 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/idavoll/generic_backend.py	Sun Apr 24 17:23:12 2005 +0000
+++ b/idavoll/generic_backend.py	Sun Apr 24 17:24:35 2005 +0000
@@ -5,7 +5,7 @@
 from twisted.xish import utility
 from twisted.internet import defer
 from zope.interface import implements
-import backend
+import backend, storage
 
 def _get_affiliation(node, entity):
     d = node.get_affiliation(entity)
@@ -103,6 +103,7 @@
             d = defer.succeed(None)
 
         d.addCallback(self._do_notify, node.id, items, deliver_payloads)
+        return d
 
     def _do_notify(self, result, node_id, items, deliver_payloads):
         if items and not deliver_payloads:
@@ -154,12 +155,12 @@
 
         d = node.add_subscription(subscriber, 'subscribed')
         d.addCallback(lambda _: 'subscribed')
-        d.addErrback(self._get_subscription, node)
+        d.addErrback(self._get_subscription, node, subscriber)
         d.addCallback(self._return_subscription, affiliation, node.id)
         return d
 
-    def _get_subscription(self, failure, node):
-        failure.Trap(storage.SubscriptionExists)
+    def _get_subscription(self, failure, node, subscriber):
+        failure.trap(storage.SubscriptionExists)
         return node.get_subscription(subscriber)
 
     def _return_subscription(self, result, affiliation, node_id):
@@ -320,11 +321,11 @@
             raise backend.NodeNotPersistent
                                                                                 
         d = node.remove_items(item_ids)
-        d.addCallback(self._do_notify_retraction, node.id)
+        d.addCallback(self._do_notify_retraction, item_ids, node.id)
         return d
                                                                                 
-    def _do_notify_retraction(self, result, node_id):
-        self.parent.dispatch({ 'item_ids': result, 'node_id': node_id },
+    def _do_notify_retraction(self, result, item_ids, node_id):
+        self.parent.dispatch({ 'item_ids': item_ids, 'node_id': node_id },
                              '//event/pubsub/retract')
                                                                                 
     def purge_node(self, node_id, requestor):