Merge pull request #568 from lcd047/phpmd_syntax
Syntax highlighting function for phpmd
This commit is contained in:
commit
5117e76f99
@ -22,12 +22,48 @@ function! SyntaxCheckers_php_phpmd_IsAvailable()
|
|||||||
return executable('phpmd')
|
return executable('phpmd')
|
||||||
endfunction
|
endfunction
|
||||||
|
|
||||||
|
function! SyntaxCheckers_php_phpmd_GetHighlightRegex(item)
|
||||||
|
let term = matchstr(a:item['text'], '\C^The \S\+ \w\+\(()\)\= \(has\|is not\|utilizes\)')
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, '\C^The \S\+ \(\w\+\)\(()\)\= .*', '\1', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], '\C^Avoid \(variables with short\|excessively long variable\) names like \S\+\.')
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, '\C^Avoid \(variables with short\|excessively long variable\) names like \(\S\+\)\..*', '\2', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], '\C^Avoid using short method names like \S\+::\S\+()\.')
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, '\C^Avoid using short method names like \S\+::\(\S\+\)()\..*', '\1', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], '\C^\S\+ accesses the super-global variable ')
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, '\C accesses the super-global variable .*$', '', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], '\C^Constant \S\+ should be defined in uppercase')
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, '\C^Constant \(\S\+\) should be defined in uppercase', '\1', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], "\\C^The '\\S\\+()' method which returns ")
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, "\\C^The '\\(\\S\\+\\()' method which returns.*", '\1', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], '\C variable \S\+ should begin with ')
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, '\C.* variable \(\S\+\) should begin with .*', '\1', '')
|
||||||
|
endif
|
||||||
|
let term = matchstr(a:item['text'], "\\C^Avoid unused \\(private fields\\|local variables\\|private methods\\|parameters\\) such as '\\S\\+'")
|
||||||
|
if term != ''
|
||||||
|
return '\V'.substitute(term, "\\C^Avoid unused \\(private fields\\|local variables\\|private methods\\|parameters\\) such as '\\(\\S\\+\\)'.*", '\2', '')
|
||||||
|
endif
|
||||||
|
return ''
|
||||||
|
endfunction
|
||||||
|
|
||||||
function! SyntaxCheckers_php_phpmd_GetLocList()
|
function! SyntaxCheckers_php_phpmd_GetLocList()
|
||||||
let makeprg = syntastic#makeprg#build({
|
let makeprg = syntastic#makeprg#build({
|
||||||
\ 'exe': 'phpmd',
|
\ 'exe': 'phpmd',
|
||||||
\ 'post_args': 'text codesize,design,unusedcode,naming',
|
\ 'post_args': 'text codesize,design,unusedcode,naming',
|
||||||
\ 'subchecker': 'phpmd' })
|
\ 'subchecker': 'phpmd' })
|
||||||
let errorformat = '%E%f:%l%m'
|
let errorformat = '%E%f:%l%\s%#%m'
|
||||||
return SyntasticMake({ 'makeprg': makeprg, 'errorformat': errorformat, 'subtype' : 'Style' })
|
return SyntasticMake({ 'makeprg': makeprg, 'errorformat': errorformat, 'subtype' : 'Style' })
|
||||||
endfunction
|
endfunction
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user