[CalendarServer-changes] [5919] CalendarServer/branches/new-store/twistedcaldav/resource.py

[source_changes at macosforge.org]

Revision: 5919
          http://trac.macosforge.org/projects/calendarserver/changeset/5919
Author:   glyph at apple.com
Date:     2010-07-19 17:45:43 -0700 (Mon, 19 Jul 2010)
Log Message:
-----------
remove now-redundant error handler

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

Modified: CalendarServer/branches/new-store/twistedcaldav/resource.py
===================================================================
--- CalendarServer/branches/new-store/twistedcaldav/resource.py	2010-07-20 00:23:39 UTC (rev 5918)
+++ CalendarServer/branches/new-store/twistedcaldav/resource.py	2010-07-20 00:45:43 UTC (rev 5919)
@@ -191,9 +191,11 @@
             raise RuntimeError("No associated transaction to propagate")
         otherResource.associateWithTransaction(self._associatedTransaction)
 
+
     def transactionError(self):
         self._transactionError = True
 
+
     def renderHTTP(self, request):
         """
         Override C{renderHTTP} to commit the transaction when the resource is
@@ -210,13 +212,9 @@
                 else:
                     self._associatedTransaction.commit()
             return result
-        def failed(failure):
-            print 'renderHTTP failed!  FIXME PLEASE: handle errors here!'
-            failure.printTraceback()
-            return failure
-        # FIXME: needs a failure handler (that rolls back the transaction)
-        return d.addCallback(succeeded).addErrback(failed)
+        return d.addCallback(succeeded)
 
+
     # Begin transitional new-store resource interface:
 
     def copyDeadPropertiesTo(self, other):
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20100719/c9697cbe/attachment.html>