From e8965658b113c2497b74f8b636c9d6ce0a182165 Mon Sep 17 00:00:00 2001
From: admin <peancharoen.pslcp3@gmail.com>
Date: Sun, 29 Mar 2026 11:14:06 +0700
Subject: [PATCH] 690329:1114 Fixing bugs uuid by GPT-5.3 #03

---
 frontend/components/correspondences/form.tsx | 22 ++++++---
 frontend/components/rfas/form.tsx            | 51 +++++++++++++++-----
 2 files changed, 53 insertions(+), 20 deletions(-)

diff --git a/frontend/components/correspondences/form.tsx b/frontend/components/correspondences/form.tsx
index 1191a08..fdb37ac 100644
--- a/frontend/components/correspondences/form.tsx
+++ b/frontend/components/correspondences/form.tsx
@@ -367,13 +367,21 @@ export function CorrespondenceForm({ initialData, uuid }: { initialData?: Initia
               <SelectValue placeholder={isLoadingContracts ? 'Loading...' : 'Select Contract'} />
             </SelectTrigger>
             <SelectContent>
-              {contracts.map((c) => (
-                <SelectItem key={c.publicId} value={c.publicId || ''}>
-                  {c.contractName || c.contractCode}
-                </SelectItem>
-              ))}
+              {contracts.map((c) => {
+                const contractValue = c.publicId;
+
+                if (!contractValue) {
+                  return null;
+                }
+
+                return (
+                  <SelectItem key={contractValue} value={contractValue}>
+                    {c.contractName || c.contractCode}
+                  </SelectItem>
+                );
+              })}
               {!isLoadingContracts && contracts.length === 0 && (
-                <SelectItem value="" disabled>
+                <SelectItem value="__no_contract_available__" disabled>
                   No contracts found for this project
                 </SelectItem>
               )}
@@ -422,7 +430,7 @@ export function CorrespondenceForm({ initialData, uuid }: { initialData?: Initia
                 </SelectItem>
               ))}
               {!isLoadingDisciplines && disciplines.length === 0 && (
-                <SelectItem value="" disabled>
+                <SelectItem value="__no_discipline_available__" disabled>
                   No disciplines found for this contract
                 </SelectItem>
               )}
diff --git a/frontend/components/rfas/form.tsx b/frontend/components/rfas/form.tsx
index 1a544f1..8c854fc 100644
--- a/frontend/components/rfas/form.tsx
+++ b/frontend/components/rfas/form.tsx
@@ -53,7 +53,7 @@ type ContractOption = {
 };
 
 type DisciplineOption = {
-  publicId: string; // ADR-019: public identifier
+  publicId?: string; // ADR-019: public identifier
   id?: number; // Internal INT
   disciplineCode: string;
   codeNameEn?: string;
@@ -61,7 +61,7 @@ type DisciplineOption = {
 };
 
 type RfaTypeOption = {
-  publicId: string; // ADR-019: public identifier
+  publicId?: string; // ADR-019: public identifier
   id?: number; // Internal INT
   typeCode?: string;
   typeName?: string;
@@ -137,6 +137,10 @@ const getOptionValue = (value?: string | number): string | undefined => {
   return String(value);
 };
 
+const getMasterOptionValue = (option: { publicId?: string; id?: number }): string | undefined => {
+  return getOptionValue(option.publicId ?? option.id);
+};
+
 export function RFAForm() {
   const router = useRouter();
   const createMutation = useCreateRFA();
@@ -187,9 +191,12 @@ export function RFAForm() {
 
   const selectedContractId = watch('contractId');
   const { data: disciplinesData, isLoading: isLoadingDisciplines } = useDisciplines(selectedContractId);
-  const disciplines = dedupeByKey(extractArrayData<DisciplineOption>(disciplinesData), (discipline) => discipline.publicId);
+  const disciplines = dedupeByKey(
+    extractArrayData<DisciplineOption>(disciplinesData),
+    (discipline) => getMasterOptionValue(discipline)
+  );
   const { data: rfaTypesData, isLoading: isLoadingRfaTypes } = useRfaTypes(selectedContractId);
-  const rfaTypes = dedupeByKey(extractArrayData<RfaTypeOption>(rfaTypesData), (rfaType) => rfaType.publicId);
+  const rfaTypes = dedupeByKey(extractArrayData<RfaTypeOption>(rfaTypesData), (rfaType) => getMasterOptionValue(rfaType));
   const [shopDrawingSearch, setShopDrawingSearch] = useState('');
   const [shopDrawingPage, setShopDrawingPage] = useState(1);
   const { data: shopDrawingsData, isLoading: isLoadingShopDrawings } = useDrawings('SHOP', {
@@ -222,7 +229,7 @@ export function RFAForm() {
   const toOrganizationId = watch('toOrganizationId');
   const selectedShopDrawingRevisionIds = watch('shopDrawingRevisionIds') ?? [];
   const selectedAsBuiltDrawingRevisionIds = watch('asBuiltDrawingRevisionIds') ?? [];
-  const selectedRfaType = rfaTypes.find((rfaType) => rfaType.publicId === rfaTypeId);
+  const selectedRfaType = rfaTypes.find((rfaType) => getMasterOptionValue(rfaType) === rfaTypeId);
   const selectedRfaTypeCode = selectedRfaType?.typeCode?.toUpperCase();
   const requiresShopDrawings = selectedRfaTypeCode === 'DDW' || selectedRfaTypeCode === 'SDW';
   const requiresAsBuiltDrawings = selectedRfaTypeCode === 'ADW';
@@ -443,9 +450,19 @@ export function RFAForm() {
                 </SelectTrigger>
                 <SelectContent>
                   {disciplines.map((d) => (
-                    <SelectItem key={d.publicId} value={String(d.publicId)}>
-                      {`${d.codeNameEn || d.codeNameTh || d.disciplineCode} (${d.disciplineCode})`}
-                    </SelectItem>
+                    (() => {
+                      const disciplineValue = getMasterOptionValue(d);
+
+                      if (!disciplineValue) {
+                        return null;
+                      }
+
+                      return (
+                        <SelectItem key={disciplineValue} value={disciplineValue}>
+                          {`${d.codeNameEn || d.codeNameTh || d.disciplineCode} (${d.disciplineCode})`}
+                        </SelectItem>
+                      );
+                    })()
                   ))}
                   {!isLoadingDisciplines && disciplines.length === 0 && (
                     <SelectItem value="0" disabled>
@@ -474,11 +491,19 @@ export function RFAForm() {
                   <SelectValue placeholder={isLoadingRfaTypes ? 'Loading...' : 'Select RFA Type'} />
                 </SelectTrigger>
                 <SelectContent>
-                  {rfaTypes.map((rfaType) => (
-                    <SelectItem key={rfaType.publicId} value={String(rfaType.publicId)}>
-                      {`${rfaType.typeCode || 'RFA'} - ${rfaType.typeName || rfaType.typeNameEn || rfaType.typeNameTh || 'Unnamed Type'}`}
-                    </SelectItem>
-                  ))}
+                  {rfaTypes.map((rfaType) => {
+                    const rfaTypeValue = getMasterOptionValue(rfaType);
+
+                    if (!rfaTypeValue) {
+                      return null;
+                    }
+
+                    return (
+                      <SelectItem key={rfaTypeValue} value={rfaTypeValue}>
+                        {`${rfaType.typeCode || 'RFA'} - ${rfaType.typeName || rfaType.typeNameEn || rfaType.typeNameTh || 'Unnamed Type'}`}
+                      </SelectItem>
+                    );
+                  })}
                 </SelectContent>
               </Select>
               {errors.rfaTypeId && <p className="text-sm text-destructive mt-1">{errors.rfaTypeId.message}</p>}