Skip to content

Commit

Permalink
Changed toast to warning - although condition to trigger should not b…
Browse files Browse the repository at this point in the history
…e possible
  • Loading branch information
alansley committed Aug 28, 2024
1 parent 413bc0b commit 736b531
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -310,10 +310,11 @@ class EditClosedGroupActivity : PassphraseRequiredActionBarActivity() {
val userPublicKey = TextSecurePreferences.getLocalNumber(this)!!
val userAsRecipient = Recipient.from(this, Address.fromSerialized(userPublicKey), false)

// There's presently way in the UI to get into the state whereby you could remove yourself from the group when removing any other members
// (you can't unselect yourself - the only way to leave is to "Leave Group" from the menu) - but it's possible that this was not always
// the case - so we can leave this in as defensive code in-case something goes screwy.
if (!members.contains(userAsRecipient) && !members.map { it.address.toString() }.containsAll(originalMembers.minus(userPublicKey))) {
// ACL TODO - Need a proper string for this
val message = "Can't leave while adding or removing other members."
return Toast.makeText(this@EditClosedGroupActivity, message, Toast.LENGTH_LONG).show()
return Log.w("EditClosedGroup", "Can't leave group while adding or removing other members.")
}

if (isClosedGroup) {
Expand Down

0 comments on commit 736b531

Please sign in to comment.