ConverterLocator's default converter has on line 130:

if (value == null || "".equals(value))
{
    return null;
}

This doesn't look right to me. I don't want the default converter to be converting "" strings to null - surely it should leave them alone and just be doing type conversions?

Can anyone explain the logic for this? There is no comment explaining why the "" check, and I'm loathe to just change it for fear of breaking things.

That said, the reason I noticed is that this behaviour wasn't the same before the recent converter changes, so whoever implemented this in the first place has also "broken things". :-)

Al

Reply via email to