Option to not open first found file automatically

[cpixl]

As asked in #13, I'm filling a new issue for this topic.
Maybe it would be better to have on option to not open the file when searching (and maybe it could be the default), as the user may be only looking for the existence of some search term on the project, and don't intend to lose focus from the current code (yeah, it's possible to go back to where it was by pressing <c-o>, but sometimes it's inconvenient to use it everytime when using ferret).

[sergei-dyshel]

Makes sense. Had to move to another plugin just because of this option.

[cpixl]

Recently I switched to CtrlSF. It gives me more options and Edit Mode (which is more user-friendly in most of the cases).

[wincent]

Wow, CtrlSF is pretty impressive.

I'm going to close this for now as it's not a pain point that I feel. If anybody wants to fix it, I think these are the options:

• Use dispatch.vim, which doesn't jump to the first match by default (this is what I do).
• If you don't want to use dispatch.vim, you could patch the cexpr call in Ferret to be cgetexpr instead (make it a configuration option if you want), and then use g:FerretQFHandler to implement any custom behavior that you want (like copen | wincmd p or whatever).

I'd be open to pull requests.

[ntnn]

I don't remember the code well, but I guess you set 'grepprg' and 'grepformat' - if so you'd just have to put an exclamation mark after the final grep.

[wincent]

Yeah @ntnn, something like that. I quickly hacked a sample up in 47ca7cd to show one way of doing it (too flakey to actually merge). We're using cexpr under the covers but switching to :grep would indeed enable a pretty straightforward switch to :grep!.

[sergei-dyshel]

Thanks for quick response!

[kriansa]

Hello @wincent,

Sorry to bump this closed issue, but is there something we could do to help to get 47ca7cd merged? Sometimes it's annoying to close the current buffer every time I do an :Ack and the result is not the first one listed.

Thanks for this great project.

[wincent]

@kriansa: yeah, I think we can merge something like that (will need to redo it it no longer applies cleanly). Lots of edge cases to deal with, but it will have to be something like this:

diff --git a/autoload/ferret/private/shared.vim b/autoload/ferret/private/shared.vim
index 14ad5a1..ff1d929 100644
--- a/autoload/ferret/private/shared.vim
+++ b/autoload/ferret/private/shared.vim
@@ -1,19 +1,43 @@
 " Copyright 2015-present Greg Hurrell. All rights reserved.
 " Licensed under the terms of the BSD 2-clause license.
 
+" Convenience function to get/sanitize the current g:FerretAutojump setting.
+function! s:autojump()
+  let l:autojump=get(g:, 'FerretAutojump', 1)
+  if l:autojump != 0 && l:autojump != 1 && l:autojump != 2
+    let l:autojump=0
+  endif
+  return l:autojump
+endfunction
+
 function! ferret#private#shared#finalize_search(output, ack)
   let l:original_errorformat=&errorformat
+  let l:autojump=s:autojump()
+  if a:ack
+    let l:prefix='c' " Will use cexpr, cgetexpr.
+    let l:handler=get(g:, 'FerretQFHandler', 'botright copen')
+    let l:post='qf'
+  else
+    let l:prefix='l' " Will use lexpr, lgetexpr.
+    let l:handler=get(g:, 'FerretLLHandler', 'lopen')
+    let l:post='location'
+  endif
   try
     let &errorformat=g:FerretFormat
-    if a:ack
-      call s:swallow('cexpr a:1', a:output)
-      execute get(g:, 'FerretQFHandler', 'botright cwindow')
-      call ferret#private#post('qf')
+    if l:autojump == 2 " Show listing and jump to first result.
+      call s:swallow(l:prefix . 'expr a:1', a:output)
     else
-      call s:swallow('lexpr a:1', a:output)
-      execute get(g:, 'FerretLLHandler', 'lwindow')
-      call ferret#private#post('location')
+      call s:swallow(l:prefix . 'getexpr a:1', a:output)
     endif
+    let l:before=winnr()
+    execute l:handler
+    if l:autojump != 1 " Show listing, but don't jump anywhere.
+      let l:after=winnr()
+      if l:before != l:after
+        execute l:before . 'wincmd w'
+      end
+    endif
+    call ferret#private#post(l:post)
   finally
     let &errorformat=l:original_errorformat
   endtry

[wincent]

Ok, so I pushed something to the "next" branch. Will let it bake there for a while before merging to master.

[kriansa]

Thank you for your quick response!