[wip] Image caching

app/build.gradle

@@ -39,7 +39,7 @@ android {
 }
 
 greendao {
-    schemaVersion 5
+    schemaVersion 6
     daoPackage 'fr.gaulupeau.apps.Poche.data.dao'
 }
 

app/src/main/AndroidManifest.xml

@@ -7,6 +7,7 @@
     <uses-permission android:name="android.permission.INTERNET" />
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
     <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
+    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
 
     <application
         android:name="fr.gaulupeau.apps.Poche.App"

app/src/main/java/fr/gaulupeau/apps/Poche/data/Settings.java

@@ -511,6 +511,14 @@ public void setAutoDownloadNewArticlesEnabled(boolean value) {
         setBoolean(R.string.pref_key_autoDlNew_enabled, value);
     }
 
+    public boolean isImageCacheEnabled() {
+        return getBoolean(R.string.pref_key_imageCache_enabled, false);
+    }
+
+    public void setImageCacheEnabled(boolean value) {
+        setBoolean(R.string.pref_key_imageCache_enabled, value);
+    }
+
     public boolean isFirstRun() {
         return getBoolean(R.string.pref_key_internal_firstRun, true);
     }

app/src/main/java/fr/gaulupeau/apps/Poche/data/dao/entities/Article.java

@@ -41,17 +41,20 @@ public class Article {
     @Property(nameInDb = "article_progress")
     private Double articleProgress;
 
+    @Property(nameInDb = "images_downloaded")
+    private Boolean imagesDownloaded;
+
     @Generated(hash = 742516792)
     public Article() {}
 
     public Article(Long id) {
         this.id = id;
     }
 
-    @Generated(hash = 1900630499)
-    public Article(Long id, Integer articleId, String content, String author, String title,
-            String url, Boolean favorite, Boolean archive, Date updateDate,
-            Double articleProgress) {
+    @Generated(hash = 1635440040)
+    public Article(Long id, Integer articleId, String content, String author,
+            String title, String url, Boolean favorite, Boolean archive,
+            Date updateDate, Double articleProgress, Boolean imagesDownloaded) {
         this.id = id;
         this.articleId = articleId;
         this.content = content;
@@ -62,6 +65,7 @@ public Article(Long id, Integer articleId, String content, String author, String
         this.archive = archive;
         this.updateDate = updateDate;
         this.articleProgress = articleProgress;
+        this.imagesDownloaded = imagesDownloaded;
     }
 
     public Long getId() {
@@ -144,4 +148,12 @@ public void setArticleProgress(Double articleProgress) {
         this.articleProgress = articleProgress;
     }
 
+    public Boolean getImagesDownloaded() {
+        return imagesDownloaded;
+    }
+
+    public void setImagesDownloaded(Boolean imagesDownloaded) {
+        this.imagesDownloaded = imagesDownloaded;
+    }
+
 }

app/src/main/java/fr/gaulupeau/apps/Poche/events/EventProcessor.java

@@ -31,7 +31,8 @@ public class EventProcessor {
 
     private static final int NOTIFICATION_ID_OTHER = 0;
     private static final int NOTIFICATION_ID_UPDATE_FEEDS_ONGOING = 1;
-    private static final int NOTIFICATION_ID_SYNC_QUEUE_ONGOING = 2;
+    private static final int NOTIFICATION_ID_FETCH_IMAGES_ONGOING = 2;
+    private static final int NOTIFICATION_ID_SYNC_QUEUE_ONGOING = 3;
 
     private Context context;
     private Settings settings;
@@ -171,13 +172,49 @@ public void onUpdateFeedsFinishedEvent(UpdateFeedsFinishedEvent event) {
 
         getNotificationManager().cancel(TAG, NOTIFICATION_ID_UPDATE_FEEDS_ONGOING);
 
-        if(event.getResult().isSuccess() && !getSettings().isFirstSyncDone()) {
-            getSettings().setFirstSyncDone(true);
+        Settings settings = getSettings();
+
+        if(event.getResult().isSuccess()) {
+            if(!getSettings().isFirstSyncDone()) {
+                settings.setFirstSyncDone(true);
+            }
+
+            if(settings.isImageCacheEnabled()) {
+                ServiceHelper.fetchImages(getContext());
+            }
         }
 
         emptyOperationID();
     }
 
+    @Subscribe(sticky = true)
+    public void onFetchImagesStartedEvent(FetchImagesStartedEvent event) {
+        Log.d(TAG, "onFetchImagesStartedEvent() started");
+
+        setOperationID(event);
+
+        Context context = getContext();
+
+        NotificationCompat.Builder notificationBuilder = new NotificationCompat.Builder(context)
+                .setSmallIcon(R.drawable.ic_action_refresh)
+                .setContentTitle(context.getString(R.string.notification_downloadingImages))
+                .setOngoing(true);
+
+        getNotificationManager().notify(TAG, NOTIFICATION_ID_FETCH_IMAGES_ONGOING,
+                notificationBuilder.setProgress(0, 0, true).build());
+    }
+
+    @Subscribe
+    public void onFetchImagesFinishedEvent(FetchImagesFinishedEvent event) {
+        Log.d(TAG, "onFetchImagesFinishedEvent() started");
+
+        checkOperationID(event);
+
+        getNotificationManager().cancel(TAG, NOTIFICATION_ID_FETCH_IMAGES_ONGOING);
+
+        emptyOperationID();
+    }
+
     @Subscribe(sticky = true)
     public void onSyncQueueStartedEvent(SyncQueueStartedEvent event) {
         Log.d(TAG, "onSyncQueueStartedEvent() started");

app/src/main/java/fr/gaulupeau/apps/Poche/events/FetchImagesFinishedEvent.java

@@ -0,0 +1,11 @@
+package fr.gaulupeau.apps.Poche.events;
+
+import fr.gaulupeau.apps.Poche.service.ActionRequest;
+
+public class FetchImagesFinishedEvent extends BackgroundOperationEvent {
+
+    public FetchImagesFinishedEvent(ActionRequest request) {
+        super(request);
+    }
+
+}

app/src/main/java/fr/gaulupeau/apps/Poche/events/FetchImagesStartedEvent.java

@@ -0,0 +1,11 @@
+package fr.gaulupeau.apps.Poche.events;
+
+import fr.gaulupeau.apps.Poche.service.ActionRequest;
+
+public class FetchImagesStartedEvent extends BackgroundOperationEvent {
+
+    public FetchImagesStartedEvent(ActionRequest request) {
+        super(request);
+    }
+
+}

app/src/main/java/fr/gaulupeau/apps/Poche/network/FeedUpdater.java

@@ -277,7 +277,10 @@ private void processFeed(ArticleDao articleDao, InputStream is,
                         continue;
                     }
 
-                    if(updateType == UpdateType.Fast && latestID != null && latestID >= id) break;
+                    if(updateType == UpdateType.Fast && latestID != null && latestID >= id) {
+                        Log.d(TAG, "processFeed(): update type fast, everything up to date");
+                        break;
+                    }
 
                     Article article = articleDao.queryBuilder()
                             .where(ArticleDao.Properties.ArticleId.eq(id)).build().unique();
@@ -307,6 +310,7 @@ private void processFeed(ArticleDao articleDao, InputStream is,
                         article.setArchive(feedType == FeedType.Archive);
                         article.setFavorite(feedType == FeedType.Favorite);
                     }
+                    article.setImagesDownloaded(false);
 
                     if(event != null) {
                         ArticlesChangedEvent.ChangeType changeType = existing