diff options
| author | Surinna Curtis <ekiru.0@gmail.com> | 2017-09-13 17:42:52 -0500 |
|---|---|---|
| committer | Surinna Curtis <ekiru.0@gmail.com> | 2017-09-13 21:47:30 -0500 |
| commit | 211f0a951375a33ed4278eef030e76a6bde9c396 (patch) | |
| tree | 8b8e3f413f0ee1c98d36cc68050844899a8bf733 | |
| parent | 4a527154b7e52653d4638d7368248eb150452f19 (diff) | |
add an explanatory comment to AccountInteractions
| -rw-r--r-- | app/models/concerns/account_interactions.rb | 1 |
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! |