From dd5f4872fce794ed75e158d1beeaf91fc16ac5af Mon Sep 17 00:00:00 2001
From: infinite-persistence <inf.persistence@gmail.com>
Date: Fri, 25 Mar 2022 14:36:24 +0800
Subject: [PATCH] Minor cleanup, no functional change

- Self-documenting variable.
- Remove incorrect comments.
- Remove redundant variables.
---
 ui/component/claimListDiscover/view.jsx  |  6 +-----
 ui/component/claimTilesDiscover/view.jsx | 15 ++++-----------
 2 files changed, 5 insertions(+), 16 deletions(-)

diff --git a/ui/component/claimListDiscover/view.jsx b/ui/component/claimListDiscover/view.jsx
index a76b02f2b..ab3fa1d17 100644
--- a/ui/component/claimListDiscover/view.jsx
+++ b/ui/component/claimListDiscover/view.jsx
@@ -642,11 +642,7 @@ function ClaimListDiscover(props: Props) {
 
   useFetchViewCount(fetchViewCount, finalUris, claimsByUri, doFetchViewCount);
 
-  const shouldFetchUserMemberships = true;
-  const arrayOfContentUris = finalUris;
-  const convertClaimUrlsToIds = claimsByUri;
-
-  useGetUserMemberships(shouldFetchUserMemberships, arrayOfContentUris, convertClaimUrlsToIds, doFetchUserMemberships);
+  useGetUserMemberships(true, finalUris, claimsByUri, doFetchUserMemberships);
 
   React.useEffect(() => {
     if (shouldPerformSearch) {
diff --git a/ui/component/claimTilesDiscover/view.jsx b/ui/component/claimTilesDiscover/view.jsx
index 3716420dc..591db3422 100644
--- a/ui/component/claimTilesDiscover/view.jsx
+++ b/ui/component/claimTilesDiscover/view.jsx
@@ -81,10 +81,8 @@ function ClaimTilesDiscover(props: Props) {
     doFetchUserMemberships,
   } = props;
 
-  // reference to the claim-grid
-  const sectionRef = React.useRef();
-  // determine the index where the ad should be injected
-  const injectedIndex = useLastVisibleItem(injectedItem, sectionRef);
+  const listRef = React.useRef();
+  const injectedIndex = useLastVisibleItem(injectedItem, listRef);
 
   const prevUris = React.useRef();
   const claimSearchUris = claimSearchResults || [];
@@ -119,12 +117,9 @@ function ClaimTilesDiscover(props: Props) {
   // --------------------------------------------------------------------------
   // --------------------------------------------------------------------------
 
-  // populate the view counts for the current claim uris
   useFetchViewCount(fetchViewCount, uris, claimsByUri, doFetchViewCount);
 
-  const shouldFetchUserMemberships = true;
-
-  useGetUserMemberships(shouldFetchUserMemberships, uris, claimsByUri, doFetchUserMemberships);
+  useGetUserMemberships(true, uris, claimsByUri, doFetchUserMemberships);
 
   // Run `doClaimSearch`
   React.useEffect(() => {
@@ -135,11 +130,10 @@ function ClaimTilesDiscover(props: Props) {
   }, [doClaimSearch, shouldPerformSearch, optionsStringified]);
 
   return (
-    <ul ref={sectionRef} className="claim-grid">
+    <ul ref={listRef} className="claim-grid">
       {finalUris && finalUris.length
         ? finalUris.map((uri, i) => {
             if (uri) {
-              // if indexes match, inject ad in place of tile (aka replace it)
               if (injectedIndex === i && injectedItem && injectedItem.replace) {
                 return <React.Fragment key={uri}>{injectedItem.node}</React.Fragment>;
               }
@@ -147,7 +141,6 @@ function ClaimTilesDiscover(props: Props) {
               return (
                 <React.Fragment key={uri}>
                   {injectedIndex === i && injectedItem && injectedItem.node}
-                  {/* inject ad */}
                   <ClaimPreviewTile
                     showNoSourceClaims={hasNoSource || showNoSourceClaims}
                     uri={uri}