Use double for metadata numbers

src/main/java/com/box/sdk/BoxFile.java

@@ -1117,7 +1117,7 @@ public Metadata setMetadata(String templateName, String scope, Metadata metadata
                 for (JsonValue value : metadata.getOperations()) {
                     if (value.asObject().get("value").isNumber()) {
                         metadataToUpdate.add(value.asObject().get("path").asString(),
-                                value.asObject().get("value").asFloat());
+                                value.asObject().get("value").asDouble());
                     } else if (value.asObject().get("value").isString()) {
                         metadataToUpdate.add(value.asObject().get("path").asString(),
                                 value.asObject().get("value").asString());

src/main/java/com/box/sdk/BoxFolder.java

@@ -876,7 +876,7 @@ public Metadata setMetadata(String templateName, String scope, Metadata metadata
                 for (JsonValue value : metadata.getOperations()) {
                     if (value.asObject().get("value").isNumber()) {
                         metadataToUpdate.add(value.asObject().get("path").asString(),
-                                value.asObject().get("value").asFloat());
+                                value.asObject().get("value").asDouble());
                     } else if (value.asObject().get("value").isString()) {
                         metadataToUpdate.add(value.asObject().get("path").asString(),
                                 value.asObject().get("value").asString());

src/main/java/com/box/sdk/Metadata.java

@@ -185,6 +185,18 @@ public Metadata add(String path, float value) {
         return this;
     }
 
+    /**
+     * Adds a new metadata value.
+     * @param path the path that designates the key. Must be prefixed with a "/".
+     * @param value the value.
+     * @return this metadata object.
+     */
+    public Metadata add(String path, double value) {
+        this.values.add(this.pathToProperty(path), value);
+        this.addOp("add", path, value);
+        return this;
+    }
+
     /**
      * Adds a new metadata value of array type.
      * @param path the path to the field.
@@ -225,6 +237,18 @@ public Metadata replace(String path, float value) {
         return this;
     }
 
+    /**
+     * Replaces an existing metadata value.
+     * @param path the path that designates the key. Must be prefixed with a "/".
+     * @param value the value.
+     * @return this metadata object.
+     */
+    public Metadata replace(String path, double value) {
+        this.values.set(this.pathToProperty(path), value);
+        this.addOp("replace", path, value);
+        return this;
+    }
+
     /**
      * Replaces an existing metadata value of array type.
      * @param path the path that designates the key. Must be prefixed with a "/".
@@ -452,6 +476,22 @@ private void addOp(String op, String path, float value) {
                 .add("value", value));
     }
 
+    /**
+     * Adds a patch operation.
+     * @param op the operation type. Must be add, replace, remove, or test.
+     * @param path the path that designates the key. Must be prefixed with a "/".
+     * @param value the value to be set.
+     */
+    private void addOp(String op, String path, double value) {
+        if (this.operations == null) {
+            this.operations = new JsonArray();
+        }
+
+        this.operations.add(new JsonObject()
+                .add("op", op)
+                .add("path", path)
+                .add("value", value));
+    }
     /**
      * Adds a new patch operation for array values.
      * @param op the operation type. Must be add, replace, remove, or test.

src/test/Fixtures/BoxFile/UpdateMetadataOnFile200.json

@@ -1,7 +1,9 @@
 {
-  "test":"text",
-  "test2":2,
-  "test3":["first", "second", "third"],
+  "test1":"text",
+  "test2":["first", "second", "third"],
+  "test3":2,
+  "test4":1.23456794E9,
+  "test5":2.33333333333333344E17,
   "$type":"account-e68798f0-347f-4882-a1ae-f65b032780ac",
   "$parent":"file_12345",
   "$id":"0db91053-c355-4fc7-b69c-9d511974bbc4",

src/test/Fixtures/BoxFolder/UpdateMetadataOnFolder200.json

@@ -1,7 +1,9 @@
 {
-  "test":"text",
-  "test2":2,
-  "test3":["first", "second", "third"],
+  "test1":"text",
+  "test2":["first", "second", "third"],
+  "test3":2,
+  "test4":1.23456794E9,
+  "test5":2.33333333333333344E17,
   "$type":"account-e68798f0-347f-4882-a1ae-f65b032780ac",
   "$parent":"folder_12345",
   "$id":"0db91053-c355-4fc7-b69c-9d511974bbc4",

src/test/java/com/box/sdk/BoxFileTest.java

@@ -1857,38 +1857,54 @@ public void testSetMetadataReturnsCorrectly() throws IOException {
         String putResult = "";
         final String fileID = "12345";
         final String metadataURL = "/files/" + fileID + "/metadata/enterprise/testtemplate";
-        ArrayList<String> values = new ArrayList<String>();
-        values.add("first");
-        values.add("second");
-        values.add("third");
+        ArrayList<String> secondValueArray = new ArrayList<String>();
+        secondValueArray.add("first");
+        secondValueArray.add("second");
+        secondValueArray.add("third");
 
         postResult = TestConfig.getFixture("/BoxException/BoxResponseException409");
         putResult = TestConfig.getFixture("/BoxFile/UpdateMetadataOnFile200");
 
-        JsonArray array = new JsonArray()
+        final String firstValue = "text";
+        JsonArray secondValueJson = new JsonArray()
                 .add("first")
                 .add("second")
                 .add("third");
+        final int thirdValue = 2;
+        final float fourthValue = 1234567890f;
+        final double fifthValue = 233333333333333340.0;
 
         JsonObject firstAttribute = new JsonObject()
                 .add("op", "add")
-                .add("path", "/test")
-                .add("value", "text");
+                .add("path", "/test1")
+                .add("value", firstValue);
 
         JsonObject secondAttribute = new JsonObject()
                 .add("op", "add")
                 .add("path", "/test2")
-                .add("value", 2);
+                .add("value", secondValueJson);
 
         JsonObject thirdAttribute = new JsonObject()
                 .add("op", "add")
                 .add("path", "/test3")
-                .add("value", array);
+                .add("value", thirdValue);
+
+        JsonObject fourthAttribute = new JsonObject()
+                .add("op", "add")
+                .add("path", "/test4")
+                .add("value", fourthValue);
+
+        JsonObject fifthAttribute = new JsonObject()
+                .add("op", "add")
+                .add("path", "/test5")
+                .add("value", fifthValue);
 
         JsonArray jsonArray = new JsonArray()
                 .add(firstAttribute)
                 .add(secondAttribute)
-                .add(thirdAttribute);
+                .add(thirdAttribute)
+                .add(fourthAttribute)
+                .add(fifthAttribute);
 
         WIRE_MOCK_CLASS_RULE.stubFor(WireMock.post(WireMock.urlPathEqualTo(metadataURL))
                 .willReturn(WireMock.aResponse()
@@ -1907,16 +1923,22 @@ public void testSetMetadataReturnsCorrectly() throws IOException {
         BoxFile file = new BoxFile(this.api, "12345");
 
         Metadata metadata = new Metadata()
-                .add("/test", "text")
-                .add("/test2", 2)
-                .add("/test3", values);
+            .add("/test1", firstValue)
+            .add("/test2", secondValueArray)
+            .add("/test3", thirdValue)
+            .add("/test4", fourthValue)
+            .add("/test5", fifthValue);
 
         Metadata metadataValues = file.setMetadata("testtemplate", "enterprise", metadata);
 
         Assert.assertEquals("file_12345", metadataValues.getParentID());
         Assert.assertEquals("testtemplate", metadataValues.getTemplateName());
         Assert.assertEquals("enterprise_11111", metadataValues.getScope());
-        Assert.assertEquals("text", metadataValues.getString("/test"));
+        Assert.assertEquals(firstValue, metadataValues.getString("/test1"));
+        Assert.assertEquals(secondValueJson, metadataValues.getValue("/test2"));
+        Assert.assertEquals((double) thirdValue, metadataValues.getFloat("/test3"), 0);
+        Assert.assertEquals((double) fourthValue, metadataValues.getFloat("/test4"), 4);
+        Assert.assertEquals((double) fifthValue, metadataValues.getFloat("/test5"), 0);
     }
 
     @Test

src/test/java/com/box/sdk/BoxFolderTest.java

@@ -1847,38 +1847,54 @@ public void testSetMetadataReturnsCorrectly() throws IOException {
         String putResult = "";
         final String folderID = "12345";
         final String metadataURL = "/folders/" + folderID + "/metadata/enterprise/testtemplate";
-        ArrayList<String> values = new ArrayList<String>();
-        values.add("first");
-        values.add("second");
-        values.add("third");
+        ArrayList<String> secondValueArray = new ArrayList<String>();
+        secondValueArray.add("first");
+        secondValueArray.add("second");
+        secondValueArray.add("third");
 
         postResult = TestConfig.getFixture("/BoxException/BoxResponseException409");
         putResult = TestConfig.getFixture("/BoxFolder/UpdateMetadataOnFolder200");
 
-        JsonArray array = new JsonArray()
+        final String firstValue = "text";
+        JsonArray secondValueJson = new JsonArray()
                 .add("first")
                 .add("second")
                 .add("third");
+        final int thirdValue = 2;
+        final float fourthValue = 1234567890f;
+        final double fifthValue = 233333333333333340.0;
 
         JsonObject firstAttribute = new JsonObject()
                 .add("op", "add")
-                .add("path", "/test")
+                .add("path", "/test1")
                 .add("value", "text");
 
         JsonObject secondAttribute = new JsonObject()
                 .add("op", "add")
                 .add("path", "/test2")
-                .add("value", 2);
+                .add("value", secondValueJson);
 
         JsonObject thirdAttribute = new JsonObject()
                 .add("op", "add")
                 .add("path", "/test3")
-                .add("value", array);
+                .add("value", thirdValue);
+
+        JsonObject fourthAttribute = new JsonObject()
+                .add("op", "add")
+                .add("path", "/test4")
+                .add("value", fourthValue);
+
+        JsonObject fifthAttribute = new JsonObject()
+                .add("op", "add")
+                .add("path", "/test5")
+                .add("value", fifthValue);
 
         JsonArray jsonArray = new JsonArray()
                 .add(firstAttribute)
                 .add(secondAttribute)
-                .add(thirdAttribute);
+                .add(thirdAttribute)
+                .add(fourthAttribute)
+                .add(fifthAttribute);
 
         WIRE_MOCK_CLASS_RULE.stubFor(WireMock.post(WireMock.urlPathEqualTo(metadataURL))
                 .willReturn(WireMock.aResponse()
@@ -1897,16 +1913,22 @@ public void testSetMetadataReturnsCorrectly() throws IOException {
         BoxFolder folder = new BoxFolder(this.api, "12345");
 
         Metadata metadata = new Metadata()
-                .add("/test", "text")
-                .add("/test2", 2)
-                .add("/test3", values);
+            .add("/test1", firstValue)
+            .add("/test2", secondValueArray)
+            .add("/test3", thirdValue)
+            .add("/test4", fourthValue)
+            .add("/test5", fifthValue);
 
         Metadata metadataValues = folder.setMetadata("testtemplate", "enterprise", metadata);
 
         Assert.assertEquals("folder_12345", metadataValues.getParentID());
         Assert.assertEquals("testtemplate", metadataValues.getTemplateName());
         Assert.assertEquals("enterprise_11111", metadataValues.getScope());
-        Assert.assertEquals("text", metadataValues.getString("/test"));
+        Assert.assertEquals(firstValue, metadataValues.getString("/test1"));
+        Assert.assertEquals(secondValueJson, metadataValues.getValue("/test2"));
+        Assert.assertEquals((double) thirdValue, metadataValues.getFloat("/test3"), 0);
+        Assert.assertEquals((double) fourthValue, metadataValues.getFloat("/test4"), 4);
+        Assert.assertEquals((double) fifthValue, metadataValues.getFloat("/test5"), 0);
     }
 
     @Test(expected = BoxDeserializationException.class)

src/test/java/com/box/sdk/MetadataTest.java

@@ -194,6 +194,78 @@ public void testMetadataDate(String dateString, Long dateLong) {
         }
     }
 
+    @Test
+    @Category(IntegrationTest.class)
+    public void testMetadataPrecisionFloat() {
+        final float expectedValueFloat = 1234567890f;
+        BoxAPIConnection api = new BoxAPIConnection(TestConfig.getAccessToken());
+
+        long timestamp = Calendar.getInstance().getTimeInMillis();
+        String templateKey = "precision" + timestamp;
+        String fieldKey = "testPrecision";
+
+        List<MetadataTemplate.Field> fields = new ArrayList<MetadataTemplate.Field>();
+        MetadataTemplate.Field valueField = new MetadataTemplate.Field();
+        valueField.setKey(fieldKey);
+        valueField.setType("float");
+        valueField.setDisplayName("Value Field");
+        fields.add(valueField);
+
+        MetadataTemplate template = MetadataTemplate.createMetadataTemplate(api, "enterprise",
+                templateKey, "Precision " + timestamp, false, fields);
+
+        Assert.assertEquals("float", template.getFields().get(0).getType());
+
+        // Add template to item
+        Metadata mdValues = new Metadata();
+        mdValues.add("/" + fieldKey, expectedValueFloat);
+        BoxFolder.Info folder = BoxFolder.getRootFolder(api).createFolder("Metadata Precision Test " + timestamp);
+        Metadata actualMD = folder.getResource().createMetadata(templateKey, mdValues);
+
+        Assert.assertEquals(templateKey, actualMD.getTemplateName());
+
+        final double actualValueDouble = actualMD.getFloat("/" + fieldKey);
+
+        // Instead of "hard-coding" the delta to 4.0, let's calculate it and then validate it
+        final double delta = actualValueDouble - (double) expectedValueFloat;
+        Assert.assertEquals(4.0, delta, 0);
+
+        // Now that we know delta is 4.0, when can use it for this validation
+        Assert.assertEquals((double) expectedValueFloat, actualValueDouble, delta);
+    }
+
+    @Test
+    @Category(IntegrationTest.class)
+    public void testMetadataPrecisionDouble() {
+        final double valueDouble = 233333333333333340.0;
+        BoxAPIConnection api = new BoxAPIConnection(TestConfig.getAccessToken());
+
+        long timestamp = Calendar.getInstance().getTimeInMillis();
+        String templateKey = "precision" + timestamp;
+        String fieldKey = "testPrecision";
+
+        List<MetadataTemplate.Field> fields = new ArrayList<MetadataTemplate.Field>();
+        MetadataTemplate.Field valueField = new MetadataTemplate.Field();
+        valueField.setKey(fieldKey);
+        valueField.setType("float");
+        valueField.setDisplayName("Value Field");
+        fields.add(valueField);
+
+        MetadataTemplate template = MetadataTemplate.createMetadataTemplate(api, "enterprise",
+                templateKey, "Precision " + timestamp, false, fields);
+
+        Assert.assertEquals("float", template.getFields().get(0).getType());
+
+        // Add template to item
+        Metadata mdValues = new Metadata();
+        mdValues.add("/" + fieldKey, valueDouble);
+        BoxFolder.Info folder = BoxFolder.getRootFolder(api).createFolder("Metadata Precision Test " + timestamp);
+        Metadata actualMD = folder.getResource().createMetadata(templateKey, mdValues);
+
+        Assert.assertEquals(templateKey, actualMD.getTemplateName());
+        Assert.assertEquals(valueDouble, actualMD.getFloat("/" + fieldKey), 0);
+    }
+
     @Test
     @Category(IntegrationTest.class)
     public void testMultiSelectMetadataCRUD() {
@@ -384,7 +456,7 @@ public void getValues(String dateString, Long dateLong) {
         Assert.assertEquals(stringValue, md.getString("/documentType"));
 
         Assert.assertEquals(intValue, md.getValue("/capacity").asInt());
-        Assert.assertEquals((float) intValue, md.getFloat("/capacity"), 0);
+        Assert.assertEquals((double) intValue, md.getFloat("/capacity"), 0);
 
         try {
             Assert.assertEquals(dateValue, md.getDate("/deadline"));