Merge pull request #337 from FromDoppler/dop-1520-save-content-links

Dop 1520 save content links

Author: mmosquera

Doppler.HtmlEditorApi.Test/Domain/DopplerHtmlDocumentTest.cs

@@ -307,6 +307,22 @@ public void GetTrackableUrls_should_not_map_field_names()
             link => Assert.Equal("ftp://|*|3*|*", link));
     }
 
+    [Fact]
+    public void GetTrackableUrls_remove_encoding_on_sanitize_to_save()
+    {
+        // Arrange
+        var input = "<p><a href=\"https://midomain.com/?param1=aa&amp;param2=bb\">This is a link with parameters</a></p>";
+        var htmlDocument = new DopplerHtmlDocument(input);
+        htmlDocument.GetDopplerContent();
+        htmlDocument.SanitizeTrackableLinks();
+
+        // Act
+        var links = htmlDocument.GetTrackableUrls();
+
+        // Assert
+        Assert.Equal("https://midomain.com/?param1=aa&param2=bb", links.FirstOrDefault());
+    }
+
     [Theory]
     [InlineDataAttribute(
         @"

Doppler.HtmlEditorApi/Domain/DopplerHtmlDocument.cs

@@ -151,7 +151,7 @@ private static string SanitizedUrl(string url)
         var domain = match.Groups["domain"].Value
             .FallbackIfNullOrEmpty(match.Groups["domainWithoutScheme"].Value)
             .ToLowerInvariant();
-        var rest = match.Groups["rest"].Value;
+        var rest = match.Groups["rest"].Value.Replace("&", "&");
         var sanitizedUrl = $"{scheme}{domain}{rest}";
 
         return sanitizedUrl;