squeegily/pleroma
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Poll notification: only notify local users

Browse Source
This commit is contained in:
Alex Gleason 2021-07-18 11:03:56 -05:00
parent f314726ea7
commit d06e0a3312
No known key found for this signature in database
GPG Key ID: 7211D1F99744FBB7
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/pleroma/notification.ex
View File

@ -471,9 +471,11 @@ defmodule Pleroma.Notification do
end end
notifications = notifications =
Enum.map([actor | voters], fn ap_id -> Enum.reduce([actor | voters], [], fn ap_id, acc ->
with %User{} = user <- User.get_by_ap_id(ap_id) do with %User{local: true} = user <- User.get_by_ap_id(ap_id) do
create_notification(activity, user, type: "poll") [create_notification(activity, user, type: "poll") | acc]
else
_ -> acc
end end
end) end)

2
test/pleroma/notification_test.exs
View File

@ -139,7 +139,7 @@ defmodule Pleroma.NotificationTest do
{:ok, notifications} = Notification.create_poll_notifications(activity) {:ok, notifications} = Notification.create_poll_notifications(activity)
assert [user1.id, user3.id, user2.id] == Enum.map(notifications, & &1.user_id) assert [user2.id, user3.id, user1.id] == Enum.map(notifications, & &1.user_id)
end end
describe "CommonApi.post/2 notification-related functionality" do describe "CommonApi.post/2 notification-related functionality" do