[PATCH] Fixes org-rmail-follow-link

[PATCH] Fixes org-rmail-follow-link

[T.F. Torrey]

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

With creating links working in Rmail, now I see that following links
fails with the error "Message not found".

The error comes from the use of the function widen in
org-rmail-follow-link. In an RMAIL buffer, the widen function only
widens to the current message. The function to widen to the entire
buffer is rmail-widen.

The attached patch replaces widen with rmail-widen and also removes the
following two lines (added by Bastien as part of the previous patch re
Rmail), because rmail-widen already displays full headers, so testing
and toggling them on has no effect.

GNU Emacs 24.1.50.1 (i686-pc-linux-gnu, GTK+ Version 3.4.2) of
2012-07-29 on doubah, modified by Debian

Org-mode version 7.8.11 (release_7.8.11-392-g9dae6f-git @ mixed
installation! /usr/local/share/emacs/site-lisp/ and
/home/tftorrey/.emacs.d/src/org-mode/lisp/)

Best to all,
Terry
-- 
T.F. Torrey


[-- Attachment #2: Patch for org-rmail-follow-link --]
[-- Type: text/plain, Size: 465 bytes --]

diff --git a/lisp/org-rmail.el b/lisp/org-rmail.el
index 9148a8e..d28a420 100644
--- a/lisp/org-rmail.el
+++ b/lisp/org-rmail.el
@@ -101,9 +101,7 @@
 	(rmail (if (string= folder "RMAIL") rmail-file-name folder))
 	(setq message-number
 	      (save-restriction
-		(widen)
-		(when (eq rmail-header-style 'normal)
-		  (rmail-toggle-header -1))
+		(rmail-widen)
 		(goto-char (point-max))
 		(if (re-search-backward
 		     (concat "^Message-ID:\\s-+" (regexp-quote

Re: [PATCH] Fixes org-rmail-follow-link

[Bastien]

tftorrey@tftorrey.com (T.F. Torrey) writes:

> With creating links working in Rmail, now I see that following links
> fails with the error "Message not found".
>
> The error comes from the use of the function widen in
> org-rmail-follow-link. In an RMAIL buffer, the widen function only
> widens to the current message. The function to widen to the entire
> buffer is rmail-widen.
>
> The attached patch replaces widen with rmail-widen and also removes the
> following two lines (added by Bastien as part of the previous patch re
> Rmail), because rmail-widen already displays full headers, so testing
> and toggling them on has no effect.

Applied, thanks.

Please send a properly formatted ChangeLog next time:
http://orgmode.org/worg/org-contribute.html#sec-5

Best,

-- 
 Bastien