From 26f8b501e77b7bd9a73028637e82de2f2605dd3a Mon Sep 17 00:00:00 2001
From: sushilicious <*>
Date: Tue, 3 Aug 2021 13:46:56 -0700
Subject: Made CleanStringParser more robust

Now it can handle [...] at beginning of string
---
 Emby.Naming/Video/CleanStringParser.cs | 44 +++++++++++++++++++++++++++-------
 1 file changed, 36 insertions(+), 8 deletions(-)

(limited to 'Emby.Naming/Video/CleanStringParser.cs')

diff --git a/Emby.Naming/Video/CleanStringParser.cs b/Emby.Naming/Video/CleanStringParser.cs
index 4eef3ebc5e..0518095705 100644
--- a/Emby.Naming/Video/CleanStringParser.cs
+++ b/Emby.Naming/Video/CleanStringParser.cs
@@ -25,26 +25,54 @@ namespace Emby.Naming.Video
                 return false;
             }
 
-            var len = expressions.Count;
-            for (int i = 0; i < len; i++)
+            // Iteratively remove extra cruft until we're left with the string
+            // we want.
+            newName = ReadOnlySpan<char>.Empty;
+            const int maxTries = 100;       // This is just a precautionary
+                                            // measure. Should not be neccesary.
+            var loopCounter = 0;
+            for (; loopCounter < maxTries; loopCounter++)
             {
-                if (TryClean(name, expressions[i], out newName))
+                bool cleaned = false;
+                var len = expressions.Count;
+                for (int i = 0; i < len; i++)
+                {
+                    if (TryClean(name, expressions[i], out newName))
+                    {
+                        cleaned = true;
+                        name = newName.ToString();
+                        break;
+                    }
+                }
+
+                if (!cleaned)
                 {
-                    return true;
+                    break;
                 }
             }
 
-            newName = ReadOnlySpan<char>.Empty;
-            return false;
+            if (loopCounter > 0)
+            {
+                newName = name.AsSpan();
+            }
+
+            return newName != ReadOnlySpan<char>.Empty;
         }
 
         private static bool TryClean(string name, Regex expression, out ReadOnlySpan<char> newName)
         {
             var match = expression.Match(name);
             int index = match.Index;
-            if (match.Success && index != 0)
+            if (match.Success)
             {
-                newName = name.AsSpan().Slice(0, match.Index);
+                var found = match.Groups.TryGetValue("cleaned", out var cleaned);
+                if (!found || cleaned == null)
+                {
+                    newName = ReadOnlySpan<char>.Empty;
+                    return false;
+                }
+
+                newName = name.AsSpan().Slice(cleaned.Index, cleaned.Length);
                 return true;
             }
 
-- 
cgit v1.2.3