From 69af53ddbabfa4d2b3cd6f4e1fb49dc0a697aa23 Mon Sep 17 00:00:00 2001
From: Alex Ning <chineseperson5@gmail.com>
Date: Tue, 8 Dec 2020 00:18:04 +0800
Subject: [PATCH] Prevent inserting read posts into database multiple times
 when clicking posts.

---
 .../adapters/PostRecyclerViewAdapter.java     | 30 +++++++++++--------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/ml/docilealligator/infinityforreddit/adapters/PostRecyclerViewAdapter.java b/app/src/main/java/ml/docilealligator/infinityforreddit/adapters/PostRecyclerViewAdapter.java
index dfed1b2d..76df281b 100644
--- a/app/src/main/java/ml/docilealligator/infinityforreddit/adapters/PostRecyclerViewAdapter.java
+++ b/app/src/main/java/ml/docilealligator/infinityforreddit/adapters/PostRecyclerViewAdapter.java
@@ -1880,14 +1880,16 @@ public class PostRecyclerViewAdapter extends PagedListAdapter<Post, RecyclerView
         }
 
         void markPostRead(Post post) {
-            post.markAsRead();
-            cardView.setBackgroundTintList(ColorStateList.valueOf(mReadPostCardViewBackgroundColor));
-            titleTextView.setTextColor(mReadPostTitleColor);
-            if (this instanceof PostTextTypeViewHolder) {
-                ((PostTextTypeViewHolder) this).contentTextView.setTextColor(mReadPostContentColor);
-            }
-            if (mActivity != null && mActivity instanceof MarkPostAsReadInterface) {
-                ((MarkPostAsReadInterface) mActivity).markPostAsRead(post);
+            if (!post.isRead()) {
+                post.markAsRead();
+                cardView.setBackgroundTintList(ColorStateList.valueOf(mReadPostCardViewBackgroundColor));
+                titleTextView.setTextColor(mReadPostTitleColor);
+                if (this instanceof PostTextTypeViewHolder) {
+                    ((PostTextTypeViewHolder) this).contentTextView.setTextColor(mReadPostContentColor);
+                }
+                if (mActivity != null && mActivity instanceof MarkPostAsReadInterface) {
+                    ((MarkPostAsReadInterface) mActivity).markPostAsRead(post);
+                }
             }
         }
     }
@@ -2933,11 +2935,13 @@ public class PostRecyclerViewAdapter extends PagedListAdapter<Post, RecyclerView
         }
 
         void markPostRead(Post post) {
-            post.markAsRead();
-            itemView.setBackgroundColor(mReadPostCardViewBackgroundColor);
-            titleTextView.setTextColor(mReadPostTitleColor);
-            if (mActivity != null && mActivity instanceof MarkPostAsReadInterface) {
-                ((MarkPostAsReadInterface) mActivity).markPostAsRead(post);
+            if (!post.isRead()) {
+                post.markAsRead();
+                itemView.setBackgroundColor(mReadPostCardViewBackgroundColor);
+                titleTextView.setTextColor(mReadPostTitleColor);
+                if (mActivity != null && mActivity instanceof MarkPostAsReadInterface) {
+                    ((MarkPostAsReadInterface) mActivity).markPostAsRead(post);
+                }
             }
         }
     }