Change MarkdigExtensions configuration property to be a comma delimit…

…ed string instead of + delimited as Markdig requires for its `Configure()` method.
RickStrahl · Sep 5, 2017 · a19ea88 · a19ea88
1 parent 4d4b891
commit a19ea88
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/MarkdownMonster/_Classes/Configuration/MarkdownOptionsConfiguration.cs b/MarkdownMonster/_Classes/Configuration/MarkdownOptionsConfiguration.cs
@@ -80,12 +80,12 @@ public string MarkdigExtensions
             get { return _markdigExtensions; }
             set
             {
-                if (value == _markdigExtensions) return;
+                if (value == _markdigExtensions) return;                
                 _markdigExtensions = value;
                 OnPropertyChanged();
             }
         }
-        private string _markdigExtensions = "emphasisextras+pipetables+gridtables+footers+footnotes+citations+attributes";
+        private string _markdigExtensions = "emphasisextras,pipetables,gridtables,footers,footnotes,citations,attributes";
 
         #endregion
 

diff --git a/MarkdownMonster/_Classes/MarkdownParser/MarkdownParserMarkDig.cs b/MarkdownMonster/_Classes/MarkdownParser/MarkdownParserMarkDig.cs
@@ -122,15 +122,13 @@ protected virtual MarkdownPipelineBuilder CreatePipelineBuilder()
             {
                 if (!string.IsNullOrWhiteSpace(options.MarkdigExtensions))
                 {
-                    builder = builder.Configure(options.MarkdigExtensions);
+                    builder = builder.Configure(options.MarkdigExtensions.Replace(",","+"));
                 }
             }
             catch(ArgumentException ex)
             {
                 // One or more of the extension options is invalid. 
-                // Processing of the extensions stopped at this point.
-                // Log an error.
-                mmApp.Log(ex);
+                mmApp.Log("Failed to load Markdig extensions: " + options.MarkdigExtensions + "\r\n" + ex.Message,ex);
             }
 
             return builder;