🎨 #1571 对微信模版消息行业信息不一致的变态问题尽可能的进行优化

binarywang · May 23, 2020 · 4eb1d1d · 4eb1d1d
1 parent bcefec0
commit 4eb1d1d
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 17 deletions.
diff --git a/...in-java-mp/src/main/java/me/chanjar/weixin/mp/bean/template/WxMpTemplateIndustryEnum.java b/...in-java-mp/src/main/java/me/chanjar/weixin/mp/bean/template/WxMpTemplateIndustryEnum.java
@@ -41,9 +41,9 @@ public enum WxMpTemplateIndustryEnum {
    */
   BANK("金融业", "银行", 7),
   /**
-   * 金融业 - 基金|理财|信托
+   * 金融业 - 证券|基金|理财|信托（实际是这个）
    */
-  FUND("金融业", "基金|理财|信托", 8),
+  FUND("金融业", "证券基金理财信托", 8),
   /**
    * 金融业 - 保险
    */
@@ -93,9 +93,9 @@ public enum WxMpTemplateIndustryEnum {
    */
   MUSEUM("政府与公共事业", "博物馆", 20),
   /**
-   * 政府与公共事业 - 公共事业|非盈利机构
+   * 政府与公共事业 - 公共事业非盈利机构
    */
-  PUBLIC_WORKS_NONPROFIT("政府与公共事业", "公共事业|非盈利机构", 21),
+  PUBLIC_WORKS_NONPROFIT("政府与公共事业", "公共事业非盈利机构", 21),
   /**
    * 医药护理 - 医药医疗
    */
@@ -125,7 +125,7 @@ public enum WxMpTemplateIndustryEnum {
    */
   THE_PLANE_RELATED("交通工具", "飞机相关", 28),
   /**
-   * 房地产 - 建筑
+   * 房地产 - 房地产|建筑（实际上是这个）
    */
   ARCHITECTURE("房地产", "建筑", 29),
   /**
@@ -153,9 +153,9 @@ public enum WxMpTemplateIndustryEnum {
    */
   AUTHENTICATION("商业服务", "认证", 35),
   /**
-   * 商业服务 - 会计|审计
+   * 商业服务 - 审计
    */
-  AUDIT("商业服务", "会计|审计", 36),
+  AUDIT("商业服务", "审计", 36),
   /**
    * 文体娱乐 - 传媒
    */

diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/util/json/WxMpIndustryGsonAdapter.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/util/json/WxMpIndustryGsonAdapter.java
@@ -29,6 +29,11 @@ public WxMpTemplateIndustry deserialize(JsonElement jsonElement, Type type, Json
 
   private WxMpTemplateIndustryEnum convertFromJson(JsonObject json) {
     String secondClass = GsonHelper.getString(json, "second_class");
+    final WxMpTemplateIndustryEnum industryEnum = WxMpTemplateIndustryEnum.findBySecondary(secondClass);
+    if (industryEnum != null) {
+      return industryEnum;
+    }
+
     if (secondClass.contains("|")) {
       secondClass = secondClass.split("\\|")[1];
     }

diff --git a/...n-java-mp/src/test/java/me/chanjar/weixin/mp/api/impl/WxMpTemplateMsgServiceImplTest.java b/...n-java-mp/src/test/java/me/chanjar/weixin/mp/api/impl/WxMpTemplateMsgServiceImplTest.java
@@ -53,8 +53,8 @@ public void testGetIndustry() throws Exception {
 
   @Test
   public void testSetIndustry() throws Exception {
-    WxMpTemplateIndustry industry = new WxMpTemplateIndustry(WxMpTemplateIndustryEnum.findByCode(1),
-      WxMpTemplateIndustryEnum.findByCode(4));
+    WxMpTemplateIndustry industry = new WxMpTemplateIndustry(WxMpTemplateIndustryEnum.findByCode(29),
+      WxMpTemplateIndustryEnum.findByCode(8));
     boolean result = this.wxService.getTemplateMsgService().setIndustry(industry);
     Assert.assertTrue(result);
   }

diff --git a/...in-java-mp/src/test/java/me/chanjar/weixin/mp/bean/template/WxMpTemplateIndustryTest.java b/...in-java-mp/src/test/java/me/chanjar/weixin/mp/bean/template/WxMpTemplateIndustryTest.java
@@ -3,7 +3,6 @@
 import org.testng.annotations.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.testng.Assert.*;
 
 /**
  * 测试类.
@@ -13,11 +12,21 @@
  */
 public class WxMpTemplateIndustryTest {
 
-    @Test
-    public void testFromJson() {
-      String json="{\"primary_industry\":{\"first_class\":\"IT科技\",\"second_class\":\"互联网|电子商务\"},\"secondary_industry\":{\"first_class\":\"房地产\",\"second_class\":\"房地产|建筑\"}}";
-      final WxMpTemplateIndustry industry = WxMpTemplateIndustry.fromJson(json);
-      assertThat(industry).isNotNull();
-      System.out.println(industry);
-    }
+  @Test
+  public void testFromJson() {
+    String json = "{\"primary_industry\":{\"first_class\":\"IT科技\",\"second_class\":\"互联网|电子商务\"}," +
+      "\"secondary_industry\":{\"first_class\":\"房地产\",\"second_class\":\"房地产|建筑\"}}";
+    final WxMpTemplateIndustry industry = WxMpTemplateIndustry.fromJson(json);
+    assertThat(industry).isNotNull();
+    System.out.println(industry);
+  }
+
+  @Test
+  public void testFromJson_another_example() {
+    String json = "{\"primary_industry\":{\"first_class\":\"金融业\",\"second_class\":\"基金理财信托\"}," +
+      "\"secondary_industry\":{\"first_class\":\"房地产\",\"second_class\":\"建筑\"}}";
+    final WxMpTemplateIndustry industry = WxMpTemplateIndustry.fromJson(json);
+    assertThat(industry).isNotNull();
+    System.out.println(industry);
+  }
 }