Skip to content

Commit

Permalink
Merge pull request #5731 from t3chguy/t3chguy/fix-room-type-misuse
Browse files Browse the repository at this point in the history
Don't wrongly show non-space invites in the space panel
  • Loading branch information
bmarty authored Apr 13, 2022
2 parents 5075775 + fd3bbab commit 2761b35
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
1 change: 1 addition & 0 deletions changelog.d/5731.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Don't wrongly show non-space invites in the space panel.
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ import org.matrix.android.sdk.api.session.room.RoomSortOrder
import org.matrix.android.sdk.api.session.room.accountdata.RoomAccountDataTypes
import org.matrix.android.sdk.api.session.room.model.Membership
import org.matrix.android.sdk.api.session.room.roomSummaryQueryParams
import org.matrix.android.sdk.api.session.room.spaceSummaryQueryParams
import org.matrix.android.sdk.api.session.room.summary.RoomAggregateNotificationCount
import org.matrix.android.sdk.api.session.space.SpaceOrderUtils
import org.matrix.android.sdk.api.session.space.model.SpaceOrderContent
Expand Down Expand Up @@ -272,11 +273,9 @@ class SpaceListViewModel @AssistedInject constructor(@Assisted initialState: Spa
}

private fun observeSpaceSummaries() {
val spaceSummaryQueryParams = roomSummaryQueryParams {
val params = spaceSummaryQueryParams {
memberships = listOf(Membership.JOIN, Membership.INVITE)
displayName = QueryStringValue.IsNotEmpty
excludeType = listOf(/**RoomType.MESSAGING,$*/
null)
}

val flowSession = session.flow()
Expand All @@ -288,7 +287,7 @@ class SpaceListViewModel @AssistedInject constructor(@Assisted initialState: Spa
it.getOrNull()
},
flowSession
.liveSpaceSummaries(spaceSummaryQueryParams),
.liveSpaceSummaries(params),
session
.accountDataService()
.getLiveRoomAccountDataEvents(setOf(RoomAccountDataTypes.EVENT_TYPE_SPACE_ORDER))
Expand Down

0 comments on commit 2761b35

Please sign in to comment.