From fb90ca074ec0a6fc7e90098a3eb26132643e6899 Mon Sep 17 00:00:00 2001
From: Neil Alexander <neilalexander@users.noreply.github.com>
Date: Thu, 7 Jan 2021 14:23:38 +0000
Subject: [PATCH] Don't advance position on remove again unless needed

---
 eduserver/cache/cache.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/eduserver/cache/cache.go b/eduserver/cache/cache.go
index 974808df8..d2fd26302 100644
--- a/eduserver/cache/cache.go
+++ b/eduserver/cache/cache.go
@@ -151,8 +151,6 @@ func (t *EDUCache) RemoveUser(userID, roomID string) int64 {
 	t.Lock()
 	defer t.Unlock()
 
-	t.latestSyncPosition++
-
 	roomData, ok := t.data[roomID]
 	if !ok {
 		return t.latestSyncPosition
@@ -163,6 +161,8 @@ func (t *EDUCache) RemoveUser(userID, roomID string) int64 {
 		return t.latestSyncPosition
 	}
 
+	t.latestSyncPosition++
+
 	timer.Stop()
 	delete(roomData.userSet, userID)