From 60bf819a5607793f653706b5ef29c9f38f9501f0 Mon Sep 17 00:00:00 2001
From: mattwr18 <mattwr18@gmail.com>
Date: Mon, 9 Dec 2019 17:41:38 +0100
Subject: [PATCH] Remove counts to make less expensive db requests

---
 webapp/components/User/User.vue                   |  8 ++++++--
 .../components/features/ReportRow/ReportRow.vue   | 10 ++++++++--
 webapp/graphql/Moderation.js                      | 15 ---------------
 3 files changed, 14 insertions(+), 19 deletions(-)

diff --git a/webapp/components/User/User.vue b/webapp/components/User/User.vue
index 2b1a62d5f..ade52ea5e 100644
--- a/webapp/components/User/User.vue
+++ b/webapp/components/User/User.vue
@@ -8,7 +8,7 @@
   <dropdown v-else :class="{ 'disabled-content': user.disabled }" placement="top-start" offset="0">
     <template slot="default" slot-scope="{ openMenu, closeMenu, isOpen }">
       <nuxt-link :to="userLink" :class="['user', isOpen && 'active']">
-        <div @mouseover="openMenu(true)" @mouseleave="closeMenu(true)">
+        <div @mouseover="openInfoMenu" @mouseleave="closeMenu(true)">
           <hc-avatar v-if="showAvatar" class="avatar" :user="user" />
           <div>
             <ds-text class="userinfo">
@@ -26,7 +26,7 @@
         </div>
       </nuxt-link>
     </template>
-    <template slot="popover">
+    <template slot="popover" v-if="showCounts">
       <div style="min-width: 250px">
         <hc-badges v-if="user.badges && user.badges.length" :badges="user.badges" />
         <ds-text
@@ -106,6 +106,7 @@ export default {
     showAvatar: { type: Boolean, default: true },
     trunc: { type: Number, default: 18 }, // "-1" is no trunc
     dateTime: { type: [Date, String], default: null },
+    showCounts: { type: Boolean, default: true },
   },
   computed: {
     ...mapGetters({
@@ -142,6 +143,9 @@ export default {
       this.user.followedByCount = followedByCount
       this.user.followedByCurrentUser = followedByCurrentUser
     },
+    openInfoMenu() {
+      if (this.showCounts) this.openMenu(true)
+    },
   },
 }
 </script>
diff --git a/webapp/components/features/ReportRow/ReportRow.vue b/webapp/components/features/ReportRow/ReportRow.vue
index 6dce9c317..e5391d311 100644
--- a/webapp/components/features/ReportRow/ReportRow.vue
+++ b/webapp/components/features/ReportRow/ReportRow.vue
@@ -19,7 +19,7 @@
       <!-- Content Column -->
       <td class="ds-table-col" data-test="report-content">
         <client-only v-if="isUser">
-          <hc-user :user="report.resource" :showAvatar="false" :trunc="30" />
+          <hc-user :user="report.resource" :showAvatar="false" :trunc="30" :showCounts="false" />
         </client-only>
         <nuxt-link v-else class="title" :to="linkTarget">
           {{ linkText | truncate(50) }}
@@ -29,7 +29,12 @@
       <!-- Author Column -->
       <td class="ds-table-col" data-test="report-author">
         <client-only v-if="!isUser">
-          <hc-user :user="report.resource.author" :showAvatar="false" :trunc="30" />
+          <hc-user
+            :user="report.resource.author"
+            :showAvatar="false"
+            :trunc="30"
+            :showCounts="false"
+          />
         </client-only>
         <span v-else>—</span>
       </td>
@@ -46,6 +51,7 @@
             :showAvatar="false"
             :trunc="30"
             :date-time="report.updatedAt"
+            :showCounts="false"
           />
         </client-only>
       </td>
diff --git a/webapp/graphql/Moderation.js b/webapp/graphql/Moderation.js
index b4048672f..c63df921e 100644
--- a/webapp/graphql/Moderation.js
+++ b/webapp/graphql/Moderation.js
@@ -29,9 +29,6 @@ export const reportsListQuery = () => {
             id
             slug
             name
-            followedByCount
-            contributionsCount
-            commentedCount
           }
         }
         resource {
@@ -42,9 +39,6 @@ export const reportsListQuery = () => {
             name
             disabled
             deleted
-            followedByCount
-            contributionsCount
-            commentedCount
           }
           ... on Comment {
             id
@@ -57,9 +51,6 @@ export const reportsListQuery = () => {
               name
               disabled
               deleted
-              followedByCount
-              contributionsCount
-              commentedCount
             }
             post {
               id
@@ -81,9 +72,6 @@ export const reportsListQuery = () => {
               name
               disabled
               deleted
-              followedByCount
-              contributionsCount
-              commentedCount
             }
           }
         }
@@ -94,9 +82,6 @@ export const reportsListQuery = () => {
             name
             disabled
             deleted
-            followedByCount
-            contributionsCount
-            commentedCount
           }
           createdAt
           reasonCategory