Exclude unregistered group members from invalid collisions check.

Fixes #13866
This commit is contained in:
Alex Hart 2025-01-06 13:38:05 -04:00
parent 0655b6f767
commit de6b9bc4ef

View file

@ -177,7 +177,12 @@ class NameCollisionTables(
val groupMembers: Optional<List<RecipientId>> = SignalDatabase.groups.getGroup(recipientId).map { it.members }
val invalidCollisions: Set<ReviewRecipient> = collisions.filter {
groupMembers.isPresent && (it.recipient.id !in groupMembers.get())
if (groupMembers.isPresent) {
val notAMember = it.recipient.id !in groupMembers.get()
val unregistered = it.recipient.isUnregistered
notAMember || unregistered
} else false
}.toSet()
val groups = (collisions - invalidCollisions).groupBy { SqlUtil.buildCaseInsensitiveGlobPattern(it.recipient.getDisplayName(context)) }