averello/vim-lsp-mirror
1
0
Fork 0
mirror of https://github.com/prabirshrestha/vim-lsp.git synced 2025-12-14 20:35:59 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix capability parsing when textDocumentSync.save is a boolean (#804)

Browse Source 
clangd recently started sending `true`; vim-lsp emits E715: Dictionary required
when preparing to save. A boolean is valid here:

https://microsoft.github.io/language-server-protocol/specifications/specification-current/#textDocument_didSave

> Server Capability:
>
> property name (optional): textDocumentSync.save
> property type: boolean | SaveOptions
This commit is contained in:
Sam McCall
2020-04-26 21:48:53 +02:00
committed by GitHub
parent e37f5e31f4
commit 386ee33c3c
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
autoload/lsp/capabilities.vim
View File

@@ -108,12 +108,11 @@ function! lsp#capabilities#get_text_document_save_registration_options(server_na
let l:capabilities = lsp#get_server_capabilities(a:server_name)
if !empty(l:capabilities) && has_key(l:capabilities, 'textDocumentSync')
if type(l:capabilities['textDocumentSync']) == type({})
if has_key(l:capabilities['textDocumentSync'], 'save')
return [1, {
\ 'includeText': has_key(l:capabilities['textDocumentSync']['save'], 'includeText') ? l:capabilities['textDocumentSync']['save']['includeText'] : 0,
\ }]
let l:save_options = get(l:capabilities['textDocumentSync'], 'save', 0)
if type(l:save_options) == type({})
return [1, {'includeText': get(save_options, 'includeText', 0)}]
else
return [0, { 'includeText': 0 }]
return [l:save_options ? 1 : 0, {'includeText': 0 }]
endif
else
return [1, { 'includeText': 0 }]