Move some logic to named functions

trento-project · Jul 11, 2023 · 3614c2e · 3614c2e
1 parent 3c1dd36
commit 3614c2e
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/assets/js/components/HostsList.jsx b/assets/js/components/HostsList.jsx
@@ -58,6 +58,16 @@ function HostsList() {
 
   const dispatch = useDispatch();
 
+  const openDeregistrationModal = (host) => {
+    setHostToDeregister(host);
+    setCleanUpModalOpen(true);
+  };
+
+  const cleanUpHost = (host) => {
+    setCleanUpModalOpen(false);
+    dispatch(deregisterHost(host));
+  };
+
   const config = {
     pagination: true,
     usePadding: false,
@@ -191,8 +201,7 @@ function HostsList() {
             <CleanUpButton
               cleaning={item.deregistering}
               onClick={() => {
-                setHostToDeregister(item);
-                setCleanUpModalOpen(true);
+                openDeregistrationModal(item);
               }}
             />
           ),
@@ -232,8 +241,7 @@ function HostsList() {
         hostname={hostToDeregister?.hostname}
         isOpen={!!cleanUpModalOpen}
         onCleanUp={() => {
-          setCleanUpModalOpen(false);
-          dispatch(deregisterHost(hostToDeregister));
+          cleanUpHost(hostToDeregister);
         }}
         onCancel={() => {
           setCleanUpModalOpen(false);