about summary refs log tree commit diff
diff options
context:
space:
mode:
authorSurinna Curtis <ekiru.0@gmail.com>2017-09-13 17:42:52 -0500
committerSurinna Curtis <ekiru.0@gmail.com>2017-09-13 21:47:30 -0500
commit211f0a951375a33ed4278eef030e76a6bde9c396 (patch)
tree8b8e3f413f0ee1c98d36cc68050844899a8bf733
parent4a527154b7e52653d4638d7368248eb150452f19 (diff)
add an explanatory comment to AccountInteractions
-rw-r--r--app/models/concerns/account_interactions.rb1
1 files changed, 1 insertions, 0 deletions
diff --git a/app/models/concerns/account_interactions.rb b/app/models/concerns/account_interactions.rb
index 4a4265df2..6f69ce1d4 100644
--- a/app/models/concerns/account_interactions.rb
+++ b/app/models/concerns/account_interactions.rb
@@ -76,6 +76,7 @@ module AccountInteractions
 
   def mute!(other_account, notifications: true)
     mute = mute_relationships.create_with(hide_notifications: notifications).find_or_create_by!(target_account: other_account)
+    # When toggling a mute between hiding and allowing notifications, the mute will already exist, so the find_or_create_by! call will return the existing Mute without updating the hide_notifications attribute. Therefore, we check that hide_notifications? is what we want and set it if it isn't.
     if mute.hide_notifications? != notifications
       mute.hide_notifications = notifications
       mute.save!