Merge pull request #153 from ctolkien/turkish-i

Fixes handling of `Turkish I`

Author: ctolkien

src/Slugify.Core/SlugHelper.cs

@@ -34,7 +34,7 @@ public virtual string GenerateSlug(string inputString)
         var normalizedInput = inputString.Normalize(NormalizationForm.FormD);
 
         normalizedInput = Config.TrimWhitespace ? normalizedInput.Trim() : normalizedInput;
-        normalizedInput = Config.ForceLowerCase ? normalizedInput.ToLower() : normalizedInput;
+        normalizedInput = Config.ForceLowerCase ? normalizedInput.ToLowerInvariant() : normalizedInput;
 
         var sb = new StringBuilder(normalizedInput);
 

src/Slugify.Core/Slugify.Core.csproj

@@ -2,7 +2,7 @@
   <PropertyGroup>
     <AssemblyTitle>Slugify Core</AssemblyTitle>
 
-    <VersionPrefix>5.1.0</VersionPrefix>
+    <VersionPrefix>5.1.1</VersionPrefix>
     <TargetFramework>netstandard2.0</TargetFramework>
 	<LangVersion>latest</LangVersion>
     <Nullable>enable</Nullable>
@@ -12,6 +12,7 @@
     <PackageId>Slugify.Core</PackageId>
     <PackageTags>URL;Slug;Web;Slugify</PackageTags>
     <PackageReleaseNotes>
+		5.1.1 - Fixes issue with Turkish `i` character.
 		5.1.0 - Adds support for MaximumLength to limit the length of the slug. Thanks @ntbm
 		5.0.0 - Rewrite using newer language features. Added support for Non-Ascii languages for SlugHelperForNonAsciiLanguages
 		4.0.0 - Bug fix relase from 3.0.0

tests/Slugify.Core.Tests/SlugHelperTests.cs

@@ -704,17 +704,27 @@ public void TestHandlingOfUnicodeCharacters(ISlugHelper helper)
     }
 
 
-    [Fact(Skip = "We are not culture aware and do not support this.")]
+    [Fact]
     public void TurkishEncodingOfI()
     {
-        //Set culture to Turkish
-        CultureInfo.CurrentCulture = new CultureInfo("tr-TR");
-        const string original = "FIFA 18";
-        const string expected = "fıfa 18";
+        var defaultCulture = CultureInfo.CurrentCulture;
 
-        var helper = CreateNonAscii();
+        try
+        {
+            //Set culture to Turkish
+            CultureInfo.CurrentCulture = new CultureInfo("tr-TR");
+            const string original = "FIFA 18";
+            const string expected = "fifa-18";
 
-        Assert.Equal(expected, helper.GenerateSlug(original));
+            var helper = CreateNonAscii();
+
+            Assert.Equal(expected, helper.GenerateSlug(original));
+        }
+        finally
+        {
+            //Reset culture
+            CultureInfo.CurrentCulture = defaultCulture;
+        }
 
     }