search

joonty / vim-phpqa

PHP QA tools for Vim
http://www.joncairns.com
MIT License
290 stars 32 forks source link

issues when saving (phpcs and phpmd are run) #3

Closed tipu closed 12 years ago

tipu commented 12 years ago

I'm on a mac, my binaries for phpmd/phpcs are in the path. Whenever I save a file, I get this error at the bottom:

Error detected while processing function phpqa:RunAll..phpqa#PhpQaTools..phpqa#Init..91_AddSignsWrapper..91_AddSignsActual: line 38: E158: Invalid buffer name: ".expand("%:p")

Everything does work fine, but this disrupts the work flow as it requires an extra step whenever saving files. Let me know if any more details are needed.

stephpy commented 12 years ago

I had this problem too, but it should be fixed with

https://github.com/joonty/vim-phpqa/commit/0547ceb9e9428a54c39ad8e0ccade665f3a42723

Have you vim compiled with perl ? Your repository is updated ?

tipu commented 12 years ago

This is a stock VIM I got from either brew or macports. I am unsure if it was compiled with perl, however I have not done anything involving that.

joonty commented 12 years ago

Try running this from a terminal:

vim --version | grep perl

If you get nothing printed out then you haven't got perl compiled into your vim installation. Take a look at closed issue #1 for more info. You can uninstall your current vim and reinstall from ports using:

sudo port install vim +perl +python +tcl +ruby

Hope that helps! I use the perl version of this script, so the non-perl version will always be less well tested I'm afraid.

stephpy commented 12 years ago

I've not vim compiled with perl and the lasts commits should fix that, if vim is not compiled with perl, it'll not execute line which make error

E158: Invalid buffer name: ".expand("%:p")

joonty commented 12 years ago

Incidentally, I'm working on a new version that doesn't use Perl for marking the file with signs. This should work on the default Mac Vim. The code is sitting on the branch dev/improved, so feel free to pull it down and try it.