Fix order of checks

This commit is contained in:
Kegan Dougal 2020-10-02 16:02:09 +01:00
parent 7b47e8456e
commit 7b1c227155

View file

@ -318,10 +318,6 @@ func (r *messagesReq) filterHistoryVisible(events []gomatrixserverlib.HeaderedEv
hisVisEvent = &queryRes.StateEvents[i]
}
}
if membershipEvent == nil {
wasJoined = false
break
}
if hisVisEvent == nil {
return events // apply no filtering as it defaults to Shared.
}
@ -329,6 +325,10 @@ func (r *messagesReq) filterHistoryVisible(events []gomatrixserverlib.HeaderedEv
if hisVis == "shared" {
return events // apply no filtering
}
if membershipEvent == nil {
wasJoined = false
break
}
membership, err := membershipEvent.Membership()
if err != nil {
wasJoined = false