Omit category if term if blank

planet/reconstitute.py

@@ -111,8 +111,8 @@ def date(xentry, name, parsed):
 
 def category(xentry, tag):
     xtag = xentry.ownerDocument.createElement('category')
-    if tag.has_key('term') and tag.term:
-        xtag.setAttribute('term', tag.get('term'))
+    if not tag.has_key('term') or not tag.term: return
+    xtag.setAttribute('term', tag.get('term'))
     if tag.has_key('scheme') and tag.scheme:
         xtag.setAttribute('scheme', tag.get('scheme'))
     if tag.has_key('label') and tag.label:

tests/data/reconstitute/category_blank_term.xml

@@ -0,0 +1,11 @@
+<!--
+Description:  category term
+Expect:       not globals().has_key('tags')
+-->
+
+<feed xmlns="http://www.w3.org/2005/Atom">
+  <entry>
+    <category term=""/>
+  </entry>
+</feed>
+

tests/data/reconstitute/category_label.xml

@@ -5,7 +5,7 @@ Expect:       tags[0].label == 'Inbox'
 
 <feed xmlns="http://www.w3.org/2005/Atom">
   <entry>
-    <category label="Inbox"/>
+    <category term='x' label="Inbox"/>
   </entry>
 </feed>
 

tests/data/reconstitute/category_scheme.xml

@@ -5,7 +5,7 @@ Expect:       tags[0].scheme == 'http://example.com/categories'
 
 <feed xmlns="http://www.w3.org/2005/Atom">
   <entry>
-    <category scheme="http://example.com/categories"/>
+    <category term='x' scheme="http://example.com/categories"/>
   </entry>
 </feed>
 

tests/reconstitute.py

@@ -41,7 +41,7 @@ if __name__ == "__main__":
     from planet import spider
     spider.spiderPlanet(only_if_new=False)
 
-    from planet import feedparser
+    import feedparser
     for source in glob.glob(os.path.join(work, 'sources/*')):
         feed = feedparser.parse(source).feed
         if feed.has_key('title'):