From b34462340a76746e943c150403d9ae20fd55a8f1 Mon Sep 17 00:00:00 2001
From: Chris Palmeri <chrispalmeri@gmail.com>
Date: Wed, 12 Apr 2023 23:33:16 -0500
Subject: [PATCH] Strip symbols and forget thousands separator

---
 .../main/java/org/oxycblt/auxio/music/Music.kt    | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/org/oxycblt/auxio/music/Music.kt b/app/src/main/java/org/oxycblt/auxio/music/Music.kt
index 2b77da6b5..9b5abb36f 100644
--- a/app/src/main/java/org/oxycblt/auxio/music/Music.kt
+++ b/app/src/main/java/org/oxycblt/auxio/music/Music.kt
@@ -355,6 +355,9 @@ class SortName(name: String, musicSettings: MusicSettings) : Comparable<SortName
     init {
         var sortName = name
         if (musicSettings.intelligentSorting) {
+            // Strip any quotes, dots, and open parentheses from the beginning
+            sortName = sortName.replace(Regex("""^['".(]+"""), "")
+
             sortName =
                 sortName.run {
                     when {
@@ -365,16 +368,8 @@ class SortName(name: String, musicSettings: MusicSettings) : Comparable<SortName
                     }
                 }
 
-            // TODO: replace starting ' " ... (
-
-            // Zero pad the first number to (an arbitrary) five digits for better sorting
-            // Will also accept commas in between digits and strip them
-            sortName =
-                sortName.replace("""(\d+[\d,]+\d+|\d+)(.*)""".toRegex()) {
-                    val (firstNumber, remainingText) = it.destructured
-                    val onlyDigits = firstNumber.filter { c -> c.isDigit() }
-                    onlyDigits.padStart(5, '0') + remainingText
-                }
+            // Zero pad all numbers to (an arbitrary) five digits for better sorting
+            sortName = sortName.replace(Regex("""\d+""")) { it.value.padStart(5, '0') }
         }
 
         collationKey = COLLATOR.getCollationKey(sortName)