Author: sebb
Date: Mon Oct 20 06:39:17 2008
New Revision: 706279

URL: http://svn.apache.org/viewvc?rev=706279&view=rev
Log:
Trim spaces from While Controller condition before comparing against LAST, 
blank or false

Modified:
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/control/WhileController.java
    jakarta/jmeter/trunk/xdocs/changes.xml

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/control/WhileController.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/control/WhileController.java?rev=706279&r1=706278&r2=706279&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/control/WhileController.java 
(original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/control/WhileController.java 
Mon Oct 20 06:39:17 2008
@@ -53,7 +53,7 @@
      * @return true means OK to continue
      */
     private boolean endOfLoop(boolean loopEnd) {
-        String cnd = getCondition();
+        String cnd = getCondition().trim();
         log.debug("Condition string:" + cnd+".");
         boolean res;
         // If blank, only check previous sample when at end of loop

Modified: jakarta/jmeter/trunk/xdocs/changes.xml
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/xdocs/changes.xml?rev=706279&r1=706278&r2=706279&view=diff
==============================================================================
--- jakarta/jmeter/trunk/xdocs/changes.xml (original)
+++ jakarta/jmeter/trunk/xdocs/changes.xml Mon Oct 20 06:39:17 2008
@@ -88,6 +88,11 @@
 Moved the Scoping Rules sub-section from Section 3. "Building a Test Plan"  to 
Section 4. "Elements of a test plan"
 </p>
 
+<p>
+The While controller now trims leading and trailing spaces from the condition 
value before it is compared
+with LAST, blank or false.
+</p>
+
 <h3>Bug fixes</h3>
 <ul>
 <li>Bug 45199 - don't try to replace blank variables in Proxy recording</li>
@@ -138,6 +143,7 @@
 <li>Bug 43119 - Save Responses to file: optionally omit the file number</li>
 <li>Allow If Controller to use variable expressions (not just Javascript)</li>
 <li>Bug 45903 - allow Assertions to apply to sub-samples</li>
+<li>Trim spaces from While Controller condition before comparing against LAST, 
blank or false</li>
 </ul>
 
 <h3>Non-functional changes</h3>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to