trim() in NumberValidator

[Bryan Lewis <br...@maine.rr.com>]

A user of our new Tapestry apps expressed surprise that numeric inputs weren't trimmed.  Pasting a number into a textfield often leaves a trailing space, and the NumberValidator rejects it. 

The javadoc for IValidator.toObject() says, "The input string will already have been trimmed."  The input does get trimmed in the checkRequired() method, but not in the final parsing of the string.  Wouldn't it be more consistent to add this one line to NumberValidator.toObject()?

            if (value != null) value = value.trim();

I could fix it locally with my own subclass of NumberValidator, but this seems like something we'd want corrected in the core.

Thanks.