This is an automated email from the ASF dual-hosted git repository.

fschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git


The following commit(s) were added to refs/heads/master by this push:
     new b8c35f1  Blank comment tooltip is visible
b8c35f1 is described below

commit b8c35f1568b98bef2cc08665ed04da24d9ac329f
Author: Felix Schumacher <[email protected]>
AuthorDate: Tue Dec 8 18:07:18 2020 +0100

    Blank comment tooltip is visible
    
    Bugzilla Id: 64963
---
 src/core/src/main/java/org/apache/jmeter/gui/MainFrame.java | 4 ++--
 xdocs/changes.xml                                           | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/core/src/main/java/org/apache/jmeter/gui/MainFrame.java 
b/src/core/src/main/java/org/apache/jmeter/gui/MainFrame.java
index 39d8e81..9e998eb 100644
--- a/src/core/src/main/java/org/apache/jmeter/gui/MainFrame.java
+++ b/src/core/src/main/java/org/apache/jmeter/gui/MainFrame.java
@@ -668,13 +668,13 @@ public class MainFrame extends JFrame implements 
TestStateListener, Remoteable,
                         Object testElement = ((DefaultMutableTreeNode) 
treeNode).getUserObject();
                         if (testElement instanceof TestElement) {
                             String comment = ((TestElement) 
testElement).getComment();
-                            if (comment != null && comment.length() > 0) {
+                            if (StringUtils.isNotBlank(comment)) {
                                 return comment;
                             }
                         }
                     }
                 }
-                return "";
+                return null;
             }
         };
         treevar.setToolTipText("");
diff --git a/xdocs/changes.xml b/xdocs/changes.xml
index fe96cc0..538184a 100644
--- a/xdocs/changes.xml
+++ b/xdocs/changes.xml
@@ -163,6 +163,7 @@ Summary
 <ul>
   <li><bug>64957</bug>When importing example test plan JMeter displays an 
NullPointerException</li>
   <li><bug>64961</bug>Darklaf: On Windows 7, NPE in 
BasicEditorPaneUI.cleanDisplayProperties with Darklaf Intellij</li>
+  <li><bug>64963</bug>Blank comment tooltip is visible</li>
 </ul>
 
  <!--  =================== Thanks =================== -->

Reply via email to