Merge pull request #34 from DevFactory/release/consective-appends-should-reuse-fix-1
Consecutive Appends Should Reuse
This commit is contained in:
@@ -45,8 +45,8 @@ public abstract class AbstractJsonMapping implements Serializable {
|
||||
@JsonAnySetter
|
||||
protected void handleUnknown(String key, Object value) {
|
||||
StringBuilder unknown = new StringBuilder(this.getClass().getSimpleName());
|
||||
unknown.append(": Unknown property='").append(key);
|
||||
unknown.append("' value='").append(value).append("'");
|
||||
unknown.append(": Unknown property='").append(key)
|
||||
.append("' value='").append(value).append("'");
|
||||
|
||||
LOG.trace(unknown.toString());
|
||||
}
|
||||
|
||||
@@ -199,8 +199,7 @@ public class Configuration extends AbstractJsonMapping implements Serializable {
|
||||
}
|
||||
|
||||
StringBuilder sb = new StringBuilder(getBaseUrl());
|
||||
sb.append(requiredSize);
|
||||
sb.append(imagePath);
|
||||
sb.append(requiredSize).append(imagePath);
|
||||
|
||||
try {
|
||||
return new URL(sb.toString());
|
||||
|
||||
@@ -144,11 +144,9 @@ public class ApiUrl {
|
||||
// Append the key information
|
||||
urlString.append(DELIMITER_FIRST)
|
||||
.append(Param.API_KEY.getValue())
|
||||
.append(apiKey);
|
||||
|
||||
// Append the search term
|
||||
urlString.append(DELIMITER_SUBSEQUENT);
|
||||
urlString.append(Param.QUERY.getValue());
|
||||
.append(apiKey)
|
||||
.append(DELIMITER_SUBSEQUENT)// Append the search term
|
||||
.append(Param.QUERY.getValue());
|
||||
|
||||
String query = (String) params.get(Param.QUERY);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user