patch 9.1.1380: 'eventignorewin' only checked for current buffer
Problem: When an autocommand executes for a non-current buffer,
'eventignorewin' is only checked from the buffer's last
wininfo (overwrites win_ignore in the loop), not from the
value of 'eventignorewin' in all windows showing the buffer as
described (after v9.1.1084)
Solution: Fix the check and don't use wininfo, as that may only contain
windows that recently showed the buffer. Consider all the
buffer's windows in all tabpages (Sean Dewar).
closes: #17294
Signed-off-by: Sean Dewar <6256228+seandewar@users.noreply.github.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
diff --git a/src/testdir/test_autocmd.vim b/src/testdir/test_autocmd.vim
index 192c436..db55583 100644
--- a/src/testdir/test_autocmd.vim
+++ b/src/testdir/test_autocmd.vim
@@ -5314,4 +5314,80 @@
bw!
endfunc
+func Test_eventignorewin_non_current()
+ defer CleanUpTestAuGroup()
+ let s:triggered = ''
+ augroup testing
+ " Will set <abuf> to the buffer of the closing window.
+ autocmd WinClosed * let s:triggered = 'WinClosed'
+ augroup END
+ let initial_win = win_getid()
+
+ new
+ let new_buf = bufnr()
+ " Only set for one of the windows into the new buffer.
+ setlocal eventignorewin=all
+ split
+ setlocal eventignorewin=
+ let close_winnr = winnr()
+
+ " Return to the window where the buffer is non-current. WinClosed should
+ " trigger as not all windows into new_buf have 'eventignorewin' set for it.
+ call win_gotoid(initial_win)
+ call assert_notequal(new_buf, bufnr())
+ execute close_winnr 'close'
+ call assert_equal('WinClosed', s:triggered)
+
+ wincmd w
+ call assert_equal(new_buf, bufnr())
+ tab split
+ setlocal eventignorewin=
+ let close_winnr = win_getid()
+
+ " Ensure that new_buf's window in the other tabpage with 'eventignorewin'
+ " unset allows WinClosed to run when new_buf is non-current.
+ call win_gotoid(initial_win)
+ call assert_notequal(new_buf, bufnr())
+ let s:triggered = ''
+ only!
+ call assert_equal('WinClosed', s:triggered)
+ call assert_equal(1, win_findbuf(new_buf)->len())
+
+ " Create an only window to new_buf with 'eventignorewin' set.
+ tabonly!
+ execute new_buf 'sbuffer'
+ setlocal eventignorewin=all
+ wincmd p
+ call assert_equal(1, win_findbuf(new_buf)->len())
+ call assert_notequal(new_buf, bufnr())
+
+ " Closing a window unrelated to new_buf should not block WinClosed.
+ split
+ let s:triggered = ''
+ close
+ call assert_equal('WinClosed', s:triggered)
+ call assert_equal(1, win_findbuf(new_buf)->len())
+
+ " Check WinClosed is blocked when we close the only window to new_buf (that
+ " has 'eventignorewin' set) while new_buf is non-current.
+ call assert_notequal(new_buf, bufnr())
+ let s:triggered = ''
+ only!
+ call assert_equal('', s:triggered)
+ call assert_equal(0, win_findbuf(new_buf)->len())
+
+ augroup testing
+ autocmd!
+ autocmd BufNew * ++once let s:triggered = 'BufNew'
+ augroup END
+
+ " Buffer not shown in a window, 'eventignorewin' should not block (and
+ " can't even be set for it anyway in this case).
+ badd foo
+ call assert_equal('BufNew', s:triggered)
+
+ unlet! s:triggered
+ %bw!
+endfunc
+
" vim: shiftwidth=2 sts=2 expandtab