tin-spa 20.4.3 → 20.4.4

package/fesm2022/tin-spa.mjs

@@ -781,14 +781,14 @@ class DetailsDialogProcessor {
     static loadDetailsFromUrl(detailsConfig, formConfig) {
         let action;
         if (formConfig.loadAction) {
-            if (detailsConfig.heroField && detailsConfig.details) {
-                // console.log("DetailsConfig Hero ID Field " + detailsConfig.heroField);
-                action = { url: `${formConfig.loadAction.url}/${detailsConfig.details[detailsConfig.heroField]}` };
-            }
-            else if (formConfig.heroField && detailsConfig.details) {
+            if (formConfig.heroField && detailsConfig.details) { // Changed: prioritize formConfig.heroField - the form's own identity field takes precedence over parent's heroField
                 // console.log("FormConfig Hero ID Field " + formConfig.heroField);
                 action = { url: `${formConfig.loadAction.url}/${detailsConfig.details[formConfig.heroField]}` };
             }
+            else if (detailsConfig.heroField && detailsConfig.details) { // Changed: fallback to detailsConfig.heroField when formConfig doesn't define one
+                // console.log("DetailsConfig Hero ID Field " + detailsConfig.heroField);
+                action = { url: `${formConfig.loadAction.url}/${detailsConfig.details[detailsConfig.heroField]}` };
+            }
             else if (detailsConfig.heroValue) {
                 // console.log("Defined Hero Value " + detailsConfig.heroValue);
                 action = { url: `${formConfig.loadAction.url}/${detailsConfig.heroValue}` };
@@ -3153,7 +3153,7 @@ class AccountingService {
                 { name: 'unitPrice', type: 'money', alias: 'Unit Price', required: true }, // Changed: Unit price field (backend will populate for Product/Service)
                 { name: 'poNumber', type: 'text', alias: 'PO Number', }, // Added: PO Number for invoice bundling
             ],
-            loadAction: { url: 'invoiceitems/id/{invoiceItemID}' },
+            loadAction: { url: 'invoiceitems/id' }, // Fixed: removed placeholder - TinCore appends ID automatically
             heroField: 'invoiceItemID',
         };
         this.invoiceItemEditButton = { name: 'edit', dialog: true, action: { url: 'invoiceitems?action=edit', method: 'post' }, disabled: x => x.status != InvoiceStatus.Draft };
@@ -3174,7 +3174,7 @@ class AccountingService {
                 { name: 'description', type: 'label', alias: 'Item', readonly: true, span: true },
                 { name: 'invoiceID', type: 'select', alias: 'Move to Invoice', required: true, span: true, loadAction: { url: 'invoiceitems/list/draft-invoices' } }
             ],
-            loadAction: { url: 'invoiceitems/id/{invoiceItemID}' },
+            loadAction: { url: 'invoiceitems/id' }, // Fixed: removed placeholder - TinCore appends ID automatically
             heroField: 'invoiceItemID'
         };
         // Added: Move item details dialog config
@@ -3545,11 +3545,15 @@ class AccountingService {
                         { name: '#81C784', condition: (x) => x.status == 3 },
                         { name: '#E57373', condition: (x) => x.status == 4 }
                     ]
+                },
+                { name: 'errorMessage', type: 'text', alias: 'Error',
+                    colors: [{ name: 'red', condition: (x) => !!x.errorMessage }] // Changed: Show error details in red
                 }
             ],
             buttons: [
                 { name: 'edit', dialog: true, action: { url: 'transactionimport?action=edit', method: 'post' } },
-                { name: 'delete', action: { url: 'transactionimport?action=delete', method: 'post' }, confirm: { message: 'Delete this import item?' }, visible: (x) => x.status != 3 } // Changed: Allow deleting non-Processed items
+                { name: 'delete', action: { url: 'transactionimport?action=delete', method: 'post' }, confirm: { message: 'Delete this import item?' }, visible: (x) => x.status != 3 },
+                { name: 'reverse', display: 'Reverse', icon: { name: 'undo' }, action: { url: 'transactionimport?action=reverse', method: 'post' }, confirm: { message: 'Reverse this transaction?' }, visible: (x) => x.status == 3 } // Changed: Per-row reverse for Processed items
             ],
             loadAction: { url: 'transactionimport/all/x' }, // Changed: Load all user items (no batch filter)
             formConfig: this.transactionImportItemFormConfig
@@ -3587,6 +3591,11 @@ class AccountingService {
                     visible: (x) => (x.resolvedCount ?? 0) > 0,
                     confirm: { message: 'Post all items?' },
                     action: { url: 'transactionimport/process', method: 'post' } },
+                { name: 'undo', display: 'Undo', inDialog: true, keepOpen: true, icon: { name: 'undo' },
+                    visible: (x) => !!x.hasProcessed,
+                    confirm: { message: 'Reverse all posted transactions?' },
+                    action: { url: 'transactionimport/reverse', method: 'post' } // Changed: Batch reversal of all processed items
+                 },
                 { name: 'clear', display: 'Clear', inDialog: true, keepOpen: true, resetMode: 'create', icon: { name: 'clear_all' },
                     visible: (x) => !!x.hasItems,
                     confirm: { message: 'Clear all imported items?' },