bug fixed for JSONType.ignores. for issue alibaba#1146

wenshao · wenshao · commit aa980697c09d · 2017-04-14T17:45:54.000+08:00
diff --git a/src/main/java/com/alibaba/fastjson/util/TypeUtils.java b/src/main/java/com/alibaba/fastjson/util/TypeUtils.java
@@ -979,6 +979,10 @@ public static List<FieldInfo> computeGetters(Class<?> clazz, //
                         continue;
                     }
 
+                    if (isJSONTypeIgnore(clazz, jsonType, propertyName)) {
+                        continue;
+                    }
+
                     Field field = getField(clazz, propertyName, declaredFields);
 
                     if (field == null) {
diff --git a/src/test/java/com/alibaba/json/bvt/issue_1100/Issue1146.java b/src/test/java/com/alibaba/json/bvt/issue_1100/Issue1146.java
@@ -0,0 +1,29 @@
+package com.alibaba.json.bvt.issue_1100;
+
+import com.alibaba.fastjson.JSON;
+import com.alibaba.fastjson.annotation.JSONType;
+import junit.framework.TestCase;
+
+/**
+ * Created by wenshao on 14/04/2017.
+ */
+public class Issue1146 extends TestCase {
+    public void test_for_issue() throws Exception {
+        String json = JSON.toJSONString(new Test());
+        assertEquals("{\"zzz\":true}", json);
+    }
+
+    @JSONType(ignores = {"xxx", "yyy"})
+    public static class Test {
+
+        public boolean isXxx() {
+            return true;
+        }
+        public boolean getYyy() {
+            return true;
+        }
+        public boolean getZzz() {
+            return true;
+        }
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -979,6 +979,10 @@ public static List<FieldInfo> computeGetters(Class<?> clazz, //`
`979`	`979`	`continue;`
`980`	`980`	`}`
`981`	`981`
	`982`	`+ if (isJSONTypeIgnore(clazz, jsonType, propertyName)) {`
	`983`	`+ continue;`
	`984`	`+ }`
	`985`	`+`
`982`	`986`	`Field field = getField(clazz, propertyName, declaredFields);`
`983`	`987`
`984`	`988`	`if (field == null) {`