TopArticleReviewers: implement actor migration

Bug: T219324
MusikAnimal · May 29, 2019 · 7e347ac · 7e347ac
1 parent fe11516
commit 7e347ac
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/tasks/top_article_reviewers.rb b/tasks/top_article_reviewers.rb
@@ -81,23 +81,24 @@ def self.top_reviewers(offset = 1)
     end_time = @mb.now.strftime('%Y%m%d%H%M%S')
 
     sql = %{
-      SELECT logtemp.log_user_text AS `reviewer`,
+      SELECT actor_name AS `reviewer`,
       COUNT(DISTINCT(logtemp.log_page)) AS `reviews`
       FROM (
-        SELECT log_user_text, log_page
+        SELECT log_actor, log_page
         FROM logging_userindex
         WHERE log_timestamp BETWEEN #{start_time} AND #{end_time}
         AND log_type = 'patrol'
         AND log_action = 'patrol'
         AND log_namespace = 0
         UNION
-        SELECT log_user_text, log_page
+        SELECT log_actor, log_page
         FROM logging_userindex
         WHERE log_timestamp BETWEEN #{start_time} AND #{end_time}
         AND log_type = 'pagetriage-curation'
         AND log_action = 'reviewed'
         AND log_namespace = 0
       ) logtemp
+      JOIN actor ON actor_id = log_actor
       GROUP BY reviewer
       ORDER BY reviews DESC
       LIMIT 100;