From a76983ca0aa920d0b09aeff5733ef5e81d65e7fc Mon Sep 17 00:00:00 2001 From: Greyson Parrelli Date: Mon, 22 Nov 2021 22:36:00 -0500 Subject: [PATCH] Add logging around changes in badges on a profile. --- .../securesms/jobs/RetrieveProfileJob.java | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/RetrieveProfileJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/RetrieveProfileJob.java index df4e2456cd..a4e9be2752 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/RetrieveProfileJob.java +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/RetrieveProfileJob.java @@ -16,6 +16,7 @@ import org.signal.core.util.logging.Log; import org.signal.zkgroup.profiles.ProfileKey; import org.signal.zkgroup.profiles.ProfileKeyCredential; import org.thoughtcrime.securesms.badges.Badges; +import org.thoughtcrime.securesms.badges.models.Badge; import org.thoughtcrime.securesms.crypto.ProfileKeyUtil; import org.thoughtcrime.securesms.database.GroupDatabase; import org.thoughtcrime.securesms.database.RecipientDatabase; @@ -343,14 +344,18 @@ public class RetrieveProfileJob extends BaseJob { } } - private void setProfileBadges(@NonNull Recipient recipient, @Nullable List badges) { - if (badges == null) { + private void setProfileBadges(@NonNull Recipient recipient, @Nullable List serviceBadges) { + if (serviceBadges == null) { return; } - SignalDatabase.recipients() - .setBadges(recipient.getId(), - badges.stream().map(Badges::fromServiceBadge).collect(java.util.stream.Collectors.toList())); + List badges = serviceBadges.stream().map(Badges::fromServiceBadge).collect(java.util.stream.Collectors.toList()); + + if (badges.size() != recipient.getBadges().size()) { + Log.i(TAG, "Likely change in badges for " + recipient.getId() + ". Going from " + recipient.getBadges().size() + " badge(s) to " + badges.size() + "."); + } + + SignalDatabase.recipients().setBadges(recipient.getId(), badges); } private void setProfileKeyCredential(@NonNull Recipient recipient,