Fix XML querying bug

Fix #480

tests/storage/dav/test_main.py

@@ -0,0 +1,27 @@
+from vdirsyncer.storage.dav import _parse_xml, _merge_xml
+
+def test_xml_utilities():
+    x = _parse_xml('''<?xml version="1.0" encoding="UTF-8" ?>
+        <D:multistatus xmlns:D="DAV:">
+            <D:response>
+                <D:propstat>
+                    <D:status>HTTP/1.1 404 Not Found</D:status>
+                    <D:prop>
+                        <D:getcontenttype/>
+                    </D:prop>
+                </D:propstat>
+                <D:propstat>
+                    <D:prop>
+                        <D:resourcetype>
+                            <D:collection/>
+                        </D:resourcetype>
+                    </D:prop>
+                </D:propstat>
+            </D:response>
+        </D:multistatus>
+    ''')
+
+    response = x.find('{DAV:}response')
+    props = _merge_xml(response.findall('{DAV:}propstat/{DAV:}prop'))
+    assert props.find('{DAV:}resourcetype/{DAV:}collection') is not None
+    assert props.find('{DAV:}getcontenttype') is not None

vdirsyncer/storage/dav.py

@@ -79,7 +79,8 @@ def _parse_xml(content):
 
 def _merge_xml(items):
     rv = items[0]
-    rv.extend(items[1:])
+    for item in items[1:]:
+        rv.extend(item.getiterator())
     return rv
 
 
@@ -544,6 +545,7 @@ class DavStorage(Storage):
             if not etag:
                 dav_logger.warning('Skipping {!r}, etag property is missing.'
                                    .format(href))
+                continue
 
             contenttype = getattr(props.find('{DAV:}getcontenttype'),
                                   'text', None)