Remove full HTML when raw HTML value changes

Previously the issue body html was not being re-encoded
after edits and also the full html was never being flushed
when the raw version has changed from the last time it was
encoded.

This would cause the issue description to not update after
being edited.

Closes pockethub#353

Author: kevinsawicki

app/src/main/java/com/github/mobile/ui/issue/IssueFragment.java

@@ -500,7 +500,9 @@ public void onActivityResult(int requestCode, int resultCode, Intent data) {
 
         switch (requestCode) {
         case ISSUE_EDIT:
-            updateHeader((Issue) data.getSerializableExtra(EXTRA_ISSUE));
+            Issue editedIssue = (Issue) data.getSerializableExtra(EXTRA_ISSUE);
+            bodyImageGetter.encode(editedIssue.getId(), editedIssue.getBodyHtml());
+            updateHeader(editedIssue);
             return;
         case COMMENT_CREATE:
             Comment comment = (Comment) data
@@ -557,4 +559,4 @@ public boolean onOptionsItemSelected(MenuItem item) {
             return super.onOptionsItemSelected(item);
         }
     }
-}
+}

app/src/main/java/com/github/mobile/util/HttpImageGetter.java

@@ -132,9 +132,13 @@ public HttpImageGetter encode(final Object id, final String html) {
 
         CharSequence encoded = HtmlUtils.encode(html, loading);
         // Use default encoding if no img tags
-        if (containsImages(html))
-            rawHtmlCache.put(id, encoded);
-        else {
+        if (containsImages(html)) {
+            CharSequence currentEncoded = rawHtmlCache.put(id, encoded);
+            // Remove full html if raw html has changed
+            if (currentEncoded == null
+                || !currentEncoded.toString().equals(encoded.toString()))
+              fullHtmlCache.remove(id);
+        } else {
             rawHtmlCache.remove(id);
             fullHtmlCache.put(id, encoded);
         }
@@ -184,7 +188,6 @@ protected CharSequence run(Account account) throws Exception {
 
             @Override
             protected void onSuccess(final CharSequence html) throws Exception {
-                rawHtmlCache.remove(id);
                 fullHtmlCache.put(id, html);
 
                 if (id.equals(view.getTag()))