[jira] [Commented] (WICKET-5692) Misleading message in PropertyValidator

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WICKET-5692?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14295388#comment-14295388 ] 

ASF GitHub Bot commented on WICKET-5692:
----------------------------------------

Github user tkruse closed the pull request at:

    https://github.com/apache/wicket/pull/77


> Misleading message in PropertyValidator
> ---------------------------------------
>
>                 Key: WICKET-5692
>                 URL: https://issues.apache.org/jira/browse/WICKET-5692
>             Project: Wicket
>          Issue Type: Improvement
>            Reporter: Thibault Kruse
>            Assignee: Martin Grigorov
>            Priority: Minor
>             Fix For: 6.18.0, 7.0.0-M4
>
>
> in org.apache.wicket.bean.validation.PropertyValidator.java:
> {code}
> if (property_ == null)
> 			{
> 				throw new IllegalStateException(
> 					"Could not resolve Property from component: " +
> 						component +
> 						". Either specify the Property in the constructor or use a model that works in combination with a " +
> 						IPropertyResolver.class.getSimpleName() +
> 						" to resolve the Property automatically");
> 			}
> {code}
> The error message is misleading, as other causes might lead to a result of null. As PropertyResolvers can be of any type, the message should not assume any given cause, but may hint at common errors.
> In particular in my case when Using a bean Foo having a field bar of Type Bar, and a Model on an instance foo of Foo having null for field bar.
> Instead, it would be nice if the Error Message made an attempt to cast the component model to AbstractPropertyModel and print the propertyExpression, as well as the type of the modelObject, but without failure if the cast is not possible, and reporting the modelObject is null if so.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)