Reviewers: conroy, rjrjr, jat, tbroyer,
Description:
Fixes two issues
Issue 5460: Scaffold app expenses does not store decimal values
Issue 5663: DoubleParser rounds values to integers
Thanks to [email protected]
Please review this at http://gwt-code-reviews.appspot.com/1181802/show
Affected files:
M user/src/com/google/gwt/text/client/DoubleParser.java
M user/test/com/google/gwt/text/client/DoubleParserTest.java
Index: user/src/com/google/gwt/text/client/DoubleParser.java
===================================================================
--- user/src/com/google/gwt/text/client/DoubleParser.java (revision 9348)
+++ user/src/com/google/gwt/text/client/DoubleParser.java (working copy)
@@ -46,7 +46,7 @@
}
try {
- return
Math.rint(NumberFormat.getDecimalFormat().parse(object.toString()));
+ return NumberFormat.getDecimalFormat().parse(object.toString());
} catch (NumberFormatException e) {
throw new ParseException(e.getMessage(), 0);
}
Index: user/test/com/google/gwt/text/client/DoubleParserTest.java
===================================================================
--- user/test/com/google/gwt/text/client/DoubleParserTest.java (revision
9348)
+++ user/test/com/google/gwt/text/client/DoubleParserTest.java (working
copy)
@@ -30,7 +30,7 @@
}
public void testOuroborus() throws ParseException {
- assertEquals("123",
DoubleRenderer.instance().render(DoubleParser.instance().parse("123")));
+ assertEquals("123.5",
DoubleRenderer.instance().render(DoubleParser.instance().parse("123.5")));
}
public void testNull() throws ParseException {
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
