From fd97007364d03121b417cecdde750b0b6f214bb2 Mon Sep 17 00:00:00 2001 From: Molly Struve Date: Mon, 30 Nov 2020 08:18:19 -0500 Subject: [PATCH] Bug Fix:Remove nil cached_tag_lists when Collecting Tags (#11648) --- app/workers/follows/update_points_worker.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/workers/follows/update_points_worker.rb b/app/workers/follows/update_points_worker.rb index 1dd4852cf..3b9322ce8 100644 --- a/app/workers/follows/update_points_worker.rb +++ b/app/workers/follows/update_points_worker.rb @@ -31,10 +31,10 @@ module Follows last_100_long_page_view_article_ids = user.page_views.where(time_tracked_in_seconds: 45..) .pluck(:article_id).last(100) articles = Article.where(id: last_100_reactable_ids + last_100_long_page_view_article_ids) - tags = articles.pluck(:cached_tag_list).flat_map { |list| list.split(", ") } + tags = articles.pluck(:cached_tag_list).compact.flat_map { |list| list.split(", ") } occurrences = tags.count(tag.name) bonus = inverse_popularity_bonus(tag) - Math.log(occurrences + bonus + 1) # +1 is purelt to avoid log(0) => -infinity + Math.log(occurrences + bonus + 1) # +1 is purely to avoid log(0) => -infinity end def adjust_other_tag_follows_of_user(user_id)