[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] lisp/org-capture.el: use link ID's for %K
From: |
Kenny Ballou |
Subject: |
Re: [PATCH] lisp/org-capture.el: use link ID's for %K |
Date: |
Fri, 08 Dec 2023 11:47:29 -0700 |
User-agent: |
mu4e 1.10.8; emacs 30.0.50 |
On 2023-12-08 13:56 GMT, Ihor Radchenko wrote:
> Kenny Ballou <kb@devnulllabs.io> 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-org-use-id' may have a number of non-nil values with
> slightly different behaviour: t, create-if-interactive,
> create-if-interactive-and-no-custom-id, use-existing, nil.
>
> Your patch will only work as expected for t and nil values, but not for
> others. You may consider leveraging `org-store-link' to account for all
> the possibilities.
This worked for the value of `use-existing', which is how I tested it.
But I see your point, the behavior should be more flexible to the other
values of `org-id-link-to-org-use-id'.
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.
If you have any other specific suggestions I'm missing, please let me
know. Otherwise, look for a new patch, hopefully soon!
--
-Kenny
- [PATCH] lisp/org-capture.el: use link ID's for %K, Kenny Ballou, 2023/12/10
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K, Ihor Radchenko, 2023/12/08
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K,
Kenny Ballou <=
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K, Ihor Radchenko, 2023/12/08
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K, Kenny Ballou, 2023/12/10
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K, Ihor Radchenko, 2023/12/10
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K, Kenny Ballou, 2023/12/11
- [PATCH] lisp/org-capture.el: use link ID's for %K, Kenny Ballou, 2023/12/11
- Message not available
- Message not available
- Re: [PATCH] lisp/org-capture.el: use link ID's for %K, Kenny Ballou, 2023/12/11
- [PATCH] lisp/org-capture.el: use `org-store-link' for %K values, Kenny Ballou, 2023/12/11
- [no subject], Kenny Ballou, 2023/12/11
- [PATCH] lisp/org-capture.el: use `org-store-link' for %K values, Kenny Ballou, 2023/12/11
- Re: [PATCH] lisp/org-capture.el: use `org-store-link' for %K values, Ihor Radchenko, 2023/12/12