Bug: Clipboard history is empty when using %^C in capture template [9.0.5 (9.0.5-elpaplus @ /home/mir/.emacs.d/elpa/org-plus-contrib-20170210/)]

Bug: Clipboard history is empty when using %^C in capture template [9.0.5 (9.0.5-elpaplus @ /home/mir/.emacs.d/elpa/org-plus-contrib-20170210/)]

[Allen Li]

Using the %^C expansion in a capture template brings up a minibuffer
prompt to read/select a clipboard value, but there are no values to
select from.

The relevant code is in the function org-capture-fill-template.

 ("C"
  (cond
   ((= (length clipboards) 1) (insert (car clipboards)))
   ((> (length clipboards) 1)
    (insert (read-string "Clipboard/kill value: "
                         (car clipboards)
                         '(clipboards . 1)
                         (car clipboards))))))

It looks like the intention is to put the list of possible clipboard
values into the minibuffer history for read-string, but in practice I
get an error saying the history is empty.

I suspect that the problem is that clipboard is lexically bound, and
since read-string takes a symbol for the history, it cannot get the
lexical value for clipboards from the symbol argument.

Emacs  : GNU Emacs 25.1.1 (x86_64-unknown-linux-gnu, GTK+ Version 3.22.7)
 of 2017-02-07
Package: Org mode version 9.0.5 (9.0.5-elpaplus @ /home/mir/.emacs.d/elpa/org-plus-contrib-20170210/)

Re: Bug: Clipboard history is empty when using %^C in capture template [9.0.5 (9.0.5-elpaplus @ /home/mir/.emacs.d/elpa/org-plus-contrib-20170210/)]

[Nicolas Goaziou]

Hello,

Allen Li <darkfeline@felesatra.moe> writes:

> Using the %^C expansion in a capture template brings up a minibuffer
> prompt to read/select a clipboard value, but there are no values to
> select from.
>
> The relevant code is in the function org-capture-fill-template.
>
>  ("C"
>   (cond
>    ((= (length clipboards) 1) (insert (car clipboards)))
>    ((> (length clipboards) 1)
>     (insert (read-string "Clipboard/kill value: "
>                          (car clipboards)
>                          '(clipboards . 1)
>                          (car clipboards))))))
>
> It looks like the intention is to put the list of possible clipboard
> values into the minibuffer history for read-string, but in practice I
> get an error saying the history is empty.
>
> I suspect that the problem is that clipboard is lexically bound, and
> since read-string takes a symbol for the history, it cannot get the
> lexical value for clipboards from the symbol argument.

Thank you for the report.

I think I fixed it in maint. Could you confirm it?

Regards,

-- 
Nicolas Goaziou