diff --git a/crates/apub/src/protocol/objects/group.rs b/crates/apub/src/protocol/objects/group.rs index f077a5011..2b32915af 100644 --- a/crates/apub/src/protocol/objects/group.rs +++ b/crates/apub/src/protocol/objects/group.rs @@ -112,7 +112,7 @@ impl Group { actor_id: Some(self.id.into()), local: Some(false), private_key: None, - hidden: Some(false), + hidden: None, public_key: self.public_key.public_key_pem, last_refreshed_at: Some(naive_now()), icon: self.icon.map(|i| i.url.into()), @@ -143,7 +143,7 @@ impl Group { actor_id: Some(self.id.into()), local: None, private_key: None, - hidden: Some(false), + hidden: None, public_key: Some(self.public_key.public_key_pem), last_refreshed_at: Some(naive_now()), icon: Some(self.icon.map(|i| i.url.into())), diff --git a/crates/db_views_actor/src/community_view.rs b/crates/db_views_actor/src/community_view.rs index abab023a6..ee6066c22 100644 --- a/crates/db_views_actor/src/community_view.rs +++ b/crates/db_views_actor/src/community_view.rs @@ -173,22 +173,19 @@ impl<'a> CommunityQuery<'a> { if !self.is_mod_or_admin.unwrap_or(true) { query = query .filter(community::removed.eq(false)) - .filter(community::deleted.eq(false)); + .filter(community::deleted.eq(false)) + .filter( + community::hidden + .eq(false) + .or(community_follower::person_id.eq(person_id_join)), + ); } match self.sort.unwrap_or(SortType::Hot) { SortType::New => query = query.order_by(community::published.desc()), SortType::TopAll => query = query.order_by(community_aggregates::subscribers.desc()), SortType::TopMonth => query = query.order_by(community_aggregates::users_active_month.desc()), - SortType::Hot => { - query = query.order_by(community_aggregates::hot_rank.desc()); - // Don't show hidden communities in Hot (trending) - query = query.filter( - community::hidden - .eq(false) - .or(community_follower::person_id.eq(person_id_join)), - ); - } + SortType::Hot => query = query.order_by(community_aggregates::hot_rank.desc()), // Covers all other sorts _ => query = query.order_by(community_aggregates::users_active_month.desc()), };