Create arranger_sort, lyricist_sort, performer and performer_sort tags

beets/autotag/__init__.py

@@ -47,12 +47,20 @@ def apply_item_metadata(item, track_info):
 
     if track_info.lyricist is not None:
         item.lyricist = track_info.lyricist
+    if track_info.lyricist_sort is not None:
+        item.lyricist_sort = track_info.lyricist_sort
     if track_info.composer is not None:
         item.composer = track_info.composer
     if track_info.composer_sort is not None:
         item.composer_sort = track_info.composer_sort
     if track_info.arranger is not None:
         item.arranger = track_info.arranger
+    if track_info.arranger_sort is not None:
+        item.arranger_sort = track_info.arranger_sort
+    if track_info.performer is not None:
+        item.performer = track_info.performer
+    if track_info.performer_sort is not None:
+        item.performer_sort = track_info.performer_sort
 
     # At the moment, the other metadata is left intact (including album
     # and track number). Perhaps these should be emptied?
@@ -155,11 +163,19 @@ def apply_metadata(album_info, mapping):
 
         if track_info.lyricist is not None:
             item.lyricist = track_info.lyricist
+        if track_info.lyricist_sort is not None:
+            item.lyricist_sort = track_info.lyricist_sort
         if track_info.composer is not None:
             item.composer = track_info.composer
         if track_info.composer_sort is not None:
             item.composer_sort = track_info.composer_sort
         if track_info.arranger is not None:
             item.arranger = track_info.arranger
+        if track_info.arranger_sort is not None:
+            item.arranger_sort = track_info.arranger_sort
+        if track_info.performer is not None:
+            item.performer = track_info.performer
+        if track_info.performer_sort is not None:
+            item.performer_sort = track_info.performer_sort
 
         item.track_alt = track_info.track_alt

beets/autotag/hooks.py

@@ -156,8 +156,10 @@ def __init__(self, title, track_id, artist=None, artist_id=None,
                  length=None, index=None, medium=None, medium_index=None,
                  medium_total=None, artist_sort=None, disctitle=None,
                  artist_credit=None, data_source=None, data_url=None,
-                 media=None, lyricist=None, composer=None, composer_sort=None,
-                 arranger=None, track_alt=None):
+                 media=None, lyricist=None, lyricist_sort=None,
+                 composer=None, composer_sort=None, arranger=None,
+                 arranger_sort=None, track_alt=None, performer=None,
+                 performer_sort=None):
         self.title = title
         self.track_id = track_id
         self.artist = artist
@@ -174,9 +176,13 @@ def __init__(self, title, track_id, artist=None, artist_id=None,
         self.data_source = data_source
         self.data_url = data_url
         self.lyricist = lyricist
+        self.lyricist_sort = lyricist_sort
         self.composer = composer
         self.composer_sort = composer_sort
         self.arranger = arranger
+        self.arranger_sort = arranger_sort
+        self.performer = performer
+        self.performer_sort = performer_sort
         self.track_alt = track_alt
 
     # As above, work around a bug in python-musicbrainz-ngs.

beets/autotag/mb.py

@@ -208,6 +208,7 @@ def track_info(recording, index=None, medium=None, medium_index=None,
         info.length = int(recording['length']) / (1000.0)
 
     lyricist = []
+    lyricist_sort = []
     composer = []
     composer_sort = []
     for work_relation in recording.get('work-relation-list', ()):
@@ -219,24 +220,60 @@ def track_info(recording, index=None, medium=None, medium_index=None,
                 type = artist_relation['type']
                 if type == 'lyricist':
                     lyricist.append(artist_relation['artist']['name'])
+                    lyricist_sort.append(
+                        artist_relation['artist']['sort-name'])
                 elif type == 'composer':
                     composer.append(artist_relation['artist']['name'])
                     composer_sort.append(
                         artist_relation['artist']['sort-name'])
     if lyricist:
         info.lyricist = u', '.join(lyricist)
+        info.lyricist_sort = u', '.join(lyricist_sort)
     if composer:
         info.composer = u', '.join(composer)
         info.composer_sort = u', '.join(composer_sort)
 
     arranger = []
+    arranger_sort = []
+    performer = []
+    performer_sort = []
+    performer_tmp = {}
+    performer_types = ['performer', 'instrument', 'vocal',
+                                    'conductor', 'performing orchestra',
+                                    'chorus master', 'concertmaster'
+                       ]
     for artist_relation in recording.get('artist-relation-list', ()):
         if 'type' in artist_relation:
-            type = artist_relation['type']
-            if type == 'arranger':
+            rel_type = artist_relation['type']
+            if rel_type == 'arranger':
                 arranger.append(artist_relation['artist']['name'])
+                arranger_sort.append(
+                    artist_relation['artist']['sort-name'])
+            if rel_type in performer_types and \
+                    artist_relation.get('direction') == 'backward':
+                artist_tmp = artist_relation['artist']['name']
+                performer_tmp[artist_tmp] = []
+                if 'attribute-list' in artist_relation:
+                    for attribute in artist_relation['attribute-list']:
+                        performer_tmp[artist_tmp].append(attribute)
+                else:
+                    performer_tmp[artist_tmp].append(artist_relation['type'])
+                performer_tmp[artist_relation['artist']['sort-name']] =\
+                    performer_tmp[artist_tmp]
+                performer.append(artist_tmp +
+                                 '(' + u', '.join(performer_tmp[artist_tmp]) +
+                                 ')')
+                performer_sort.append(artist_relation['artist']['sort-name'] +
+                                      '(' +
+                                      u', '.join(performer_tmp[artist_tmp]) +
+                                      ')')
     if arranger:
         info.arranger = u', '.join(arranger)
+        info.arranger_sort = u', '.join(arranger_sort)
+
+    if performer:
+        info.performer = u', '.join(performer)
+        info.performer_sort = u', '.join(performer_sort)
 
     info.decode()
     return info

beets/library.py

@@ -436,9 +436,13 @@ class Item(LibModel):
         'albumartist_credit':   types.STRING,
         'genre':                types.STRING,
         'lyricist':             types.STRING,
+        'lyricist_sort':        types.STRING,
         'composer':             types.STRING,
         'composer_sort':        types.STRING,
         'arranger':             types.STRING,
+        'arranger_sort':        types.STRING,
+        'performer':            types.STRING,
+        'performer_sort':       types.STRING,
         'grouping':             types.STRING,
         'year':                 types.PaddedInt(4),
         'month':                types.PaddedInt(2),

docs/changelog.rst

@@ -208,11 +208,9 @@ There are some changes for developers:
   missing values into type-specific null-like values. This should help in
   cases where a string field is unexpectedly `None` sometimes instead of just
   showing up as an empty string. :bug:`2605`
-* Refactored the move functions the `beets.library` module and the
-  `manipulate_files` function in `beets.importer` to use a single parameter
-  describing the file operation instead of multiple Boolean flags.
-  There is a new numerated type describing how to move, copy, or link files.
-  :bug:`2682`
+
+* Added new tags arranger_sort, lyricist_sort, performer and performer_sort. 
+  Thanks to :user:`dosoe`. :bug:`2563`
 
 
 1.4.5 (June 20, 2017)

test/_common.py

@@ -67,8 +67,13 @@ def item(lib=None):
         album=u'the album',
         genre=u'the genre',
         lyricist=u'the lyricist',
+        lyricist_sort=u'the lyricist sort name',
         composer=u'the composer',
+        composer_sort=u'the composer sort name',
         arranger=u'the arranger',
+        arranger_sort=u'the arranger sort name',
+        performer=u'the performer and his role',
+        performer_sort=u'the performer sort name and his role',
         grouping=u'the grouping',
         year=1,
         month=2,