Handle blank templates correctly

mwtemplates/templateeditor2.py

@@ -152,7 +152,9 @@ def __init__(self, editor, doc):
     def _templates(self):
         templates = []
         for node in self.doc.findall('.//template'):
-            templates.append(Template(node, self.editor))
+            q = Template(node, self.editor)
+            if q.key != '':
+                templates.append(q)
         return templates
 
     # def add(self, node):
@@ -435,6 +437,8 @@ def __repr__(self):
     def key(self):
         tmp = self._name.strip()
         tmp = re.sub(r'^(?:[Mm]al|[Tt]emplate):', '', tmp)
+        if len(tmp) == 0:
+            return ''
         return tmp[0].upper() + tmp[1:]
 
     @property

setup.py

@@ -12,7 +12,7 @@
 README = open(os.path.join(here, 'README.rst')).read()
 
 setup(name='mwtemplates',
-      version='0.3.1',
+      version='0.3.2',
       description='MediaWiki template parser and editor',
       long_description=README,
       classifiers=[

tests/templateeditor2_test.py

@@ -354,5 +354,17 @@ def test_delete_template(self):
         dp.templates['Infoboks A'][0].remove()  # Note that 'del dp.templates['Infoboks A'][0]' currently does not work
         self.assertEqual(dp.wikitext(), text2)
 
+    def test_blank_template(self):
+        # Should not crash...
+        text = 'Lorem ipsum {{}} dolor sit amet'
+        te = TemplateEditor(text)
+        self.assertEqual(len(te.templates), 0)
+
+    def test_blank_template_roundtrip(self):
+        # Check that a parameter can be deleted
+        text = 'Lorem ipsum {{ }} dolor sit amet'
+        dp = TemplateEditor(text)
+        self.assertEqual(dp.wikitext(), text)
+
 if __name__ == '__main__':
     unittest.main()