From 0802ff8624311d9e56f45ca751154abc03c6f372 Mon Sep 17 00:00:00 2001 From: Scott Jackson Date: Thu, 4 Sep 2014 17:39:05 -0700 Subject: Fixes for GenericEntryUpdater --- .../daneren2005/dsub/domain/MusicDirectory.java | 9 +++++++++ .../daneren2005/dsub/service/CachedMusicService.java | 20 ++++++++++---------- 2 files changed, 19 insertions(+), 10 deletions(-) (limited to 'src') diff --git a/src/github/daneren2005/dsub/domain/MusicDirectory.java b/src/github/daneren2005/dsub/domain/MusicDirectory.java index a738f9bf..5b6bab26 100644 --- a/src/github/daneren2005/dsub/domain/MusicDirectory.java +++ b/src/github/daneren2005/dsub/domain/MusicDirectory.java @@ -139,6 +139,15 @@ public class MusicDirectory implements Serializable { private Bookmark bookmark; private int type = 0; private int closeness; + + public Entry() { + + } + public Entry(Artist artist) { + this.id = artist.getId(); + this.title = artist.getName(); + this.directory = true; + } public void loadMetadata(File file) { try { diff --git a/src/github/daneren2005/dsub/service/CachedMusicService.java b/src/github/daneren2005/dsub/service/CachedMusicService.java index 58de141f..783f0b4d 100644 --- a/src/github/daneren2005/dsub/service/CachedMusicService.java +++ b/src/github/daneren2005/dsub/service/CachedMusicService.java @@ -20,6 +20,7 @@ package github.daneren2005.dsub.service; import java.io.File; import java.util.ArrayList; +import java.util.Arrays; import java.util.Iterator; import java.util.List; import java.util.ListIterator; @@ -1252,7 +1253,7 @@ public class CachedMusicService implements MusicService { boolean isTagBrowsing = Util.isTagBrowsing(context, musicService.getInstance(context)); // Run through each entry, trying to update the directory it is in - List songs; + final List songs = new ArrayList(); for(final Entry entry: entries) { if(isTagBrowsing) { // If starring album, needs to reference artist instead @@ -1283,39 +1284,38 @@ public class CachedMusicService implements MusicService { new IndexesUpdater(context, cacheName) { @Override public boolean checkResult(Artist check) { - return checkResult(entry, new Entry(check)); + return GenericEntryUpdater.this.checkResult(entry, new Entry(check)); } @Override public void updateResult(List objects, Artist result) { - updateResult(new Entry(result)); + GenericEntryUpdater.this.updateResult(new Entry(result)); } }.execute(); } else { new MusicDirectoryUpdater(context, cacheName, parent) { @Override public boolean checkResult(Entry check) { - return checkResult(entry, check); + return GenericEntryUpdater.this.checkResult(entry, check); } @Override public void updateResult(List objects, Entry result) { - updateResult(result); + GenericEntryUpdater.this.updateResult(result); } }.execute(); } if(entry instanceof PodcastEpisode) { - PodcastEpisode episode = (PodcastEpisode) entry; - new MusicDirectoryUpdater(context, cacheName, "p-" + entry.getId()) { + new MusicDirectoryUpdater(context, cacheName, "p-" + entry.getParent()) { @Override public boolean checkResult(Entry check) { - return checkResult(entry, check); + return GenericEntryUpdater.this.checkResult(entry, check); } @Override public void updateResult(List objects, Entry result) { - updateResult(result); + GenericEntryUpdater.this.updateResult(result); } }.execute(); } else if(!entry.isDirectory()) { @@ -1329,7 +1329,7 @@ public class CachedMusicService implements MusicService { @Override public boolean checkResult(Entry check) { for(Entry entry: songs) { - if(checkResult(entry, check)) { + if(GenericEntryUpdater.this.checkResult(entry, check)) { return true; } } -- cgit v1.2.3