2017-08-10, 00:50
(2017-08-09, 19:49)DaveBlake Wrote: New test build KodiSetup-20170809-a130a8e139-InfoSettings
The only part changed is the processing of "Set for all artists/albums shown". It should now apply info provider settings correctly when viewing "artists" or "albums" type smart playlists, or if the a filter has been applied to the node (from the "filter" on the sideblade). So have another look at playlists, and filtering
OK so I ran tests on a fresh MyMusic67 and two tests (one album and one artist) that failed on the previous build now pass. Also I now get a confirm dialog "Do you want to refresh info" after changing info provider for both album and artist shown in smart playlists.
Specific test case:
1 Set default info provider in settings/media to local info both albums and artists
2. Add new music source and scan (infosettings is empty and both album and artist idInfoSetting is 0)
3. Open music album view
4. On context menu for first album select "change default info provider" and set to "1ting" / "use for all" / "rescan no"
5. Observe infosettings is empty and both album and artist idInfoSettings is 0 (PASS)
6. Open music album smart playlist view.
7. On context menu for first album select "change info provider for albums shown" and set to "local info only" / "set for all shown" / "rescan no"
8. Observe infosettings has new entry 1 for metadata.local and album idInfoSettings is 1 for all albums in the smart playlist (PASS)
9. Open music artist view
10. On context menu for first artist select "change default info provider" and set to "Encyclopaedia Metallica" / "use for all" / "rescan no"
11. Observe infosettings is unchanged (1 entry) and artist idInfoSettings remains 0 (PASS)
12. Open music artist smart playlist view
13. On context menu for first artist select "change info provider for artists shown" and set to "tadb artist scraper" / "set for all shown" / "rescan no"
14. Observe infosettings has new entry 2 for metadata.artists.theaudiodb.com and artist idInfoSettings is 2 for all artists in smart playlist. (PASS)
scott s.
.