修复删除个性化菜单的接口 #127

Author: binarywang

weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/WxMpMenuService.java

@@ -24,20 +24,20 @@ public interface WxMpMenuService {
   /**
    * <pre>
    * 自定义菜单删除接口
-   * 详情请见: http://mp.weixin.qq.com/wiki/index.php?title=自定义菜单删除接口
+   * 详情请见: https://mp.weixin.qq.com/wiki?t=resource/res_main&id=mp1421141015&token=&lang=zh_CN
    * </pre>
    */
   void menuDelete() throws WxErrorException;
 
   /**
    * <pre>
    * 删除个性化菜单接口
-   * 详情请见: http://mp.weixin.qq.com/wiki/0/c48ccd12b69ae023159b4bfaa7c39c20.html
+   * 详情请见: https://mp.weixin.qq.com/wiki?t=resource/res_main&id=mp1455782296&token=&lang=zh_CN
    * </pre>
    *
-   * @param menuid
+   * @param menuId 个性化菜单的menuid
    */
-  void menuDelete(String menuid) throws WxErrorException;
+  void menuDelete(String menuId) throws WxErrorException;
 
   /**
    * <pre>

weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java

@@ -45,10 +45,12 @@ public void menuDelete() throws WxErrorException {
   }
 
   @Override
-  public void menuDelete(String menuid) throws WxErrorException {
+  public void menuDelete(String menuId) throws WxErrorException {
     String url = API_URL_PREFIX + "/delconditional";
-    String result = this.wxMpService.get(url, "menuid=" + menuid);
-    log.debug("根据MeunId({})删除菜单结果：{}", menuid, result);
+    JsonObject jsonObject = new JsonObject();
+    jsonObject.addProperty("menuid", menuId);
+    String result = this.wxMpService.post(url, jsonObject.toString());
+    log.debug("根据MeunId({})删除菜单结果：{}", menuId, result);
   }
 
   @Override
@@ -70,7 +72,7 @@ public WxMenu menuGet() throws WxErrorException {
   public WxMenu menuTryMatch(String userid) throws WxErrorException {
     String url = API_URL_PREFIX + "/trymatch";
     JsonObject jsonObject = new JsonObject();
-    jsonObject.addProperty("user_id",userid);
+    jsonObject.addProperty("user_id", userid);
     try {
       String resultContent = this.wxMpService.post(url, jsonObject.toString());
       return WxMenu.fromJson(resultContent);

weixin-java-mp/src/test/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImplTest.java

@@ -100,6 +100,12 @@ public void testMenuDelete() throws WxErrorException {
     this.wxService.getMenuService().menuDelete();
   }
 
+  @Test
+  public void testDeleteConditionalMenu() throws WxErrorException {
+    String menuId = "123";
+    this.wxService.getMenuService().menuDelete(menuId);
+  }
+
   @DataProvider(name="menu")
   public Object[][] getMenu() {
     WxMenu menu = new WxMenu();