2018-02-09, 01:09
Currently in v18 and below, there are settings options for both video and music to show an "all items" entry in library container views. One case is in musicvideo artists node, selecting an artist displays a list of that artist's albums in the container. Per code, when the number of albums for the artist is <=1 the "all albums" item is not included.
A use case exists where musicvideos have assigned artist but not album. In this case in default skin in the artists node there is a label of "appearances nn" for each artist item that shows the number of musicvideo items with that artist. But selecting the artist results in an empty container due to no album title strings in the library and thus the test for showing "all albums" fails. A work around does exist of using the musicvideo "titles" node and filtering on artist to display a container of all the artist's musicvideos.
An alternative is to PR the code so in the case of node type NODE_TYPE_MUSICVIDEOS_ALBUM in VideoFileItemListModifier to show the "all albums" item when the number of musicvideo albums = 0. But I don't know if this has other impacts or if creating this exception just for musicvideo albums makes sense.
scott s.
.
A use case exists where musicvideos have assigned artist but not album. In this case in default skin in the artists node there is a label of "appearances nn" for each artist item that shows the number of musicvideo items with that artist. But selecting the artist results in an empty container due to no album title strings in the library and thus the test for showing "all albums" fails. A work around does exist of using the musicvideo "titles" node and filtering on artist to display a container of all the artist's musicvideos.
An alternative is to PR the code so in the case of node type NODE_TYPE_MUSICVIDEOS_ALBUM in VideoFileItemListModifier to show the "all albums" item when the number of musicvideo albums = 0. But I don't know if this has other impacts or if creating this exception just for musicvideo albums makes sense.
scott s.
.