feat(author-search): ✨ search/upgrade multi-author entries to single …

…author entries for example: `Jason Anspach, Nick Cole, Doc Spears` -> `Jason Anspach`
djdembeck · Oct 3, 2021 · e5d70dd · e5d70dd
1 parent a18cda6
commit e5d70dd
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,6 +1,7 @@
 {
     "conventionalCommits.scopes": [
         "search",
-        "update"
+        "update",
+        "author-search"
     ]
 }
diff --git a/Contents/Code/__init__.py b/Contents/Code/__init__.py
@@ -72,13 +72,13 @@ def search(self, results, media, lang, manual):
         if not result:
             log.warn(
                 'No results found for query "%s"',
-                media.artist
+                search_helper.media.artist
             )
             return
         log.debug(
             'Found %s result(s) for query "%s"',
             len(result),
-            media.artist
+            search_helper.media.artist
         )
 
         info = self.process_results(search_helper, result)

diff --git a/Contents/Code/search_tools.py b/Contents/Code/search_tools.py
@@ -213,10 +213,19 @@ def parse_api_response(self, api_response):
 
     def validate_author_name(self):
         """
-            Checks a list of known bad author names.
+            Checks for combined authors and a list of known bad author names.
             If matched, author name is set to None to prevent
             it being used in search query.
         """
+        if ',' in self.media.artist:
+            split_authors = self.media.artist.split(',')
+            log.info(
+                'Merging multi-author "' +
+                self.media.artist +
+                '" into top-level author "' +
+                split_authors[0] + '"'
+            )
+            self.media.artist = split_authors[0]
         strings_to_check = [
             "[Unknown Artist]"
         ]