You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/06/09 18:57:47 UTC

svn commit: r545777 - /jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/extractor/XPathExtractor.java

Author: sebb
Date: Sat Jun  9 09:57:46 2007
New Revision: 545777

URL: http://svn.apache.org/viewvc?view=rev&rev=545777
Log:
Avoid NPE if node has no childred

Modified:
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/extractor/XPathExtractor.java

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/extractor/XPathExtractor.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/extractor/XPathExtractor.java?view=diff&rev=545777&r1=545776&r2=545777
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/extractor/XPathExtractor.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/extractor/XPathExtractor.java Sat Jun  9 09:57:46 2007
@@ -195,9 +195,13 @@
 	        for (int i = 0 ; i < length; i++) {
 	            Node match = matches.item(i);
 				if ( match instanceof Element){
-				   // elements have empty nodeValue, but we are usually
-				   // interested in their content
-				   val = match.getFirstChild().getNodeValue();
+				// elements have empty nodeValue, but we are usually interested in their content
+				   final Node firstChild = match.getFirstChild();
+				   if (firstChild != null) {
+					   val = firstChild.getNodeValue();
+				   } else {
+					   val = match.getNodeValue(); // TODO is this correct?
+				   }
 				} else {				
 				   val = match.getNodeValue();
 				}



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org