You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sc...@apache.org on 2005/08/01 20:53:52 UTC
svn commit: r226856 -
/myfaces/tomahawk/trunk/src/java/org/apache/myfaces/custom/datascroller/HtmlDataScroller.java
Author: schof
Date: Mon Aug 1 11:53:49 2005
New Revision: 226856
URL: http://svn.apache.org/viewcvs?rev=226856&view=rev
Log:
Patch for MYFACES-330 (Mike Kienenberger)
Modified:
myfaces/tomahawk/trunk/src/java/org/apache/myfaces/custom/datascroller/HtmlDataScroller.java
Modified: myfaces/tomahawk/trunk/src/java/org/apache/myfaces/custom/datascroller/HtmlDataScroller.java
URL: http://svn.apache.org/viewcvs/myfaces/tomahawk/trunk/src/java/org/apache/myfaces/custom/datascroller/HtmlDataScroller.java?rev=226856&r1=226855&r2=226856&view=diff
==============================================================================
--- myfaces/tomahawk/trunk/src/java/org/apache/myfaces/custom/datascroller/HtmlDataScroller.java (original)
+++ myfaces/tomahawk/trunk/src/java/org/apache/myfaces/custom/datascroller/HtmlDataScroller.java Mon Aug 1 11:53:49 2005
@@ -254,17 +254,18 @@
else
{
forComp = findComponent(forStr);
- if (forComp == null)
- {
- log.warn("could not find UIData referenced by attribute dataScroller@for = '"
- + forStr + "'");
- }
}
- if (!(forComp instanceof UIData))
+ if (forComp == null)
{
throw new IllegalArgumentException(
- "uiComponent referenced by attribute dataScroller@for must be of type "
- + UIData.class.getName());
+ "could not find UIData referenced by attribute dataScroller@for = '"
+ + forStr + "'");
+ }
+ else if (!(forComp instanceof UIData))
+ {
+ throw new IllegalArgumentException(
+ "uiComponent referenced by attribute dataScroller@for = '" + forStr + "' must be of type "
+ + UIData.class.getName() + ", not type " + forComp.getClass().getName());
}
return (UIData) forComp;
}