[CalendarServer-changes] [2491] CalendarServer/trunk/twistedcaldav/directory/calendaruserproxy.py

source_changes at macosforge.org source_changes at macosforge.org
Sat May 24 20:02:37 PDT 2008


Revision: 2491
          http://trac.macosforge.org/projects/calendarserver/changeset/2491
Author:   wsanchez at apple.com
Date:     2008-05-24 20:02:36 -0700 (Sat, 24 May 2008)

Log Message:
-----------
Don't create a variable you never intend to use.
yield doesn't return, use returnValue.

Modified Paths:
--------------
    CalendarServer/trunk/twistedcaldav/directory/calendaruserproxy.py

Modified: CalendarServer/trunk/twistedcaldav/directory/calendaruserproxy.py
===================================================================
--- CalendarServer/trunk/twistedcaldav/directory/calendaruserproxy.py	2008-05-24 07:35:32 UTC (rev 2490)
+++ CalendarServer/trunk/twistedcaldav/directory/calendaruserproxy.py	2008-05-25 03:02:36 UTC (rev 2491)
@@ -189,10 +189,9 @@
         # Map the principals to UIDs.
         uids = [p.principalUID() for p in principals]
 
-        _ignore = yield self._index().setGroupMembers(self.uid, uids)
+        yield self._index().setGroupMembers(self.uid, uids)
         changed = yield self.parent.cacheNotifier.changed()
-        yield True
-        return
+        returnValue(True)
 
     ##
     # HTTP

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20080524/8c1acf4b/attachment.htm 


More information about the calendarserver-changes mailing list