From 0214caeaeaa7af72b33e51cdc7069878217a3f96 Mon Sep 17 00:00:00 2001 From: strawberry Date: Wed, 3 Apr 2024 16:42:35 -0400 Subject: [PATCH] always allow `count` to be filled in search response i fail to see any reason why we would always want this to be None Signed-off-by: strawberry --- src/api/client_server/search.rs | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/api/client_server/search.rs b/src/api/client_server/search.rs index c93038ea..d2a305d7 100644 --- a/src/api/client_server/search.rs +++ b/src/api/client_server/search.rs @@ -167,11 +167,7 @@ pub async fn search_events_route(body: Ruma) -> Resu Ok(search_events::v3::Response::new(ResultCategories { room_events: ResultRoomEvents { - count: if cfg!(feature = "element_hacks") { - Some((results.len() as u32).into()) - } else { - None - }, // TODO: set this to none. Element shouldn't depend on it (strawberry: why not?) + count: Some((results.len() as u32).into()), groups: BTreeMap::new(), // TODO next_batch, results,