On 2023-12-08 13:56 GMT, Ihor Radchenko wrote: > Kenny Ballou <k...@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