From 6f470d46196c6eaaa9125f6de02215f22c77e537 Mon Sep 17 00:00:00 2001
From: Jabar Jeremy <24471994+jabahum@users.noreply.github.com>
Date: Mon, 8 Jan 2024 12:39:31 +0300
Subject: [PATCH] U4X-297 : Complete and Submit for review buttons don't work
 (#53)

* U4X-297 : Complete and Submit for review buttons don't work

* fix : fix onComplete

* fix : onSubmit for review
---
 src/stock-items/types.ts                      |  3 +
 .../add-stock-operation.component.tsx         | 86 ++++---------------
 .../stock-operation-submission.component.tsx  | 18 ++--
 .../stock-operation.utils.tsx                 | 14 +++
 ...ck-operations-approve-button.component.tsx |  1 +
 ...ions-approve-dispatch-button.component.tsx |  1 +
 ...ock-operations-cancel-button.component.tsx |  1 +
 ...k-operations-complete-button.component.tsx |  1 +
 ...ns-completed-dispatch-button.component.tsx |  1 +
 .../stock-operations-dialog.component.tsx     | 20 ++++-
 ...perations-issue-stock-button.component.tsx |  1 +
 ...ock-operations-reject-button.component.tsx |  1 +
 ...ock-operations-return-button.component.tsx |  1 +
 13 files changed, 74 insertions(+), 75 deletions(-)

diff --git a/src/stock-items/types.ts b/src/stock-items/types.ts
index ac3ba980..c05501dc 100644
--- a/src/stock-items/types.ts
+++ b/src/stock-items/types.ts
@@ -3,3 +3,6 @@ import { StockOperationDTO } from "../core/api/types/stockOperation/StockOperati
 
 export type SaveStockItem = (item: StockItemDTO) => Promise<void>;
 export type SaveStockOperation = (item: StockOperationDTO) => Promise<void>;
+export type SaveStockOperationAction = (
+  item: StockOperationDTO
+) => Promise<void>;
diff --git a/src/stock-operations/add-stock-operation/add-stock-operation.component.tsx b/src/stock-operations/add-stock-operation/add-stock-operation.component.tsx
index 5fee0891..6e93ccfb 100644
--- a/src/stock-operations/add-stock-operation/add-stock-operation.component.tsx
+++ b/src/stock-operations/add-stock-operation/add-stock-operation.component.tsx
@@ -9,7 +9,10 @@ import { AddStockOperationProps } from "./types";
 import { useInitializeStockOperations } from "./add-stock-operation.resource";
 import { AccordionSkeleton } from "@carbon/react";
 import { closeOverlay } from "../../core/components/overlay/hook";
-import { addOrEditStockOperation } from "../stock-operation.utils";
+import {
+  addOrEditStockOperation,
+  showActionDialogButton,
+} from "../stock-operation.utils";
 import StockOperationApprovalButton from "../stock-operations-dialog/stock-operations-approve-button.component";
 import StockOperationRejectButton from "../stock-operations-dialog/stock-operations-reject-button.component";
 import StockOperationReturnButton from "../stock-operations-dialog/stock-operations-return-button.component";
@@ -18,18 +21,6 @@ import StockOperationPrintButton from "../stock-operations-dialog/stock-operatio
 import StockOperationApproveDispatchButton from "../stock-operations-dialog/stock-operations-approve-dispatch-button.component";
 import StockOperationCompleteDispatchButton from "../stock-operations-dialog/stock-operations-completed-dispatch-button.component";
 import StockOperationIssueStockButton from "../stock-operations-dialog/stock-operations-issue-stock-button.component";
-import {
-  OperationType,
-  StockOperationType,
-  StockOperationTypeCanCapturePurchasePrice,
-  StockOperationTypeIsNegativeQtyAllowed,
-  StockOperationTypeIsQuantityOptional,
-  StockOperationTypeRequiresActualBatchInformation,
-  StockOperationTypeRequiresBatchUuid,
-  StockOperationTypeRequiresDispatchAcknowledgement,
-  StockOperationTypeRequiresStockAdjustmentReason,
-  operationFromString,
-} from "../../core/api/types/stockOperation/StockOperationType";
 
 const AddStockOperation: React.FC<AddStockOperationProps> = (props) => {
   const { t } = useTranslation();
@@ -44,21 +35,8 @@ const AddStockOperation: React.FC<AddStockOperationProps> = (props) => {
   );
   const [selectedIndex, setSelectedIndex] = useState(0);
 
-  const [stockOperationType, setStockOperationType] = useState<
-    StockOperationType | null | undefined
-  >(null);
-  const [isNegativeQtyAllowed, setIsNegativeQtyAllowed] = useState(false);
-  const [requiresBatchUuid, setRequiresBatchUuid] = useState(false);
-  const [requiresActualBatchInformation, setRequiresActualBatchInformation] =
-    useState(false);
-  const [isQuantityOptional, setIsQuantityOptional] = useState(false);
-  const [canCapturePurchasePrice, setCanCapturePurchasePrice] = useState(false);
-  const [requireStockAdjustmentReason, setRequireStockAdjustmentReason] =
-    useState(false);
   const [requiresDispatchAcknowledgement, setRequiresDispatchAcknowledgement] =
     useState(false);
-  const [allowExpiredBatchNumbers, setAllowExpiredBatchNumbers] =
-    useState(false);
 
   if (isLoading) return <AccordionSkeleton />;
   if (isError) {
@@ -130,14 +108,18 @@ const AddStockOperation: React.FC<AddStockOperationProps> = (props) => {
             onGoBack: () => {
               setSelectedIndex(1);
             },
-            onComplete: () => {
-              // TODO: Update
+            onComplete: async () => {
+              await showActionDialogButton("Complete", false, props?.model);
             },
-            onSubmit: () => {
-              // TODO: Update
+            onSubmit: async () => {
+              await showActionDialogButton("Submit", false, props?.model);
             },
-            onDispatch: () => {
-              // TODO: Update
+            onDispatch: async () => {
+              await showActionDialogButton(
+                "Dispatch Approval",
+                false,
+                props?.model
+              );
             },
           }}
         />
@@ -146,39 +128,6 @@ const AddStockOperation: React.FC<AddStockOperationProps> = (props) => {
     },
   ];
 
-  // const currentStockOperationType = props.operations?.find(
-  //   (p) => p.uuid === props.operation?.uuid
-  // );
-
-  // console.info("data operations-->", props);
-
-  // console.info("data-->", currentStockOperationType);
-  // const operationType: OperationType = operationFromString(
-  //   currentStockOperationType?.operationType ?? ""
-  // );
-
-  // setStockOperationType(currentStockOperationType);
-  // setIsNegativeQtyAllowed(
-  //   StockOperationTypeIsNegativeQtyAllowed(operationType)
-  // );
-  // setRequiresBatchUuid(StockOperationTypeRequiresBatchUuid(operationType));
-  // setRequiresActualBatchInformation(
-  //   StockOperationTypeRequiresActualBatchInformation(operationType)
-  // );
-  // setIsQuantityOptional(StockOperationTypeIsQuantityOptional(operationType));
-  // setCanCapturePurchasePrice(
-  //   StockOperationTypeCanCapturePurchasePrice(operationType)
-  // );
-  // setRequireStockAdjustmentReason(
-  //   StockOperationTypeRequiresStockAdjustmentReason(operationType)
-  // );
-  // setRequiresDispatchAcknowledgement(
-  //   StockOperationTypeRequiresDispatchAcknowledgement(operationType)
-  // );
-  // setAllowExpiredBatchNumbers(
-  //   currentStockOperationType?.allowExpiredBatchNumbers ?? false
-  // );
-
   return (
     <>
       <div
@@ -295,8 +244,11 @@ const AddStockOperation: React.FC<AddStockOperationProps> = (props) => {
                   <StockOperationIssueStockButton operation={props.model} />
                 </div>
               )}
-
-              {canPrint && (
+              {(props.model?.permission?.isRequisitionAndCanIssueStock ||
+                props.model?.operationType === "stockissue" ||
+                props.model?.operationType === "requisition" ||
+                props.model?.operationType === "receipt" ||
+                props.model?.operationType === "transferout") && (
                 <div style={{ margin: "2px" }}>
                   <StockOperationPrintButton operation={props.model} />
                 </div>
diff --git a/src/stock-operations/add-stock-operation/stock-operation-submission.component.tsx b/src/stock-operations/add-stock-operation/stock-operation-submission.component.tsx
index 75f32e24..290f7462 100644
--- a/src/stock-operations/add-stock-operation/stock-operation-submission.component.tsx
+++ b/src/stock-operations/add-stock-operation/stock-operation-submission.component.tsx
@@ -1,7 +1,10 @@
 import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { StockOperationDTO } from "../../core/api/types/stockOperation/StockOperationDTO";
-import { SaveStockOperation } from "../../stock-items/types";
+import {
+  SaveStockOperation,
+  SaveStockOperationAction,
+} from "../../stock-items/types";
 import { StockOperationType } from "../../core/api/types/stockOperation/StockOperationType";
 import { InitializeResult } from "./types";
 import {
@@ -31,9 +34,9 @@ interface StockOperationSubmissionProps {
   actions: {
     onGoBack: () => void;
     onSave?: SaveStockOperation;
-    onComplete: () => void;
-    onSubmit: () => void;
-    onDispatch: () => void;
+    onComplete: SaveStockOperationAction;
+    onSubmit: SaveStockOperationAction;
+    onDispatch: SaveStockOperationAction;
   };
 }
 
@@ -98,6 +101,7 @@ const StockOperationSubmission: React.FC<StockOperationSubmissionProps> = ({
                 <Button
                   name="complete"
                   type="button"
+                  style={{ margin: "4px" }}
                   className="submitButton"
                   kind="primary"
                   onClick={actions.onComplete}
@@ -110,6 +114,7 @@ const StockOperationSubmission: React.FC<StockOperationSubmissionProps> = ({
                 <Button
                   name="dispatch"
                   type="button"
+                  style={{ margin: "4px" }}
                   className="submitButton"
                   kind="primary"
                   onClick={actions.onDispatch}
@@ -122,12 +127,13 @@ const StockOperationSubmission: React.FC<StockOperationSubmissionProps> = ({
                 <Button
                   name="submit"
                   type="button"
+                  style={{ margin: "4px" }}
                   className="submitButton"
                   kind="primary"
                   onClick={actions.onSubmit}
                   renderIcon={SendFilled}
                 >
-                  {t("submit", "Submit")}
+                  {t("submit", "Submit For Review")}
                 </Button>
               )}
             </>
@@ -136,6 +142,7 @@ const StockOperationSubmission: React.FC<StockOperationSubmissionProps> = ({
             name="save"
             type="button"
             className="submitButton"
+            style={{ margin: "4px" }}
             disabled={isSaving}
             onClick={async () => {
               setIsSaving(true);
@@ -150,6 +157,7 @@ const StockOperationSubmission: React.FC<StockOperationSubmissionProps> = ({
           {!isSaving && (
             <Button
               type="button"
+              style={{ margin: "4px" }}
               className="cancelButton"
               kind="tertiary"
               onClick={actions.onGoBack}
diff --git a/src/stock-operations/stock-operation.utils.tsx b/src/stock-operations/stock-operation.utils.tsx
index 8d9f11e0..8cc2d4f6 100644
--- a/src/stock-operations/stock-operation.utils.tsx
+++ b/src/stock-operations/stock-operation.utils.tsx
@@ -2,6 +2,7 @@ import { closeOverlay, launchOverlay } from "../core/components/overlay/hook";
 import React from "react";
 import {
   FetchResponse,
+  showModal,
   showNotification,
   showToast,
 } from "@openmrs/esm-framework";
@@ -90,3 +91,16 @@ export function getStockOperationUniqueId() {
     .toString(36)
     .substring(2, 16)}`;
 }
+
+export const showActionDialogButton = async (
+  title: string,
+  requireReason: boolean,
+  operation: StockOperationDTO
+) => {
+  const dispose = showModal("stock-operation-dialog", {
+    title: title,
+    operation: operation,
+    requireReason: requireReason,
+    closeModal: () => dispose(),
+  });
+};
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-approve-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-approve-button.component.tsx
index ea6ed246..53bd75f0 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-approve-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-approve-button.component.tsx
@@ -18,6 +18,7 @@ const StockOperationApprovalButton: React.FC<
     const dispose = showModal("stock-operation-dialog", {
       title: "Approve",
       operation: operation,
+      requireReason: false,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-approve-dispatch-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-approve-dispatch-button.component.tsx
index cd45b92a..bb624eb3 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-approve-dispatch-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-approve-dispatch-button.component.tsx
@@ -18,6 +18,7 @@ const StockOperationApproveDispatchButton: React.FC<
     const dispose = showModal("stock-operation-dialog", {
       title: "Approve Dispatch",
       operation: operation,
+      requireReason: true,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-cancel-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-cancel-button.component.tsx
index b2315f6c..f6b9847b 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-cancel-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-cancel-button.component.tsx
@@ -18,6 +18,7 @@ const StockOperationCancelButton: React.FC<StockOperationCancelButtonProps> = ({
     const dispose = showModal("stock-operation-dialog", {
       title: "Cancel",
       operation: operation,
+      requireReason: true,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-complete-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-complete-button.component.tsx
index bc684292..07e72ef0 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-complete-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-complete-button.component.tsx
@@ -18,6 +18,7 @@ const StockOperationCompleteButton: React.FC<
     const dispose = showModal("stock-operation-dialog", {
       title: "Complete",
       operation: operation,
+      requireReason: true,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-completed-dispatch-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-completed-dispatch-button.component.tsx
index b1df0f0a..52e79ff2 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-completed-dispatch-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-completed-dispatch-button.component.tsx
@@ -18,6 +18,7 @@ const StockOperationCompleteDispatchButton: React.FC<
     const dispose = showModal("stock-operation-dialog", {
       title: "Complete Dispatch",
       operation: operation,
+      requireReason: false,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-dialog.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-dialog.component.tsx
index 2b188646..b3390b05 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-dialog.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-dialog.component.tsx
@@ -20,16 +20,18 @@ import { closeOverlay } from "../../core/components/overlay/hook";
 
 interface StockOperationDialogProps {
   title: string;
+  requireReason: boolean;
   operation: StockOperationDTO;
   closeModal: () => void;
 }
 
 const StockOperationDialog: React.FC<StockOperationDialogProps> = ({
   title,
+  requireReason,
   operation,
   closeModal,
 }) => {
-  const confirmType = title.toLocaleLowerCase();
+  const confirmType = title.toLocaleLowerCase().trim();
 
   const { t } = useTranslation();
 
@@ -45,6 +47,18 @@ const StockOperationDialog: React.FC<StockOperationDialogProps> = ({
     let actionName: StopOperationActionType | null = null;
 
     switch (confirmType) {
+      case "submit":
+        actionName = "SUBMIT";
+        break;
+      case "dispatch":
+        actionName = "DISPATCH";
+        break;
+      case "complete":
+        actionName = "COMPLETE";
+        break;
+      case "completedispatch":
+        actionName = "COMPLETE";
+        break;
       case "cancel":
         actionName = "CANCEL";
         break;
@@ -68,7 +82,7 @@ const StockOperationDialog: React.FC<StockOperationDialogProps> = ({
 
     const payload: StopOperationAction = {
       name: actionName,
-      uuid: operation.uuid,
+      uuid: operation?.uuid,
       reason: notes,
     };
 
@@ -117,7 +131,7 @@ const StockOperationDialog: React.FC<StockOperationDialogProps> = ({
               </h5>
             </section>
             <br />
-            {title !== "Approve" && (
+            {requireReason && (
               <section className={styles.section}>
                 <TextArea
                   labelText={t("notes", "Please explain the reason:")}
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-issue-stock-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-issue-stock-button.component.tsx
index 3a88085b..6dcfa5a8 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-issue-stock-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-issue-stock-button.component.tsx
@@ -18,6 +18,7 @@ const StockOperationIssueStockButton: React.FC<
     const dispose = showModal("stock-operation-dialog", {
       title: "Issue Stock",
       operation: operation,
+      requireReason: true,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-reject-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-reject-button.component.tsx
index 04bdaab4..52d2aa00 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-reject-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-reject-button.component.tsx
@@ -24,6 +24,7 @@ const StockOperationRejectButton: React.FC<StockOperationRejectButtonProps> = ({
     const dispose = showModal("stock-operation-dialog", {
       title: "Reject",
       operation: operation,
+      requireReason: true,
       closeModal: () => dispose(),
     });
   }, [operation]);
diff --git a/src/stock-operations/stock-operations-dialog/stock-operations-return-button.component.tsx b/src/stock-operations/stock-operations-dialog/stock-operations-return-button.component.tsx
index 1a9aa605..5925a2d8 100644
--- a/src/stock-operations/stock-operations-dialog/stock-operations-return-button.component.tsx
+++ b/src/stock-operations/stock-operations-dialog/stock-operations-return-button.component.tsx
@@ -24,6 +24,7 @@ const StockOperationReturnButton: React.FC<StockOperationReturnButtonProps> = ({
     const dispose = showModal("stock-operation-dialog", {
       title: "Return",
       operation: operation,
+      requireReason: true,
       closeModal: () => dispose(),
     });
   }, [operation]);