about summary refs log tree commit diff
path: root/app/models
diff options
context:
space:
mode:
authorSurinna Curtis <ekiru.0@gmail.com>2017-09-13 18:32:10 -0500
committerSurinna Curtis <ekiru.0@gmail.com>2017-09-13 21:47:30 -0500
commitf9d7b8a94f4a89d76081a6265103f6d7439be250 (patch)
tree6e91787f0181b1b7ec3e658f2ee35710e2071aaa /app/models
parent211f0a951375a33ed4278eef030e76a6bde9c396 (diff)
Refactor handling of default params for muting to make code cleaner
Diffstat (limited to 'app/models')
-rw-r--r--app/models/concerns/account_interactions.rb3
1 files changed, 2 insertions, 1 deletions
diff --git a/app/models/concerns/account_interactions.rb b/app/models/concerns/account_interactions.rb
index 6f69ce1d4..0afdebf89 100644
--- a/app/models/concerns/account_interactions.rb
+++ b/app/models/concerns/account_interactions.rb
@@ -74,7 +74,8 @@ module AccountInteractions
     block_relationships.find_or_create_by!(target_account: other_account)
   end
 
-  def mute!(other_account, notifications: true)
+  def mute!(other_account, notifications: nil)
+    notifications = true if notifications.nil?
     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