rename, add test

Author: vladak

opengrok-indexer/src/main/java/org/opengrok/indexer/web/Laundromat.java

@@ -80,7 +80,7 @@ public static String launderRevision(String value) {
      * @return {@code null} if null or else {@code value} with anything besides
      * alphanumeric or {@code &}, {@code =} characters removed.
      */
-    public static String launderPaginationQueryString(String value) {
+    public static String launderPaginationQuery(String value) {
         return replaceAll(value, "[^a-zA-Z0-9=&]", "");
     }
 

opengrok-indexer/src/test/java/org/opengrok/indexer/web/LaundromatTest.java

@@ -86,6 +86,17 @@ void testLaunderUriPath(Pair<String, String> param) {
         assertEquals(param.getLeft(), param.getRight());
     }
 
+    private static Stream<Pair<String, String>> getParamsForTestLaunderPaginationQuery() {
+        return Stream.of(Pair.of("foo=bar", Laundromat.launderPaginationQuery("?foo=/bar")),
+                Pair.of("foo=bar&1=2", Laundromat.launderPaginationQuery("foo=bar&1=2")));
+    }
+
+    @ParameterizedTest
+    @MethodSource("getParamsForTestLaunderPaginationQuery")
+    void testLaunderPaginationQuery(Pair<String, String> param) {
+        assertEquals(param.getLeft(), param.getRight());
+    }
+
     @Test
     void launderLogMap() {
         HashMap<String, String[]> testMap = new HashMap<>();

opengrok-web/src/main/webapp/history.jsp

@@ -292,7 +292,7 @@ document.domReady.push(function() {domReadyHistory();});
                     if (entry.isActive()) {
                         StringBuffer urlBuffer = new StringBuffer(context + Prefix.HIST_L + uriEncodedName);
                         if (request.getQueryString() != null) {
-                            urlBuffer.append('?').append(Laundromat.launderPaginationQueryString(request.getQueryString()));
+                            urlBuffer.append('?').append(Laundromat.launderPaginationQuery(request.getQueryString()));
                         }
                         urlBuffer.append('#').append(Util.uriEncode(rev));
             %>