Merge remote-tracking branch 'azumukupoe/artistdelimiter'

Author: Googolplexed0

README.md

@@ -93,6 +93,7 @@ Set arguments in the commandline like this: `-sc False` or `--codec mp3`. Wrap a
 | `MD_SAVE_GENRES`             | `--md-save-genres`                       | False     | Whether genres should be saved in metadata                                                |
 | `MD_ALLGENRES`               | `--md-allgenres`                         | False     | Save all relevant genres in metadata                                                      |
 | `MD_GENREDELIMITER`          | `--md-genredelimiter`                    | ,         | Delimiter character used to split genres in metadata                                      |
+| `MD_ARTISTDELIMITER`         | `--md-artistdelimiter`                   | ,         | Delimiter character used to split artists in metadata                                     |
 | `SKIP_EXISTING_FILES`        | `-ie`, `--skip-existing`                 | True      | Skip songs with the same name                                                             |
 | `SKIP_PREVIOUSLY_DOWNLOADED` | `-ip`, `--skip-previously-downloaded`    | False     | Use the global song_archive file to skip previously downloaded songs                      |
 | `RETRY_ATTEMPTS`             | `--retry-attempts`                       | 1         | Number of times Zotify will retry a failed request                                        |

zotify/config.py

@@ -32,6 +32,7 @@
 MD_SAVE_GENRES = 'MD_SAVE_GENRES'
 MD_ALLGENRES = 'MD_ALLGENRES'
 MD_GENREDELIMITER = 'MD_GENREDELIMITER'
+MD_ARTISTDELIMITER = 'MD_ARTISTDELIMITER'
 PRINT_PROGRESS_INFO = 'PRINT_PROGRESS_INFO'
 PRINT_WARNINGS = 'PRINT_WARNINGS'
 RETRY_ATTEMPTS = 'RETRY_ATTEMPTS'
@@ -91,7 +92,8 @@
     MD_DISC_TRACK_TOTALS:       { 'default': 'True',                    'type': bool,   'arg': ('--md-disc-track-totals'                 ,) },
     MD_SAVE_GENRES:             { 'default': 'False',                   'type': bool,   'arg': ('--md-save-genres'                       ,) },
     MD_ALLGENRES:               { 'default': 'False',                   'type': bool,   'arg': ('--md-allgenres'                         ,) },
-    MD_GENREDELIMITER:          { 'default': ',',                       'type': str,    'arg': ('--md-genredelimiter'                    ,) },
+    MD_GENREDELIMITER:          { 'default': ', ',                      'type': str,    'arg': ('--md-genredelimiter'                    ,) },
+    MD_ARTISTDELIMITER:         { 'default': ', ',                      'type': str,    'arg': ('--md-artistdelimiter'                   ,) },
     SKIP_EXISTING:              { 'default': 'True',                    'type': bool,   'arg': ('-ie', '--skip-existing'                 ,) },
     SKIP_PREVIOUSLY_DOWNLOADED: { 'default': 'False',                   'type': bool,   'arg': ('-ip', '--skip-previously-downloaded'    ,) },
     RETRY_ATTEMPTS:             { 'default': '1',                       'type': int,    'arg': ('--retry-attempts'                       ,) },
@@ -327,6 +329,10 @@ def get_all_genres(cls) -> bool:
     def get_all_genres_delimiter(cls) -> bool:
         return cls.get(MD_GENREDELIMITER)
 
+    @classmethod
+    def get_artist_delimiter(cls) -> bool:
+        return cls.get(MD_ARTISTDELIMITER)
+    
     @classmethod
     def get_output(cls, mode: str) -> str:
         v = cls.get(OUTPUT)

zotify/utils.py

@@ -194,7 +194,7 @@ def set_audio_tags(filename, artists, genres, name, album_name, album_artist, re
 
 def conv_artist_format(artists) -> str:
     """ Returns converted artist format """
-    return ', '.join(artists)
+    return Zotify.CONFIG.get_artist_delimiter().join(artists)
 
 
 def set_music_thumbnail(filename: PurePath, image_url, mode: str) -> None: