From d3a6d5974a98cdd6ce1944b67cffa0666c62c56b Mon Sep 17 00:00:00 2001 From: Scott Jackson Date: Tue, 15 Jul 2014 18:23:33 -0700 Subject: No point in inheriting from RESTMusicService, use a single error message --- .../dsub/service/OfflineMusicService.java | 167 +++++++++++++++------ 1 file changed, 120 insertions(+), 47 deletions(-) diff --git a/src/github/daneren2005/dsub/service/OfflineMusicService.java b/src/github/daneren2005/dsub/service/OfflineMusicService.java index 04955cc3..05d17444 100644 --- a/src/github/daneren2005/dsub/service/OfflineMusicService.java +++ b/src/github/daneren2005/dsub/service/OfflineMusicService.java @@ -33,8 +33,12 @@ import android.content.Context; import android.content.SharedPreferences; import android.graphics.Bitmap; import android.util.Log; + +import org.apache.http.HttpResponse; + import github.daneren2005.dsub.domain.Artist; import github.daneren2005.dsub.domain.Bookmark; +import github.daneren2005.dsub.domain.ChatMessage; import github.daneren2005.dsub.domain.Genre; import github.daneren2005.dsub.domain.Indexes; import github.daneren2005.dsub.domain.PodcastEpisode; @@ -46,10 +50,13 @@ import github.daneren2005.dsub.domain.Playlist; import github.daneren2005.dsub.domain.PodcastChannel; import github.daneren2005.dsub.domain.SearchCritera; import github.daneren2005.dsub.domain.SearchResult; +import github.daneren2005.dsub.domain.Share; import github.daneren2005.dsub.domain.User; +import github.daneren2005.dsub.domain.Version; import github.daneren2005.dsub.util.Constants; import github.daneren2005.dsub.util.FileUtil; import github.daneren2005.dsub.util.ProgressListener; +import github.daneren2005.dsub.util.SilentBackgroundTask; import github.daneren2005.dsub.util.Util; import java.io.*; import java.util.Comparator; @@ -58,10 +65,16 @@ import java.util.SortedSet; /** * @author Sindre Mehus */ -public class OfflineMusicService extends RESTMusicService { +public class OfflineMusicService implements MusicService { private static final String TAG = OfflineMusicService.class.getSimpleName(); + private static final String ERRORMSG = "Not available in offline mode"; - @Override + @Override + public void ping(Context context, ProgressListener progressListener) throws Exception { + + } + + @Override public boolean isLicenseValid(Context context, ProgressListener progressListener) throws Exception { return true; } @@ -140,12 +153,12 @@ public class OfflineMusicService extends RESTMusicService { @Override public MusicDirectory getArtist(String id, String name, boolean refresh, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Artist by tags not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public MusicDirectory getAlbum(String id, String name, boolean refresh, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Album by tags not available in offline mode"); + throw new OfflineException(ERRORMSG); } private String getName(File file) { @@ -234,9 +247,29 @@ public class OfflineMusicService extends RESTMusicService { } } - @Override + @Override + public HttpResponse getDownloadInputStream(Context context, MusicDirectory.Entry song, long offset, int maxBitrate, SilentBackgroundTask task) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public String getMusicUrl(Context context, MusicDirectory.Entry song, int maxBitrate) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public Version getLocalVersion(Context context) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public Version getLatestVersion(Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override public List getMusicFolders(boolean refresh, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Music folders not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override @@ -304,7 +337,12 @@ public class OfflineMusicService extends RESTMusicService { return new SearchResult(artists, albums, songs); } - + + @Override + public MusicDirectory getStarredList(Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + private void recursiveAlbumSearch(String artistName, File file, SearchCritera criteria, Context context, List albums, List songs) { int closeness; for(File albumFile : FileUtil.listMediaFiles(file)) { @@ -446,37 +484,37 @@ public class OfflineMusicService extends RESTMusicService { @Override public void createPlaylist(String id, String name, List entries, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Playlists not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void deletePlaylist(String id, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Playlists not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void addToPlaylist(String id, List toAdd, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Updating playlist not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void removeFromPlaylist(String id, List toRemove, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Removing from playlist not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void overwritePlaylist(String id, String name, int toRemove, List toAdd, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Overwriting playlist not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void updatePlaylist(String id, String name, String comment, boolean pub, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Updating playlist not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public Lyrics getLyrics(String artist, String title, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Lyrics not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override @@ -509,12 +547,12 @@ public class OfflineMusicService extends RESTMusicService { @Override public MusicDirectory getAlbumList(String type, int size, int offset, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Album lists not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public MusicDirectory getAlbumList(String type, String extra, int size, int offset, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Album lists not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override @@ -524,42 +562,42 @@ public class OfflineMusicService extends RESTMusicService { @Override public String getVideoStreamUrl(String format, int maxBitrate, Context context, String id) throws Exception { - return null; + throw new OfflineException(ERRORMSG); } @Override public String getHlsUrl(String id, int bitRate, Context context) throws Exception { - return null; + throw new OfflineException(ERRORMSG); } @Override public RemoteStatus updateJukeboxPlaylist(List ids, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Jukebox not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public RemoteStatus skipJukebox(int index, int offsetSeconds, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Jukebox not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public RemoteStatus stopJukebox(Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Jukebox not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public RemoteStatus startJukebox(Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Jukebox not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public RemoteStatus getJukeboxStatus(Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Jukebox not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public RemoteStatus setJukeboxGain(float gain, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Jukebox not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override @@ -586,15 +624,45 @@ public class OfflineMusicService extends RESTMusicService { offlineEditor.putInt(Constants.OFFLINE_STAR_COUNT, stars); offlineEditor.commit(); } - + + @Override + public List getShares(Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public List createShare(List ids, String description, Long expires, Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public void deleteShare(String id, Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public void updateShare(String id, String description, Long expires, Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public List getChatMessages(Long since, Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + + @Override + public void addChatMessage(String message, Context context, ProgressListener progressListener) throws Exception { + throw new OfflineException(ERRORMSG); + } + @Override public List getGenres(boolean refresh, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Getting Genres not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public MusicDirectory getSongsByGenre(String genre, int count, int offset, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Getting Songs By Genre not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override @@ -615,7 +683,12 @@ public class OfflineMusicService extends RESTMusicService { return result; } - + + @Override + public String getCoverArtUrl(Context context, MusicDirectory.Entry entry) throws Exception { + throw new OfflineException(ERRORMSG); + } + @Override public List getPodcastChannels(boolean refresh, Context context, ProgressListener progressListener) throws Exception { List channels = new ArrayList(); @@ -647,97 +720,97 @@ public class OfflineMusicService extends RESTMusicService { @Override public void refreshPodcasts(Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Getting Podcasts not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void createPodcastChannel(String url, Context context, ProgressListener progressListener) throws Exception{ - throw new OfflineException("Getting Podcasts not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void deletePodcastChannel(String id, Context context, ProgressListener progressListener) throws Exception{ - throw new OfflineException("Getting Podcasts not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void downloadPodcastEpisode(String id, Context context, ProgressListener progressListener) throws Exception{ - throw new OfflineException("Getting Podcasts not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void deletePodcastEpisode(String id, Context context, ProgressListener progressListener) throws Exception{ - throw new OfflineException("Getting Podcasts not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void setRating(String id, int rating, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Setting ratings not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public List getBookmarks(boolean refresh, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Getting bookmarks not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void createBookmark(String id, int position, String comment, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Creating bookmarks not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void deleteBookmark(String id, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Deleting bookmarks not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public User getUser(boolean refresh, String username, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Getting user not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public List getUsers(boolean refresh, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Getting users not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void createUser(User user, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Creating users not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void updateUser(User user, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Updating users not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void deleteUser(String username, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Deleting users not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void changeEmail(String username, String email, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Changing email not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void changePassword(String username, String password, Context context, ProgressListener progressListener) throws Exception { - throw new OfflineException("Changing passwords not available in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public Bitmap getAvatar(String username, int size, Context context, ProgressListener progressListener) throws Exception { - return null; + throw new OfflineException(ERRORMSG); } @Override public int processOfflineSyncs(final Context context, final ProgressListener progressListener) throws Exception{ - throw new OfflineException("Offline scrobble cached can not be processes while in offline mode"); + throw new OfflineException(ERRORMSG); } @Override public void setInstance(Integer instance) throws Exception{ - throw new OfflineException("Offline servers only have one instance"); + throw new OfflineException(ERRORMSG); } private void listFilesRecursively(File parent, List children) { -- cgit v1.2.3