commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Niall Pemberton (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (BEANUTILS-23) Misleading error message in ConvertingWrapDynaBean
Date Fri, 27 Jul 2007 11:31:30 GMT

     [ https://issues.apache.org/jira/browse/BEANUTILS-23?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Niall Pemberton resolved BEANUTILS-23.
--------------------------------------

    Resolution: Fixed

> Misleading error message in ConvertingWrapDynaBean
> --------------------------------------------------
>
>                 Key: BEANUTILS-23
>                 URL: https://issues.apache.org/jira/browse/BEANUTILS-23
>             Project: Commons BeanUtils
>          Issue Type: Bug
>    Affects Versions: 1.7.0
>         Environment: Operating System: other
> Platform: All
>            Reporter: Aslak Helles√ły
>            Priority: Minor
>             Fix For: 1.8.0
>
>
> ConvertingWrapDynaBean.set(String,Object) swallows any throwable and rethrows a 
> (sometimes) misleading IllegalArgumentException without revealing the initial 
> cause.
> There should be more catch clauses in the method, possibly catching some 
> reflection exceptions. Anyway, the initial cause should be printStackTrace()d 
> or included in the IllegalArgumentException.
> I discovered this when I tried to set a property in a bean with jelly, and my 
> set method throws an exception:
> java.lang.IllegalArgumentException: Property 'expr' has no write method
>         at org.apache.commons.beanutils.ConvertingWrapDynaBean.set
> (ConvertingWrapDynaBean.java:116)
>         at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:275)
>         at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:135)
>         at org.apache.commons.jelly.impl.DynamicTag.doTag(DynamicTag.java:113)
>         at org.apache.commons.jelly.impl.StaticTagScript.run
> (StaticTagScript.java:151)
>         at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:135)
> (The setExpr(String) method is there allright, but it throws an exception if 
> the input is bad.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message