Merge pull request #646 from NUTFes/develop

購入報告すり抜けバグ対応リリース
NUTFes · Sep 29, 2023 · 834e477 · 834e477
2 parents 298251b + c372a96
commit 834e477
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 14 deletions.
diff --git a/view/next-project/src/components/purchasereports/DetailModal.tsx b/view/next-project/src/components/purchasereports/DetailModal.tsx
@@ -66,10 +66,12 @@ const DetailModal: FC<ModalProps> = (props) => {
   // 購入報告の合計金額を計算
   const TotalFee = (purchaseReport: PurchaseReport, purchaseItems: PurchaseItem[]) => {
     let totalFee = 0;
-    purchaseItems.map((purchaseItem: PurchaseItem) => {
-      totalFee += purchaseItem.price * purchaseItem.quantity;
-    });
-    totalFee += purchaseReport.addition - purchaseReport.discount;
+    if (purchaseItems) {
+      purchaseItems.map((purchaseItem: PurchaseItem) => {
+        totalFee += purchaseItem.price * purchaseItem.quantity;
+      });
+      totalFee += purchaseReport.addition - purchaseReport.discount;
+    }
     return totalFee;
   };
 
@@ -161,7 +163,7 @@ const DetailModal: FC<ModalProps> = (props) => {
             </thead>
             <tbody className='w-full border border-x-white-0 border-b-primary-1 border-t-white-0'>
               {/* <div className='flex items-start'> */}
-              {props.purchaseReportViewItem?.purchaseItems.map((purchaseItem) => (
+              {props.purchaseReportViewItem?.purchaseItems?.map((purchaseItem) => (
                 <tr key={purchaseItem.id} className='w-full'>
                   <td className='border-b py-3'>
                     <div className='text-center text-sm text-black-300'>{purchaseItem.item}</div>

diff --git a/view/next-project/src/pages/purchasereports/index.tsx b/view/next-project/src/pages/purchasereports/index.tsx
@@ -93,21 +93,24 @@ export default function PurchaseReports(props: Props) {
 
   const TotalFee = useCallback((purchaseReport: PurchaseReport, purchaseItems: PurchaseItem[]) => {
     let totalFee = 0;
-    purchaseItems.map((purchaseItem: PurchaseItem) => {
-      if (purchaseItem.financeCheck) {
-        totalFee += purchaseItem.price * purchaseItem.quantity;
-      }
-    });
-    totalFee += purchaseReport.addition - purchaseReport.discount;
-    return totalFee;
+    if (purchaseItems) {
+      purchaseItems.map((purchaseItem: PurchaseItem) => {
+        if (purchaseItem.financeCheck) {
+          totalFee += purchaseItem.price * purchaseItem.quantity;
+        }
+      });
+      totalFee += purchaseReport.addition - purchaseReport.discount;
+      return totalFee;
+    }
   }, []);
 
   // すべてのpurchaseReportの合計金額
   const totalReportFee = useMemo(() => {
     if (filteredPurchaseReportViews) {
       let totalFee = 0;
       filteredPurchaseReportViews.map((purchaseReportView: PurchaseReportView) => {
-        totalFee += TotalFee(purchaseReportView.purchaseReport, purchaseReportView.purchaseItems);
+        totalFee +=
+          TotalFee(purchaseReportView.purchaseReport, purchaseReportView.purchaseItems) || 0;
       });
       return totalFee;
     }
@@ -333,7 +336,7 @@ export default function PurchaseReports(props: Props) {
                         )}
                       >
                         {/* name (個数/finasucheck) */}
-                        {purchaseReportViewItem.purchaseItems.map((purchaseItem, index) => (
+                        {purchaseReportViewItem.purchaseItems?.map((purchaseItem, index) => (
                           <div key={index}>
                             {`${purchaseItem.financeCheck ? '○' : 'x'} ${purchaseItem.item} (${
                               purchaseItem.quantity