You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by ma...@apache.org on 2006/11/23 09:57:33 UTC

svn commit: r478515 - /myfaces/core/trunk/api/src/main/java/javax/faces/component/_SelectItemsIterator.java

Author: manolito
Date: Thu Nov 23 00:57:32 2006
New Revision: 478515

URL: http://svn.apache.org/viewvc?view=rev&rev=478515
Log:
MYFACES-1420 "Null Pointer Exception in SelectItemsIterator.next() if binding is null" resolved

Modified:
    myfaces/core/trunk/api/src/main/java/javax/faces/component/_SelectItemsIterator.java

Modified: myfaces/core/trunk/api/src/main/java/javax/faces/component/_SelectItemsIterator.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/component/_SelectItemsIterator.java?view=diff&rev=478515&r1=478514&r2=478515
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/component/_SelectItemsIterator.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/component/_SelectItemsIterator.java Thu Nov 23 00:57:32 2006
@@ -173,7 +173,8 @@
                                 .getValueBinding("value");
                 throw new IllegalArgumentException(
                 _collectionLabel + " referenced by UISelectItems with binding '"
-                + binding.getExpressionString()
+                + (binding == null ? null : binding
+                                        .getExpressionString())
                 + "' and Component-Path : " + getPathToComponent(_currentUISelectItems)
                 + " does not contain Objects of type SelectItem");
             }