You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sk...@apache.org on 2006/01/16 03:25:19 UTC
svn commit: r369311 -
/myfaces/impl/trunk/src/main/java/org/apache/myfaces/el/PropertyResolverImpl.java
Author: skitching
Date: Sun Jan 15 18:25:14 2006
New Revision: 369311
URL: http://svn.apache.org/viewcvs?rev=369311&view=rev
Log:
Improve error message
Modified:
myfaces/impl/trunk/src/main/java/org/apache/myfaces/el/PropertyResolverImpl.java
Modified: myfaces/impl/trunk/src/main/java/org/apache/myfaces/el/PropertyResolverImpl.java
URL: http://svn.apache.org/viewcvs/myfaces/impl/trunk/src/main/java/org/apache/myfaces/el/PropertyResolverImpl.java?rev=369311&r1=369310&r2=369311&view=diff
==============================================================================
--- myfaces/impl/trunk/src/main/java/org/apache/myfaces/el/PropertyResolverImpl.java (original)
+++ myfaces/impl/trunk/src/main/java/org/apache/myfaces/el/PropertyResolverImpl.java Sun Jan 15 18:25:14 2006
@@ -297,11 +297,18 @@
{
try
{
- if (base == null || property == null ||
- property instanceof String && ((String)property).length() == 0)
+ if (base == null)
{
- throw new PropertyNotFoundException("Bean is null");
+ throw new PropertyNotFoundException("Base bean is null.");
}
+ else if (property == null)
+ {
+ throw new PropertyNotFoundException("Property name is null.");
+ }
+ else if (property instanceof String && ((String)property).length() == 0)
+ {
+ throw new PropertyNotFoundException("Property name is an empty String.");
+ }
if (base instanceof Map)
{