Carsten Dominik <carsten.domi...@gmail.com> writes:
> Hi Sebastian - I am confused - which is the patch I should apply?


Sorry Carsten. The I CCed you with - the second one. It takes into
account, that the title could be empty too.

It's in the mail you're replying to.


>>  [[url][ ]]
>>
>>
>> Test:
>>
>>
>> (let ((orglink (org-make-link-string
>>                "http://www.google.de"; " ")))
>>  (insert orglink))
>> Which is invisible in Org-files!!!
>
> I have fixed this also, in a diffeerent way.
>
> - Carsten


Great.



This patch:


>> diff --git a/lisp/org-protocol.el b/lisp/org-protocol.el
>> index 5c65fb0..0d40c2c 100644
>> --- a/lisp/org-protocol.el
>> +++ b/lisp/org-protocol.el
>> @@ -470,9 +470,10 @@ Now template ?b will be used."
>>              (url (org-protocol-sanitize-uri (car parts)))
>>              (type (if (string-match "^\\([a-z]+\\):" url)
>>                        (match-string 1 url)))
>> -             (title (cadr parts))
>> -             (region (caddr parts))
>> -             (orglink (org-make-link-string url title))
>> +             (title (or (cadr parts) ""))
>> +             (region (or (caddr parts) ""))
>> +             (orglink (org-make-link-string
>> +                   url (if (string-match "[^[:space:]]" title) title url)))
>>              remember-annotation-functions)
>>         (setq org-stored-links
>>               (cons (list url title) org-stored-links))



Best

  Sebastian


_______________________________________________
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode

Reply via email to