FINERACT-1547: Cleanup unnecessary credit bureau dependency in infrastructure core
This commit is contained in:
committed by
Arnold Gálovics
parent
83eb0842dc
commit
da03d73704
+4
-5
@@ -21,7 +21,6 @@ package org.apache.fineract.infrastructure.core.data;
|
|||||||
import java.io.Serializable;
|
import java.io.Serializable;
|
||||||
import java.util.HashMap;
|
import java.util.HashMap;
|
||||||
import java.util.Map;
|
import java.util.Map;
|
||||||
import org.apache.fineract.infrastructure.creditbureau.data.CreditBureauReportData;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Represents the successful result of an REST API call that results in processing a command.
|
* Represents the successful result of an REST API call that results in processing a command.
|
||||||
@@ -38,7 +37,7 @@ public class CommandProcessingResult implements Serializable {
|
|||||||
private final Long subResourceId;
|
private final Long subResourceId;
|
||||||
private final String transactionId;
|
private final String transactionId;
|
||||||
private final Map<String, Object> changes;
|
private final Map<String, Object> changes;
|
||||||
private final CreditBureauReportData creditBureauReportData;
|
private final Map<String, Object> creditBureauReportData;
|
||||||
@SuppressWarnings("unused")
|
@SuppressWarnings("unused")
|
||||||
private final String resourceIdentifier;
|
private final String resourceIdentifier;
|
||||||
private final Long productId;
|
private final Long productId;
|
||||||
@@ -55,7 +54,7 @@ public class CommandProcessingResult implements Serializable {
|
|||||||
|
|
||||||
public static CommandProcessingResult fromDetails(final Long commandId, final Long officeId, final Long groupId, final Long clientId,
|
public static CommandProcessingResult fromDetails(final Long commandId, final Long officeId, final Long groupId, final Long clientId,
|
||||||
final Long loanId, final Long savingsId, final String resourceIdentifier, final Long entityId, final Long gsimId,
|
final Long loanId, final Long savingsId, final String resourceIdentifier, final Long entityId, final Long gsimId,
|
||||||
final Long glimId, final CreditBureauReportData creditBureauReportData, final String transactionId,
|
final Long glimId, final Map<String, Object> creditBureauReportData, final String transactionId,
|
||||||
final Map<String, Object> changes, final Long productId, final Boolean rollbackTransaction, final Long subResourceId) {
|
final Map<String, Object> changes, final Long productId, final Boolean rollbackTransaction, final Long subResourceId) {
|
||||||
return new CommandProcessingResult(commandId, officeId, groupId, clientId, loanId, savingsId, resourceIdentifier, entityId,
|
return new CommandProcessingResult(commandId, officeId, groupId, clientId, loanId, savingsId, resourceIdentifier, entityId,
|
||||||
transactionId, changes, productId, gsimId, glimId, creditBureauReportData, rollbackTransaction, subResourceId);
|
transactionId, changes, productId, gsimId, glimId, creditBureauReportData, rollbackTransaction, subResourceId);
|
||||||
@@ -117,7 +116,7 @@ public class CommandProcessingResult implements Serializable {
|
|||||||
private CommandProcessingResult(final Long commandId, final Long officeId, final Long groupId, final Long clientId, final Long loanId,
|
private CommandProcessingResult(final Long commandId, final Long officeId, final Long groupId, final Long clientId, final Long loanId,
|
||||||
final Long savingsId, final String resourceIdentifier, final Long resourceId, final String transactionId,
|
final Long savingsId, final String resourceIdentifier, final Long resourceId, final String transactionId,
|
||||||
final Map<String, Object> changesOnly, final Long productId, final Long gsimId, final Long glimId,
|
final Map<String, Object> changesOnly, final Long productId, final Long gsimId, final Long glimId,
|
||||||
final CreditBureauReportData creditBureauReportData, Boolean rollbackTransaction, final Long subResourceId) {
|
final Map<String, Object> creditBureauReportData, Boolean rollbackTransaction, final Long subResourceId) {
|
||||||
this.commandId = commandId;
|
this.commandId = commandId;
|
||||||
this.officeId = officeId;
|
this.officeId = officeId;
|
||||||
this.groupId = groupId;
|
this.groupId = groupId;
|
||||||
@@ -200,7 +199,7 @@ public class CommandProcessingResult implements Serializable {
|
|||||||
return this.transactionId;
|
return this.transactionId;
|
||||||
}
|
}
|
||||||
|
|
||||||
public CreditBureauReportData getCreditReport() {
|
public Map<String, Object> getCreditReport() {
|
||||||
return this.creditBureauReportData;
|
return this.creditBureauReportData;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+2
-3
@@ -19,7 +19,6 @@
|
|||||||
package org.apache.fineract.infrastructure.core.data;
|
package org.apache.fineract.infrastructure.core.data;
|
||||||
|
|
||||||
import java.util.Map;
|
import java.util.Map;
|
||||||
import org.apache.fineract.infrastructure.creditbureau.data.CreditBureauReportData;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Represents the successful result of an REST API call that results in processing a command.
|
* Represents the successful result of an REST API call that results in processing a command.
|
||||||
@@ -39,7 +38,7 @@ public class CommandProcessingResultBuilder {
|
|||||||
private Long glimId;
|
private Long glimId;
|
||||||
private String transactionId;
|
private String transactionId;
|
||||||
private Map<String, Object> changes;
|
private Map<String, Object> changes;
|
||||||
private CreditBureauReportData creditBureauReportData;
|
private Map<String, Object> creditBureauReportData;
|
||||||
private Long productId;
|
private Long productId;
|
||||||
private boolean rollbackTransaction = false;
|
private boolean rollbackTransaction = false;
|
||||||
|
|
||||||
@@ -119,7 +118,7 @@ public class CommandProcessingResultBuilder {
|
|||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
|
|
||||||
public CommandProcessingResultBuilder withCreditReport(final CreditBureauReportData withCreditReport) {
|
public CommandProcessingResultBuilder withCreditReport(final Map<String, Object> withCreditReport) {
|
||||||
this.creditBureauReportData = withCreditReport;
|
this.creditBureauReportData = withCreditReport;
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
|
|||||||
+27
@@ -54,4 +54,31 @@ public final class CreditBureauReportData implements Serializable {
|
|||||||
this.closedAccounts = closedAccounts;
|
this.closedAccounts = closedAccounts;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public String getName() {
|
||||||
|
return name;
|
||||||
|
}
|
||||||
|
|
||||||
|
public String getGender() {
|
||||||
|
return gender;
|
||||||
|
}
|
||||||
|
|
||||||
|
public String getAddress() {
|
||||||
|
return address;
|
||||||
|
}
|
||||||
|
|
||||||
|
public String getCreditScore() {
|
||||||
|
return creditScore;
|
||||||
|
}
|
||||||
|
|
||||||
|
public String getBorrowerInfo() {
|
||||||
|
return borrowerInfo;
|
||||||
|
}
|
||||||
|
|
||||||
|
public String[] getOpenAccounts() {
|
||||||
|
return openAccounts;
|
||||||
|
}
|
||||||
|
|
||||||
|
public String[] getClosedAccounts() {
|
||||||
|
return closedAccounts;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+6
-1
@@ -22,6 +22,7 @@ import java.io.File;
|
|||||||
import java.nio.charset.StandardCharsets;
|
import java.nio.charset.StandardCharsets;
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
import java.util.Map;
|
||||||
import java.util.Objects;
|
import java.util.Objects;
|
||||||
import java.util.Optional;
|
import java.util.Optional;
|
||||||
import javax.persistence.PersistenceException;
|
import javax.persistence.PersistenceException;
|
||||||
@@ -103,7 +104,11 @@ public class CreditReportWritePlatformServiceImpl implements CreditReportWritePl
|
|||||||
CreditBureauReportData reportobj = this.thitsaWorksCreditBureauIntegrationWritePlatformService
|
CreditBureauReportData reportobj = this.thitsaWorksCreditBureauIntegrationWritePlatformService
|
||||||
.getCreditReportFromThitsaWorks(command);
|
.getCreditReportFromThitsaWorks(command);
|
||||||
|
|
||||||
return new CommandProcessingResultBuilder().withCreditReport(reportobj).build();
|
Map<String, Object> reportMap = Map.of("name", reportobj.getName(), "gender", reportobj.getGender(), "address",
|
||||||
|
reportobj.getAddress(), "creditScore", reportobj.getCreditScore(), "borrowerInfo", reportobj.getBorrowerInfo(),
|
||||||
|
"openAccounts", reportobj.getOpenAccounts(), "closedAccounts", reportobj.getClosedAccounts());
|
||||||
|
|
||||||
|
return new CommandProcessingResultBuilder().withCreditReport(reportMap).build();
|
||||||
}
|
}
|
||||||
|
|
||||||
baseDataValidator.reset().failWithCode("creditBureau.has.not.been.Integrated");
|
baseDataValidator.reset().failWithCode("creditBureau.has.not.been.Integrated");
|
||||||
|
|||||||
Reference in New Issue
Block a user