forked from github-mirrors/ale
Fix crash for Reek 6.0.4 (#4010)
When `let g:ale_ruby_reek_show_wiki_link = 1`, Reek linter is crashed because `wiki_link` attribute does not exist in result. It seems that `wiki_link` is now replaced with `documentation_link` in recent version of Reek
This commit is contained in:
parent
091592bfb0
commit
e4ec2e4dc7
|
@ -19,6 +19,10 @@ function! ale_linters#ruby#reek#GetCommand(buffer, version) abort
|
|||
\ . l:display_name_args
|
||||
endfunction
|
||||
|
||||
function! s:GetDocumentationLink(error) abort
|
||||
return get(a:error, 'documentation_link', get(a:error, 'wiki_link', ''))
|
||||
endfunction
|
||||
|
||||
function! s:BuildText(buffer, error) abort
|
||||
let l:parts = []
|
||||
|
||||
|
@ -29,7 +33,7 @@ function! s:BuildText(buffer, error) abort
|
|||
call add(l:parts, a:error.message)
|
||||
|
||||
if ale#Var(a:buffer, 'ruby_reek_show_wiki_link')
|
||||
call add(l:parts, '[' . a:error.wiki_link . ']')
|
||||
call add(l:parts, '[' . s:GetDocumentationLink(a:error) . ']')
|
||||
endif
|
||||
|
||||
return join(l:parts, ' ')
|
||||
|
|
Loading…
Reference in New Issue