From c5ddee59337a3b8b2b28b7a00e5ca5d4d3c70f03 Mon Sep 17 00:00:00 2001
From: sb-cecilialiu <cecilia.liu@icf.com>
Date: Thu, 7 Nov 2024 06:53:23 -0600
Subject: [PATCH] MAT7698 edit parameter

---
 src/AceEditor/madie-ace-editor.tsx            |  4 +++
 src/CqlBuilderPanel/CqlBuilderPanel.tsx       |  2 ++
 .../Parameters/EditParameterDialog.tsx        |  4 ++-
 .../Parameters/ParameterBuilder.tsx           | 35 ++++---------------
 .../CqlEditorWithTerminology.tsx              |  2 ++
 5 files changed, 18 insertions(+), 29 deletions(-)

diff --git a/src/AceEditor/madie-ace-editor.tsx b/src/AceEditor/madie-ace-editor.tsx
index a81a9078..4b756e9d 100644
--- a/src/AceEditor/madie-ace-editor.tsx
+++ b/src/AceEditor/madie-ace-editor.tsx
@@ -31,6 +31,10 @@ export interface EditorPropsType {
   onChange?: (value: string) => void;
   handleApplyCode?: (code: string) => void;
   handleApplyParameter?: (parameter: Parameter) => void;
+  handleParameterEdit?: (
+    parameter: Parameter,
+    parameterToApply: Parameter
+  ) => void;
   handleApplyValueSet?: (vs: ValueSetForSearch) => void;
   handleApplyDefinition?: (def: Definition) => void;
   handleDefinitionEdit?: (lib: SelectedLibrary, def: Definition) => void;
diff --git a/src/CqlBuilderPanel/CqlBuilderPanel.tsx b/src/CqlBuilderPanel/CqlBuilderPanel.tsx
index f6ea9c7a..63d864d9 100644
--- a/src/CqlBuilderPanel/CqlBuilderPanel.tsx
+++ b/src/CqlBuilderPanel/CqlBuilderPanel.tsx
@@ -28,6 +28,7 @@ export default function CqlBuilderPanel({
   handleEditLibrary,
   handleApplyCode,
   handleApplyParameter,
+  handleParameterEdit,
   handleApplyValueSet,
   handleApplyDefinition,
   handleDefinitionEdit,
@@ -211,6 +212,7 @@ export default function CqlBuilderPanel({
           <Parameters
             canEdit={canEdit}
             handleApplyParameter={handleApplyParameter}
+            handleParameterEdit={handleParameterEdit}
             cqlBuilderLookupsTypes={cqlBuilderLookupsTypes}
             isCQLUnchanged={isCQLUnchanged}
             cql={measureStoreCql}
diff --git a/src/CqlBuilderPanel/Parameters/EditParameterDialog.tsx b/src/CqlBuilderPanel/Parameters/EditParameterDialog.tsx
index c959aad0..62041835 100644
--- a/src/CqlBuilderPanel/Parameters/EditParameterDialog.tsx
+++ b/src/CqlBuilderPanel/Parameters/EditParameterDialog.tsx
@@ -20,7 +20,9 @@ const EditParameterDialog = ({
 }: PropTypes) => {
   const updatedParameter = {
     parameterName: parameter?.name,
-    expressionValue: parameter?.logic,
+    expression:
+      parameter?.logic?.slice(0, 1).toUpperCase() +
+      parameter?.logic?.slice(1, parameter?.logic?.length),
   } as Parameter;
 
   return (
diff --git a/src/CqlBuilderPanel/Parameters/ParameterBuilder.tsx b/src/CqlBuilderPanel/Parameters/ParameterBuilder.tsx
index 749fe37a..0a28e228 100644
--- a/src/CqlBuilderPanel/Parameters/ParameterBuilder.tsx
+++ b/src/CqlBuilderPanel/Parameters/ParameterBuilder.tsx
@@ -9,7 +9,7 @@ import AceEditor from "react-ace";
 
 export interface Parameter {
   parameterName?: string;
-  expressionValue?: string;
+  expression?: string;
 }
 
 export interface ParameterProps {
@@ -27,16 +27,13 @@ export default function ParameterBuilder({
   parameter,
   setOpenParameterDialog,
 }: ParameterProps) {
-  const [editorHeight, setEditorHeight] = useState("50px");
+  const [editorHeight, setEditorHeight] = useState("180px");
   const textAreaRef = useRef(null);
-  const [expressionEditorValue, setExpressionEditorValue] = useState(
-    parameter?.expressionValue || ""
-  );
 
   const formik = useFormik({
     initialValues: {
       parameterName: parameter?.parameterName || "",
-      expression: parameter?.expressionValue || "",
+      expression: parameter?.expression || "",
     },
     validationSchema: ParameterSchemaValidator,
     enableReinitialize: true,
@@ -44,13 +41,6 @@ export default function ParameterBuilder({
   });
   const { resetForm } = formik;
 
-  const isEditDialogFormDirty = () => {
-    if (parameter?.expressionValue !== expressionEditorValue || formik.dirty) {
-      return false;
-    }
-    return true;
-  };
-
   return (
     <div>
       <form id="parameter-form" onSubmit={formik.handleSubmit}>
@@ -103,12 +93,7 @@ export default function ParameterBuilder({
             <Button
               variant="outline"
               data-testid="parameter-cancel-btn"
-              disabled={
-                (!formik.dirty &&
-                  expressionEditorValue ===
-                    (parameter?.expressionValue || "")) ||
-                !canEdit
-              }
+              disabled={!formik.dirty || !canEdit}
               tw="mr-4"
               onClick={() => {
                 resetForm();
@@ -119,21 +104,15 @@ export default function ParameterBuilder({
             </Button>
             <Button
               data-testid={`parameter-save-btn`}
-              disabled={
-                (!formik.dirty &&
-                  expressionEditorValue ===
-                    (parameter?.expressionValue || "")) ||
-                !canEdit
-              }
+              disabled={!formik.dirty || !canEdit}
               onClick={() => {
                 const parameterToApply: Parameter = {
                   parameterName: formik.values.parameterName,
-                  expressionValue: expressionEditorValue,
+                  expression: formik.values.expression,
                 };
                 resetForm();
-                setExpressionEditorValue("");
-                formik.setFieldValue("parameterName", "");
                 // call handleParameterEdit
+                handleParameterEdit(parameter, parameterToApply);
                 onClose();
               }}
             >
diff --git a/src/cqlEditorWithTerminology/CqlEditorWithTerminology.tsx b/src/cqlEditorWithTerminology/CqlEditorWithTerminology.tsx
index 73d883de..676bf9e6 100644
--- a/src/cqlEditorWithTerminology/CqlEditorWithTerminology.tsx
+++ b/src/cqlEditorWithTerminology/CqlEditorWithTerminology.tsx
@@ -15,6 +15,7 @@ const CqlEditorWithTerminology = ({
   handleDefinitionDelete,
   handleApplyCode,
   handleApplyParameter,
+  handleParameterEdit,
   handleApplyValueSet,
   handleApplyDefinition,
   handleApplyLibrary,
@@ -108,6 +109,7 @@ const CqlEditorWithTerminology = ({
               isCQLUnchanged={isCQLUnchanged}
               handleApplyCode={handleApplyCode}
               handleApplyParameter={handleApplyParameter}
+              handleParameterEdit={handleParameterEdit}
               handleApplyValueSet={handleApplyValueSet}
               handleApplyDefinition={handleApplyDefinition}
               handleDefinitionEdit={handleDefinitionEdit}