(fix) O3-5200 Add validation so that once a bill has been "posted" its line items cannot be modified

src/bill-item-actions/edit-bill-item.modal.tsx

@@ -88,6 +88,14 @@ const EditBillLineItemModal: React.FC<EditBillLineItemModalProps> = ({ bill, clo
   }, [quantity, price]);
 
   const onSubmit = async (data: BillLineItemForm) => {
+    if (bill?.status === 'PENDING') {
+      showSnackbar({
+        title: t('cannotEditThisBill', 'You can not edit this bill'),
+        subtitle: t('onlyPendingBillsCanBeEdited', 'Only pending bills can be edited'),
+        kind: 'error',
+      });
+      return;
+    }
     const url = `${apiBasePath}bill`;
 
     const newItem = {

src/billing.resource.ts

@@ -21,9 +21,11 @@ import type {
 
 export const mapBillProperties = (bill: PatientInvoice): MappedBill => {
   const activeLineItems = bill?.lineItems?.filter((item) => !item.voided) || [];
+  const isSpecialStatus = bill.status === 'POSTED' || bill.status === 'PAID' || bill.status === 'ADJUSTED';
 
-  const status =
-    activeLineItems.length > 0
+  const status = isSpecialStatus
+    ? bill.status
+    : activeLineItems.length > 0
       ? activeLineItems.some((item) => item.paymentStatus === 'PENDING')
         ? 'PENDING'
         : 'PAID'

src/invoice/invoice-table.component.tsx

@@ -85,6 +85,7 @@ const InvoiceTable: React.FC<InvoiceTableProps> = ({ bill, isLoadingBill }) => {
   const tableRows: Array<typeof DataTableRow> = useMemo(
     () =>
       filteredLineItems?.map((item, index) => {
+        const canEditBill = bill?.status === 'PENDING';
         return {
           no: `${index + 1}`,
           id: `${item.uuid}`,
@@ -96,20 +97,32 @@ const InvoiceTable: React.FC<InvoiceTableProps> = ({ bill, isLoadingBill }) => {
           total: convertToCurrency(item.price * item.quantity, defaultCurrency),
           actionButton: (
             <span>
-              <Button
-                data-testid={`edit-button-${item.uuid}`}
-                renderIcon={Edit}
-                hasIconOnly
-                kind="ghost"
-                iconDescription={t('editThisBillItem', 'Edit this bill item')}
-                tooltipPosition="left"
-                onClick={() => handleSelectBillItem(item)}
-              />
+              {canEditBill ? (
+                <Button
+                  data-testid={`edit-button-${item.uuid}`}
+                  renderIcon={Edit}
+                  hasIconOnly
+                  kind="ghost"
+                  iconDescription={t('editThisBillItem', 'Edit this bill item')}
+                  tooltipPosition="left"
+                  onClick={() => handleSelectBillItem(item)}
+                />
+              ) : (
+                <Button
+                  data-tested={`edit-button-${item.uuid}`}
+                  renderIcon={Edit}
+                  hasIconOnly
+                  kind="ghost"
+                  disabled
+                  iconDescription={t('cannotEditThisBill', 'You can not edit this bill')}
+                  tooltipPosition="left"
+                />
+              )}
             </span>
           ),
         };
       }) ?? [],
-    [filteredLineItems, bill?.receiptNumber, defaultCurrency, t, handleSelectBillItem],
+    [filteredLineItems, bill?.receiptNumber, defaultCurrency, t, handleSelectBillItem, bill?.status],
   );
 
   if (isLoadingBill) {

translations/en.json

@@ -226,5 +226,7 @@
   "uuidRequired": "UUID is required",
   "validationError": "Validation error",
   "visitTime": "Visit time",
-  "waiverForm": "Waiver form"
+  "waiverForm": "Waiver form",
+  "cannotEditThisBill" : "You can not edit this bill",
+  "onlyPendingBillsCanBeEdited": "Only PENDING bills can be edited"
 }