diff --git a/app/src/main/java/com/kabouzeid/gramophone/glide/artistimage/ArtistImageLoader.java b/app/src/main/java/com/kabouzeid/gramophone/glide/artistimage/ArtistImageLoader.java index e7798c7f..fe08678a 100644 --- a/app/src/main/java/com/kabouzeid/gramophone/glide/artistimage/ArtistImageLoader.java +++ b/app/src/main/java/com/kabouzeid/gramophone/glide/artistimage/ArtistImageLoader.java @@ -19,48 +19,32 @@ import java.io.InputStream; public class ArtistImageLoader implements StreamModelLoader { private Context context; - private LastFMRestClient lastFMRestClient; + private LastFMRestClient lastFMClient; private ModelLoader urlLoader; public ArtistImageLoader(Context context, LastFMRestClient lastFMRestClient, ModelLoader urlLoader) { this.context = context; - this.lastFMRestClient = lastFMRestClient; + this.lastFMClient = lastFMRestClient; this.urlLoader = urlLoader; } @Override public DataFetcher getResourceFetcher(ArtistImage model, int width, int height) { - return new ArtistImageFetcher(context, lastFMRestClient, model, urlLoader, width, height); + return new ArtistImageFetcher(context, lastFMClient, model, urlLoader, width, height); } public static class Factory implements ModelLoaderFactory { - private static volatile LastFMRestClient internalClient; - private LastFMRestClient client; + private LastFMRestClient lastFMClient; private OkHttpUrlLoader.Factory okHttpFactory; - - private static LastFMRestClient getInternalClient(Context context) { - if (internalClient == null) { - synchronized (Factory.class) { - if (internalClient == null) { - internalClient = new LastFMRestClient(context); - } - } - } - return internalClient; - } - - /** - * Constructor for a new Factory that runs requests using a static singleton client. - */ public Factory(Context context) { - client = getInternalClient(context); okHttpFactory = new OkHttpUrlLoader.Factory(); + lastFMClient = new LastFMRestClient(context); } @Override public ModelLoader build(Context context, GenericLoaderFactory factories) { - return new ArtistImageLoader(context, client, okHttpFactory.build(context, factories)); + return new ArtistImageLoader(context, lastFMClient, okHttpFactory.build(context, factories)); } @Override diff --git a/app/src/main/java/com/kabouzeid/gramophone/lastfm/rest/LastFMRestClient.java b/app/src/main/java/com/kabouzeid/gramophone/lastfm/rest/LastFMRestClient.java index a249af26..85110f5e 100644 --- a/app/src/main/java/com/kabouzeid/gramophone/lastfm/rest/LastFMRestClient.java +++ b/app/src/main/java/com/kabouzeid/gramophone/lastfm/rest/LastFMRestClient.java @@ -12,7 +12,6 @@ import com.squareup.okhttp.Response; import java.io.File; import java.io.IOException; -import java.util.concurrent.TimeUnit; import retrofit.GsonConverterFactory; import retrofit.Retrofit; @@ -33,9 +32,6 @@ public class LastFMRestClient { okHttpClient.setCache(new Cache(cacheDir, 1024 * 1024 * 10)); } - okHttpClient.setConnectTimeout(15, TimeUnit.SECONDS); - okHttpClient.setReadTimeout(20, TimeUnit.SECONDS); - okHttpClient.interceptors().add(new Interceptor() { @Override public Response intercept(Chain chain) throws IOException {