Skip to content

Commit faed436

Browse files
authored
Merge pull request #69 from serhiyivanov/support-message-templating-fields
added message templating related fields
2 parents 2735db4 + 84d55db commit faed436

File tree

5 files changed

+34
-4
lines changed

5 files changed

+34
-4
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<modelVersion>4.0.0</modelVersion>
77
<groupId>com.taboola</groupId>
88
<artifactId>backstage-api-java-client</artifactId>
9-
<version>1.1.2</version>
9+
<version>1.1.3</version>
1010

1111
<name>${project.groupId}:${project.artifactId}</name>
1212
<description>Backstage API Java Client</description>

src/main/java/com/taboola/backstage/Backstage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ public static class BackstageBuilder {
181181
private static final String DEFAULT_BACKSTAGE_HOST = "https://backstage.taboola.com/backstage/";
182182
private static final String DEFAULT_AUTH_BACKSTAGE_HOST = "https://authentication.taboola.com/authentication/";
183183
private static final String DEFAULT_USER_AGENT = "Taboola Java Client";
184-
private static final String VERSION = "1.1.2";
184+
private static final String VERSION = "1.1.3";
185185
private static final SerializationConfig DEFAULT_SERIALIZATION_CONFIG = new SerializationConfig();
186186

187187
private String baseUrl;

src/main/java/com/taboola/backstage/exceptions/BackstageAPIException.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ private static String parseError(APIError error) {
3535
return "";
3636
}
3737

38-
return String.format(". Response payload status [%s], message [%s], offending field [%s], message code [%s]", error.getHttpStatus(), error.getMessage(), error.getOffendingField(), error.getMessageCode());
38+
return String.format(". Response payload status [%s], message [%s], offending field [%s], message code [%s], message code template [%s], template parameters [%s] ", error.getHttpStatus(), error.getMessage(), error.getOffendingField(), error.getMessageCode(), error.getMessageCodeEnglishTemplate(), error.getTemplateParameters());
3939
}
4040

4141
public APIError getError() {

src/main/java/com/taboola/backstage/model/APIError.java

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
package com.taboola.backstage.model;
22

3+
import java.util.Collection;
4+
import java.util.List;
5+
36
/**
47
* Created by vladi
58
* Date: 10/12/2017
@@ -14,6 +17,8 @@ public class APIError {
1417
private String message;
1518
private String offendingField;
1619
private String messageCode;
20+
private String messageCodeEnglishTemplate;
21+
private List<String> templateParameters;
1722

1823
public APIError() {
1924
//for jackson
@@ -25,10 +30,16 @@ public APIError(String message, int httpStatus) {
2530
}
2631

2732
public APIError(String message, int httpStatus, String offendingField, String messageCode) {
33+
this(message, httpStatus, offendingField, messageCode, null, null);
34+
}
35+
36+
public APIError(String message, int httpStatus, String offendingField, String messageCode, String messageCodeEnglishTemplate, List<String> templateParameters) {
2837
this.message = message;
2938
this.httpStatus = httpStatus;
3039
this.offendingField = offendingField;
3140
this.messageCode = messageCode;
41+
this.messageCodeEnglishTemplate = messageCodeEnglishTemplate;
42+
this.templateParameters = templateParameters;
3243
}
3344

3445
public int getHttpStatus() {
@@ -63,13 +74,32 @@ public void setMessageCode(String messageCode) {
6374
this.messageCode = messageCode;
6475
}
6576

77+
public String getMessageCodeEnglishTemplate() {
78+
return messageCodeEnglishTemplate;
79+
}
80+
81+
public void setMessageCodeEnglishTemplate(String messageCodeEnglishTemplate) {
82+
this.messageCodeEnglishTemplate = messageCodeEnglishTemplate;
83+
}
84+
85+
public Collection<String> getTemplateParameters() {
86+
return templateParameters;
87+
}
88+
89+
public void setTemplateParameters(List<String> templateParameters) {
90+
this.templateParameters = templateParameters;
91+
}
92+
6693
@Override
6794
public String toString() {
6895
return "APIError{" +
6996
"httpStatus=" + httpStatus +
7097
", message='" + message + '\'' +
7198
", offendingField='" + offendingField + '\'' +
7299
", messageCode='" + messageCode + '\'' +
100+
", messageCodeEnglishTemplate='" + messageCodeEnglishTemplate + '\'' +
101+
", templateParameters=" + templateParameters +
73102
'}';
74103
}
104+
75105
}

src/test/java/com/taboola/backstage/internal/factories/BackstageAPIExceptionFactoryTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ public void testParseError_whenErrorMsgContainsPercentSign_expectingValidFormatF
3535
apiError = testInstance.normalizeError(apiError);
3636
BackstageAPIException ex = new BackstageAPIRequestException(400, apiError);
3737
logger.info(ex);
38-
assertEquals("Failed to perform API call with response code [400]. Response payload status [400], message [Cpc boost value must be between -99% and 100%], offending field [null], message code [null]", ex.getMessage());
38+
assertEquals("Failed to perform API call with response code [400]. Response payload status [400], message [Cpc boost value must be between -99% and 100%], offending field [null], message code [null], message code template [null], template parameters [null] ", ex.getMessage());
3939
}
4040
}

0 commit comments

Comments
 (0)