Skip to content

Commit 61ade2a

Browse files
committed
Improved: SeoContextFilter.java is not able to handle query strings
(OFBIZ-11278) Removes a trailing blank and uses empty to check request::getParameterMap result Thanks: Mathieu to spot "empty to check request::getParameterMap result"
1 parent 09e583b commit 61ade2a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

applications/product/src/main/java/org/apache/ofbiz/product/category/SeoContextFilter.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -101,15 +101,15 @@ public void doFilter(ServletRequest request, ServletResponse response, FilterCha
101101
String uri = httpRequest.getRequestURI();
102102

103103
Map<String, String[]> parameterMap =request.getParameterMap();
104-
if (parameterMap != null) {
104+
if (!parameterMap.isEmpty()) {
105105
List<BasicNameValuePair> params = new ArrayList<BasicNameValuePair>();
106106
request.getParameterMap().forEach((name, values) -> {
107107
for(String value : values) {
108108
params.add(new BasicNameValuePair(name, value));
109109
}
110110
});
111111
String queryString = URLEncodedUtils.format(params, Charset.forName("UTF-8"));
112-
uri = uri + "?" + queryString;
112+
uri = uri + "?" + queryString;
113113
}
114114

115115
boolean forwarded = forwardUri(httpResponse, uri);

0 commit comments

Comments
 (0)