fix: encode url contain special char

Author: shawndenggh

client/src/main/java/cn/vika/client/api/model/ApiQueryParam.java

@@ -128,7 +128,7 @@ private String formatSort(String fieldName, Order order) {
         sortMap.put("field", fieldName);
         sortMap.put("order", order.name().toLowerCase());
         try {
-            return new ObjectMapper().writeValueAsString(sortMap);
+            return UrlEncoder.encodeURIComponent(new ObjectMapper().writeValueAsString(sortMap));
         }
         catch (JsonProcessingException e) {
             throw new IllegalArgumentException("can't format sort parameter", e);

client/src/test/java/cn/vika/client/api/model/ApiQueryParamTest.java

@@ -37,7 +37,7 @@ public void testParamWithSort() {
         ApiQueryParam queryParam = new ApiQueryParam()
                 .withSort("id", Order.DESC);
         Map<String, String> queryMap = queryParam.toMap();
-        assertThat(queryMap).containsOnlyKeys("sort[]");
+        assertThat(queryMap).containsOnlyKeys("sort%5B%5D");
     }
 
     @Test
@@ -46,7 +46,7 @@ public void testParamWithSortMulti() {
                 .withSort("id", Order.DESC)
                 .withSort("name", Order.DESC);
         Map<String, String> queryMap = queryParam.toMap();
-        assertThat(queryMap).containsOnlyKeys("sort[].0", "sort[].1");
+        assertThat(queryMap).containsOnlyKeys("sort%5B%5D.0", "sort%5B%5D.1");
     }
 
     @Test

core/src/test/java/cn/vika/core/utils/MapUtilTest.java

@@ -1,10 +1,11 @@
 package cn.vika.core.utils;
 
+import java.util.LinkedHashMap;
 import java.util.Map;
 
-import org.assertj.core.util.Maps;
 import org.junit.jupiter.api.Test;
 
+import static cn.vika.core.utils.UrlEncoder.encodeURIComponent;
 import static org.assertj.core.api.Assertions.assertThat;
 
 /**
@@ -14,13 +15,14 @@ public class MapUtilTest {
 
     @Test
     public void testExtractKeyToVariables() {
-        Map<String, String> uriVariables = Maps.newHashMap("page", "1");
+        Map<String, String> uriVariables = new LinkedHashMap<>(6);
+        uriVariables.put("page", "1");
         uriVariables.put("pageSize", "10");
-        uriVariables.put("sort[].0", "{\"field\":\"id\",\"order\":\"desc\"}");
-        uriVariables.put("sort[].1", "{\"field\":\"name\",\"order\":\"desc\"}");
+        uriVariables.put(encodeURIComponent("sort[].0"), encodeURIComponent("{\"field\":\"id\",\"order\":\"desc\"}"));
+        uriVariables.put(encodeURIComponent("sort[].1"), encodeURIComponent("{\"field\":\"name\",\"order\":\"desc\"}"));
         uriVariables.put("recordIds.0", "rec0");
         uriVariables.put("recordIds.1", "rec1");
         String urlParam = MapUtil.extractKeyToVariables(uriVariables);
-        assertThat(urlParam).isEqualTo("?recordIds={recordIds.0}&sort%5B%5D={sort[].0}&sort%5B%5D={sort[].1}&recordIds={recordIds.1}&pageSize={pageSize}&page={page}");
+        assertThat(urlParam).isEqualTo("?page={page}&pageSize={pageSize}&sort%5B%5D={sort%5B%5D.0}&sort%5B%5D={sort%5B%5D.1}&recordIds={recordIds.0}&recordIds={recordIds.1}");
     }
 }