issue binarywang#99 Wx*MessageRouter.isDuplicateMessage方法逻辑错误

chanjarster · chanjarster · commit 95d8bd3c47f9 · 2015-02-04T11:49:01.000+08:00
diff --git a/weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/WxCpMessageRouter.java b/weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/WxCpMessageRouter.java
@@ -204,7 +204,7 @@ protected boolean isDuplicateMessage(WxCpXmlMessage wxMessage) {
       messageId = String.valueOf(wxMessage.getCreateTime())
           + "-" +String.valueOf(wxMessage.getAgentId() == null ? "" : wxMessage.getAgentId())
           + "-" + wxMessage.getFromUserName()
-          + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEvent());
+          + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEventKey());
       ;
     } else {
       messageId = String.valueOf(wxMessage.getMsgId());
diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/WxMpMessageRouter.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/WxMpMessageRouter.java
@@ -203,7 +203,7 @@ protected boolean isDuplicateMessage(WxMpXmlMessage wxMessage) {
     if (wxMessage.getMsgId() == null) {
       messageId = String.valueOf(wxMessage.getCreateTime())
           + "-" + wxMessage.getFromUserName()
-          + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEvent());
+          + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEventKey());
       ;
     } else {
       messageId = String.valueOf(wxMessage.getMsgId());

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,7 @@ protected boolean isDuplicateMessage(WxCpXmlMessage wxMessage) {`
`204`	`204`	`messageId = String.valueOf(wxMessage.getCreateTime())`
`205`	`205`	`+ "-" +String.valueOf(wxMessage.getAgentId() == null ? "" : wxMessage.getAgentId())`
`206`	`206`	`+ "-" + wxMessage.getFromUserName()`
`207`		`- + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEvent());`
	`207`	`+ + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEventKey());`
`208`	`208`	`;`
`209`	`209`	`} else {`
`210`	`210`	`messageId = String.valueOf(wxMessage.getMsgId());`
Original file line number	Diff line number	Diff line change
`@@ -203,7 +203,7 @@ protected boolean isDuplicateMessage(WxMpXmlMessage wxMessage) {`
`203`	`203`	`if (wxMessage.getMsgId() == null) {`
`204`	`204`	`messageId = String.valueOf(wxMessage.getCreateTime())`
`205`	`205`	`+ "-" + wxMessage.getFromUserName()`
`206`		`- + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEvent());`
	`206`	`+ + "-" + String.valueOf(wxMessage.getEventKey() == null ? "" : wxMessage.getEventKey());`
`207`	`207`	`;`
`208`	`208`	`} else {`
`209`	`209`	`messageId = String.valueOf(wxMessage.getMsgId());`