Simplify logic.

This commit is contained in:
akwizgran
2022-08-08 12:30:20 +01:00
parent d3a06cf2c0
commit 42243f73f4

View File

@@ -330,29 +330,26 @@ class MailboxClientManager implements Service, EventListener {
for (Entry<ContactId, Updates> e : contactUpdates.entrySet()) { for (Entry<ContactId, Updates> e : contactUpdates.entrySet()) {
ContactId c = e.getKey(); ContactId c = e.getKey();
Updates u = e.getValue(); Updates u = e.getValue();
boolean isOwnMailboxUsable = if (!isOwnMailboxUsable(ownProperties, u.remote)) {
isOwnMailboxUsable(ownProperties, u.remote); // Our mailbox isn't usable for communicating with this
// contact, so don't assign/reassign this contact
continue;
}
if (isContactMailboxUsable(u.remote)) { if (isContactMailboxUsable(u.remote)) {
// The contact has a usable mailbox, so the contact should // The contact has a usable mailbox, so the contact should
// currently be assigned to the contact's mailbox for upload // currently be assigned to the contact's mailbox for upload
// and download // and download. Reassign the contact to our mailbox for
if (isOwnMailboxUsable) { // download
// Reassign the contact to our mailbox for download MailboxClient contactClient =
MailboxClient contactClient = requireNonNull(contactClients.get(c));
requireNonNull(contactClients.get(c)); contactClient.deassignContactForDownload(c);
contactClient.deassignContactForDownload(c); assignContactToOwnMailboxForDownload(c, u);
assignContactToOwnMailboxForDownload(c, u); } else {
}
// Else the contact remains assigned to the contact's mailbox
// for download
} else if (isOwnMailboxUsable) {
// The contact doesn't have a usable mailbox, so assign the // The contact doesn't have a usable mailbox, so assign the
// contact to our mailbox for upload and download // contact to our mailbox for upload and download
assignContactToOwnMailboxForUpload(c, u); assignContactToOwnMailboxForUpload(c, u);
assignContactToOwnMailboxForDownload(c, u); assignContactToOwnMailboxForDownload(c, u);
} }
// Else the contact doesn't have a usable mailbox and neither do we,
// so the contact remains unassigned for upload and download
} }
} }