Ryo TAKAISHI <[email protected]> writes:
> I fixed and added document in docstring & org.texi.
Thank you.
> + @r{If sexp's attr is link keyword (@code{%:keyword}),}
> + @r{it will be expanded using @code{org-store-link-plist}.}
Documentation should not contain a reference to `org-store-link-plist':
it is an internal variable, therefore, it is of no use for the end user.
I suggest something along the lines of:
Evaluate Elisp SEXP and replace with the results@footnote{For
convenience, %:keyword (see above) placeholders within the expression
will be expanded.}. The sexp must return a string.
> %(sexp) Evaluate elisp `(sexp)' and replace with the result.
> + If sexp's attr is link keyword (%:keyword), it will
> + be expanded using `org-store-link-plist`.
Ditto.
Otherwise the code is fine. Would you provide a complete patch, i.e.
with `git format-patch'? The changelog entry may be:
org-capture: Expand keywords within %(sexp) placeholder in template
* org-capture.el (org-capture--expand-keyword-in-embedded-elisp): New
function.
(org-capture-expand-embedded-elisp): Use new function.
Regards,
