vim/ale
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #1222 from srathbone/master

Browse Source 
Fix PHP linter to support PHP 7.2 lint output
This commit is contained in:
w0rp 2017-12-17 12:41:28 +00:00 committed by GitHub
commit fc151d7b30
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ale_linters/php/php.vim
View File

@ -4,12 +4,14 @@
function! ale_linters#php#php#Handle(buffer, lines) abort
" Matches patterns like the following:
"
" Parse error: syntax error, unexpected ';', expecting ']' in - on line 15
let l:pattern = '\v^%(Fatal|Parse) error:\s+(.+unexpected ''(.+)%(expecting.+)@<!''.*|.+) in - on line (\d+)'
" PHP 7.1<= - Parse error: syntax error, unexpected ';', expecting ']' in - on line 15
" PHP 7.2>= - Parse error: syntax error, unexpected ';', expecting ']' in Standard input code on line 15
let l:pattern = '\v^%(Fatal|Parse) error:\s+(.+unexpected ''(.+)%(expecting.+)@<!''.*|.+) in %(-|Standard input code) on line (\d+)'
let l:output = []
for l:match in ale#util#GetMatches(a:lines, l:pattern)
let l:col = empty(l:match[2]) ? 0 : stridx(getline(l:match[3]), l:match[2]) + 1
let l:obj = {
\ 'lnum': l:match[3] + 0,
\ 'col': l:col,

12
test/handler/test_php_handler.vader
View File

@ -54,6 +54,18 @@ Execute (The php handler should ignore lines starting with 'PHP Parse error'):
\ "PHP Parse error: syntax error, This line should be ignored completely in - on line 1",
\ ])
Execute (The php handler should handle lines containing 'Standard input code'):
AssertEqual
\ [
\ {
\ 'lnum': 47,
\ 'col': 0,
\ 'text': "Invalid numeric literal",
\ },
\ ],
\ ale_linters#php#php#Handle(347, [
\ "Parse error: Invalid numeric literal in Standard input code on line 47",
\ ])
Execute (The php handler should parse lines without column indication):
AssertEqual
\ [