diff options
| author | w0rp <devw0rp@gmail.com> | 2017-07-27 00:45:25 +0100 |
|---|---|---|
| committer | w0rp <devw0rp@gmail.com> | 2017-07-27 00:45:25 +0100 |
| commit | fa33faad9ed6da220394bd2abec2e1bdcd19edba (patch) | |
| tree | 288fa6a1f8d2231d3e7b3b8e2915730906d6920e /ale_linters/ruby/brakeman.vim | |
| parent | db4d68eae75c071b2a6521fe8587102f5b781efe (diff) | |
| download | ale-fa33faad9ed6da220394bd2abec2e1bdcd19edba.zip | |
#810 - Handle output which is not JSON in many linters
Diffstat (limited to 'ale_linters/ruby/brakeman.vim')
| -rw-r--r-- | ale_linters/ruby/brakeman.vim | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/ale_linters/ruby/brakeman.vim b/ale_linters/ruby/brakeman.vim index 565c8c70..0070c15c 100644 --- a/ale_linters/ruby/brakeman.vim +++ b/ale_linters/ruby/brakeman.vim @@ -5,20 +5,10 @@ let g:ale_ruby_brakeman_options = \ get(g:, 'ale_ruby_brakeman_options', '') function! ale_linters#ruby#brakeman#Handle(buffer, lines) abort - if len(a:lines) == 0 - return [] - endif - - try - let l:result = json_decode(join(a:lines, '')) - catch /E474/ - " Ignore invalid JSON - return [] - endtry - let l:output = [] + let l:json = ale#util#FuzzyJSONDecode(a:lines, {}) - for l:warning in l:result.warnings + for l:warning in get(l:json, 'warnings', []) " Brakeman always outputs paths relative to the Rails app root let l:rails_root = ale#ruby#FindRailsRoot(a:buffer) let l:warning_file = l:rails_root . '/' . l:warning.file |