Avoid unneccessary recipient refreshes.

signalapp · Dec 5, 2019 · 2961a37 · 2961a37
1 parent 5e2a4fb
commit 2961a37
Showing 1 changed file with 15 additions and 10 deletions.
diff --git a/src/org/thoughtcrime/securesms/database/RecipientDatabase.java b/src/org/thoughtcrime/securesms/database/RecipientDatabase.java
@@ -659,36 +659,41 @@ public void setBlocked(@NonNull RecipientId id, boolean blocked) {
   public void setMessageRingtone(@NonNull RecipientId id, @Nullable Uri notification) {
     ContentValues values = new ContentValues();
     values.put(MESSAGE_RINGTONE, notification == null ? null : notification.toString());
-    update(id, values);
-    Recipient.live(id).refresh();
+    if (update(id, values)) {
+      Recipient.live(id).refresh();
+    }
   }
 
   public void setCallRingtone(@NonNull RecipientId id, @Nullable Uri ringtone) {
     ContentValues values = new ContentValues();
     values.put(CALL_RINGTONE, ringtone == null ? null : ringtone.toString());
-    update(id, values);
-    Recipient.live(id).refresh();
+    if (update(id, values)) {
+      Recipient.live(id).refresh();
+    }
   }
 
   public void setMessageVibrate(@NonNull RecipientId id, @NonNull VibrateState enabled) {
     ContentValues values = new ContentValues();
     values.put(MESSAGE_VIBRATE, enabled.getId());
-    update(id, values);
-    Recipient.live(id).refresh();
+    if (update(id, values)) {
+      Recipient.live(id).refresh();
+    }
   }
 
   public void setCallVibrate(@NonNull RecipientId id, @NonNull VibrateState enabled) {
     ContentValues values = new ContentValues();
     values.put(CALL_VIBRATE, enabled.getId());
-    update(id, values);
-    Recipient.live(id).refresh();
+    if (update(id, values)) {
+      Recipient.live(id).refresh();
+    }
   }
 
   public void setMuted(@NonNull RecipientId id, long until) {
     ContentValues values = new ContentValues();
     values.put(MUTE_UNTIL, until);
-    update(id, values);
-    Recipient.live(id).refresh();
+    if (update(id, values)) {
+      Recipient.live(id).refresh();
+    }
   }
 
   public void setSeenFirstInviteReminder(@NonNull RecipientId id) {