2007-08-21 [paul] 2.10.0cvs139
[claws.git] / src / messageview.c
index 53bc7561fbb4f7e2de6fa2181c6ff1b7a3b5f643..6a4331c0d275398f9403b12044940d534076330f 100644 (file)
@@ -1,10 +1,10 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2006 Hiroyuki Yamamoto and the Sylpheed-Claws team
+ * Copyright (C) 1999-2007 Hiroyuki Yamamoto and the Claws Mail team
  *
  * This program 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 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -13,8 +13,8 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 #include "defs.h"
 #include "filtering.h"
 #include "partial_download.h"
 #include "gedit-print.h"
+#include "uri_opener.h"
 #include "inc.h"
 #include "log.h"
+#include "combobox.h"
 
 static GList *messageview_list = NULL;
 
@@ -145,11 +147,15 @@ static void create_filter_cb              (gpointer        data,
 static void create_processing_cb       (gpointer        data,
                                         guint           action,
                                         GtkWidget      *widget);
+static void open_urls_cb               (gpointer        data,
+                                        guint           action,
+                                        GtkWidget      *widget);
 
 static void about_cb                   (gpointer        data,
                                         guint           action,
                                         GtkWidget      *widget);
-static void messageview_update         (MessageView *msgview);
+static void messageview_update         (MessageView    *msgview,
+                                        MsgInfo        *old_msginfo);
 static gboolean messageview_update_msg (gpointer source, gpointer data);
 
 static GList *msgview_list = NULL;
@@ -325,6 +331,8 @@ static GtkItemFactoryEntry msgview_entries[] =
        {N_("/_Tools/Create processing rule/by _Subject"),
                                        NULL, create_processing_cb, FILTER_BY_SUBJECT, NULL},
        {N_("/_Tools/---"),             NULL, NULL, 0, "<Separator>"},
+       {N_("/_Tools/List _URLs..."),   "<shift><control>U", open_urls_cb, 0, NULL},
+       {N_("/_Tools/---"),             NULL, NULL, 0, "<Separator>"},
        {N_("/_Tools/Actio_ns"),        NULL, NULL, 0, "<Branch>"},
 
        {N_("/_Help"),                  NULL, NULL, 0, "<Branch>"},
@@ -398,7 +406,7 @@ void messageview_add_toolbar(MessageView *msgview, GtkWidget *window)
        GtkWidget *handlebox;
        GtkWidget *vbox;
        GtkWidget *menubar;
-       GtkWidget *statusbar;
+       GtkWidget *statusbar = NULL;
        guint n_menu_entries;
 
        vbox = gtk_vbox_new(FALSE, 0);
@@ -418,15 +426,22 @@ void messageview_add_toolbar(MessageView *msgview, GtkWidget *window)
        }
        gtk_box_pack_start(GTK_BOX(vbox), handlebox, FALSE, FALSE, 0);
        gtk_widget_realize(handlebox);
+#ifdef MAEMO
+       msgview->toolbar = toolbar_create(TOOLBAR_MSGVIEW, window,
+                                         (gpointer)msgview);
+       msgview->statusbar = NULL;
+       msgview->statusbar_cid = 0;
+#else
        msgview->toolbar = toolbar_create(TOOLBAR_MSGVIEW, handlebox,
                                          (gpointer)msgview);
-
        statusbar = gtk_statusbar_new();
        gtk_widget_show(statusbar);
        gtk_box_pack_end(GTK_BOX(vbox), statusbar, FALSE, FALSE, 0);
        msgview->statusbar = statusbar;
        msgview->statusbar_cid = gtk_statusbar_get_context_id
                (GTK_STATUSBAR(statusbar), "Message View");
+#endif
+
 
        msgview->handlebox = handlebox;
        msgview->menubar   = menubar;
@@ -445,8 +460,8 @@ static MessageView *messageview_create_with_new_window_visible(MainWindow *mainw
        GtkWidget *window;
        static GdkGeometry geometry;
 
-       window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
-       gtk_window_set_title(GTK_WINDOW(window), _("Sylpheed-Claws - Message View"));
+       window = gtkut_window_new(GTK_WINDOW_TOPLEVEL, "messageview");
+       gtk_window_set_title(GTK_WINDOW(window), _("Claws Mail - Message View"));
        gtk_window_set_resizable(GTK_WINDOW(window), TRUE);
 
        if (!geometry.min_height) {
@@ -466,9 +481,12 @@ static MessageView *messageview_create_with_new_window_visible(MainWindow *mainw
                         msgview);
        g_signal_connect(G_OBJECT(window), "delete_event",
                         G_CALLBACK(messageview_delete_cb), msgview);
+#ifdef MAEMO
+       maemo_connect_key_press_to_mainwindow(GTK_WINDOW(window));
+#else
        g_signal_connect(G_OBJECT(window), "key_press_event",
                         G_CALLBACK(key_pressed), msgview);
-
+#endif
        messageview_add_toolbar(msgview, window);
 
        if (show) {
@@ -540,6 +558,8 @@ static gint disposition_notification_send(MsgInfo *msginfo)
         gchar *addr;
         gchar *addrp;
        gchar *foo = NULL;
+       gboolean queued_removed = FALSE;
+       
        if (!msginfo->extradata)
                return -1;
        if (!msginfo->extradata->returnreceiptto && 
@@ -568,7 +588,7 @@ static gint disposition_notification_send(MsgInfo *msginfo)
        if (ok != 0) {
                AlertValue val;
                gchar *message;
-               message = g_strdup_printf(
+               message = g_markup_printf_escaped(
                  _("The notification address to which the return receipt is\n"
                    "to be sent does not correspond to the return path:\n"
                    "Notification address: %s\n"
@@ -616,8 +636,8 @@ static gint disposition_notification_send(MsgInfo *msginfo)
        }
 
        /* write to temporary file */
-       g_snprintf(tmp, sizeof(tmp), "%s%ctmpmsg%d",
-                  get_rc_dir(), G_DIR_SEPARATOR, (gint)msginfo);
+       g_snprintf(tmp, sizeof(tmp), "%s%ctmpmsg%p",
+                  get_rc_dir(), G_DIR_SEPARATOR, msginfo);
 
        if ((fp = g_fopen(tmp, "wb")) == NULL) {
                FILE_OP_ERROR(tmp, "fopen");
@@ -666,7 +686,7 @@ static gint disposition_notification_send(MsgInfo *msginfo)
                g_free(path);
        }               
 
-       fprintf(fp, "X-Sylpheed-End-Special-Headers: 1\n");
+       fprintf(fp, "X-Claws-End-Special-Headers: 1\n");
        
        /* Date */
        get_rfc822_date(buf, sizeof(buf));
@@ -714,26 +734,22 @@ static gint disposition_notification_send(MsgInfo *msginfo)
        }
                
        if (prefs_common.work_offline && 
-           !inc_offline_should_override(
-               _("Sylpheed-Claws needs network access in order "
+           !inc_offline_should_override(TRUE,
+               _("Claws Mail needs network access in order "
                  "to send this email.")))
                return 0;
 
        /* send it */
        path = folder_item_fetch_msg(queue, num);
-       ok = procmsg_send_message_queue(path, &foo, queue, num);
+       ok = procmsg_send_message_queue(path, &foo, queue, num, &queued_removed);
        g_free(path);
        g_free(foo);
-       folder_item_remove_msg(queue, num);
+       if (ok == 0 && !queued_removed)
+               folder_item_remove_msg(queue, num);
 
        return ok;
 }
 
-GList *messageview_get_window_list(void)
-{
-       return messageview_list;
-}
-
 static gboolean find_encrypted_func(GNode *node, gpointer data)
 {
        MimeInfo *mimeinfo = (MimeInfo *) node->data;
@@ -770,6 +786,7 @@ gint messageview_show(MessageView *messageview, MsgInfo *msginfo,
                messageview->mimeview->textview->stop_loading = TRUE;
                return 0;
        }
+
        if (messageview->toolbar)
                toolbar_set_learn_button
                        (messageview->toolbar,
@@ -780,10 +797,6 @@ gint messageview_show(MessageView *messageview, MsgInfo *msginfo,
                         MSG_IS_SPAM(msginfo->flags)?LEARN_HAM:LEARN_SPAM);
 
        if (messageview->toolbar) {
-               if (messageview->toolbar->learn_ham_btn)
-                       gtk_widget_set_sensitive(
-                               messageview->toolbar->learn_ham_btn, 
-                               procmsg_spam_can_learn());
                if (messageview->toolbar->learn_spam_btn)
                        gtk_widget_set_sensitive(
                                messageview->toolbar->learn_spam_btn, 
@@ -848,11 +861,18 @@ gint messageview_show(MessageView *messageview, MsgInfo *msginfo,
        textview_set_all_headers(messageview->mimeview->textview, 
                        messageview->all_headers);
 
+#ifdef MAEMO
+       maemo_window_full_screen_if_needed(GTK_WINDOW(messageview->window));
+#endif
+       if (messageview->window) {
+               gtk_window_set_title(GTK_WINDOW(messageview->window), 
+                               _("Claws Mail - Message View"));
+               GTK_EVENTS_FLUSH();
+       }
        mimeview_show_message(messageview->mimeview, mimeinfo, file);
        
-       if (messageview->window)
-               gtk_window_set_title(GTK_WINDOW(messageview->window), 
-                               _("Sylpheed-Claws - Message View"));
+       messageview_set_position(messageview, 0);
+
        if (messageview->window && msginfo->subject) {
                subject = g_strdup(msginfo->subject);
                if (!g_utf8_validate(subject, -1, NULL)) {
@@ -867,13 +887,17 @@ gint messageview_show(MessageView *messageview, MsgInfo *msginfo,
                g_free(subject);
        }
 
+       if (msginfo && msginfo->folder) {
+               msginfo->folder->last_seen = msginfo->msgnum;   
+       }
+
        main_create_mailing_list_menu(messageview->mainwin, messageview->msginfo);
 
-       if (messageview->msginfo->extradata
+       if (messageview->msginfo && messageview->msginfo->extradata
            && messageview->msginfo->extradata->partial_recv)
                partial_recv_show(messageview->noticeview, 
                                  messageview->msginfo);
-       else if (messageview->msginfo->extradata &&
+       else if (messageview->msginfo && messageview->msginfo->extradata &&
            (messageview->msginfo->extradata->dispositionnotificationto || 
             messageview->msginfo->extradata->returnreceiptto) &&
            !MSG_IS_RETRCPT_SENT(messageview->msginfo->flags) &&
@@ -884,8 +908,11 @@ gint messageview_show(MessageView *messageview, MsgInfo *msginfo,
                noticeview_hide(messageview->noticeview);
 
        mimeinfo = procmime_mimeinfo_next(mimeinfo);
-       if (mimeinfo && (mimeinfo->type != MIMETYPE_TEXT || 
-           strcasecmp(mimeinfo->subtype, "plain"))) {
+       if (!all_headers && mimeinfo 
+                       && (mimeinfo->type != MIMETYPE_TEXT || 
+           strcasecmp(mimeinfo->subtype, "plain")) 
+                       && (mimeinfo->type != MIMETYPE_MULTIPART || 
+           strcasecmp(mimeinfo->subtype, "signed"))) {
                if (strcasecmp(mimeinfo->subtype, "html"))
                        mimeview_show_part(messageview->mimeview,mimeinfo);
                else if (prefs_common.invoke_plugin_on_html)
@@ -911,6 +938,8 @@ void messageview_reflect_prefs_pixmap_theme(void)
 
 void messageview_clear(MessageView *messageview)
 {
+       if (!messageview)
+               return;
        procmsg_msginfo_free(messageview->msginfo);
        messageview->msginfo = NULL;
        messageview->filtered = FALSE;
@@ -924,6 +953,12 @@ void messageview_destroy(MessageView *messageview)
        debug_print("destroy messageview\n");
        messageview_list = g_list_remove(messageview_list, messageview);
 
+       if (messageview->mainwin->summaryview->messageview == messageview)
+               messageview->mainwin->summaryview->messageview = NULL;
+
+       if (messageview->mainwin->summaryview->ext_messageview == messageview)
+               messageview->mainwin->summaryview->ext_messageview = NULL;
+
        if (!messageview->deferred_destroy) {
                hooks_unregister_hook(MSGINFO_UPDATE_HOOKLIST,
                              messageview->msginfo_update_callback_id);
@@ -1005,7 +1040,12 @@ void messageview_delete(MessageView *msgview)
                        procmsg_msginfo_set_flags(msginfo, MSG_DELETED, 0);
                        /* NOTE: does not update to next message in summaryview */
                }
-       }               
+       }
+#ifdef MAEMO
+       if (msgview->window) {
+               messageview_destroy(msgview);
+       }
+#endif
 }
 
 /* 
@@ -1013,17 +1053,15 @@ void messageview_delete(MessageView *msgview)
  *        leave unchanged if summaryview is empty
  * \param pointer to MessageView
  */    
-static void messageview_update(MessageView *msgview)
+static void messageview_update(MessageView *msgview, MsgInfo *old_msginfo)
 {
        SummaryView *summaryview = (SummaryView*)msgview->mainwin->summaryview;
 
        g_return_if_fail(summaryview != NULL);
        
        if (summaryview->selected) {
-               GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
-               MsgInfo *msginfo = gtk_ctree_node_get_row_data(ctree, 
-                                                     summaryview->selected);
-               if (msginfo == NULL)
+               MsgInfo *msginfo = summary_get_selected_msg(summaryview);
+               if (msginfo == NULL || msginfo == old_msginfo)
                        return;
 
                messageview_show(msgview, msginfo, 
@@ -1055,17 +1093,13 @@ MimeInfo *messageview_get_selected_mime_part(MessageView *messageview)
 
 void messageview_copy_clipboard(MessageView *messageview)
 {
-       TextView *text;
-
-       text = messageview_get_current_textview(messageview);
+       gchar *text = messageview_get_selection(messageview);
        if (text) {
-               GtkTextView *textview = GTK_TEXT_VIEW(text->text);
-               GtkTextBuffer *buffer = gtk_text_view_get_buffer(textview);
-               GtkClipboard *clipboard
-                       = gtk_clipboard_get(GDK_SELECTION_CLIPBOARD);
-
-               gtk_text_buffer_copy_clipboard(buffer, clipboard);
+               gtk_clipboard_set_text(
+                       gtk_clipboard_get(GDK_SELECTION_CLIPBOARD),
+                       text, -1);
        }
+       g_free(text);
 }
 
 void messageview_select_all(MessageView *messageview)
@@ -1098,6 +1132,13 @@ gboolean messageview_search_string(MessageView *messageview, const gchar *str,
 {
        TextView *text;
 
+       if (messageview->mimeview->type == MIMEVIEW_VIEWER) {
+               MimeViewer *viewer = messageview->mimeview->mimeviewer;
+               if (viewer && viewer->text_search) {
+                       return viewer->text_search(viewer, FALSE, str, case_sens);
+               }
+       }
+
        text = messageview_get_current_textview(messageview);
        if (text)
                return textview_search_string(text, str, case_sens);
@@ -1110,6 +1151,13 @@ gboolean messageview_search_string_backward(MessageView *messageview,
 {
        TextView *text;
 
+       if (messageview->mimeview->type == MIMEVIEW_VIEWER) {
+               MimeViewer *viewer = messageview->mimeview->mimeviewer;
+               if (viewer && viewer->text_search) {
+                       return viewer->text_search(viewer, TRUE, str, case_sens);
+               }
+       }
+
        text = messageview_get_current_textview(messageview);
        if (text)       
                return textview_search_string_backward(text,
@@ -1124,7 +1172,7 @@ gboolean messageview_is_visible(MessageView *messageview)
        return messageview->visible;
 }
 
-void messageview_save_as(MessageView *messageview)
+static void messageview_save_as(MessageView *messageview)
 {
        gchar *filename = NULL;
        MsgInfo *msginfo;
@@ -1137,7 +1185,19 @@ void messageview_save_as(MessageView *messageview)
                Xstrdup_a(filename, msginfo->subject, return);
                subst_for_filename(filename);
        }
-       dest = filesel_select_file_save(_("Save as"), filename);
+       if (filename && !g_utf8_validate(filename, -1, NULL)) {
+               gchar *oldstr = filename;
+               filename = conv_codeset_strdup(filename,
+                                              conv_get_locale_charset_str(),
+                                              CS_UTF_8);
+               if (!filename) {
+                       g_warning("messageview_save_as(): failed to convert character set.");
+                       filename = g_strdup(oldstr);
+               }
+               dest = filesel_select_file_save(_("Save as"), filename);
+               g_free(filename);
+       } else
+               dest = filesel_select_file_save(_("Save as"), filename);
        if (!dest) return;
        if (is_file_exist(dest)) {
                AlertValue aval;
@@ -1191,28 +1251,6 @@ static gboolean key_pressed(GtkWidget *widget, GdkEventKey *event,
        return FALSE;
 }
 
-void messageview_toggle_view_real(MessageView *messageview)
-{
-       MainWindow *mainwin = messageview->mainwin;
-       union CompositeWin *cwin = &mainwin->win;
-       GtkWidget *vpaned = NULL;
-       GtkWidget *container = NULL;
-       
-       switch (mainwin->type) {
-       case SEPARATE_NONE:
-               vpaned = cwin->sep_none.vpaned;
-               container = cwin->sep_none.hpaned;
-               break;
-       case SEPARATE_FOLDER:
-               vpaned = cwin->sep_folder.vpaned;
-               container = mainwin->vbox_body;
-               break;
-       case SEPARATE_MESSAGE:
-       case SEPARATE_BOTH:
-               return;
-       }
-}
-
 static void return_receipt_show(NoticeView *noticeview, MsgInfo *msginfo)
 {
        gchar *addr = NULL;
@@ -1232,10 +1270,12 @@ static void return_receipt_show(NoticeView *noticeview, MsgInfo *msginfo)
        }
 
        if (from_me) {
+               noticeview_set_icon(noticeview, STOCK_PIXMAP_NOTICE_WARN);
                noticeview_set_text(noticeview, _("You asked for a return receipt in this message."));
                noticeview_set_button_text(noticeview, NULL);
                noticeview_set_button_press_callback(noticeview, NULL, NULL);
        } else {
+               noticeview_set_icon(noticeview, STOCK_PIXMAP_NOTICE_WARN);
                noticeview_set_text(noticeview, _("This message asks for a return receipt."));
                noticeview_set_button_text(noticeview, _("Send receipt"));
                noticeview_set_button_press_callback(noticeview,
@@ -1322,6 +1362,7 @@ static void partial_recv_show(NoticeView *noticeview, MsgInfo *msginfo)
                }
        }
        
+       noticeview_set_icon(noticeview, STOCK_PIXMAP_NOTICE_WARN);
        noticeview_set_text(noticeview, text);
        g_free(text);
        noticeview_set_button_text(noticeview, button1);
@@ -1361,26 +1402,22 @@ static void partial_recv_unmark_clicked(NoticeView *noticeview,
 
 static void select_account_cb(GtkWidget *w, gpointer data)
 {
-       *(gint*)data = GPOINTER_TO_INT(g_object_get_data(G_OBJECT(w), MENU_VAL_ID));
+       *(gint*)data = combobox_get_active_data(GTK_COMBO_BOX(w));
 }
-       
+
 static PrefsAccount *select_account_from_list(GList *ac_list)
 {
        GtkWidget *optmenu;
-       GtkWidget *menu;
        gint account_id;
 
        g_return_val_if_fail(ac_list != NULL, NULL);
        g_return_val_if_fail(ac_list->data != NULL, NULL);
        
-       optmenu = gtk_option_menu_new();
-       menu = gtkut_account_menu_new(ac_list, 
-                       G_CALLBACK(select_account_cb), 
+       optmenu = gtkut_account_menu_new(ac_list,
+                       G_CALLBACK(select_account_cb),
                        &account_id);
-       if (!menu)
+       if (!optmenu)
                return NULL;
-       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), menu);
-       gtk_option_menu_set_history(GTK_OPTION_MENU(optmenu), 0);
        account_id = ((PrefsAccount *) ac_list->data)->account_id;
        if (alertpanel_with_widget(
                                _("Return Receipt Notification"),
@@ -1388,8 +1425,8 @@ static PrefsAccount *select_account_from_list(GList *ac_list)
                                  "accounts.\n"
                                  "Please choose which account do you want to "
                                  "use for sending the receipt notification:"),
-                               _("_Send Notification"), _("+_Cancel"), NULL,
-                               FALSE, optmenu) != G_ALERTDEFAULT)
+                               _("_Cancel"), _("_Send Notification"), NULL,
+                               FALSE, G_ALERTDEFAULT, optmenu) != G_ALERTALTERNATE)
                return NULL;
        return account_find_from_id(account_id);
 }
@@ -1450,7 +1487,7 @@ static void save_as_cb(gpointer data, guint action, GtkWidget *widget)
 }
 
 #ifdef USE_GNOMEPRINT
-static void print_mimeview(MimeView *mimeview) 
+static void print_mimeview(MimeView *mimeview, gint sel_start, gint sel_end, gint partnum
 {
        if (!mimeview 
        ||  !mimeview->textview
@@ -1459,11 +1496,32 @@ static void print_mimeview(MimeView *mimeview)
                                     "contain text."));
        else {
                gtk_widget_realize(mimeview->textview->text);
+               if (partnum > 0) {
+                       mimeview_select_part_num(mimeview, partnum);
+               }
+               if (mimeview->type == MIMEVIEW_VIEWER) {
+                       MimeViewer *viewer = mimeview->mimeviewer;
+                       if (viewer && viewer->print) {
+                               viewer->print(viewer);
+                               return;
+                       }
+               }
+               if (sel_start != -1 && sel_end != -1) {
+                       GtkTextIter start, end;
+                       GtkTextView *text = GTK_TEXT_VIEW(mimeview->textview->text);
+                       GtkTextBuffer *buffer = gtk_text_view_get_buffer(text);
+
+                       gtk_text_buffer_get_iter_at_offset(buffer, &start, sel_start);
+                       gtk_text_buffer_get_iter_at_offset(buffer, &end, sel_end);
+                       gtk_text_buffer_select_range(buffer, &start, &end);
+               }
+
                gedit_print(GTK_TEXT_VIEW(mimeview->textview->text));
        }
 }
 
-void messageview_print(MsgInfo *msginfo, gboolean all_headers) 
+void messageview_print(MsgInfo *msginfo, gboolean all_headers, 
+                       gint sel_start, gint sel_end, gint partnum) 
 {
        PangoFontDescription *font_desc = NULL;
        MessageView *tmpview = messageview_create_with_new_window_visible(
@@ -1485,7 +1543,8 @@ void messageview_print(MsgInfo *msginfo, gboolean all_headers)
        tmpview->all_headers = all_headers;
        if (msginfo && messageview_show(tmpview, msginfo, 
                tmpview->all_headers) >= 0) {
-                       print_mimeview(tmpview->mimeview);
+                       print_mimeview(tmpview->mimeview, 
+                               sel_start, sel_end, partnum);
        }
        messageview_destroy(tmpview);
 }
@@ -1497,6 +1556,8 @@ static void print_cb(gpointer data, guint action, GtkWidget *widget)
 #ifndef USE_GNOMEPRINT
        gchar *cmdline = NULL;
        gchar *p;
+#else
+       gint sel_start = -1, sel_end = -1, partnum = 0;
 #endif
 
        if (!messageview->msginfo) return;
@@ -1516,7 +1577,11 @@ static void print_cb(gpointer data, guint action, GtkWidget *widget)
        procmsg_print_message(messageview->msginfo, cmdline);
        g_free(cmdline);
 #else
-       messageview_print(messageview->msginfo, messageview->all_headers);
+       partnum = mimeview_get_selected_part_num(messageview->mimeview);
+       textview_get_selection_offsets(messageview->mimeview->textview,
+               &sel_start, &sel_end);
+       messageview_print(messageview->msginfo, messageview->all_headers, 
+               sel_start, sel_end, partnum);
 #endif
 }
 
@@ -1589,6 +1654,13 @@ static void show_all_header_cb(gpointer data, guint action, GtkWidget *widget)
        MessageView *messageview = (MessageView *)data;
        MsgInfo *msginfo = messageview->msginfo;
 
+       if (messageview->mimeview->textview &&
+           messageview->mimeview->textview->loading) {
+               return;
+       }
+       if (messageview->updating)
+               return;
+
        messageview->all_headers = 
                        GTK_CHECK_MENU_ITEM(widget)->active;
        if (!msginfo) return;
@@ -1713,6 +1785,12 @@ static void create_processing_cb(gpointer data, guint action,
                                    (PrefsFilterType)action, 1);
 }
 
+static void open_urls_cb(gpointer data, guint action, GtkWidget *widget)
+{
+       MessageView *messageview = (MessageView *)data;
+       messageview_list_urls(messageview);
+}
+
 static void about_cb(gpointer data, guint action, GtkWidget *widget)
 {
        about_show();
@@ -1727,8 +1805,9 @@ static gboolean messageview_update_msg(gpointer source, gpointer data)
                return FALSE;
 
        if (msginfo_update->flags & MSGINFO_UPDATE_DELETED) {
+               MsgInfo *old_msginfo = messageview->msginfo;
                messageview_clear(messageview);
-               messageview_update(messageview);
+               messageview_update(messageview, old_msginfo);
        }
 
        return FALSE;
@@ -1745,12 +1824,7 @@ void messageview_set_menu_sensitive(MessageView *messageview)
        if (!messageview->menubar) return;
        ifactory = gtk_item_factory_from_widget(messageview->menubar);
        if (!ifactory) return;
-       if (messageview->mainwin->type == SEPARATE_MESSAGE) {
-               menuitem = gtk_item_factory_get_widget(ifactory, "/View/All headers");
-               gtk_check_menu_item_set_active
-                       (GTK_CHECK_MENU_ITEM(menuitem),
-                        messageview->mimeview->textview->show_all_headers);
-       }
+
        if (prefs_common.hide_quotes) {
                menuitem = NULL;
                if (prefs_common.hide_quotes == 1)
@@ -1774,13 +1848,13 @@ void messageview_learn (MessageView *msgview, gboolean is_spam)
                if (procmsg_spam_learner_learn(msgview->msginfo, NULL, TRUE) == 0)
                        procmsg_msginfo_set_flags(msgview->msginfo, MSG_SPAM, 0);
                else
-                       log_error(_("An error happened while learning.\n"));
+                       log_error(LOG_PROTOCOL, _("An error happened while learning.\n"));
                
        } else {
                if (procmsg_spam_learner_learn(msgview->msginfo, NULL, FALSE) == 0)
                        procmsg_msginfo_unset_flags(msgview->msginfo, MSG_SPAM, 0);
                else
-                       log_error(_("An error happened while learning.\n"));
+                       log_error(LOG_PROTOCOL, _("An error happened while learning.\n"));
        }
        if (msgview->toolbar)
                toolbar_set_learn_button
@@ -1791,3 +1865,21 @@ void messageview_learn (MessageView *msgview, gboolean is_spam)
                        (msgview->mainwin->toolbar,
                         MSG_IS_SPAM(msgview->msginfo->flags)?LEARN_HAM:LEARN_SPAM);
 }
+
+void messageview_list_urls (MessageView        *msgview)
+{
+       GSList *cur = msgview->mimeview->textview->uri_list;
+       GSList *newlist = NULL;
+       for (; cur; cur = cur->next) {
+               ClickableText *uri = (ClickableText *)cur->data;
+               if (uri->uri &&
+                   (!g_ascii_strncasecmp(uri->uri, "ftp.", 4) ||
+                    !g_ascii_strncasecmp(uri->uri, "www.", 4) ||
+                    !g_ascii_strncasecmp(uri->uri, "http:", 5) ||
+                    !g_ascii_strncasecmp(uri->uri, "https:", 6)))
+                       newlist = g_slist_prepend(newlist, uri);
+       }
+       newlist = g_slist_reverse(newlist);
+       uri_opener_open(msgview, newlist);
+       g_slist_free(newlist);
+}