Bug #13232: fixing technical error if no content language is used whe…

…n displaying all public attachment versions
Silverpeas · Nov 8, 2022 · 9971d2e · 9971d2e
1 parent b2a6bf0
commit 9971d2e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/core-war/src/main/webapp/attachment/jsp/publicVersions.jsp b/core-war/src/main/webapp/attachment/jsp/publicVersions.jsp
@@ -99,13 +99,14 @@
   <view:browseBar extraInformations="${requestScope.Document.title}" clickable="false"/>
 
   <%
-    final String arrayUrl = UriBuilder.fromPath("ViewAllVersions")
+    UriBuilder uriBuilder = UriBuilder.fromPath("ViewAllVersions")
         .queryParam("DocId", id)
         .queryParam("ComponentId", componentId)
-        .queryParam("fromAlias", fromAlias)
-        .queryParam("Language", contentLanguage)
-        .build().toString();
-    ArrayPane arrayPane = gef.getArrayPane("List", arrayUrl, request, session);
+        .queryParam("fromAlias", fromAlias);
+    if (StringUtil.isDefined(contentLanguage)) {
+      uriBuilder = uriBuilder.queryParam("Language", contentLanguage);
+    }
+    ArrayPane arrayPane = gef.getArrayPane("List", uriBuilder.build().toString(), request, session);
 
 // header of the array
     ArrayColumn arrayColumn_version = arrayPane.addArrayColumn(messages.getString("version"));