From bdf0d3618db779a03cccf88d67a87b504ac33fe0 Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Mon, 18 Jan 2021 14:59:02 -0600
Subject: [PATCH] Refactor mod report modals

---
 app/soapbox/actions/moderation.js             | 49 +++++++++++++++++++
 .../features/admin/components/report.js       | 38 +++-----------
 2 files changed, 55 insertions(+), 32 deletions(-)
 create mode 100644 app/soapbox/actions/moderation.js

diff --git a/app/soapbox/actions/moderation.js b/app/soapbox/actions/moderation.js
new file mode 100644
index 000000000..75de6cc9f
--- /dev/null
+++ b/app/soapbox/actions/moderation.js
@@ -0,0 +1,49 @@
+import { defineMessages } from 'react-intl';
+import { openModal } from 'soapbox/actions/modal';
+import { deactivateUsers, deleteUsers } from 'soapbox/actions/admin';
+import snackbar from 'soapbox/actions/snackbar';
+
+const messages = defineMessages({
+  deactivateUserPrompt: { id: 'confirmations.admin.deactivate_user.message', defaultMessage: 'You are about to deactivate {acct}. Deactivating a user is a reversible action.' },
+  deactivateUserConfirm: { id: 'confirmations.admin.deactivate_user.confirm', defaultMessage: 'Deactivate {acct}' },
+  userDeactivated: { id: 'admin.reports.user_deactivated_message', defaultMessage: '{acct} was deactivated' },
+  deleteUserPrompt: { id: 'confirmations.admin.delete_user.message', defaultMessage: 'You are about to delete {acct}. THIS IS A DESTRUCTIVE ACTION THAT CANNOT BE UNDONE.' },
+  deleteUserConfirm: { id: 'confirmations.admin.delete_user.confirm', defaultMessage: 'Delete {acct}' },
+  userDeleted: { id: 'admin.reports.user_deleted_message', defaultMessage: '{acct} was deleted' },
+});
+
+export function deactivateUserModal(intl, accountId, afterConfirm = () => {}) {
+  return function(dispatch, getState) {
+    const state = getState();
+    const acct = state.getIn(['accounts', accountId, 'acct']);
+    dispatch(openModal('CONFIRM', {
+      message: intl.formatMessage(messages.deactivateUserPrompt, { acct: `@${acct}` }),
+      confirm: intl.formatMessage(messages.deactivateUserConfirm, { acct: `@${acct}` }),
+      onConfirm: () => {
+        dispatch(deactivateUsers([acct])).then(() => {
+          const message = intl.formatMessage(messages.userDeactivated, { acct: `@${acct}` });
+          dispatch(snackbar.success(message));
+          afterConfirm();
+        }).catch(() => {});
+      },
+    }));
+  };
+}
+
+export function deleteUserModal(intl, accountId, afterConfirm = () => {}) {
+  return function(dispatch, getState) {
+    const state = getState();
+    const acct = state.getIn(['accounts', accountId, 'acct']);
+    dispatch(openModal('CONFIRM', {
+      message: intl.formatMessage(messages.deleteUserPrompt, { acct: `@${acct}` }),
+      confirm: intl.formatMessage(messages.deleteUserConfirm, { acct: `@${acct}` }),
+      onConfirm: () => {
+        dispatch(deleteUsers([acct])).then(() => {
+          const message = intl.formatMessage(messages.userDeleted, { acct: `@${acct}` });
+          dispatch(snackbar.success(message));
+          afterConfirm();
+        }).catch(() => {});
+      },
+    }));
+  };
+}
diff --git a/app/soapbox/features/admin/components/report.js b/app/soapbox/features/admin/components/report.js
index a1e7d82bb..4ade81a30 100644
--- a/app/soapbox/features/admin/components/report.js
+++ b/app/soapbox/features/admin/components/report.js
@@ -9,20 +9,14 @@ import Button from 'soapbox/components/button';
 import DropdownMenu from 'soapbox/containers/dropdown_menu_container';
 import Accordion from 'soapbox/features/ui/components/accordion';
 import ReportStatus from './report_status';
-import { closeReports, deactivateUsers, deleteUsers } from 'soapbox/actions/admin';
+import { closeReports } from 'soapbox/actions/admin';
 import snackbar from 'soapbox/actions/snackbar';
-import { openModal } from 'soapbox/actions/modal';
+import { deactivateUserModal, deleteUserModal } from 'soapbox/actions/moderation';
 
 const messages = defineMessages({
   reportClosed: { id: 'admin.reports.report_closed_message', defaultMessage: 'Report on {acct} was closed' },
   deactivateUser: { id: 'admin.reports.actions.deactivate_user', defaultMessage: 'Deactivate {acct}' },
-  deactivateUserPrompt: { id: 'confirmations.admin.deactivate_user.message', defaultMessage: 'You are about to deactivate {acct}. Deactivating a user is a reversible action.' },
-  deactivateUserConfirm: { id: 'confirmations.admin.deactivate_user.confirm', defaultMessage: 'Deactivate {acct}' },
-  userDeactivated: { id: 'admin.reports.user_deactivated_message', defaultMessage: '{acct} was deactivated' },
   deleteUser: { id: 'admin.reports.actions.delete_user', defaultMessage: 'Delete {acct}' },
-  deleteUserPrompt: { id: 'confirmations.admin.delete_user.message', defaultMessage: 'You are about to delete {acct}. THIS IS A DESTRUCTIVE ACTION THAT CANNOT BE UNDONE.' },
-  deleteUserConfirm: { id: 'confirmations.admin.delete_user.confirm', defaultMessage: 'Delete {acct}' },
-  userDeleted: { id: 'admin.reports.user_deleted_message', defaultMessage: '{acct} was deleted' },
 });
 
 export default @connect()
@@ -60,34 +54,14 @@ class Report extends ImmutablePureComponent {
 
   handleDeactivateUser = () => {
     const { intl, dispatch, report } = this.props;
-    const nickname = report.getIn(['account', 'acct']);
-    dispatch(openModal('CONFIRM', {
-      message: intl.formatMessage(messages.deactivateUserPrompt, { acct: `@${nickname}` }),
-      confirm: intl.formatMessage(messages.deactivateUserConfirm, { acct: `@${nickname}` }),
-      onConfirm: () => {
-        dispatch(deactivateUsers([nickname])).then(() => {
-          const message = intl.formatMessage(messages.userDeactivated, { acct: `@${nickname}` });
-          dispatch(snackbar.success(message));
-        }).catch(() => {});
-        this.handleCloseReport();
-      },
-    }));
+    const accountId = report.getIn(['account', 'id']);
+    dispatch(deactivateUserModal(intl, accountId, () => this.handleCloseReport()));
   }
 
   handleDeleteUser = () => {
     const { intl, dispatch, report } = this.props;
-    const nickname = report.getIn(['account', 'acct']);
-    dispatch(openModal('CONFIRM', {
-      message: intl.formatMessage(messages.deleteUserPrompt, { acct: `@${nickname}` }),
-      confirm: intl.formatMessage(messages.deleteUserConfirm, { acct: `@${nickname}` }),
-      onConfirm: () => {
-        dispatch(deleteUsers([nickname])).then(() => {
-          const message = intl.formatMessage(messages.userDeleted, { acct: `@${nickname}` });
-          dispatch(snackbar.success(message));
-        }).catch(() => {});
-        this.handleCloseReport();
-      },
-    }));
+    const accountId = report.getIn(['account', 'id']);
+    dispatch(deleteUserModal(intl, accountId, () => this.handleCloseReport()));
   }
 
   handleAccordionToggle = setting => {