Anomaly RM#89736: Batch accounting Cutoff does not link anomaly

axelor · Feb 14, 2025 · ca28bcb · ca28bcb
1 parent 6eeec00
commit ca28bcb
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 21 deletions.
diff --git a/...or-account/src/main/java/com/axelor/apps/account/service/batch/BatchAccountingCutOff.java b/...or-account/src/main/java/com/axelor/apps/account/service/batch/BatchAccountingCutOff.java
@@ -60,13 +60,18 @@ public BatchAccountingCutOff(
 
   @Override
   protected void process() {
-    AccountingBatch accountingBatch = batch.getAccountingBatch();
+    try {
+      AccountingBatch accountingBatch = batch.getAccountingBatch();
 
-    LocalDate moveDate = accountingBatch.getMoveDate();
-    int accountingCutOffTypeSelect = accountingBatch.getAccountingCutOffTypeSelect();
-    this.updateBatch(moveDate, accountingCutOffTypeSelect);
+      LocalDate moveDate = accountingBatch.getMoveDate();
+      int accountingCutOffTypeSelect = accountingBatch.getAccountingCutOffTypeSelect();
+      this.updateBatch(moveDate, accountingCutOffTypeSelect);
 
-    super.process();
+      super.process();
+    } catch (Exception e) {
+      TraceBackService.trace(e, null, batch.getId());
+      incrementAnomaly();
+    }
   }
 
   @Override

diff --git a/...main/java/com/axelor/apps/supplychain/service/batch/BatchAccountingCutOffSupplyChain.java b/...main/java/com/axelor/apps/supplychain/service/batch/BatchAccountingCutOffSupplyChain.java
@@ -73,25 +73,30 @@ public BatchAccountingCutOffSupplyChain(
 
   @Override
   protected void process() {
-    AccountingBatch accountingBatch = batch.getAccountingBatch();
-
-    LocalDate moveDate = accountingBatch.getMoveDate();
-    int accountingCutOffTypeSelect = accountingBatch.getAccountingCutOffTypeSelect();
-    updateBatch(moveDate, accountingCutOffTypeSelect);
+    try {
+      AccountingBatch accountingBatch = batch.getAccountingBatch();
 
-    if (accountingCutOffTypeSelect
-        < AccountingBatchRepository.ACCOUNTING_CUT_OFF_TYPE_PREPAID_EXPENSES) {
-      if (this.recordIdList == null) {
-        this._processStockMovesByQuery(accountingBatch);
-      } else {
-        this._processStockMovesByIds(accountingBatch);
-      }
-    } else {
-      if (this.recordIdList == null) {
-        this._processByQuery(accountingBatch);
+      LocalDate moveDate = accountingBatch.getMoveDate();
+      int accountingCutOffTypeSelect = accountingBatch.getAccountingCutOffTypeSelect();
+      updateBatch(moveDate, accountingCutOffTypeSelect);
+
+      if (accountingCutOffTypeSelect
+          < AccountingBatchRepository.ACCOUNTING_CUT_OFF_TYPE_PREPAID_EXPENSES) {
+        if (this.recordIdList == null) {
+          this._processStockMovesByQuery(accountingBatch);
+        } else {
+          this._processStockMovesByIds(accountingBatch);
+        }
       } else {
-        this._processByIds(accountingBatch);
+        if (this.recordIdList == null) {
+          this._processByQuery(accountingBatch);
+        } else {
+          this._processByIds(accountingBatch);
+        }
       }
+    } catch (Exception e) {
+      TraceBackService.trace(e, null, batch.getId());
+      incrementAnomaly();
     }
   }
 

diff --git a/changelogs/unreleased/89736.yml b/changelogs/unreleased/89736.yml
@@ -0,0 +1,3 @@
+---
+title: "Accounting batch: fixed the issue where batch accounting cut-off does not link anomaly."
+module: axelor-account