summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorw0rp <devw0rp@gmail.com>2017-08-08 07:29:10 +0100
committerw0rp <devw0rp@gmail.com>2017-08-08 07:29:10 +0100
commit5010ddc28fdced941925563bbc55f9219dfa4267 (patch)
tree6393065da7a6176db5d305563c9bd919ba6dfc62
parent16cfedf04a46a3d498dfd4f7293c555562d0776e (diff)
downloadale-5010ddc28fdced941925563bbc55f9219dfa4267.zip
Fix #833 - Do not open windows on save when the option is off
-rw-r--r--autoload/ale/list.vim2
-rw-r--r--test/test_list_opening.vader8
2 files changed, 9 insertions, 1 deletions
diff --git a/autoload/ale/list.vim b/autoload/ale/list.vim
index f7c8ddd6..ae13e10a 100644
--- a/autoload/ale/list.vim
+++ b/autoload/ale/list.vim
@@ -18,7 +18,7 @@ function! s:ShouldOpen(buffer) abort
let l:saved = getbufvar(a:buffer, 'ale_save_event_fired', 0)
return (type(l:val) == type(1) && l:val == 1)
- \ || (l:val ==# 'on_save' && l:saved)
+ \ || (type(l:val) == type('') && l:val ==# 'on_save' && l:saved)
endfunction
function! ale#list#SetLists(buffer, loclist) abort
diff --git a/test/test_list_opening.vader b/test/test_list_opening.vader
index 253efa46..024e55ae 100644
--- a/test/test_list_opening.vader
+++ b/test/test_list_opening.vader
@@ -197,3 +197,11 @@ Execute(The ale_open_list='on_save' option should work):
call ale#list#SetLists(bufnr('%'), g:loclist)
" Now the list should have opened.
Assert ale#list#IsQuickfixOpen()
+
+Execute(The window shouldn't open on save when ale_open_list=0):
+ let b:ale_open_list = 0
+ let b:ale_save_event_fired = 1
+
+ call ale#list#SetLists(bufnr('%'), g:loclist)
+ " Now the list should have opened.
+ Assert !ale#list#IsQuickfixOpen()