Tested and working... thanks. phil
P.S.: I know it's a pita but what about a 'release' ? On Mon, May 10, 2010 at 1:39 PM, Philippe Van Dyck <[email protected]> wrote: > ok sorry, got it, testing with preproduction data right now > > phil > > > On Mon, May 10, 2010 at 12:34 PM, Philippe Van Dyck <[email protected]>wrote: > >> I will test it but I can't find it... could you please confirm that it is >> in the HEAD branch of the git repository ? >> I suppose that we are talking about changes in NumberType ? >> >> cheers, >> >> phil >> >> >> On Mon, May 10, 2010 at 2:29 AM, Niclas Hedhman <[email protected]>wrote: >> >>> On Sun, May 9, 2010 at 10:56 PM, Philippe Van Dyck <[email protected]> >>> wrote: >>> > Hi all, >>> > I use a lot of BigDecimal (it is the base to my Money value). >>> > Did you push your changes to the code repository yet ? >>> > Are you sure you could handle it in a compatible way ? >>> >>> Pretty sure. There are tests for it as well. >>> >>> Since the Object coming into the fromJSON() method in NumberType will >>> vary from either an Integer, Double or String depending on the size of >>> the number, I just put in a >>> >>> new BigDecimal( "" + json ) >>> >>> which deals with it correctly. >>> >>> It has been committed if you would like to try it out. >>> >>> >>> Cheers >>> -- >>> Niclas Hedhman, Software Developer >>> http://www.qi4j.org - New Energy for Java >>> >>> I live here; http://tinyurl.com/2qq9er >>> I work here; http://tinyurl.com/2ymelc >>> I relax here; http://tinyurl.com/2cgsug >>> >>> _______________________________________________ >>> qi4j-dev mailing list >>> [email protected] >>> http://lists.ops4j.org/mailman/listinfo/qi4j-dev >>> >> >> >
_______________________________________________ qi4j-dev mailing list [email protected] http://lists.ops4j.org/mailman/listinfo/qi4j-dev
