diff --git a/matrix_sdk_base/src/client.rs b/matrix_sdk_base/src/client.rs index 515ba1b3..21b14609 100644 --- a/matrix_sdk_base/src/client.rs +++ b/matrix_sdk_base/src/client.rs @@ -100,7 +100,7 @@ fn hoist_room_event_prev_content(event: &mut EventJson) -> Option { - if let Some(prev) = prev_content.deserialize().ok() { + if let Ok(prev) = prev_content.deserialize() { member.prev_content = Some(prev) } @@ -123,7 +123,7 @@ fn hoist_state_event_prev_content(event: &EventJson) -> Option { - if let Some(prev) = prev_content.deserialize().ok() { + if let Ok(prev) = prev_content.deserialize() { member.prev_content = Some(prev) } diff --git a/matrix_sdk_base/src/models/room.rs b/matrix_sdk_base/src/models/room.rs index be412840..805ae6c2 100644 --- a/matrix_sdk_base/src/models/room.rs +++ b/matrix_sdk_base/src/models/room.rs @@ -589,9 +589,7 @@ impl Room { // TODO: This would not be handled correctly as all the MemberEvents have the `prev_content` // inside of `unsigned` field. match event.membership_change() { - Invited | Joined => { - self.add_member(event) - } + Invited | Joined => self.add_member(event), Kicked | Banned | KickedAndBanned | InvitationRejected | Left => { self.remove_member(event) }