LSP: Don't override insertText with newText if insertText is not empty

Waqar Ahmedrequested to merge
work/lsp-comp-newtext into master

newText can contain a value that makes no sense if it is not used in combination with the textEdit.range. E.g.,

string.length vs string..length

server sent .length as completion..

Merge request reports