diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java index 28dbff91..a296fffd 100644 --- a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java +++ b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java @@ -1,5 +1,8 @@ package me.chanjar.weixin.mp.api.impl; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + import me.chanjar.weixin.common.bean.menu.WxMenu; import me.chanjar.weixin.common.exception.WxErrorException; import me.chanjar.weixin.common.util.http.SimpleGetRequestExecutor; @@ -12,6 +15,7 @@ import me.chanjar.weixin.mp.api.WxMpService; */ public class WxMpMenuServiceImpl implements WxMpMenuService { private static final String API_URL_PREFIX = "https://api.weixin.qq.com/cgi-bin/menu"; + private static Logger log = LoggerFactory.getLogger(WxMpMenuServiceImpl.class); private WxMpService wxMpService; @@ -21,25 +25,38 @@ public class WxMpMenuServiceImpl implements WxMpMenuService { @Override public void menuCreate(WxMenu menu) throws WxErrorException { + String menuJson = menu.toJson(); + String url = API_URL_PREFIX + "/create"; if (menu.getMatchRule() != null) { - String url = API_URL_PREFIX + "/addconditional"; - this.wxMpService.execute(new SimplePostRequestExecutor(), url, menu.toJson()); - } else { - String url = API_URL_PREFIX + "/create"; - this.wxMpService.execute(new SimplePostRequestExecutor(), url, menu.toJson()); + url = API_URL_PREFIX + "/addconditional"; + } + if(log.isTraceEnabled()){ + log.trace("开始创建菜单:{}", menuJson); + } + + String result = this.wxMpService.execute(new SimplePostRequestExecutor(), url,menuJson); + + if(log.isDebugEnabled()){ + log.debug("创建菜单:{},结果:{}", menuJson, result); } } @Override public void menuDelete() throws WxErrorException { String url = API_URL_PREFIX + "/delete"; - this.wxMpService.execute(new SimpleGetRequestExecutor(), url, null); + String result = this.wxMpService.execute(new SimpleGetRequestExecutor(), url, null); + if(log.isDebugEnabled()){ + log.debug("删除菜单结果:{}", result); + } } @Override public void menuDelete(String menuid) throws WxErrorException { String url = API_URL_PREFIX + "/delconditional"; - this.wxMpService.execute(new SimpleGetRequestExecutor(), url, "menuid=" + menuid); + String result = this.wxMpService.execute(new SimpleGetRequestExecutor(), url, "menuid=" + menuid); + if(log.isDebugEnabled()){ + log.debug("根据MeunId({})删除菜单结果:{}", menuid,result); + } } @Override