Fixed TOC Option parsing.

The new option parser assumes bool values if the default is bool or
None. As the "title" option is not a bool value, it should default to an
empty string rather than None. Fixes #347.

Author: Waylan Limberg

markdown/extensions/toc.py

@@ -216,9 +216,9 @@ def __init__(self, *args, **kwargs):
             "slugify" : [slugify,
                 "Function to generate anchors based on header text - "
                 "Defaults to the headerid ext's slugify function."],
-            "title" : [None,
+            "title" : ["",
                 "Title to insert into TOC <div> - "
-                "Defaults to None"],
+                "Defaults to an empty string"],
             "anchorlink" : [0,
                 "1 if header should be a self link - "
                 "Defaults to 0"],

tests/test_apis.py

@@ -590,7 +590,7 @@ def create_config_file(self, config):
         with os.fdopen(fd, 'w') as fp:
             fp.write(config)
 
-    def testExtensonConfigOption(self):
+    def testExtensionConfigOption(self):
         config = {
         'markdown.extensions.wikilinks': {
             'base_url': 'http://example.com/',
@@ -606,6 +606,19 @@ def testExtensonConfigOption(self):
         self.default_options['extension_configs'] = config
         self.assertEqual(options, self.default_options)
 
+    def textBoolExtensionConfigOption(self):
+        config = {
+        'markdown.extensions.toc': {
+            'title': 'Some Title',
+            'anchorlink': True,
+            'permalink': True
+            }
+        }
+        self.create_config_file(config)
+        options, logging_level = parse_options(['-c', self.tempfile])
+        self.default_options['extension_configs'] = config
+        self.assertEqual(options, self.default_options)
+
     def testExtensonConfigOptionAsJSON(self):
         config = {
         'markdown.extensions.wikilinks': {