[ http://issues.apache.org/jira/browse/HIVEMIND-124?page=all ]

Richard Hensley updated HIVEMIND-124:
-------------------------------------

    Attachment: patch_with_smarttranslator_specialcase.txt

I've attach a patch that special cases the SmartTranslator in 
BuilderPropertyFacet. The SmartTranslator dependant unit test that was failing 
with the first patch is now working correctly. The test that is included in the 
patch is also working correctly.

> 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, patch_with_smarttranslator_specialcase.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]

Reply via email to