--- On Fri, 2/12/10, Adam Heath <[email protected]> wrote: > From: Adam Heath <[email protected]> > Subject: Re: svn commit: r908676 - > /ofbiz/trunk/framework/base/src/org/ofbiz/base/test/BaseUnitTests.java > To: [email protected] > Date: Friday, February 12, 2010, 10:16 AM > Adam Heath wrote: > > Adrian Crum wrote: > >> Actually, that error message was caused by rev > 908700. > > > > How? That added an Enum converter; I didn't > think ofbiz actually used > > enums mutch. > > Ok, I see it; Is this code trying to convert a String to an > Enum? > That's not supported. To make that work, you also > need to pass in a > the target class. Perhaps we should remove the single > arg variant of > convert.
Why would we want to do that? Also, why not extend AbstractConverter like the other classes? The exception goes away if the converter is written like the rest. Also, StringToEnum.convert(String obj) should throw ConversionException.
