Added WithInkscapeEnabled to the ConfigurationFiles that can be used to enable Inkscape in the delegates.

Author: dlemstra

src/Magick.NET.Core/Configuration/IConfigurationFiles.cs

@@ -69,4 +69,10 @@ public interface IConfigurationFiles
     /// Gets the type-ghostscript configuration.
     /// </summary>
     IConfigurationFile TypeGhostscript { get; }
+
+    /// <summary>
+    /// Creates a new configuration with Inkscape enabled.
+    /// </summary>
+    /// <returns>A new configuration with Inkscape enabled.</returns>
+    public IConfigurationFiles WithInkscapeEnabled();
 }

src/Magick.NET/Configuration/ConfigurationFile.cs

@@ -13,6 +13,12 @@ public ConfigurationFile(string fileName)
         Data = LoadData();
     }
 
+    public ConfigurationFile(IConfigurationFile configurationFile)
+    {
+        FileName = configurationFile.FileName;
+        Data = configurationFile.Data;
+    }
+
     public string FileName { get; }
 
     public string Data { get; set; }

src/Magick.NET/Configuration/ConfigurationFiles.cs

@@ -10,25 +10,45 @@ namespace ImageMagick.Configuration;
 /// </summary>
 public sealed class ConfigurationFiles : IConfigurationFiles
 {
-    private ConfigurationFiles()
+    private ConfigurationFiles(
+        IConfigurationFile colors,
+        IConfigurationFile configure,
+        IConfigurationFile delegates,
+        IConfigurationFile english,
+        IConfigurationFile locale,
+        IConfigurationFile log,
+        IConfigurationFile policy,
+        IConfigurationFile thresholds,
+        IConfigurationFile type,
+        IConfigurationFile typeGhostscript)
     {
-        Colors = new ConfigurationFile("colors.xml");
-        Configure = new ConfigurationFile("configure.xml");
-        Delegates = new ConfigurationFile("delegates.xml");
-        English = new ConfigurationFile("english.xml");
-        Locale = new ConfigurationFile("locale.xml");
-        Log = new ConfigurationFile("log.xml");
-        Policy = new ConfigurationFile("policy.xml");
-        Thresholds = new ConfigurationFile("thresholds.xml");
-        Type = new ConfigurationFile("type.xml");
-        TypeGhostscript = new ConfigurationFile("type-ghostscript.xml");
+        Colors = colors;
+        Configure = configure;
+        Delegates = delegates;
+        English = english;
+        Locale = locale;
+        Log = log;
+        Policy = policy;
+        Thresholds = thresholds;
+        Type = type;
+        TypeGhostscript = typeGhostscript;
     }
 
     /// <summary>
     /// Gets the default configuration.
     /// </summary>
     public static IConfigurationFiles Default
-        => new ConfigurationFiles();
+        => new ConfigurationFiles(
+            colors: new ConfigurationFile("colors.xml"),
+            configure: new ConfigurationFile("configure.xml"),
+            delegates: new ConfigurationFile("delegates.xml"),
+            english: new ConfigurationFile("english.xml"),
+            locale: new ConfigurationFile("locale.xml"),
+            log: new ConfigurationFile("log.xml"),
+            policy: new ConfigurationFile("policy.xml"),
+            thresholds: new ConfigurationFile("thresholds.xml"),
+            type: new ConfigurationFile("type.xml"),
+            typeGhostscript: new ConfigurationFile("type-ghostscript.xml"));
 
     /// <summary>
     /// Gets all the configuration files.
@@ -105,4 +125,26 @@ IConfigurationFiles IConfigurationFiles.Default
     /// Gets the type-ghostscript configuration.
     /// </summary>
     public IConfigurationFile TypeGhostscript { get; }
+
+    /// <summary>
+    /// Creates a new configuration with Inkscape enabled.
+    /// </summary>
+    /// <returns>A new configuration with Inkscape enabled.</returns>
+    public IConfigurationFiles WithInkscapeEnabled()
+    {
+        var delegates = new ConfigurationFile(Delegates);
+        delegates.Data = delegates.Data.Replace(@"<!--<delegate decode=""svg:decode"" stealth=""True"" command=""&quot;inkscape&quot; &quot;%s&quot; - -export-filename=&quot;%s&quot; - -export-dpi=&quot;%s&quot; - -export-background=&quot;%s&quot; - -export-background-opacity=&quot;%s&quot; &gt; &quot;%s&quot; 2&gt;&amp;1""/>-->", @"<delegate decode=""svg:decode"" stealth=""True"" command=""&quot;inkscape&quot; &quot;%s&quot; --export-filename=&quot;%s&quot; --export-dpi=&quot;%s&quot; --export-background=&quot;%s&quot; --export-background-opacity=&quot;%s&quot; &gt; &quot;%s&quot; 2&gt;&amp;1""/>");
+
+        return new ConfigurationFiles(
+            colors: new ConfigurationFile(Colors),
+            configure: new ConfigurationFile(Configure),
+            delegates: new ConfigurationFile(delegates),
+            english: new ConfigurationFile(English),
+            locale: new ConfigurationFile(Locale),
+            log: new ConfigurationFile(Log),
+            policy: new ConfigurationFile(Policy),
+            thresholds: new ConfigurationFile(Thresholds),
+            type: new ConfigurationFile(Type),
+            typeGhostscript: new ConfigurationFile(TypeGhostscript));
+    }
 }

tests/Magick.NET.Tests/Configuration/ConfigurationFilesTests/TheWithInkscapeEnabledMethod.cs

@@ -0,0 +1,43 @@
+// Copyright Dirk Lemstra https://github.com/dlemstra/Magick.NET.
+// Licensed under the Apache License, Version 2.0.
+
+using ImageMagick.Configuration;
+using Xunit;
+
+namespace Magick.NET.Tests;
+
+public partial class ConfigurationFilesTests
+{
+    public class TheWithInkscapeEnabledMethod
+    {
+        [Fact]
+        public void ShouldReturnConfigurationWithInkScapeEnabled()
+        {
+            var defaultConfiguration = ConfigurationFiles.Default;
+            defaultConfiguration.Colors.Data = "colors";
+            defaultConfiguration.Configure.Data = "configure";
+            defaultConfiguration.Delegates.Data = defaultConfiguration.Delegates.Data.Replace("\"browse\"", "\"foobar\"");
+            defaultConfiguration.English.Data = "english";
+            defaultConfiguration.Locale.Data = "locale";
+            defaultConfiguration.Log.Data = "log";
+            defaultConfiguration.Policy.Data = "policy";
+            defaultConfiguration.Thresholds.Data = "thresholds";
+            defaultConfiguration.Type.Data = "type";
+            defaultConfiguration.TypeGhostscript.Data = "typeghostscript";
+
+            var configuration = defaultConfiguration.WithInkscapeEnabled();
+
+            Assert.Contains(" <delegate decode=\"svg:decode\" stealth=\"True\" command=\"&quot;inkscape&quot; &quot;%s&quot; --export-filename=&quot;%s&quot; --export-dpi=&quot;%s&quot; --export-background=&quot;%s&quot; --export-background-opacity=&quot;%s&quot; &gt; &quot;%s&quot; 2&gt;&amp;1\"/>", configuration.Delegates.Data);
+            Assert.Contains("\"foobar\"", configuration.Delegates.Data);
+            Assert.Equal("colors", configuration.Colors.Data);
+            Assert.Equal("configure", configuration.Configure.Data);
+            Assert.Equal("english", configuration.English.Data);
+            Assert.Equal("locale", configuration.Locale.Data);
+            Assert.Equal("log", configuration.Log.Data);
+            Assert.Equal("policy", configuration.Policy.Data);
+            Assert.Equal("thresholds", configuration.Thresholds.Data);
+            Assert.Equal("type", configuration.Type.Data);
+            Assert.Equal("typeghostscript", configuration.TypeGhostscript.Data);
+        }
+    }
+}