Corrección de NPE al exportar txt del Banco del Caribe

adempiere · Oct 20, 2022 · fa81f72 · fa81f72
1 parent 45b79a8
commit fa81f72
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/financial/src/main/java/base/org/erpya/lve/bank/exp/Bancaribe_2022.java b/financial/src/main/java/base/org/erpya/lve/bank/exp/Bancaribe_2022.java
@@ -126,10 +126,13 @@ public int exportToFile(List<MPaySelectionCheck> checks, File file, StringBuffer
 								businessPartnerEmail = "";
 							}
 							//	String Account Type
-							String businessPartnerAccountType = "CTE";
-							if(businessPartnerAccount.getBankAccountType().equals(MBPBankAccount.BANKACCOUNTTYPE_Savings)) {
-								businessPartnerAccountType = "AHO";
-							}
+							AtomicReference<String> businessPartnerAccountType = new AtomicReference<String>("CTE");
+							Optional<String> maybeBusinessPartnerAccountType = Optional.ofNullable(businessPartnerAccount.getBankAccountType());
+							maybeBusinessPartnerAccountType.ifPresent(accountType -> {
+								if (accountType.equals(MBPBankAccount.BANKACCOUNTTYPE_Savings))
+									businessPartnerAccountType.set("AHO");
+							});
+
 							//	Validate
 							if(Optional.ofNullable(businessPartnerAccountNo).isPresent()
 									&& Optional.ofNullable(businessPartnerName).isPresent()) {
@@ -147,7 +150,7 @@ public int exportToFile(List<MPaySelectionCheck> checks, File file, StringBuffer
 									.append(SEPARATOR)							//	Blank Space
 									.append(businessPartnerAccountNo)			//  Third Party Account Code
 									.append(SEPARATOR)							//	Blank Space
-									.append(businessPartnerAccountType)			//  Account Type Saving = AHO, Checking = CTE
+									.append(businessPartnerAccountType.get())			//  Account Type Saving = AHO, Checking = CTE
 									.append(SEPARATOR)							//	Blank Space
 									.append(currencyType)						//  Currency Type 0 = VES
 									.append(SEPARATOR)							//	Blank Space