Created attachment 127806 [details] [review]
Add %album_artist% placeholder

Adds the ``ITrackInfo.DisplayAlbumArtistName`` property, akin to ``DisplayArtistName``. Adds a new placeholder, %album_artist%, and modifies the album templates to use it rather than %artist%.

Adding album_artist is good, but I think we should have it also be an alias for artist, and change what is now called artist to track_artist.

This is a separate/bigger issue, but we should have a much more customizable pattern editor, and have some smartness that takes care of multi-disc albums appropriately, possibly different patterns for compilations (eg albumartist/album/01 - track_artist - title.mp3 where the track_artist - part wouldn't be there for a non-compilation).  Any way, sorry, separate issue, just brain dumping.

> Adding album_artist is good, but I think we should have it also be an alias for
> artist, and change what is now called artist to track_artist.

That was my first thought at first, hence the bug title. But it conflicts with the terminology used for metadata elsewhere in Banshee, including the user interface and track metadata properties.

Change the terminology to Album Artist, but leave the identifier "artist".  The user doesn't see that key/name any way, they only see the label.

Created attachment 127965 [details] [review]
Add %album_artist% placeholder, modify %artist%

> Adding album_artist is good, but I think we should have it also be an alias for
> artist, and change what is now called artist to track_artist.

> Change the terminology to Album Artist, but leave the identifier "artist".

Done and done

Created attachment 127968 [details] [review]
Add %album_artist% placeholder, modify %artist% (v3)

> It would also be nice to update the pattern descriptions in LibrarySchema.cs
> while you're at that, rename the "Artist" label to "Track Artist"

Thanks a lot John, committed.

*** Bug 557608 has been marked as a duplicate of this bug. ***

*** Bug 587532 has been marked as a duplicate of this bug. ***