.mobile.erb registered with eruby_subtype does not indent html correctly about vim-ruby HOT 9 CLOSED

tpope: Following your suggestion from this closed issue in vim-rails, running :echo GetErubyIndent() returns -1

ref.: tpope/vim-rails#90 (comment)

from vim-ruby.

I've uninstalled every plugin but pathogen and vim-rails, and I still have the issue. I found a workaround though by reloading the buffer (:e!). Once I reload, indentation works (for this buffer).

from vim-ruby.

The issue seems to be with b:eruby_subtype_indent that is set to the default "indent(prevnonblank(v:lnum-1))" value, even if b:eruby_subtype is indeed "html".

Calling ":let v:lnum = line(".")|echo GetErubyIndent()" on a line does indeed return the correct indent value, even before reloading.

Once I :e!, b:eruby_subtype_indent it is correctly set to "HtmlIndentGet(v:lnum)"

from vim-ruby.

I suspect it's a funky ordering issue. Without getting too deep into that, here's a newer alternative you can try:

autocmd BufNewFile,BufRead *.mobile.erb set filetype=eruby.html

If it doesn't work, check if the 'filetype' is eruby.html or just eruby. That will provide further ordering clues.

from vim-ruby.

It worked, thanks. Let me know if you want to do some further debugging, else feel free to close the issue.

from vim-ruby.

Hmmm. It worked, but it broke other plugins (in this case, Syntastic) filetype detection. In this case Syntastic now detects these files as plain HTML, not as eruby. This might be an issue with Syntastic though.

from vim-ruby.

The other thing you could try is moving that first autocmd around in your vimrc to see if you can get the issue to go away that way. Other than that, I would ask @scrooloose what he thinks about the Syntastic issue.

from vim-ruby.

Moving the autocmd around (top/bottom/around the other autocmds, before/after pathogen's infection) did not work, but I tried to reset the filetype after setting the subtype and it did work, and did not break Syntastic.

Looks like the perfect workaround for now.

Thanks a lot for the support Tim.

autocmd BufNewFile,BufRead *.mobile.erb let b:eruby_subtype='html'
autocmd BufNewFile,BufRead *.mobile.erb set filetype=eruby

from vim-ruby.

awesome, thanks for that.

from vim-ruby.