You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by iv...@apache.org on 2007/11/12 06:17:31 UTC

svn commit: r594023 - /wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java

Author: ivaynberg
Date: Sun Nov 11 21:17:20 2007
New Revision: 594023

URL: http://svn.apache.org/viewvc?rev=594023&view=rev
Log:
better error message

Modified:
    wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java?rev=594023&r1=594022&r2=594023&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/lang/PropertyResolver.java Sun Nov 11 21:17:20 2007
@@ -116,8 +116,9 @@
 		}
 		if (object == null)
 		{
-			throw new WicketRuntimeException("Null object setting value: " + value +
-				" with expression: " + expression);
+			throw new WicketRuntimeException(
+				"Attempted to set property value on a null object. Property expression: " +
+					expression + " Value: " + value);
 		}
 
 		ObjectAndGetSetter setter = getObjectAndGetSetter(expression, object, CREATE_NEW_VALUE);
@@ -223,8 +224,7 @@
 				index = getNextDotIndex(expressionBracketsSeperated, index + 1);
 				if (index != -1)
 				{
-					String indexExpression = expressionBracketsSeperated
-						.substring(lastIndex, index);
+					String indexExpression = expressionBracketsSeperated.substring(lastIndex, index);
 					getAndSetter = getGetAndSetter(indexExpression, clz);
 				}
 				else