svn commit: r1516146 - /jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java

[pm...@apache.org]

Author: pmouawad
Date: Wed Aug 21 12:32:53 2013
New Revision: 1516146

URL: http://svn.apache.org/r1516146
Log:
Bug 55459 - Elements using ComboStringEditor lose the input value if user selects another Test Element
Bug 55103 - why does the focus listener now get called after modifyTestElement?
Bugzilla Id: 55459

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java?rev=1516146&r1=1516145&r2=1516146&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java Wed Aug 21 12:32:53 2013
@@ -144,6 +144,11 @@ public class JMeterTreeListener implemen
     public void valueChanged(TreeSelectionEvent e) {
         log.debug("value changed, updating currentPath");
         currentPath = e.getNewLeadSelectionPath();
+        // Call requestFocusInWindow to ensure current component loses focus and
+        // all values are correctly saved
+        // see https://issues.apache.org/bugzilla/show_bug.cgi?id=55103
+        // see https://issues.apache.org/bugzilla/show_bug.cgi?id=55459
+        tree.requestFocusInWindow();
         actionHandler.actionPerformed(new ActionEvent(this, 3333, ActionNames.EDIT)); // $NON-NLS-1$
     }