From: Ihor Radchenko <yantar92@gmail.com>
To: Max Nikulin <manikulin@gmail.com>
Cc: Bastien <bzg@gnu.org>, Fr Ml <fr_ml@t-online.de>,
emacs-orgmode@gnu.org
Subject: Re: Bug: org-store-link uses CUSTOM_ID instead of target point [9.4.4 (release_9.4.4 @ /usr/share/emacs/27.2/lisp/org/)]
Date: Tue, 13 Sep 2022 21:05:14 +0800 [thread overview]
Message-ID: <87edwfctmd.fsf@localhost> (raw)
In-Reply-To: <e2c807a7-1924-6f08-9e63-4f70aee9d3b5@gmail.com>
Max Nikulin <manikulin@gmail.com> writes:
>> Fixed in maint, thanks a lot for reporting this and Ihor for
>> confirming the bug.
>
> Bastien, unfortunately your fix caused duplication of stored links like
> "file:~/org/file.org::#custom_id" when point is outside of <<target>>.
> Earlier #CUSTOM_ID link was stored in addition to
> "file:~/org/file.org::*Heading" search link. My suggestion is to revert
> your patch and to just reset custom-id variable when <<target>> link is
> stored. Another effect or your patch, that I consider unintentional, is
> storing
> [[file:~/org/file.org::#custom_id][file:~/org/file.org::#custom_id]]
> instead of [[file:~/org/file.org::#custom_id][Heading]]. I prefer
> "original" behavior.
>
> Third patch is intended to avoid links inserted as
> [[target][file:~/org/file.org::target]] in the case of same file. I
> suppose, just [[target]] is better. Current variant looks unbalanced and
> misleading. Of course, you are free to skip last patch.
Thanks for reminding about this unresolved patch!
Applied all three patches onto main.
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=543a23a57d2947cd01c906d134ae9c5c8d0907c4
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=f7b8510283537bda4eba3b54fce5eafc7cec9993
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=c3d6672cfdbff8c9dd4c2ec70886ad3f62153d07
--
Ihor Radchenko,
Org mode contributor,
Learn more about Org mode at https://orgmode.org/.
Support Org development at https://liberapay.com/org-mode,
or support my work at https://liberapay.com/yantar92
next prev parent reply other threads:[~2022-09-13 13:06 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-04 19:17 Bug: org-store-link uses CUSTOM_ID instead of target point [9.4.4 (release_9.4.4 @ /usr/share/emacs/27.2/lisp/org/)] Fr Ml
2021-05-05 2:34 ` Ihor Radchenko
2021-05-06 12:41 ` Bastien
2021-11-06 12:51 ` Max Nikulin
2022-08-14 9:58 ` Max Nikulin
2022-09-13 13:05 ` Ihor Radchenko [this message]
2021-05-06 20:08 ` Table alignment problem Fr Ml
2021-05-07 14:50 ` Ihor Radchenko
2021-05-08 6:37 ` Jeremie Juste
2021-05-08 7:45 ` Fr Ml
2021-05-08 7:48 ` Fr Ml
2021-05-08 8:24 ` Ihor Radchenko
[not found] ` <5ebfdc6f-6e40-1470-4379-4a5a2b666aa7@t-online.de>
[not found] ` <87czu1pivb.fsf@localhost>
[not found] ` <6c4c8084-3800-c085-b724-f5653d7c20bd@t-online.de>
[not found] ` <87eeeh17fq.fsf@localhost>
[not found] ` <8975eded-5a22-cb9c-b85e-f3b523f16411@t-online.de>
[not found] ` <871ragff66.fsf@localhost>
[not found] ` <84936763-9384-7e6c-5304-b94217298b9a@t-online.de>
[not found] ` <87lf8o9pgn.fsf@localhost>
[not found] ` <eb465a73-691d-01be-135e-f58a9ff36173@t-online.de>
[not found] ` <87im3s9ie8.fsf@localhost>
[not found] ` <2c2f2204-14b1-49b2-658b-2b8b75ab86b9@t-online.de>
2021-05-15 9:26 ` [PATCH] Pixel-wise table alignment Ihor Radchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87edwfctmd.fsf@localhost \
--to=yantar92@gmail.com \
--cc=bzg@gnu.org \
--cc=emacs-orgmode@gnu.org \
--cc=fr_ml@t-online.de \
--cc=manikulin@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).