[CalendarServer-changes] [3361] CalendarServer/trunk/twistedcaldav/method
source_changes at macosforge.org
source_changes at macosforge.org
Mon Nov 10 18:28:06 PST 2008
Revision: 3361
http://trac.macosforge.org/projects/calendarserver/changeset/3361
Author: cdaboo at apple.com
Date: 2008-11-10 18:28:06 -0800 (Mon, 10 Nov 2008)
Log Message:
-----------
Minor clean-up.
Modified Paths:
--------------
CalendarServer/trunk/twistedcaldav/method/delete.py
CalendarServer/trunk/twistedcaldav/method/put_common.py
Modified: CalendarServer/trunk/twistedcaldav/method/delete.py
===================================================================
--- CalendarServer/trunk/twistedcaldav/method/delete.py 2008-11-11 01:38:43 UTC (rev 3360)
+++ CalendarServer/trunk/twistedcaldav/method/delete.py 2008-11-11 02:28:06 UTC (rev 3361)
@@ -113,16 +113,12 @@
# Do some clean up
yield self.deletedCalendar(request)
-
- if lock:
- yield lock.release()
except MemcacheLockTimeoutError:
raise HTTPError(StatusResponse(responsecode.CONFLICT, "Resource: %s currently in use on the server." % (self.uri,)))
- except Exception, e:
+ finally:
if lock:
yield lock.clean()
- raise e
returnValue(response)
Modified: CalendarServer/trunk/twistedcaldav/method/put_common.py
===================================================================
--- CalendarServer/trunk/twistedcaldav/method/put_common.py 2008-11-11 01:38:43 UTC (rev 3360)
+++ CalendarServer/trunk/twistedcaldav/method/put_common.py 2008-11-11 02:28:06 UTC (rev 3361)
@@ -147,9 +147,9 @@
def __init__(self, index, uid, uri, internal_request):
if internal_request:
- self._lock = None
+ self.lock = None
else:
- self._lock = MemcacheLock("ImplicitUIDLock", uid, timeout=60.0)
+ self.lock = MemcacheLock("ImplicitUIDLock", uid, timeout=60.0)
self.reserved = False
self.index = index
self.uid = uid
@@ -159,9 +159,9 @@
def reserve(self):
# Implicit lock
- if self._lock:
+ if self.lock:
try:
- yield self._lock.acquire()
+ yield self.lock.acquire()
except MemcacheLockTimeoutError:
raise HTTPError(StatusResponse(responsecode.CONFLICT, "Resource: %s currently in use on the server." % (self.uri,)))
@@ -184,8 +184,8 @@
yield pause
if self.uri and not self.reserved:
- if self._lock:
- yield self._lock.release()
+ if self.lock:
+ yield self.lock.release()
raise HTTPError(StatusResponse(responsecode.CONFLICT, "Resource: %s currently in use in calendar." % (self.uri,)))
@inlineCallbacks
@@ -193,8 +193,8 @@
if self.reserved:
yield self.index.unreserveUID(self.uid)
self.reserved = False
- if self._lock:
- yield self._lock.clean()
+ if self.lock:
+ yield self.lock.clean()
def __init__(
self,
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20081110/b0db23b3/attachment.html>
More information about the calendarserver-changes
mailing list