Skip to content

Commit a6c1d4e

Browse files
author
nsingh@pivotal.io
committed
PT 159667257 - Re-introduce hack to bypass LSP4E completion sorting
1 parent cca89dc commit a6c1d4e

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

headless-services/commons/commons-language-server/src/main/java/org/springframework/ide/vscode/commons/languageserver/completion/VscodeCompletionEngineAdapter.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@
3131
import org.slf4j.Logger;
3232
import org.slf4j.LoggerFactory;
3333
import org.springframework.ide.vscode.commons.languageserver.completion.DocumentEdits.TextReplace;
34+
import org.springframework.ide.vscode.commons.languageserver.util.LspClient;
35+
import org.springframework.ide.vscode.commons.languageserver.util.LspClient.Client;
3436
import org.springframework.ide.vscode.commons.languageserver.util.SimpleLanguageServer;
3537
import org.springframework.ide.vscode.commons.languageserver.util.SimpleTextDocumentService;
3638
import org.springframework.ide.vscode.commons.languageserver.util.SortKeys;
@@ -164,6 +166,12 @@ private Mono<CompletionList> getCompletionsMono(TextDocumentPositionParams param
164166
}
165167
}
166168
list.setItems(items);
169+
//This is a hack. It bypasses lsp4e filtering / sorting because it
170+
// it behaves totally bonkers. See: https://bugs.eclipse.org/bugs/show_bug.cgi?id=535823
171+
// See also https://www.pivotaltracker.com/story/show/159667257
172+
if (LspClient.currentClient()==Client.ECLIPSE) {
173+
list.setIsIncomplete(true);
174+
}
167175
return list;
168176
})
169177
.doOnNext(x -> log.info("Got {} completions", x.getItems().size()))

0 commit comments

Comments
 (0)