fix getBigDecimal with Boolean, for issue #2745 (#2746)

* fix #2745, support Boolean for getInteger, getBigInteger and getBigDecimal.

Author: Cooperzzy

core/src/main/java/com/alibaba/fastjson2/JSONArray.java

@@ -490,6 +490,10 @@ public Integer getInteger(int index) {
             return Integer.parseInt(str);
         }
 
+        if (value instanceof Boolean) {
+            return (boolean) value ? Integer.valueOf(1) : Integer.valueOf(0);
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to Integer");
     }
 
@@ -768,6 +772,10 @@ public BigInteger getBigInteger(int index) {
             return new BigInteger(str);
         }
 
+        if (value instanceof Boolean) {
+            return (boolean) value ? BigInteger.ONE : BigInteger.ZERO;
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to BigInteger");
     }
 

core/src/main/java/com/alibaba/fastjson2/JSONObject.java

@@ -620,6 +620,10 @@ public Integer getInteger(String key) {
             return Integer.parseInt(str);
         }
 
+        if (value instanceof Boolean) {
+            return (boolean) value ? Integer.valueOf(1) : Integer.valueOf(0);
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to Integer");
     }
 
@@ -978,6 +982,10 @@ public BigInteger getBigInteger(String key) {
             return new BigInteger(str);
         }
 
+        if (value instanceof Boolean) {
+            return (boolean) value ? BigInteger.ONE : BigInteger.ZERO;
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to BigInteger");
     }
 

core/src/test/java/com/alibaba/fastjson2/issues_2700/Issue2745.java

@@ -0,0 +1,30 @@
+package com.alibaba.fastjson2.issues_2700;
+
+import com.alibaba.fastjson2.JSONArray;
+import com.alibaba.fastjson2.JSONObject;
+import org.junit.jupiter.api.Test;
+
+import java.math.BigDecimal;
+import java.math.BigInteger;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+public class Issue2745 {
+    @Test
+    public void Test1() throws Exception {
+        JSONArray jsonArray = new JSONArray();
+        jsonArray.add(true);
+        assertEquals(BigDecimal.ONE, jsonArray.getBigDecimal(0));
+        assertEquals(BigInteger.ONE, jsonArray.getBigInteger(0));
+        assertEquals(Integer.valueOf(1), jsonArray.getInteger(0));
+    }
+
+    @Test
+    public void Test2() throws Exception {
+        JSONObject obj = new JSONObject();
+        obj.put("bool", true);
+        assertEquals(BigDecimal.ONE, obj.getBigDecimal("bool"));
+        assertEquals(BigInteger.ONE, obj.getBigInteger("bool"));
+        assertEquals(Integer.valueOf(1), obj.getInteger("bool"));
+    }
+}

fastjson1-compatible/src/main/java/com/alibaba/fastjson/JSONArray.java

@@ -210,6 +210,10 @@ public BigDecimal getBigDecimal(int index) {
             return toBigDecimal(str);
         }
 
+        if (value instanceof Boolean) {
+            return (Boolean) value ? BigDecimal.ONE : BigDecimal.ZERO;
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to BigDecimal");
     }
 
@@ -261,6 +265,10 @@ public Integer getInteger(int index) {
             return Integer.parseInt(str);
         }
 
+        if (value instanceof Boolean) {
+            return (Boolean) value ? Integer.valueOf(1) : Integer.valueOf(0);
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to Integer");
     }
 
@@ -650,6 +658,10 @@ public BigInteger getBigInteger(int index) {
             return new BigInteger(str);
         }
 
+        if (value instanceof Boolean) {
+            return (Boolean) value ? BigInteger.ONE : BigInteger.ZERO;
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to BigInteger");
     }
 

fastjson1-compatible/src/main/java/com/alibaba/fastjson/JSONObject.java

@@ -525,6 +525,10 @@ public Integer getInteger(String key) {
             return Integer.parseInt(str);
         }
 
+        if (value instanceof Boolean) {
+            return (Boolean) value ? Integer.valueOf(1) : Integer.valueOf(0);
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to Integer");
     }
 
@@ -756,6 +760,10 @@ public BigDecimal getBigDecimal(String key) {
             return toBigDecimal((String) value);
         }
 
+        if (value instanceof Boolean) {
+            return (Boolean) value ? BigDecimal.ONE : BigDecimal.ZERO;
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to BigDecimal");
     }
 
@@ -789,6 +797,10 @@ public BigInteger getBigInteger(String key) {
             return new BigInteger(str);
         }
 
+        if (value instanceof Boolean) {
+            return (Boolean) value ? BigInteger.ONE : BigInteger.ZERO;
+        }
+
         throw new JSONException("Can not cast '" + value.getClass() + "' to BigInteger");
     }
 

fastjson1-compatible/src/test/java/com/alibaba/fastjson/issues_compatible/Issue2745.java

@@ -0,0 +1,30 @@
+package com.alibaba.fastjson.issues_compatible;
+
+import com.alibaba.fastjson.JSONArray;
+import com.alibaba.fastjson.JSONObject;
+import org.junit.jupiter.api.Test;
+
+import java.math.BigDecimal;
+import java.math.BigInteger;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+public class Issue2745 {
+    @Test
+    public void Test1() throws Exception {
+        JSONArray jsonArray = new JSONArray();
+        jsonArray.add(true);
+        assertEquals(BigDecimal.ONE, jsonArray.getBigDecimal(0));
+        assertEquals(BigInteger.ONE, jsonArray.getBigInteger(0));
+        assertEquals(Integer.valueOf(1), jsonArray.getInteger(0));
+    }
+
+    @Test
+    public void Test2() throws Exception {
+        JSONObject obj = new JSONObject();
+        obj.put("bool", true);
+        assertEquals(BigDecimal.ONE, obj.getBigDecimal("bool"));
+        assertEquals(BigInteger.ONE, obj.getBigInteger("bool"));
+        assertEquals(Integer.valueOf(1), obj.getInteger("bool"));
+    }
+}