On Thu, Oct 02 2014, Ian Main <imain@stemwinder.org> wrote:

> This patch adds URI handling to the vim client.  You can now press
> 'u' by default and the client will parse the current line and find
> any URIs available.  If there are more than one it opens the one
> under the cursor or else it opens the only one available.  It also
> supports mailto: URI's and will compose a new message when activated.
>
> By default xdg-open is used for everything but mailto: which generally
> does the right thing afaict.

This patch LGTM.

>
>     Ian

Tomi

> ---
>  vim/notmuch.txt |  1 +
>  vim/notmuch.vim | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++-------
>  2 files changed, 53 insertions(+), 7 deletions(-)
>
> diff --git a/vim/notmuch.txt b/vim/notmuch.txt
> index 4374102..d336406 100644
> --- a/vim/notmuch.txt
> +++ b/vim/notmuch.txt
> @@ -72,6 +72,7 @@ q	Quit view
>  A	Archive (-inbox -unread)
>  I	Mark as read (-unread)
>  t	Tag (prompted)
> +u	Open URI
>  s	Search
>  p	Save patches
>  r	Reply
> diff --git a/vim/notmuch.vim b/vim/notmuch.vim
> index 331e930..de82bb9 100644
> --- a/vim/notmuch.vim
> +++ b/vim/notmuch.vim
> @@ -12,7 +12,7 @@ let g:notmuch_folders_maps = {
>  	\ '<Enter>':	'folders_show_search()',
>  	\ 's':		'folders_search_prompt()',
>  	\ '=':		'folders_refresh()',
> -	\ 'c':		'compose()',
> +	\ 'c':		'compose("")',
>  	\ }
>  
>  let g:notmuch_search_maps = {
> @@ -25,7 +25,7 @@ let g:notmuch_search_maps = {
>  	\ 's':		'search_search_prompt()',
>  	\ '=':		'search_refresh()',
>  	\ '?':		'search_info()',
> -	\ 'c':		'compose()',
> +	\ 'c':		'compose("")',
>  	\ }
>  
>  let g:notmuch_show_maps = {
> @@ -37,10 +37,11 @@ let g:notmuch_show_maps = {
>  	\ 'e':		'show_extract_msg()',
>  	\ 's':		'show_save_msg()',
>  	\ 'p':		'show_save_patches()',
> +	\ 'u':		'show_open_uri()',
>  	\ 'r':		'show_reply()',
>  	\ '?':		'show_info()',
>  	\ '<Tab>':	'show_next_msg()',
> -	\ 'c':		'compose()',
> +	\ 'c':		'compose("")',
>  	\ }
>  
>  let g:notmuch_compose_maps = {
> @@ -59,6 +60,7 @@ let s:notmuch_datetime_format_default = '%d.%m.%y %H:%M:%S'
>  let s:notmuch_reader_default = 'mutt -f %s'
>  let s:notmuch_sendmail_default = 'sendmail'
>  let s:notmuch_folders_count_threads_default = 0
> +let s:notmuch_open_uri_default = 'xdg-open'
>  
>  function! s:new_file_buffer(type, fname)
>  	exec printf('edit %s', a:fname)
> @@ -135,8 +137,8 @@ function! s:show_reply()
>  	startinsert!
>  endfunction
>  
> -function! s:compose()
> -	ruby open_compose
> +function! s:compose(to_email)
> +	ruby open_compose(VIM::evaluate('a:to_email'))
>  	let b:compose_done = 0
>  	call s:set_map(g:notmuch_compose_maps)
>  	autocmd BufDelete <buffer> call s:on_compose_delete()
> @@ -159,6 +161,45 @@ ruby << EOF
>  EOF
>  endfunction
>  
> +function! s:show_open_uri()
> +	let line = getline(".")
> +	let pos = getpos(".")
> +	let col = pos[2]
> +ruby << EOF
> +	m = get_message
> +	line = VIM::evaluate('line')
> +	col = VIM::evaluate('col') - 1
> +	uris = URI.extract(line)
> +	wanted_uri = nil
> +	if uris.length == 1
> +		wanted_uri = uris[0]
> +	else
> +		uris.each do |uri|
> +			# Check to see the URI is at the present cursor location
> +			idx = line.index(uri)
> +			if col >= idx and col <= idx + uri.length
> +				wanted_uri = uri
> +				break
> +			end
> +		end
> +	end
> +
> +	if wanted_uri
> +		uri = URI.parse(wanted_uri)
> +		if uri.class == URI::MailTo
> +			vim_puts("Composing new email to #{uri.to}.")
> +			VIM::command("call s:compose('#{uri.to}')")
> +		else
> +			vim_puts("Opening #{uri.to_s}.")
> +			cmd = VIM::evaluate('g:notmuch_open_uri')
> +			system(cmd, uri.to_s)
> +		end
> +	else
> +		vim_puts('URI not found.')
> +	end
> +EOF
> +endfunction
> +
>  function! s:show_open_msg()
>  ruby << EOF
>  	m = get_message
> @@ -404,6 +445,10 @@ function! s:set_defaults()
>  		endif
>  	endif
>  
> +	if !exists('g:notmuch_open_uri')
> +		let g:notmuch_open_uri = s:notmuch_open_uri_default
> +	endif
> +
>  	if !exists('g:notmuch_reader')
>  		if exists('g:notmuch_rb_reader')
>  			let g:notmuch_reader = g:notmuch_rb_reader
> @@ -611,11 +656,11 @@ ruby << EOF
>  		open_compose_helper(lines, cur)
>  	end
>  
> -	def open_compose()
> +	def open_compose(to_email)
>  		lines = []
>  
>  		lines << "From: #{$email}"
> -		lines << "To: "
> +		lines << "To: #{to_email}"
>  		cur = lines.count
>  
>  		lines << "Cc: "
> -- 
> 1.9.3
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch