From: David ‘Bombe’ Roden <bombe@pterodactylus.net>
Date: Tue, 21 Feb 2012 18:03:51 +0000 (+0100)
Subject: Always fire element-was-read events, even if element is not new.
X-Git-Tag: 0.8^2~26
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;h=36a0267479a22637095f901fc51fa99320329e14;p=Sone.git

Always fire element-was-read events, even if element is not new.

Sometimes elements are marked as read but not removed from the
notifications.
---

diff --git a/src/main/java/net/pterodactylus/sone/core/Core.java b/src/main/java/net/pterodactylus/sone/core/Core.java
index 108b05c..da2a2a1 100644
--- a/src/main/java/net/pterodactylus/sone/core/Core.java
+++ b/src/main/java/net/pterodactylus/sone/core/Core.java
@@ -1650,8 +1650,8 @@ public class Core extends AbstractService implements IdentityListener, UpdateLis
 	public void markPostKnown(Post post) {
 		post.setKnown(true);
 		synchronized (knownPosts) {
+			coreListenerManager.fireMarkPostKnown(post);
 			if (knownPosts.add(post.getId())) {
-				coreListenerManager.fireMarkPostKnown(post);
 				touchConfiguration();
 			}
 		}
@@ -1789,8 +1789,8 @@ public class Core extends AbstractService implements IdentityListener, UpdateLis
 	public void markReplyKnown(PostReply reply) {
 		reply.setKnown(true);
 		synchronized (knownReplies) {
+			coreListenerManager.fireMarkReplyKnown(reply);
 			if (knownReplies.add(reply.getId())) {
-				coreListenerManager.fireMarkReplyKnown(reply);
 				touchConfiguration();
 			}
 		}