From e196005b0b395b594eecc695dc7e6d8a168293a5 Mon Sep 17 00:00:00 2001
From: sballesteros <sebastien.ballesteros@gmail.com>
Date: Thu, 21 Nov 2019 21:38:57 -0800
Subject: [PATCH] handle content deletion #11

---
 src/components/moderate.js        | 73 +++++++++++++++++--------------
 src/components/moderation-card.js |  4 +-
 src/middlewares/cache.js          | 11 ++++-
 3 files changed, 51 insertions(+), 37 deletions(-)

diff --git a/src/components/moderate.js b/src/components/moderate.js
index fe96dec..d4b50fc 100644
--- a/src/components/moderate.js
+++ b/src/components/moderate.js
@@ -12,11 +12,10 @@ import ModerationCard from './moderation-card';
 export default function Moderate() {
   const [user] = useUser();
   const [bookmark, setBookmark] = useState(null);
+  const [excluded, setExcluded] = useState(new Set());
 
   const search = createModerationQs({ bookmark });
 
-  // TODO set of exluded content
-
   const [results, progress] = useActionsSearchResults(search, !!bookmark);
 
   const [isOpenedMap, setIsOpenedMap] = useState(
@@ -56,36 +55,46 @@ export default function Moderate() {
         ) : (
           <div>
             <ul className="moderate__card-list">
-              {results.rows.map(({ doc }) => (
-                <li key={getId(doc)}>
-                  <ModerationCard
-                    user={user}
-                    reviewAction={doc}
-                    isOpened={isOpenedMap[getId(doc)] || false}
-                    isLockedBy={undefined /* TODO wire */}
-                    onOpen={() => {
-                      setIsOpenedMap(
-                        results.rows.reduce((map, row) => {
-                          map[getId(row.doc)] = getId(row.doc) === getId(doc);
-                          return map;
-                        }, {})
-                      );
-                    }}
-                    onSuccess={moderationAction => {
-                      // TODO add a `exclude` params to the search qs so that we refresh the results in a safe way
-                      console.log('moderationAction', moderationAction);
-                    }}
-                    onClose={() => {
-                      setIsOpenedMap(
-                        results.rows.reduce((map, row) => {
-                          map[getId(row.doc)] = false;
-                          return map;
-                        }, {})
-                      );
-                    }}
-                  />
-                </li>
-              ))}
+              {results.rows
+                .filter(row => !excluded.has(getId(row.doc)))
+                .map(({ doc }) => (
+                  <li key={getId(doc)}>
+                    <ModerationCard
+                      user={user}
+                      reviewAction={doc}
+                      isOpened={isOpenedMap[getId(doc)] || false}
+                      isLockedBy={undefined /* TODO wire */}
+                      onOpen={() => {
+                        setIsOpenedMap(
+                          results.rows.reduce((map, row) => {
+                            map[getId(row.doc)] = getId(row.doc) === getId(doc);
+                            return map;
+                          }, {})
+                        );
+                      }}
+                      onSuccess={(moderationActionType, reviewActionId) => {
+                        if (
+                          moderationActionType ===
+                            'ModerateRapidPREreviewAction' ||
+                          moderationActionType ===
+                            'IgnoreReportRapidPREreviewAction'
+                        ) {
+                          setExcluded(
+                            new Set(Array.from(excluded).concat(reviewActionId))
+                          );
+                        }
+                      }}
+                      onClose={() => {
+                        setIsOpenedMap(
+                          results.rows.reduce((map, row) => {
+                            map[getId(row.doc)] = false;
+                            return map;
+                          }, {})
+                        );
+                      }}
+                    />
+                  </li>
+                ))}
             </ul>
           </div>
         )}
diff --git a/src/components/moderation-card.js b/src/components/moderation-card.js
index 722d57f..0d6173d 100644
--- a/src/components/moderation-card.js
+++ b/src/components/moderation-card.js
@@ -276,7 +276,6 @@ function ModerationCardModal({
                       moderationReason: ref.current.value
                     },
                     body => {
-                      onSuccess(body);
                       setFrame('success');
                     }
                   );
@@ -324,7 +323,6 @@ function ModerationCardModal({
                       moderationReason: ref.current.value
                     },
                     body => {
-                      onSuccess(body);
                       setFrame('success');
                     }
                   );
@@ -373,7 +371,6 @@ function ModerationCardModal({
                       moderationReason: ref.current.value
                     },
                     body => {
-                      onSuccess(body);
                       setFrame('success');
                     }
                   );
@@ -396,6 +393,7 @@ function ModerationCardModal({
             <Controls>
               <Button
                 onClick={() => {
+                  onSuccess(defaultFrame, getId(reviewAction));
                   onClose();
                 }}
               >
diff --git a/src/middlewares/cache.js b/src/middlewares/cache.js
index 75b3647..bb57af9 100644
--- a/src/middlewares/cache.js
+++ b/src/middlewares/cache.js
@@ -16,7 +16,10 @@ function getOnlyIfValue(obj = {}) {
       return obj._rev;
 
     default:
-      throw createError(500, 'getOnlyIfValue: invalid type for obj');
+      throw createError(
+        500,
+        `getOnlyIfValue: invalid type for obj (got ${obj['@type']})`
+      );
   }
 }
 
@@ -196,7 +199,11 @@ export function invalidate() {
           case 'ModerateRapidPREreviewAction':
           case 'RequestForRapidPREreviewAction':
           case 'RapidPREreviewAction': {
-            if (action['@type'] === 'ModerateRapidPREreviewAction') {
+            if (
+              action['@type'] === 'ModerateRapidPREreviewAction' ||
+              action['@type'] === 'ReportRapidPREreviewAction' ||
+              action['@type'] === 'IgnoreReportRapidPREreviewAction'
+            ) {
               action = action.result;
             }