pr suggestion rename isPrivate to isVisibilityPrivate

This commit is contained in:
Mauro Romito
2026-01-22 15:54:03 +01:00
committed by Mauro
parent b60a252d91
commit 758f1c03c2
4 changed files with 9 additions and 9 deletions

View File

@@ -136,7 +136,7 @@ class CreateRoomScreenViewModel: CreateRoomScreenViewModelType, CreateRoomScreen
context.$viewState
.dropFirst()
.map(\.roomAccessType)
.filter(\.isPrivate)
.filter(\.isVisibilityPrivate)
.removeDuplicates()
.receive(on: DispatchQueue.main)
.sink { [weak self] _ in
@@ -156,7 +156,7 @@ class CreateRoomScreenViewModel: CreateRoomScreenViewModelType, CreateRoomScreen
return
}
guard !state.roomAccessType.isPrivate,
guard !state.roomAccessType.isVisibilityPrivate,
let canonicalAlias = String.makeCanonicalAlias(aliasLocalPart: aliasLocalPart,
serverName: state.serverName) else {
// While is empty or private room we don't change or display the error
@@ -209,7 +209,7 @@ class CreateRoomScreenViewModel: CreateRoomScreenViewModelType, CreateRoomScreen
showLoadingIndicator()
// Better to double check the errors also when trying to create the room
if !state.roomAccessType.isPrivate {
if !state.roomAccessType.isVisibilityPrivate {
guard let canonicalAlias = String.makeCanonicalAlias(aliasLocalPart: state.aliasLocalPart,
serverName: state.serverName),
isRoomAliasFormatValid(alias: canonicalAlias) else {
@@ -261,7 +261,7 @@ class CreateRoomScreenViewModel: CreateRoomScreenViewModelType, CreateRoomScreen
isSpace: state.isSpace,
userIDs: [], // The invite users screen is shown next so we don't need to invite anyone right now.
avatarURL: avatarURL,
aliasLocalPart: state.roomAccessType.isPrivate ? nil : state.aliasLocalPart) {
aliasLocalPart: state.roomAccessType.isVisibilityPrivate ? nil : state.aliasLocalPart) {
case .success(let roomID):
guard case let .joined(roomProxy) = await userSession.clientProxy.roomForIdentifier(roomID) else {
state.bindings.alertInfo = AlertInfo(id: .failedCreatingRoom,

View File

@@ -43,7 +43,7 @@ struct CreateRoomScreen: View {
selectSpaceSection
}
roomAccessSection
if !context.viewState.roomAccessType.isPrivate {
if !context.viewState.roomAccessType.isVisibilityPrivate {
roomAliasSection
}
}

View File

@@ -1378,15 +1378,15 @@ private extension CreateRoomAccessType {
}
var visibility: RoomVisibility {
isPrivate ? .private : .public
isVisibilityPrivate ? .private : .public
}
var preset: RoomPreset {
isPrivate ? .privateChat : .publicChat
isVisibilityPrivate ? .privateChat : .publicChat
}
var historyVisibilityOverride: RoomHistoryVisibility? {
isPrivate ? .invited : nil
isVisibilityPrivate ? .invited : nil
}
var joinRuleOverride: JoinRule? {

View File

@@ -55,7 +55,7 @@ enum CreateRoomAccessType: Equatable {
case askToJoin
case `private`
var isPrivate: Bool {
var isVisibilityPrivate: Bool {
switch self {
case .private, .spaceMembers, .askToJoinWithSpaceMembers:
true