[BUG] org-open-at-point does not work anymore in comment blocks

[BUG] org-open-at-point does not work anymore in comment blocks

[Samuel Loury]

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

Hi,

On version baa2c5943a4afce71a6336cbd957139e840bd952.

Links are not followed anymore in comment blocks.

You may test it by launching (org-open-at-point) with the cursor into
the link in the following example.
--8<---------------cut here---------------start------------->8---
* Test
#+BEGIN_COMMENT
[[http://orgmode.org/]]
#+END_COMMENT
--8<---------------cut here---------------end--------------->8---

It triggers a user error in lisp/org.el at line 10641 since there is no
computed context.

Indeed, org-element-context does not appear to do anything for this
case.

I have no time left to investigate further, but I think someone out
there could find this bit of information useful.
-- 
Konubinix
GPG Key    : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE  5C36 75D2 3CED 7439 106A

[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]

Re: [BUG] org-open-at-point does not work anymore in comment blocks

[Nicolas Goaziou]

Hello,

Samuel Loury <konubinix@gmail.com> writes:

> On version baa2c5943a4afce71a6336cbd957139e840bd952.
>
> Links are not followed anymore in comment blocks.
>
> You may test it by launching (org-open-at-point) with the cursor into
> the link in the following example.
> * Test #+BEGIN_COMMENT [[http://orgmode.org/]] #+END_COMMENT
>
> It triggers a user error in lisp/org.el at line 10641 since there is no
> computed context.

This is to be expected. Commenting some part of a buffer tells the
parser to ignore it. The only purpose of comments is to deactivate some
syntax (e.g., a link, a timestamp...) in the buffer, which is exactly
what happens here.

If you want to ignore that part only during export, but still want to
use keep it active in the buffer, I suggest to use a drawer, with an
appropriate `org-export-with-drawers' value, e.g.,

  #+OPTIONS: d:(not "IGNORE_ME")
  
  :IGNORE_ME:
  [[http://orgmode.org]]
  :END:

This was discussed before on this ML. Some users made a point that
`org-open-at-point' should process links in node properties. In the case
of comments, though, I think we should be strict and use drawers
instead.


Regards,

-- 
Nicolas Goaziou

Re: [BUG] org-open-at-point does not work anymore in comment blocks

[Samuel Loury]

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

Hi,
Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

> Samuel Loury <konubinix@gmail.com> writes:
>
>> On version baa2c5943a4afce71a6336cbd957139e840bd952.
>>
>> Links are not followed anymore in comment blocks.
>>
>> You may test it by launching (org-open-at-point) with the cursor into
>> the link in the following example.
>> * Test #+BEGIN_COMMENT [[http://orgmode.org/]] #+END_COMMENT
>>
>> It triggers a user error in lisp/org.el at line 10641 since there is no
>> computed context.
>
> If you want to ignore that part only during export, but still want to
> use keep it active in the buffer, I suggest to use a drawer, with an
> appropriate `org-export-with-drawers' value, e.g.,
>
>   #+OPTIONS: d:(not "IGNORE_ME")
>   
>   :IGNORE_ME:
>   [[http://orgmode.org]]
>   :END:

Thank you for the answer. It is exactly what I want to do.
-- 
Konubinix
GPG Key    : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE  5C36 75D2 3CED 7439 106A

[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]

Re: [BUG] org-open-at-point does not work anymore in comment blocks

[Bastien]

Hi Samuel and Nicolas,

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

> This was discussed before on this ML. Some users made a point that
> `org-open-at-point' should process links in node properties. In the case
> of comments, though, I think we should be strict and use drawers
> instead.

I disagree, as you know.  We already support opening links from
standard comments (as an exception), I don't see why the exception
should not extend to #+BEGIN_COMMENT blocks.

I pushed a change so that links in comment blocks are now opened.

-- 
 Bastien