diff options
author | Adriaan Zonnenberg <amz@adriaan.xyz> | 2017-02-15 23:59:18 +0100 |
---|---|---|
committer | Adriaan Zonnenberg <amz@adriaan.xyz> | 2017-02-15 23:59:18 +0100 |
commit | b2241e991bee98e5a085384061bd5e2de4026afc (patch) | |
tree | 3a1ed5c3404611846c38ed37c7ffe04e6794958a /ale_linters/php | |
parent | 2e7050dbe2cdce030ed2f4189b9a904e67b352af (diff) | |
download | ale-b2241e991bee98e5a085384061bd5e2de4026afc.zip |
Support columns on php handler
Also added some tests for different error messages
Diffstat (limited to 'ale_linters/php')
-rw-r--r-- | ale_linters/php/php.vim | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/ale_linters/php/php.vim b/ale_linters/php/php.vim index b807fee9..1127b94a 100644 --- a/ale_linters/php/php.vim +++ b/ale_linters/php/php.vim @@ -4,8 +4,9 @@ function! ale_linters#php#php#Handle(buffer, lines) abort " Matches patterns like the following: " - " Parse error: parse error in - on line 7 - let l:pattern = 'Parse error:\s\+\(.\+\) on line \(\d\+\)' + " PHP Parse error: syntax error, unexpected ';', expecting ']' in - on line 15 + let l:pattern = ':\s\+\(.\+unexpected ''\(.\{-}\)''.*\) in - on line \(\d\+\)' + let l:output = [] for l:line in a:lines @@ -18,9 +19,9 @@ function! ale_linters#php#php#Handle(buffer, lines) abort " vcol is needed to indicate that the column is a character. call add(l:output, { \ 'bufnr': a:buffer, - \ 'lnum': l:match[2] + 0, + \ 'lnum': l:match[3] + 0, \ 'vcol': 0, - \ 'col': 1, + \ 'col': match(getline(l:match[3]), l:match[2]) + 1, \ 'text': l:match[1], \ 'type': 'E', \ 'nr': -1, |