From 1802bdb2d83fb77c6f8243ac37e023278eb653c8 Mon Sep 17 00:00:00 2001
From: Neil Alexander <neilalexander@users.noreply.github.com>
Date: Wed, 19 Oct 2022 12:43:44 +0100
Subject: [PATCH] Actually the switch is less clear

---
 userapi/consumers/clientapi.go | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/userapi/consumers/clientapi.go b/userapi/consumers/clientapi.go
index ae362569e..79f1bf06f 100644
--- a/userapi/consumers/clientapi.go
+++ b/userapi/consumers/clientapi.go
@@ -81,12 +81,7 @@ func (s *OutputReceiptEventConsumer) onMessage(ctx context.Context, msgs []*nats
 	readPos := msg.Header.Get(jetstream.EventID)
 	evType := msg.Header.Get("type")
 
-	switch {
-	case readPos == "":
-		return true
-	case evType == "m.read": // allowed
-	case evType == "m.read.private": // allowed
-	default:
+	if readPos == "" || (evType != "m.read" && evType != "m.read.private") {
 		return true
 	}