Stricter validation of items

vdirsyncer/storage/base.py

@@ -15,7 +15,7 @@ class Item(object):
 
     '''should-be-immutable wrapper class for VCALENDAR and VCARD'''
 
-    def __init__(self, raw):
+    def __init__(self, raw, needs_uid=True):
         assert isinstance(raw, utils.text_type)
         raw = raw.splitlines()
         self.uid = None
@@ -23,6 +23,10 @@ class Item(object):
         for line in raw:
             if line.startswith(u'UID:'):
                 self.uid = line[4:].strip()
+
+        if needs_uid and self.uid is None:
+            raise ValueError('Item has not uid.')
+
         self.raw = u'\n'.join(raw)
 
 

vdirsyncer/storage/http.py

@@ -50,7 +50,7 @@ def split_collection(text):
                     item.extend(timezone)
                 item.append(line)
                 if item_type != u'VTIMEZONE':
-                    yield Item(u'\n'.join(item))
+                    yield Item(u'\n'.join(item), needs_uid=False)
                 item = []
                 item_type = None
             else: