fix some bugs

cope2n-fe/src/pages/reviews2/index.tsx

@@ -126,11 +126,11 @@ const ReviewPage = () => {
     }
   }, []);
 
-  useEffect(() => {
-    if (reason) {
-      setSolution(counter_measure_map[reason]);
-    }
-  }, [reason]);
+  // useEffect(() => {
+  //   if (reason) {
+  //     setSolution(counter_measure_map[reason]);
+  //   }
+  // }, [reason]);
 
   const updateSelectedFileDataSource = (fileContent) => {
     let tempData = {};
@@ -144,13 +144,14 @@ const ReviewPage = () => {
         if (isEmpty(tempData[k][REVIEWED_RESULT])) {
           isEmptyResult = true;
         }
-        if (tempData[k][REVIEWED_RESULT].length > 0) {
+        if (Array.isArray(tempData[k][REVIEWED_RESULT]) && tempData[k][REVIEWED_RESULT].length > 0) {
           isEmptyResult = tempData[k][REVIEWED_RESULT].every((r: any) =>
             isEmpty(r),
           );
         }
         if (isEmptyResult) {
           tempData[k][REVIEWED_RESULT] = tempData[k][PREDICTED_RESULT];
+         
         }
       }
     });
@@ -174,16 +175,20 @@ const ReviewPage = () => {
 
     if (!reason) {
       setReason(null);
+      setOtherReason(null);
     } else if (REASON_BAD_QUALITY.some((r) => r.value === reason)) {
       setReason(reason);
+      setOtherReason(null);
     } else {
       setReason('other');
       setOtherReason(reason);
     }
     if (!solution) {
       setSolution(null);
+      setOtherSolution(null);
     } else if (SOLUTION_BAD_QUALITY.some((r) => r.value === solution)) {
       setSolution(solution);
+      setOtherSolution(null);
     } else {
       setSolution('other');
       setOtherSolution(solution);
@@ -198,7 +203,6 @@ const ReviewPage = () => {
       setImageLoading(false);
     }
   };
-
   const loadCurrentRequest = (requestIndex) => {
     setLoading(true);
     setImageLoading(true);
@@ -306,11 +310,18 @@ const ReviewPage = () => {
     let request_file_result = {};
     SOURCE_KEYS.forEach((k) => {
       request_file_result[k] = selectedFileDataSource[k][REVIEWED_RESULT];
+      if(Array.isArray(selectedFileDataSource[k][REVIEWED_RESULT])){
+        request_file_result[k] = selectedFileDataSource[k][REVIEWED_RESULT].toString();
+      }else{
+        request_file_result[k] = selectedFileDataSource[k][REVIEWED_RESULT];
+      }
+
+
     });
     let data = {
       request_file_result,
-      reason: reason ? reason : otherReason,
-      solution: solution ? solution : otherSolution,
+      reason: reason !== 'other' ? reason : otherReason,
+      solution: solution !== 'other' ? solution : otherSolution,
     };
     try {
       await updateRevisedDataByFile(currentRequest?.RequestID, fileId, data);
@@ -344,10 +355,10 @@ const ReviewPage = () => {
 
   const [lightBox, setLightBox] = useState(false);
 
-  const updateRevised = (fieldName) => {
+  const updateRevisedByFeedback = (fieldName) => {
     setSelectedFileDataSource((prevData) => {
       prevData[fieldName][REVIEWED_RESULT] =
-        prevData[fieldName][FEEDBACK_RESULT];
+      prevData[fieldName][FEEDBACK_RESULT];
       return {
         ...prevData,
       };
@@ -675,7 +686,7 @@ const ReviewPage = () => {
               style={{ minWidth: '120px' }}
               onClick={handleConfirmReview}
             >
-              (C) Confirm request
+            Confirm request
             </Button>
           </div>
         </div>
@@ -746,7 +757,7 @@ const ReviewPage = () => {
                       ghost
                       icon={<CopyOutlined />}
                       size='small'
-                      onClick={() => updateRevised(data)}
+                      onClick={() => updateRevisedByFeedback(data)}
                     />
                   </div>
                   <Input
@@ -785,7 +796,11 @@ const ReviewPage = () => {
                 placeholder='Select a reason'
                 style={{ width: 170, flexBasis: '50%', height: '32px' }}
                 options={REASON_BAD_QUALITY}
-                onChange={setReason}
+                onChange={(value) => {
+                  let newReason = value;
+                  setReason(newReason);
+                  setSolution(counter_measure_map[newReason]);
+                }}
                 value={reason}
               />
               {reason === 'other' && (
@@ -837,7 +852,7 @@ const ReviewPage = () => {
               }}
               onClick={submitRevisedData}
             >
-              (U) Update File
+              Update File
             </Button>
           </div>
         </div>