Bug: org-store-link misinterprets headings containing multiple links [8.3.4 (8.3.4-34-gacfd41-elpaplus @ ~/.emacs.d/elpa/org-plus-contrib-20160411/)]

Bug: org-store-link misinterprets headings containing multiple links [8.3.4 (8.3.4-34-gacfd41-elpaplus @ ~/.emacs.d/elpa/org-plus-contrib-20160411/)]

[Georgiy Tugai]

[-- Attachment #1: Type: text/plain, Size: 2429 bytes --]

Reproducing the bug
===================

  - Create a heading (of any level). The heading title should contain at
    least two links, of any type. For example, `[[#A][alpha]]'
    `[[#B][beta]]'
  - With point on the heading, call `org-store-link'.


Expected behaviour
==================

  - The "Stored: " message contains the text of the links ("Stored:
    alpha beta")
  - The stored link's description contains the text of the links ("alpha
    beta")


Actual behaviour
================

  - The "Stored: " message contains the text of the *first* link,
    repeated multiple times depending on the number of other links in
    the heading. ("Stored: alpha alpha")
  - The stored link's description matches the message ("alpha alpha")

------------------------------------------------------------------------

Emacs  : GNU Emacs 24.5.1 (x86_64-unknown-linux-gnu, GTK+ Version 3.16.6)
 of 2015-09-10 on foutrelis
Package: Org-mode version 8.3.4 (8.3.4-34-gacfd41-elpaplus @ ~/.emacs.d/elpa/org-plus-contrib-20160411/)

current state:
==============
(setq
 org-tab-first-hook '(org-hide-block-toggle-maybe org-babel-hide-result-toggle-maybe org-babel-header-arg-expand)
 org-speed-command-hook '(org-speed-command-default-hook org-babel-speed-command-hook)
 org-occur-hook '(org-first-headline-recenter)
 org-metaup-hook '(org-babel-load-in-session-maybe)
 org-confirm-shell-link-function 'yes-or-no-p
 org-after-todo-state-change-hook '(org-clock-out-if-current)
 org-src-mode-hook '(org-src-babel-configure-edit-buffer org-src-mode-configure-edit-buffer)
 org-agenda-before-write-hook '(org-agenda-add-entry-text)
 org-babel-pre-tangle-hook '(save-buffer)
 org-mode-hook '(#[nil "\300\301\302\303\304$\207" [org-add-hook change-major-mode-hook org-show-block-all append local] 5]
		 #[nil "\300\301\302\303\304$\207" [org-add-hook change-major-mode-hook org-babel-show-result-all append local]
		   5]
		 org-babel-result-hide-spec org-babel-hide-all-hashes org-eldoc-load)
 org-archive-hook '(org-attach-archive-delete-maybe)
 org-ctrl-c-ctrl-c-hook '(org-babel-hash-at-point org-babel-execute-safely-maybe)
 org-cycle-hook '(org-cycle-hide-archived-subtrees org-cycle-hide-drawers org-cycle-show-empty-lines
		  org-optimize-window-after-visibility-change)
 org-confirm-elisp-link-function 'yes-or-no-p
 org-metadown-hook '(org-babel-pop-to-session-maybe)
 org-clock-out-hook '(org-clock-remove-empty-clock-drawer)
 )

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

Re: Bug: org-store-link misinterprets headings containing multiple links [8.3.4 (8.3.4-34-gacfd41-elpaplus @ ~/.emacs.d/elpa/org-plus-contrib-20160411/)]

[Nicolas Goaziou]

Hello,

Georgiy Tugai <georgiy.tugai@gmail.com> writes:

> Reproducing the bug
> ===================
>
>   - Create a heading (of any level). The heading title should contain at
>     least two links, of any type. For example, `[[#A][alpha]]'
>     `[[#B][beta]]'
>   - With point on the heading, call `org-store-link'.
>
>
> Expected behaviour
> ==================
>
>   - The "Stored: " message contains the text of the links ("Stored:
>     alpha beta")
>   - The stored link's description contains the text of the links ("alpha
>     beta")
>
>
> Actual behaviour
> ================
>
>   - The "Stored: " message contains the text of the *first* link,
>     repeated multiple times depending on the number of other links in
>     the heading. ("Stored: alpha alpha")
>   - The stored link's description matches the message ("alpha alpha")
>
> ------------------------------------------------------------------------

Fixed. Thank you.

Regards,

-- 
Nicolas Goaziou