Commit 08cda23b authored by Andreas Jung's avatar Andreas Jung

      - WebDAV: removing an non-existing property now returns a HTTP
        200-OK response instead of 404 (now compliant with RFC 2518)
parent 8d721ad5
...@@ -107,6 +107,9 @@ Zope Changes ...@@ -107,6 +107,9 @@ Zope Changes
- Collector #285: Zope changes its working directory - Collector #285: Zope changes its working directory
to the var directory at startup to the var directory at startup
- WebDAV: removing an non-existing property now returns a HTTP
200-OK response instead of 404 (now compliant with RFC 2518)
- Fixed a bug in TM.py that would cause database adapters to hang - Fixed a bug in TM.py that would cause database adapters to hang
on errors in the second phase of the two-phase commit. on errors in the second phase of the two-phase commit.
...@@ -115,6 +118,7 @@ Zope Changes ...@@ -115,6 +118,7 @@ Zope Changes
- Collector #266: Retried requests losing track of http request - Collector #266: Retried requests losing track of http request
headers, causing Connection:Close requests to stall headers, causing Connection:Close requests to stall
>>>>>>> 1.440
Zope 2.5.1 beta 1 Zope 2.5.1 beta 1
Bugs Fixed Bugs Fixed
......
...@@ -13,7 +13,7 @@ ...@@ -13,7 +13,7 @@
"""WebDAV xml request objects.""" """WebDAV xml request objects."""
__version__='$Revision: 1.17 $'[11:-2] __version__='$Revision: 1.18 $'[11:-2]
import sys, os import sys, os
from common import absattr, aq_base, urlfix, urlbase from common import absattr, aq_base, urlfix, urlbase
...@@ -241,8 +241,9 @@ class PropPatch: ...@@ -241,8 +241,9 @@ class PropPatch:
name, ns=value name, ns=value
propset=propsets.get(ns, None) propset=propsets.get(ns, None)
if propset is None or not propset.hasProperty(name): if propset is None or not propset.hasProperty(name):
errors.append('Property not found: %s' % name) # removing a non-existing property is not an error!
status='404 Not Found' # according to RFC 2518
status='200 OK'
else: else:
try: propset._delProperty(name) try: propset._delProperty(name)
except: except:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment