diff options
author | eal <eal@waifu.club> | 2017-11-21 23:09:59 +0000 |
---|---|---|
committer | eal <eal@waifu.club> | 2017-11-21 23:09:59 +0000 |
commit | 1625021a9cd01127171709324f165899a52acfe5 (patch) | |
tree | 71cf3225fd3b6ff18102139e8618a5535cb6919e | |
parent | 9b35a4e850ca9fe1d36b9157054bcde3b4e41083 (diff) | |
parent | e9037ffc8d38d905c746a30128ab8ca930acbcd1 (diff) | |
download | pleroma-1625021a9cd01127171709324f165899a52acfe5.tar.gz |
Merge branch 'fix/self-follow' into 'develop'
Dont include the user in their own follower count.
See merge request pleroma/pleroma!29
-rw-r--r-- | lib/pleroma/user.ex | 1 | ||||
-rw-r--r-- | test/user_test.exs | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/lib/pleroma/user.ex b/lib/pleroma/user.ex index f80c0ae33..68ffe184b 100644 --- a/lib/pleroma/user.ex +++ b/lib/pleroma/user.ex @@ -265,6 +265,7 @@ defmodule Pleroma.User do def update_follower_count(%User{} = user) do follower_count_query = from u in User, where: fragment("? @> ?", u.following, ^user.follower_address), + where: u.id != ^user.id, select: count(u.id) follower_count = Repo.one(follower_count_query) diff --git a/test/user_test.exs b/test/user_test.exs index 8ec23c751..31c5962e2 100644 --- a/test/user_test.exs +++ b/test/user_test.exs @@ -71,6 +71,7 @@ defmodule Pleroma.UserTest do {:error, _} = User.unfollow(user, user) + user = Repo.get(User, user.id) assert user.following == [user.ap_id] end |