From dfc06a4cbd2319cca8141159ac5a14b43854c0cd Mon Sep 17 00:00:00 2001 From: Scott Jackson Date: Wed, 13 Aug 2014 14:24:06 -0700 Subject: Fix setStarred looking up wrong serialization with tags on --- .../daneren2005/dsub/service/CachedMusicService.java | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) (limited to 'src/github') diff --git a/src/github/daneren2005/dsub/service/CachedMusicService.java b/src/github/daneren2005/dsub/service/CachedMusicService.java index 6c4822fb..6ba5dc02 100644 --- a/src/github/daneren2005/dsub/service/CachedMusicService.java +++ b/src/github/daneren2005/dsub/service/CachedMusicService.java @@ -453,8 +453,22 @@ public class CachedMusicService implements MusicService { // If parents is null, or artist id's are set, then we are looking at artists if(parents != null && (artistId == null || artistId.size() == 0)) { + String cacheName; + + // If using tag browsing, need to do lookup off of different criteria + if(Util.isTagBrowsing(context, musicService.getInstance(context))) { + // If using id's, we are starring songs and need to use album listings + if(id != null && id.size() > 0) { + cacheName = "album"; + } else { + cacheName = "artist"; + } + } else { + cacheName = "directory"; + } + for (String parent : parents) { - new MusicDirectoryUpdater(context, "directory", parent) { + new MusicDirectoryUpdater(context, cacheName, parent) { @Override public boolean checkResult(Entry check) { for (String id : checkIds) { @@ -468,7 +482,6 @@ public class CachedMusicService implements MusicService { @Override public void updateResult(List objects, Entry result) { - Log.d(TAG, result.getId()); result.setStarred(starred); } }.execute(); -- cgit v1.2.3