improvement: ignore "the" in artist names for sorting purposes

johnnyshankman · Nov 14, 2023 · 62d29c0 · 62d29c0
1 parent f5362b1
commit 62d29c0
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/src/main/main.ts b/src/main/main.ts
@@ -139,8 +139,12 @@ ipcMain.on('select-library', async (event): Promise<any> => {
   const orderedFilesToTags: { [key: string]: SongSkeletonStructure } = {};
   Object.keys(filesToTags)
     .sort((a, b) => {
-      const artistA = filesToTags[a].common?.artist?.toLowerCase();
-      const artistB = filesToTags[b].common?.artist?.toLowerCase();
+      const artistA = filesToTags[a].common?.artist
+        ?.toLowerCase()
+        .replace(/^the /, '');
+      const artistB = filesToTags[b].common?.artist
+        ?.toLowerCase()
+        .replace(/^the /, '');
       const albumA = filesToTags[a].common?.album?.toLowerCase();
       const albumB = filesToTags[b].common?.album?.toLowerCase();
       const trackA = filesToTags[a].common?.track?.no;

diff --git a/src/renderer/App.tsx b/src/renderer/App.tsx
@@ -349,8 +349,12 @@ function MainDash() {
     setFilterDirection(filterDirection === 'asc' ? 'desc' : 'asc');
 
     const filtered = Object.keys(filteredLibrary).sort((a, b) => {
-      const artistA = filteredLibrary[a].common?.artist?.toLowerCase();
-      const artistB = filteredLibrary[b].common?.artist?.toLowerCase();
+      const artistA = filteredLibrary[a].common?.artist
+        ?.toLowerCase()
+        .replace(/^the /, '');
+      const artistB = filteredLibrary[b].common?.artist
+        ?.toLowerCase()
+        .replace(/^the /, '');
       const albumA = filteredLibrary[a].common?.album?.toLowerCase();
       const albumB = filteredLibrary[b].common?.album?.toLowerCase();
       const trackA = filteredLibrary[a].common?.track?.no;