• Chris McDonough's avatar
    Tests 15 (propnullns) and 16 (propget) from the props suite · 0e9e7a53
    Chris McDonough authored
    of litmus version 10.5 (http://www.webdav.org/neon/litmus/)
    expose a bug in Zope propertysheet access via DAV.  If a
    proppatch command sets a property with a null xmlns,
    e.g. with a PROPPATCH body like:
    
    <?xml version="1.0" encoding="utf-8" ?>
    <propertyupdate xmlns="DAV:">
    <set>
    <prop>
     <nonamespace xmlns="">randomvalue</nonamespace>
    </prop>
    </set>
    </propertyupdate>
    
    When we set properties in the null namespace, Zope turns
    around and creates (or finds) a propertysheet with the
    xml_namespace of None and sets the value on it.  The
    response to a subsequent PROPFIND for the resource will fail
    because the XML generated by dav__propstat included a bogus
    namespace declaration (xmlns="None").
    
    Fixed by amending OFS.PropertySheets.dav__propstat.
    0e9e7a53
testProperties.py 4.74 KB