* org-mew.el: Bug in creating links to messages marked for refile
@ 2008-11-17 10:07 Harri Kiiskinen
2008-11-17 11:17 ` More on org-mew Harri Kiiskinen
0 siblings, 1 reply; 3+ messages in thread
From: Harri Kiiskinen @ 2008-11-17 10:07 UTC (permalink / raw
To: emacs-orgmode
[-- Attachment #1: Type: Text/Plain, Size: 1205 bytes --]
There's a small bug in org-mew.el, which appears when trying to store
links to messages which have been marked for refile. I try to be as
clear as possible:
First, a working case:
when storing a link to a message on an IMAP server, the result is like
this:
[[mew:UTU:%inbox#20081117.115043.144292908.harri.kiiskinen@utu.fi][Email
from Harri Kiiskinen: test]]
which is exactly as it should be: First the link type ("mew:"), then
the mew case and the folder name ("UTU:%inbox"), and then the Message
ID. (This is a message I sent to myself.)
Link type: "mew:"
mew case: "UTU:"
folder: "‰inbox"
message id: "#20081117......"
Now, I set the refile mark on the message, and create a new link to
it. The result is:
[[mew:%2008#20081117.115043.144292908.harri.kiiskinen@utu.fi][Email
from Harri Kiiskinen: test]]
Now, the mew case is still set to UTU, meaning that mew will refile
the message to folder UTU:%2008, but the link that is created by
org-mew-store-link does not include the case information. We have:
Link type: mew:
case: (empty)
folder: %2008
message id: (the rest)
I'm no quite able to figure where the bug might be, but I'll keep
trying.
Harri K.
[-- Attachment #2: Type: text/plain, Size: 204 bytes --]
_______________________________________________
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* More on org-mew
2008-11-17 10:07 org-mew.el: Bug in creating links to messages marked for refile Harri Kiiskinen
@ 2008-11-17 11:17 ` Harri Kiiskinen
0 siblings, 0 replies; 3+ messages in thread
From: Harri Kiiskinen @ 2008-11-17 11:17 UTC (permalink / raw
To: emacs-orgmode
Hello,
some more info on my previous report on the broken links to refiled
mails in Mew.
From: Harri Kiiskinen <harri.kiiskinen@utu.fi>
Subject: org-mew.el: Bug in creating links to messages marked for refile
Date: Mon, 17 Nov 2008 12:07:51 +0200 (EET)
> Now, I set the refile mark on the message, and create a new link to
> it. The result is:
>
> [[mew:%2008#20081117.115043.144292908.harri.kiiskinen@utu.fi][Email
> from Harri Kiiskinen: test]]
>
> Now, the mew case is still set to UTU, meaning that mew will refile
> the message to folder UTU:%2008, but the link that is created by
> org-mew-store-link does not include the case information.
This patch seems to give correct links:
---------------------------------------------------
diff --git a/lisp/org-mew.el b/lisp/org-mew.el
index 2e6cb9b..453ce30 100644
--- a/lisp/org-mew.el
+++ b/lisp/org-mew.el
@@ -76,7 +76,10 @@
(folder-name
(if (and org-mew-link-to-refile-destination
(eq mark-info ?o)) ; marked as refile
- (nth 1 (mew-refile-get msgnum))
+ (concat
+ (mew-sinfo-get-case)
+ ":"
+ (nth 1 (mew-refile-get msgnum)))
(mew-summary-folder-name)))
message-id from to subject desc link)
(save-window-excursion
--------------------------------------------------
It uses mew-sinfo-get-case, which is buffer-local (or something), so
the refile always stays in the same 'world' as the original, to use
the Mew terminology.
The links produced to a message marked for refile and to a message already
in the folder are identical, so things should be ok:
[[mew:UTU:%2008#20081117.115043.144292908.harri.kiiskinen@utu.fi][Email from Harri Kiiskinen: test]]
[[mew:UTU:%2008#20081117.115043.144292908.harri.kiiskinen@utu.fi][Email from Harri Kiiskinen: test]]
Another problem appears, though, and this is definitively beyond me.
Now, the problem is, that if these are closed, i.e. look like this:
Email from Harri Kiiskinen: test
I can't open either with org-open-at-point. Both tell me "Message not
found". But when the links are 'open', like this:
[[mew:UTU:%2008#20081117.115043.144292908.harri.kiiskinen@utu.fi][Email
from Harri Kiiskinen: test]
and the point is on the link text ("mew:...") itself,
org-open-at-point works nicely and opens the respective message in
Mew.
Baffled,
Harri K.
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: org-mew.el: Bug in creating links to messages marked for refile
@ 2008-11-18 15:04 Tokuya Kameshima
0 siblings, 0 replies; 3+ messages in thread
From: Tokuya Kameshima @ 2008-11-18 15:04 UTC (permalink / raw
To: emacs-orgmode; +Cc: kames
[-- Attachment #1: Type: Text/Plain, Size: 1673 bytes --]
Hi Harri,
Thank you for the bug report and clear explanation. Could you try the
attached version of org-mew.el?
Thanks,
--Tokuya
> From: Harri Kiiskinen <harri.kiiskinen <at> utu.fi>
> Subject: org-mew.el: Bug in creating links to messages marked for refile
> Newsgroups: gmane.emacs.orgmode
> Date: 2008-11-17 10:07:51 GMT (1 day, 4 hours and 48 minutes ago)
>
> There's a small bug in org-mew.el, which appears when trying to store
> links to messages which have been marked for refile. I try to be as
> clear as possible:
>
> First, a working case:
>
> when storing a link to a message on an IMAP server, the result is like
> this:
>
> [[mew:UTU:%inbox#20081117.115043.144292908.harri.kiiskinen <at> utu.fi][Email
> from Harri Kiiskinen: test]]
>
> which is exactly as it should be: First the link type ("mew:"), then
> the mew case and the folder name ("UTU:%inbox"), and then the Message
> ID. (This is a message I sent to myself.)
> Link type: "mew:"
> mew case: "UTU:"
> folder: "‰inbox"
> message id: "#20081117......"
>
> Now, I set the refile mark on the message, and create a new link to
> it. The result is:
>
> [[mew:%2008#20081117.115043.144292908.harri.kiiskinen <at> utu.fi][Email
> from Harri Kiiskinen: test]]
>
> Now, the mew case is still set to UTU, meaning that mew will refile
> the message to folder UTU:%2008, but the link that is created by
> org-mew-store-link does not include the case information. We have:
> Link type: mew:
> case: (empty)
> folder: %2008
> message id: (the rest)
>
> I'm no quite able to figure where the bug might be, but I'll keep
> trying.
>
> Harri K.
[-- Attachment #2: org-mew.el --]
[-- Type: Text/Plain, Size: 5058 bytes --]
;;; org-mew.el --- Support for links to Mew messages from within Org-mode
;; Copyright (C) 2008 Free Software Foundation, Inc.
;; Author: Tokuya Kameshima <kames at fa2 dot so-net dot ne dot jp>
;; Keywords: outlines, hypermedia, calendar, wp
;; Homepage: http://orgmode.org
;; Version: 6.03pre03
;; This file is part of GNU Emacs.
;; GNU Emacs is free software: you can redistribute it and/or modify
;; it under the terms of the GNU General Public License as published by
;; the Free Software Foundation, either version 3 of the License, or
;; (at your option) any later version.
;; GNU Emacs is distributed in the hope that it will be useful,
;; but WITHOUT ANY WARRANTY; without even the implied warranty of
;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
;; GNU General Public License for more details.
;; You should have received a copy of the GNU General Public License
;; along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>.
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
;;
;;; Commentary:
;; This file implements links to Mew messages from within Org-mode.
;; Org-mode loads this module by default - if this is not what you want,
;; configure the variable `org-modules'.
;;; Code:
(require 'org)
(defgroup org-mew nil
"Options concerning the Mew link."
:tag "Org Startup"
:group 'org-link)
(defcustom org-mew-link-to-refile-destination t
"Create a link to the refile destination if the message is marked as refile."
:group 'org-mew
:type 'boolean)
;; Declare external functions and variables
(declare-function mew-cache-hit "ext:mew-cache" (fld msg &optional must-hit))
(declare-function mew-case-folder "ext:mew-func" (case folder))
(declare-function mew-header-get-value "ext:mew-header"
(field &optional as-list))
(declare-function mew-init "ext:mew" ())
(declare-function mew-refile-get "ext:mew-refile" (msg))
(declare-function mew-sinfo-get-case "ext:mew-summary" ())
(declare-function mew-summary-display "ext:mew-summary2" (&optional redisplay))
(declare-function mew-summary-folder-name "ext:mew-syntax" (&optional ext))
(declare-function mew-summary-get-mark "ext:mew-mark" ())
(declare-function mew-summary-message-number2 "ext:mew-syntax" ())
(declare-function mew-summary-pick-with-mewl "ext:mew-pick"
(pattern folder src-msgs))
(declare-function mew-summary-search-msg "ext:mew-const" (msg))
(declare-function mew-summary-set-message-buffer "ext:mew-summary3" (fld msg))
(declare-function mew-summary-visit-folder "ext:mew-summary4"
(folder &optional goend no-ls))
(declare-function mew-window-push "ext:mew" ())
(defvar mew-init-p)
(defvar mew-summary-goto-line-then-display)
;; Install the link type
(org-add-link-type "mew" 'org-mew-open)
(add-hook 'org-store-link-functions 'org-mew-store-link)
;; Implementation
(defun org-mew-store-link ()
"Store a link to a Mew folder or message."
(when (memq major-mode '(mew-summary-mode mew-virtual-mode))
(let* ((msgnum (mew-summary-message-number2))
(mark-info (mew-summary-get-mark))
(folder-name
(if (and org-mew-link-to-refile-destination
(eq mark-info ?o)) ; marked as refile
(mew-case-folder (mew-sinfo-get-case)
(nth 1 (mew-refile-get msgnum)))
(mew-summary-folder-name)))
message-id from to subject desc link)
(save-window-excursion
(if (fboundp 'mew-summary-set-message-buffer)
(mew-summary-set-message-buffer folder-name msgnum)
(set-buffer (mew-cache-hit folder-name msgnum t)))
(setq message-id (mew-header-get-value "Message-Id:"))
(setq from (mew-header-get-value "From:"))
(setq to (mew-header-get-value "To:"))
(setq subject (mew-header-get-value "Subject:")))
(org-store-link-props :type "mew" :from from :to to
:subject subject :message-id message-id)
(setq message-id (org-remove-angle-brackets message-id))
(setq desc (org-email-link-description))
(setq link (org-make-link "mew:" folder-name
"#" message-id))
(org-add-link-props :link link :description desc)
link)))
(defun org-mew-open (path)
"Follow the Mew message link specified by PATH."
(let (folder msgnum)
(cond ((string-match "\\`\\(+.*\\)+\\+\\([0-9]+\\)\\'" path) ; for Bastien's
(setq folder (match-string 1 path))
(setq msgnum (match-string 2 path)))
((string-match "\\`\\(\\(%#\\)?[^#]+\\)\\(#\\(.*\\)\\)?" path)
(setq folder (match-string 1 path))
(setq msgnum (match-string 4 path)))
(t (error "Error in Mew link")))
(require 'mew)
(mew-window-push)
(unless mew-init-p (mew-init))
(mew-summary-visit-folder folder)
(when msgnum
(if (not (string-match "\\`[0-9]+\\'" msgnum))
(let* ((pattern (concat "message-id=" msgnum))
(msgs (mew-summary-pick-with-mewl pattern folder nil)))
(setq msgnum (car msgs))))
(if (mew-summary-search-msg msgnum)
(if mew-summary-goto-line-then-display
(mew-summary-display))
(error "Message not found")))))
(provide 'org-mew)
;; arch-tag: 07ccdca7-6020-4941-a593-588a1e51b870
;;; org-mew.el ends here
[-- Attachment #3: Type: text/plain, Size: 204 bytes --]
_______________________________________________
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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-11-18 15:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-17 10:07 org-mew.el: Bug in creating links to messages marked for refile Harri Kiiskinen
2008-11-17 11:17 ` More on org-mew Harri Kiiskinen
-- strict thread matches above, loose matches on Subject: below --
2008-11-18 15:04 org-mew.el: Bug in creating links to messages marked for refile Tokuya Kameshima
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.