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 2005/11/25 01:37:31 UTC

svn commit: r348841 - /jakarta/jmeter/branches/rel-2-1/src/functions/org/apache/jmeter/functions/JavaScript.java

Author: sebb
Date: Thu Nov 24 16:37:27 2005
New Revision: 348841

URL: http://svn.apache.org/viewcvs?rev=348841&view=rev
Log:
Avoid NPE if running JavaScript on TestPlan element

Modified:
    jakarta/jmeter/branches/rel-2-1/src/functions/org/apache/jmeter/functions/JavaScript.java

Modified: jakarta/jmeter/branches/rel-2-1/src/functions/org/apache/jmeter/functions/JavaScript.java
URL: http://svn.apache.org/viewcvs/jakarta/jmeter/branches/rel-2-1/src/functions/org/apache/jmeter/functions/JavaScript.java?rev=348841&r1=348840&r2=348841&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-1/src/functions/org/apache/jmeter/functions/JavaScript.java (original)
+++ jakarta/jmeter/branches/rel-2-1/src/functions/org/apache/jmeter/functions/JavaScript.java Thu Nov 24 16:37:27 2005
@@ -81,8 +81,9 @@
 			Object result = cx.evaluateString(scope, script, "<cmd>", 1, null);
 
 			resultStr = Context.toString(result);
-			if (varName != null)
+			if (varName != null && vars != null) {// vars can be null if run from TestPlan
 				vars.put(varName, resultStr);
+            }
 
 		} catch (WrappedException e) {
 			log.error("Error processing Javascript", e);



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