fix: undo the removal of Other as this would mean a breaking API chan…

…ge. Instead mark obsolete. See also 73008ec
skwasjer · May 20, 2023 · 36dff7a · 36dff7a
1 parent 24167c4
commit 36dff7a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/IbanNet/Registry/Patterns/AsciiCategory.cs b/src/IbanNet/Registry/Patterns/AsciiCategory.cs
@@ -15,6 +15,15 @@ public enum AsciiCategory
     /// </summary>
     None = 0,
 
+    /// <summary>
+    /// No ASCII category.
+    /// </summary>
+    [Obsolete("Use None instead.", true)]
+#pragma warning disable CA1069
+    // ReSharper disable once UnusedMember.Global
+    Other = 0,
+#pragma warning restore CA1069
+
     /// <summary>
     /// The space character.
     /// </summary>

diff --git a/src/IbanNet/Registry/Patterns/PatternToken.cs b/src/IbanNet/Registry/Patterns/PatternToken.cs
@@ -121,9 +121,10 @@ private PatternToken(AsciiCategory category, int minLength, int maxLength, strin
     /// <inheritdoc />
     public override string ToString()
     {
+        string category = Enum.GetName(typeof(AsciiCategory), Category)!;
         return IsFixedLength
-            ? $"{Category}[{MaxLength}]"
-            : $"{Category}[{MinLength}-{MaxLength}]";
+            ? $"{category}[{MaxLength}]"
+            : $"{category}[{MinLength}-{MaxLength}]";
     }
 
     private static Func<char, int, bool> GetCharacterTest(AsciiCategory category)