[Zope3-checkins] CVS: Zope3/src/zope/app/services - event.py:1.31

Steve Alexander steve@cat-box.net
Tue, 17 Jun 2003 03:44:35 -0400


Update of /cvs-repository/Zope3/src/zope/app/services
In directory cvs.zope.org:/tmp/cvs-serv14002/src/zope/app/services

Modified Files:
	event.py 
Log Message:
Changed the comment to better describe the problem.


=== Zope3/src/zope/app/services/event.py 1.30 => 1.31 ===
--- Zope3/src/zope/app/services/event.py:1.30	Mon Jun 16 16:40:44 2003
+++ Zope3/src/zope/app/services/event.py	Tue Jun 17 03:44:35 2003
@@ -214,12 +214,12 @@
                 hubId = iter(hubIds).next()
                 # XXX This code path needs a unit test!
                 #     This code is also wrong.
-                #     The call to unsubscribeAll assumes that we are using
-                #     the subscriptions service. In fact, this could be
-                #     any service that implements ISubscriptionService.
-                #     The appropriate service should be sought using
-                #     self._subscribeToServiceName, and unsubscribeAll
-                #     should be called on that service.
+                #     The call to unsubscribeAll assumes that whatever class
+                #     mixes this class in provides an unsubscribeAll method
+                #     that correctly uses the self._subscribeToServiceName
+                #     to decide what it should be unsubscribing from.
+                #     This could be any service that implements
+                #     ISubscriptionService
                 unsubscribeAll(hubId, local_only=True)
 
             paths = clean_self._paths