Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-11 Thread Kenny Ballou
On 2023-12-11 19:16 GMT, Ihor Radchenko wrote: > >> +*** Capture template expansion now supports ID links >> + >> +The capture template expansion element `%K' creates links using >> +`org-store-link', which respects the values of >> +`org-id-link-to-use-id'. > > Here, we simply use Org markup. So

Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-11 Thread Kenny Ballou
On 2023-12-10 12:19 GMT, Ihor Radchenko wrote: > Kenny Ballou writes: > >> >> I ended up using `org-store-link-plist'. For some reason that I don't >> understand, the `save-excursion' -> `org-store-link' was not updating >> `org-stored-links'. > > I can see it from the code now. Basically, inst

Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-10 Thread Kenny Ballou
On 2023-12-08 19:01 GMT, Ihor Radchenko wrote: > Kenny Ballou writes: > >> I'm currently struggling through modifying it to work with >> `org-store-link'. Something, save-excursion -> org-clock-goto -> >> org-store-link -> org-insert-link is what I'm trying for now, but that's >> not quite worki

Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-10 Thread Kenny Ballou
On 2023-12-08 13:56 GMT, Ihor Radchenko wrote: > Kenny Ballou writes: >> + (if org-id-link-to-org-use-id >> + (org-link-make-string >> + (format "id:%s" (org-id-get org-clock-marker)) >> + v-k) > > `org-id-link-to-

Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-10 Thread Ihor Radchenko
Kenny Ballou writes: >> The last step is unnecessary. You may just examine `org-stored-links' >> (documented in the docstring of `org-store-link') or >> `org-store-link-plist' (undocumented, but containing more info and also >> used by `org-capture-fill-template'). > > I ended up using `org-store

Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-08 Thread Ihor Radchenko
Kenny Ballou writes: > I'm currently struggling through modifying it to work with > `org-store-link'. Something, save-excursion -> org-clock-goto -> > org-store-link -> org-insert-link is what I'm trying for now, but that's > not quite working. I'll come back to this in a few hours. The last s

Re: [PATCH] lisp/org-capture.el: use link ID's for %K

2023-12-08 Thread Ihor Radchenko
Kenny Ballou writes: > * org-capture.el (org-capture-fill-template): fill %K values using org > ID's if `org-id-link-to-org-use-id' is non-nil. > * doc/ORG-NEWS: (Template variable expansion %K) Document new behavior. Thanks for the patch! > + (if org-id-link-to-org-use-id > +