Merge pull request #3205 from matrix-org/stefan/invites_main_room_list

feat: Expose a room list filter for Invites only

Author: Hywan

bindings/matrix-sdk-ffi/src/room_list.rs

@@ -16,8 +16,9 @@ use matrix_sdk_ui::{
     room_list_service::{
         filters::{
             new_filter_all, new_filter_any, new_filter_category, new_filter_favourite,
-            new_filter_fuzzy_match_room_name, new_filter_non_left, new_filter_none,
-            new_filter_normalized_match_room_name, new_filter_unread, RoomCategory,
+            new_filter_fuzzy_match_room_name, new_filter_invite, new_filter_non_left,
+            new_filter_none, new_filter_normalized_match_room_name, new_filter_unread,
+            RoomCategory,
         },
         BoxedFilterFn,
     },
@@ -421,6 +422,7 @@ pub enum RoomListEntriesDynamicFilterKind {
     NonLeft,
     Unread,
     Favourite,
+    Invite,
     Category { expect: RoomListFilterCategory },
     None,
     NormalizedMatchRoomName { pattern: String },
@@ -460,6 +462,7 @@ impl FilterWrapper {
             Kind::NonLeft => Self(Box::new(new_filter_non_left(client))),
             Kind::Unread => Self(Box::new(new_filter_unread(client))),
             Kind::Favourite => Self(Box::new(new_filter_favourite(client))),
+            Kind::Invite => Self(Box::new(new_filter_invite(client))),
             Kind::Category { expect } => Self(Box::new(new_filter_category(client, expect.into()))),
             Kind::None => Self(Box::new(new_filter_none())),
             Kind::NormalizedMatchRoomName { pattern } => {

crates/matrix-sdk-ui/src/room_list_service/filters/invite.rs

@@ -0,0 +1,81 @@
+use matrix_sdk::{Client, RoomListEntry};
+use matrix_sdk_base::RoomState;
+
+use super::Filter;
+
+struct InviteRoomMatcher<F>
+where
+    F: Fn(&RoomListEntry) -> Option<RoomState>,
+{
+    state: F,
+}
+
+impl<F> InviteRoomMatcher<F>
+where
+    F: Fn(&RoomListEntry) -> Option<RoomState>,
+{
+    fn matches(&self, room: &RoomListEntry) -> bool {
+        if !matches!(room, RoomListEntry::Filled(_) | RoomListEntry::Invalidated(_)) {
+            return false;
+        }
+
+        if let Some(state) = (self.state)(room) {
+            state == RoomState::Invited
+        } else {
+            false
+        }
+    }
+}
+
+/// Create a new filter that will accept all filled or invalidated entries, but
+/// filters out rooms that are not invites (see
+/// [`matrix_sdk_base::RoomState::Invited`]).
+pub fn new_filter(client: &Client) -> impl Filter {
+    let client = client.clone();
+
+    let matcher = InviteRoomMatcher {
+        state: move |room| {
+            let room_id = room.as_room_id()?;
+            let room = client.get_room(room_id)?;
+            Some(room.state())
+        },
+    };
+
+    move |room_list_entry| -> bool { matcher.matches(room_list_entry) }
+}
+
+#[cfg(test)]
+mod tests {
+    use matrix_sdk::RoomListEntry;
+    use matrix_sdk_base::RoomState;
+    use ruma::room_id;
+
+    use super::InviteRoomMatcher;
+
+    #[test]
+    fn test_all_invite_kind_of_room_list_entry() {
+        // When we can't figure out the room state, nothing matches.
+        let matcher = InviteRoomMatcher { state: |_| None };
+        assert!(!matcher.matches(&RoomListEntry::Empty));
+        assert!(!matcher.matches(&RoomListEntry::Filled(room_id!("!r0:bar.org").to_owned())));
+        assert!(!matcher.matches(&RoomListEntry::Invalidated(room_id!("!r0:bar.org").to_owned())));
+
+        // When a room has been left, it doesn't match.
+        let matcher = InviteRoomMatcher { state: |_| Some(RoomState::Left) };
+        assert!(!matcher.matches(&RoomListEntry::Empty));
+        assert!(!matcher.matches(&RoomListEntry::Filled(room_id!("!r0:bar.org").to_owned())));
+        assert!(!matcher.matches(&RoomListEntry::Invalidated(room_id!("!r0:bar.org").to_owned())));
+
+        // When a room has been joined, it doesn't match.
+        let matcher = InviteRoomMatcher { state: |_| Some(RoomState::Joined) };
+        assert!(!matcher.matches(&RoomListEntry::Empty));
+        assert!(!matcher.matches(&RoomListEntry::Filled(room_id!("!r0:bar.org").to_owned())));
+        assert!(!matcher.matches(&RoomListEntry::Invalidated(room_id!("!r0:bar.org").to_owned())));
+
+        // When a room has been joined, it does match (unless it's empty).
+        let matcher = InviteRoomMatcher { state: |_| Some(RoomState::Invited) };
+        assert!(!matcher.matches(&RoomListEntry::Empty));
+        assert!(matcher.matches(&RoomListEntry::Filled(room_id!("!r0:bar.org").to_owned())));
+        assert!(matcher.matches(&RoomListEntry::Invalidated(room_id!("!r0:bar.org").to_owned())));
+    }
+}

crates/matrix-sdk-ui/src/room_list_service/filters/mod.rs

@@ -59,6 +59,7 @@ mod any;
 mod category;
 mod favourite;
 mod fuzzy_match_room_name;
+mod invite;
 mod non_left;
 mod none;
 mod normalized_match_room_name;
@@ -70,6 +71,7 @@ pub use any::new_filter as new_filter_any;
 pub use category::{new_filter as new_filter_category, RoomCategory};
 pub use favourite::new_filter as new_filter_favourite;
 pub use fuzzy_match_room_name::new_filter as new_filter_fuzzy_match_room_name;
+pub use invite::new_filter as new_filter_invite;
 use matrix_sdk::RoomListEntry;
 pub use non_left::new_filter as new_filter_non_left;
 pub use none::new_filter as new_filter_none;