Skip to content

Commit 75aa3fc

Browse files
lixizebinarywang
authored andcommitted
🎨 #3445 【企业微信】第三方应用开发消息处理判断重复的方法进行优化
1 parent 24f03e9 commit 75aa3fc

File tree

5 files changed

+8
-7
lines changed

5 files changed

+8
-7
lines changed

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/message/WxChannelMessageRouter.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ public Object route(final WxChannelMessage message, final String content, final
178178
* @param wxMessage 消息
179179
* @return 是否重复
180180
*/
181-
private boolean isMsgDuplicated(WxChannelMessage wxMessage) {
181+
protected boolean isMsgDuplicated(WxChannelMessage wxMessage) {
182182
String messageId = this.generateMessageId(wxMessage);
183183
return this.messageDuplicateChecker.isDuplicate(messageId);
184184
}
@@ -188,7 +188,7 @@ private boolean isMsgDuplicated(WxChannelMessage wxMessage) {
188188
*
189189
* @return 消息id
190190
*/
191-
private String generateMessageId(WxChannelMessage wxMessage) {
191+
protected String generateMessageId(WxChannelMessage wxMessage) {
192192
StringBuilder sb = new StringBuilder();
193193
if (wxMessage.getMsgId() == null) {
194194
sb.append(wxMessage.getCreateTime())

weixin-java-cp/src/main/java/me/chanjar/weixin/cp/message/WxCpMessageRouter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ public WxCpXmlOutMessage route(final WxCpXmlMessage wxMessage) {
258258
return this.route(wxMessage, new HashMap<>(2));
259259
}
260260

261-
private boolean isMsgDuplicated(WxCpXmlMessage wxMessage) {
261+
protected boolean isMsgDuplicated(WxCpXmlMessage wxMessage) {
262262
StringBuilder messageId = new StringBuilder();
263263
if (wxMessage.getMsgId() == null) {
264264
messageId.append(wxMessage.getCreateTime())

weixin-java-cp/src/main/java/me/chanjar/weixin/cp/tp/message/WxCpTpMessageRouter.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,7 @@ public WxCpXmlOutMessage route(final WxCpTpXmlMessage wxMessage) {
278278
return this.route(wxMessage, new HashMap<>(2));
279279
}
280280

281-
private boolean isMsgDuplicated(final String suiteId, WxCpTpXmlMessage wxMessage) {
281+
protected boolean isMsgDuplicated(final String suiteId, WxCpTpXmlMessage wxMessage) {
282282
StringBuilder messageId = new StringBuilder();
283283
messageId.append(wxMessage.getToUserName());
284284
if (wxMessage.getInfoType() != null) {
@@ -306,7 +306,8 @@ private boolean isMsgDuplicated(final String suiteId, WxCpTpXmlMessage wxMessage
306306
.append("-").append(wxMessage.getCreateTime())
307307
.append("-").append(wxMessage.getFromUserName())
308308
.append("-").append(StringUtils.trimToEmpty(wxMessage.getEvent()))
309-
.append("-").append(StringUtils.trimToEmpty(wxMessage.getEventKey()));
309+
.append("-").append(StringUtils.trimToEmpty(wxMessage.getEventKey()))
310+
.append("-").append(StringUtils.trimToEmpty(wxMessage.getExternalUserID()));
310311
}
311312
}
312313

weixin-java-miniapp/src/main/java/cn/binarywang/wx/miniapp/message/WxMaMessageRouter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ public WxMaXmlOutMessage route(final WxMaMessage wxMessage) {
173173
return this.route(wxMessage, new HashMap<>(2));
174174
}
175175

176-
private boolean isMsgDuplicated(WxMaMessage wxMessage) {
176+
protected boolean isMsgDuplicated(WxMaMessage wxMessage) {
177177
StringBuilder messageId = new StringBuilder();
178178
if (wxMessage.getMsgId() == null) {
179179
messageId.append(wxMessage.getCreateTime())

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ public WxMpXmlOutMessage route(String appid, final WxMpXmlMessage wxMessage) {
315315
return this.route(appid, wxMessage, new HashMap<>(2));
316316
}
317317

318-
private boolean isMsgDuplicated(WxMpXmlMessage wxMessage) {
318+
protected boolean isMsgDuplicated(WxMpXmlMessage wxMessage) {
319319
StringBuilder messageId = new StringBuilder();
320320
if (wxMessage.getMsgId() == null) {
321321
messageId.append(wxMessage.getCreateTime())

0 commit comments

Comments
 (0)