[CalendarServer-changes] [5683] CalendarServer/branches/new-store/twistedcaldav/static.py

source_changes at macosforge.org source_changes at macosforge.org
Fri Jun 4 11:36:12 PDT 2010


Revision: 5683
          http://trac.macosforge.org/projects/calendarserver/changeset/5683
Author:   glyph at apple.com
Date:     2010-06-04 11:36:09 -0700 (Fri, 04 Jun 2010)
Log Message:
-----------
If you've already got a filepath, don't use os.path

Modified Paths:
--------------
    CalendarServer/branches/new-store/twistedcaldav/static.py

Modified: CalendarServer/branches/new-store/twistedcaldav/static.py
===================================================================
--- CalendarServer/branches/new-store/twistedcaldav/static.py	2010-06-04 18:32:02 UTC (rev 5682)
+++ CalendarServer/branches/new-store/twistedcaldav/static.py	2010-06-04 18:36:09 UTC (rev 5683)
@@ -227,7 +227,7 @@
             log.err("Attempt to create collection where file exists: %s" % (self.fp.path,))
             raise HTTPError(StatusResponse(responsecode.NOT_ALLOWED, "File exists"))
 
-        if not os.path.isdir(os.path.dirname(self.fp.path)):
+        if not self.fp.parent().isdir():
             log.err("Attempt to create collection with no parent: %s" % (self.fp.path,))
             raise HTTPError(StatusResponse(responsecode.CONFLICT, "No parent collection"))
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20100604/73b391a7/attachment.html>


More information about the calendarserver-changes mailing list