From f4d12b625a085965b6198aa635befc452a05deb4 Mon Sep 17 00:00:00 2001 From: Kyle Drake Date: Sat, 29 Feb 2020 14:50:05 -0500 Subject: [PATCH] Move follow count to a weekly compute task due to bugs/abuse. *sigh* --- models/site.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/models/site.rb b/models/site.rb index ee5e95b4..1158e0f9 100644 --- a/models/site.rb +++ b/models/site.rb @@ -379,13 +379,15 @@ class Site < Sequel::Model follow = followings_dataset.filter(site_id: site.id).first site.events_dataset.filter(follow_id: follow.id).delete follow.delete - DB['update sites set follow_count=follow_count-1 where id=?', site.id].first if scorable_follow?(site) + # FIXME This is a being abused somehow. A weekly script now computes this. + # DB['update sites set follow_count=follow_count-1 where id=?', site.id].first if scorable_follow?(site) end false else DB.transaction do follow = add_following site_id: site.id - DB['update sites set follow_count=follow_count+1 where id=?', site.id].first if scorable_follow?(site) + # FIXME see above. + # DB['update sites set follow_count=follow_count+1 where id=?', site.id].first if scorable_follow?(site) Event.create site_id: site.id, actioning_site_id: self.id, follow_id: follow.id end