[ http://issues.apache.org/jira/browse/HIVEMIND-124?page=comments#action_65746 ] Knut Wannheden commented on HIVEMIND-124: -----------------------------------------
Richard, I can see what you're saying here. The reason for eating the ApplicationRuntimeException and returning false is that it is possible the the translator will throw one if it cannot parse the value as an instance of the expected type. The smart translator will for instance do this if it can't find a PropertyEditor for the expected type. The only solution I can think of is to also add the isAssignableToType() method to the Translator interface. Note that this would potentially also push the caching of the translated value into the Translator. What do you folks think about that? > Report a proper message when a constructor argument in BuilderFactory is > invalid > -------------------------------------------------------------------------------- > > Key: HIVEMIND-124 > URL: http://issues.apache.org/jira/browse/HIVEMIND-124 > Project: HiveMind > Type: Bug > Components: framework > Reporter: Richard Hensley > Attachments: patch.txt > > My situation is that I use constructor arguments for all my required > parameters of objects. The code inside of the BuilderFactoryLogic uses the > BuilderPropertyFactet.isAssignableFrom() method to figure out if a particular > parameter in a constructor is right. The > BuilderPropertyFactet.isAssignableFrom() eats an ApplicationRuntimeException > and returns false. This is bad when you miss-spell a service name, for > instance: > <service>service:MyServic</service> > When you meant, > <service>service:MyService</service> > The error that Hivemind throws is that it had trouble finding a valid > constructor, when the real problem was that it had problems finding your > service to even check if the constructor was valid. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
