refine code to mark item as readed; force mail to reload upon notific…

…ation
cameoh · May 15, 2016 · 616f5d3 · 616f5d3
1 parent 9736016
commit 616f5d3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/app/src/main/java/com/zfdang/zsmth_android/MailListFragment.java b/app/src/main/java/com/zfdang/zsmth_android/MailListFragment.java
@@ -412,7 +412,15 @@ public void markMailAsReaded(final int position) {
             final Mail mail = MailListContent.MAILS.get(position);
             if (!mail.isNew) return;
 
-            // this is a new mail, mark it as read in remote and local
+            // only referred post need explicit marking. for mails, there is no need to mark
+            if(TextUtils.equals(currentFolder, INBOX_LABEL) || TextUtils.equals(currentFolder, OUTBOX_LABEL)
+                    || TextUtils.equals(currentFolder, DELETED_LABEL)) {
+                mail.isNew = false;
+                recyclerView.getAdapter().notifyItemChanged(position);
+                return;
+            }
+
+            // mark it as read in remote and local
             SMTHHelper helper = SMTHHelper.getInstance();
             helper.wService.readReferPosts(currentFolder, mail.referIndex)
                     .subscribeOn(Schedulers.io())

diff --git a/app/src/main/java/com/zfdang/zsmth_android/MainActivity.java b/app/src/main/java/com/zfdang/zsmth_android/MainActivity.java
@@ -46,6 +46,7 @@
 import com.zfdang.zsmth_android.listeners.OnVolumeUpDownListener;
 import com.zfdang.zsmth_android.models.Board;
 import com.zfdang.zsmth_android.models.Mail;
+import com.zfdang.zsmth_android.models.MailListContent;
 import com.zfdang.zsmth_android.models.Topic;
 import com.zfdang.zsmth_android.newsmth.AjaxResponse;
 import com.zfdang.zsmth_android.newsmth.SMTHHelper;
@@ -350,6 +351,8 @@ protected void onNewIntent(Intent intent) {
                 } else if(message.contains(SMTHApplication.NOTIFICATION_NEW_REPLY)) {
                     mailListFragment.setCurrentFolder(MailListFragment.REPLY_LABEL);
                 }
+                // force mail fragment to reload
+                MailListContent.clear();
 
                 fm.beginTransaction().replace(R.id.content_frame, mailListFragment).commitAllowingStateLoss();
             }