You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by gi...@apache.org on 2005/03/22 07:07:42 UTC
svn commit: r158545 -
cocoon/whiteboard/forms/java/org/apache/cocoon/forms/binding/MultiValueJXPathBinding.java
Author: giacomo
Date: Mon Mar 21 22:07:40 2005
New Revision: 158545
URL: http://svn.apache.org/viewcvs?view=rev&rev=158545
Log:
fixing a NPE
Modified:
cocoon/whiteboard/forms/java/org/apache/cocoon/forms/binding/MultiValueJXPathBinding.java
Modified: cocoon/whiteboard/forms/java/org/apache/cocoon/forms/binding/MultiValueJXPathBinding.java
URL: http://svn.apache.org/viewcvs/cocoon/whiteboard/forms/java/org/apache/cocoon/forms/binding/MultiValueJXPathBinding.java?view=diff&r1=158544&r2=158545
==============================================================================
--- cocoon/whiteboard/forms/java/org/apache/cocoon/forms/binding/MultiValueJXPathBinding.java (original)
+++ cocoon/whiteboard/forms/java/org/apache/cocoon/forms/binding/MultiValueJXPathBinding.java Mon Mar 21 22:07:40 2005
@@ -108,7 +108,10 @@
public boolean createObject(JXPathContext context, Pointer pointer,
Object parent, String name, int index) {
final Object o = context.getValue(name);
- if( Collection.class.isAssignableFrom( o.getClass() ) ) {
+ if( o == null ) {
+ return false;
+ }
+ if( Collection.class.isAssignableFrom( o.getClass() ) ) {
((Collection)context.getValue(name)).add(null);
} else if( o.getClass().isArray() ) {
// not yet supported