Auto append brackets when auto complete function in DEBUG CONSOLE view (microsoft#308)

testforstephen · web-flow · commit fc45056fa03c · 2019-11-13T16:21:41.000+08:00
Signed-off-by: Jinbo Wang &lt;jinbwan@microsoft.com&gt;
diff --git a/com.microsoft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/CompletionProposalRequestor.java b/com.microsoft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/CompletionProposalRequestor.java
@@ -21,6 +21,7 @@
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
+import org.apache.commons.lang3.StringUtils;
 import org.eclipse.core.runtime.CoreException;
 import org.eclipse.core.runtime.IPath;
 import org.eclipse.jdt.core.CompletionContext;
@@ -152,10 +153,20 @@ public CompletionItem toCompletionItem(CompletionProposal proposal, int index) {
         data.put(CompletionResolveHandler.DATA_FIELD_PROPOSAL_ID, String.valueOf(index));
         $.setData(data);
         this.descriptionProvider.updateDescription(proposal, $);
+        adjustCompleteItem($);
         $.setSortText(SortTextHelper.computeSortText(proposal));
         return $;
     }
 
+    private void adjustCompleteItem(CompletionItem item) {
+        if (item.getKind() == CompletionItemKind.Function) {
+            String text = item.getInsertText();
+            if (StringUtils.isNotBlank(text) && !text.endsWith(")")) {
+                item.setInsertText(text + "()");
+            }
+        }
+    }
+
     @Override
     public void acceptContext(CompletionContext context) {
         super.acceptContext(context);
