Skip to content

Commit 571116f

Browse files
committed
CB-5400: Restructured batch custom type example; removed deprecated method calls.
1 parent 3d3b83a commit 571116f

File tree

10 files changed

+68
-112
lines changed

10 files changed

+68
-112
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
2+
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
3+
<modelVersion>4.0.0</modelVersion>
4+
5+
<parent>
6+
<groupId>com.backbase.buildingblocks</groupId>
7+
<artifactId>backbase-service-extension-starter-parent</artifactId>
8+
<version>12.1.0</version>
9+
</parent>
10+
11+
<groupId>com.backbase.dbs.paymentorder</groupId>
12+
<artifactId>lorem-ipsum-batch-upload-type-service-extension</artifactId>
13+
<version>1.0.0-SNAPSHOT</version>
14+
15+
<properties>
16+
17+
<!-- docker configuration -->
18+
<docker.image.name>harbor.backbase.eu/experimental/${project.artifactId}</docker.image.name>
19+
<docker.image.tag>${project.version}</docker.image.tag>
20+
21+
<!-- base image -->
22+
<docker.base.tag>DBS-2.20.1</docker.base.tag>
23+
<docker.base.name>repo.backbase.com/backbase-docker-releases/payment-order-service</docker.base.name>
24+
</properties>
25+
26+
<dependencyManagement>
27+
<dependencies>
28+
<dependency>
29+
<groupId>com.backbase.dbs</groupId>
30+
<artifactId>banking-services-bom</artifactId>
31+
<version>2.20.1</version>
32+
<type>pom</type>
33+
<scope>import</scope>
34+
</dependency>
35+
</dependencies>
36+
</dependencyManagement>
37+
38+
<dependencies>
39+
<dependency>
40+
<groupId>com.backbase.dbs.paymentorder</groupId>
41+
<artifactId>payment-order-service</artifactId>
42+
<classifier>classes</classifier>
43+
<scope>provided</scope>
44+
</dependency>
45+
<dependency>
46+
<groupId>org.springframework.boot</groupId>
47+
<artifactId>spring-boot-starter-test</artifactId>
48+
<scope>test</scope>
49+
</dependency>
50+
</dependencies>
51+
</project>
Lines changed: 14 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,7 @@
33
import com.backbase.buildingblocks.presentation.errors.BadRequestException;
44
import com.backbase.dbs.payment.batch.config.BatchConfiguration;
55
import com.backbase.dbs.payment.batch.model.BatchHeader;
6-
import com.backbase.dbs.payment.batch.model.BatchHeader.BatchHeaderBuilder;
76
import com.backbase.dbs.payment.batch.model.BatchPayment;
8-
import com.backbase.dbs.payment.batch.model.BatchPayment.BatchPaymentBuilder;
97
import com.backbase.dbs.payment.batch.model.BatchUploadHeader;
108
import com.backbase.dbs.payment.batch.parser.BatchParseCallback;
119
import com.backbase.dbs.payment.batch.parser.DigestBatchParser;
@@ -38,53 +36,53 @@ public String getBatchUploadType() {
3836
public void parseBatches(InputStream inputStream, BatchParseCallback callback) {
3937

4038
try (JsonParser parser = new JsonFactory().createParser(inputStream)) {
41-
callback.onBatchUpload(BatchUploadHeader.builder().batchFileType(LOREM_IPSUM_TYPE).build());
42-
BatchHeaderBuilder batchHeaderBuilder = BatchHeader.builder().currency("EUR").batchType("SEPACT");
39+
callback.onBatchUpload(new BatchUploadHeader().batchFileType(LOREM_IPSUM_TYPE));
40+
BatchHeader batchHeader = new BatchHeader().currency("EUR").batchType("SEPACT");
4341
parser.nextToken();
4442
while (parser.nextToken() != JsonToken.END_OBJECT) {
4543
switch (parser.getCurrentName()) {
4644
case "label":
47-
batchHeaderBuilder.batchName(parser.nextTextValue());
45+
batchHeader.batchName(parser.nextTextValue());
4846
break;
4947
case "priority":
5048
// priority is non-standard field and will be stored in the additions map
51-
batchHeaderBuilder.additions(Collections.singletonMap("priority", parser.nextTextValue()));
49+
batchHeader.additions(Collections.singletonMap("priority", parser.nextTextValue()));
5250
break;
5351
case "own-account":
54-
batchHeaderBuilder.originatorAccount(parser.nextTextValue());
52+
batchHeader.originatorAccount(parser.nextTextValue());
5553
break;
5654
case "requested-execution-date":
57-
batchHeaderBuilder.requestedExecutionDate(
55+
batchHeader.requestedExecutionDate(
5856
LocalDate.parse(parser.nextTextValue(), DateTimeFormatter.ofPattern("dd/MM/yyyy")));
5957
break;
6058
case "batch-total-amount":
61-
batchHeaderBuilder.batchTotalAmount(new BigDecimal(parser.nextTextValue()));
59+
batchHeader.batchTotalCreditAmount(new BigDecimal(parser.nextTextValue()));
6260
break;
6361
case "batch-payments-count":
64-
batchHeaderBuilder.batchPaymentsCount(Integer.valueOf(parser.nextTextValue()));
62+
batchHeader.batchPaymentsCount(Integer.valueOf(parser.nextTextValue()));
6563
break;
6664
case "payments":
67-
callback.onBatch(batchHeaderBuilder.build());
65+
callback.onBatch(batchHeader);
6866
parser.nextToken();
6967
while (parser.nextToken() != JsonToken.END_ARRAY) {
70-
BatchPaymentBuilder batchPaymentBuilder = BatchPayment.builder().currency("EUR")
68+
BatchPayment batchPayment = new BatchPayment().currency("EUR")
7169
.location(formatJsonLocation(parser.getCurrentLocation()));
7270
while (parser.nextToken() != JsonToken.END_OBJECT) {
7371
switch (parser.getCurrentName()) {
7472
case "counterparty-name":
75-
batchPaymentBuilder.counterpartyName(parser.nextTextValue());
73+
batchPayment.counterpartyName(parser.nextTextValue());
7674
break;
7775
case "counterparty-account":
78-
batchPaymentBuilder.counterpartyAccount(parser.nextTextValue());
76+
batchPayment.counterpartyAccount(parser.nextTextValue());
7977
break;
8078
case "amount":
81-
batchPaymentBuilder.amount(new BigDecimal(parser.nextTextValue()));
79+
batchPayment.amount(new BigDecimal(parser.nextTextValue()));
8280
break;
8381
default:
8482
throw new BadRequestException("Batch is not valid");
8583
}
8684
}
87-
callback.onPaymentRecord(batchPaymentBuilder.build());
85+
callback.onPaymentRecord(batchPayment);
8886
}
8987
callback.onBatchEnd();
9088
break;
Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,11 @@ public void validate(BatchUploadHeader batchUploadHeader, BatchUploadReport batc
5050
public void validate(BatchHeader batchHeader, BatchUploadReport batchUploadReport) {
5151
// add batch order pre-validation logic here
5252

53-
if (batchHeader.getBatchTotalAmount().compareTo(BigDecimal.ZERO) <= 0) {
53+
if (batchHeader.getBatchTotalCreditAmount().compareTo(BigDecimal.ZERO) <= 0) {
5454
// We do not propagate batch order location because the upload always has one batch order
5555
batchUploadReport.addError(
56-
ErrorFactory.createAmountIsNotPositiveError("batch-total-amount", batchHeader.getBatchTotalAmount()));
56+
ErrorFactory.createAmountIsNotPositiveError("batch-total-amount",
57+
batchHeader.getBatchTotalCreditAmount()));
5758
}
5859
}
5960

dbs/payments/payments-add-custom-batch-type/enable-batch-type-lorem-ipsum.yml

Lines changed: 0 additions & 4 deletions
This file was deleted.

dbs/payments/payments-add-custom-batch-type/generate-stub-for-batch-type-lorem-ipsum.sh

Lines changed: 0 additions & 12 deletions
This file was deleted.

dbs/payments/payments-add-custom-batch-type/lorem-ipsum-batch-type-service-extension/Dockerfile

Lines changed: 0 additions & 5 deletions
This file was deleted.

dbs/payments/payments-add-custom-batch-type/lorem-ipsum-batch-type-service-extension/pom.xml

Lines changed: 0 additions & 73 deletions
This file was deleted.

0 commit comments

Comments
 (0)