warn about privacy system 'none' and auto sign/encrypt being enabled in all applicabl...
[claws.git] / src / compose.c
index 75166b118f5fd645f72b890829471ecfb0c392b8..1df165aa9084ef46c29c9dc47d7517c4c5de8768 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Claws Mail -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2019 the Claws Mail team and Hiroyuki Yamamoto
+ * Copyright (C) 1999-2020 the Claws Mail team and Hiroyuki Yamamoto
  *
  * 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
@@ -191,6 +191,13 @@ typedef enum {
 #define COMPOSE_DRAFT_TIMEOUT_UNSET -1
 #define COMPOSE_DRAFT_TIMEOUT_FORBIDDEN -2
 
+#define COMPOSE_PRIVACY_WARNING() {                                                    \
+       alertpanel_error(_("You have opted to sign and/or encrypt this "                \
+                          "message but have not selected a privacy system.\n\n"        \
+                          "Signing and encrypting have been disabled for this "        \
+                          "message."));                                                \
+}
+
 static GdkColor default_header_bgcolor = {
        (gulong)0,
        (gushort)0,
@@ -1012,10 +1019,7 @@ Compose *compose_generic_new(PrefsAccount *account, const gchar *mailto, FolderI
 
        if (privacy_system_can_sign(compose->privacy_system) == FALSE &&
            (account->default_encrypt || account->default_sign))
-               alertpanel_error(_("You have opted to sign and/or encrypt this "
-                                  "message but have not selected a privacy system.\n\n"
-                                  "Signing and encrypting have been disabled for this "
-                                  "message."));
+               COMPOSE_PRIVACY_WARNING();
 
        /* override from name if mailto asked for it */
        if (mailto_from) {
@@ -1692,12 +1696,10 @@ static Compose *compose_generic_reply(MsgInfo *msginfo,
        g_free(s_system);
 
        if (privacy_system_can_sign(compose->privacy_system) == FALSE &&
-           ((account->default_encrypt_reply && MSG_IS_ENCRYPTED(compose->replyinfo->flags)) ||
+           ((account->default_encrypt || account->default_sign) ||
+            (account->default_encrypt_reply && MSG_IS_ENCRYPTED(compose->replyinfo->flags)) ||
             (account->default_sign_reply && MSG_IS_SIGNED(compose->replyinfo->flags))))
-               alertpanel_error(_("You have opted to sign and/or encrypt this "
-                                  "message but have not selected a privacy system.\n\n"
-                                  "Signing and encrypting have been disabled for this "
-                                  "message."));
+               COMPOSE_PRIVACY_WARNING();
 
        SIGNAL_BLOCK(textbuf);
        
@@ -1902,6 +1904,10 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
 #endif
        SIGNAL_UNBLOCK(textbuf);
        
+       if (privacy_system_can_sign(compose->privacy_system) == FALSE &&
+           (account->default_encrypt || account->default_sign))
+               COMPOSE_PRIVACY_WARNING();
+
        cursor_pos = quote_fmt_get_cursor_pos();
        if (cursor_pos == -1)
                gtk_widget_grab_focus(compose->header_last->entry);
@@ -1983,6 +1989,9 @@ static Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_
 
        compose = compose_create(account, ((MsgInfo *)msginfo_list->data)->folder, COMPOSE_FORWARD, FALSE);
        compose_apply_folder_privacy_settings(compose, ((MsgInfo *)msginfo_list->data)->folder);
+       if (privacy_system_can_sign(compose->privacy_system) == FALSE &&
+           (account->default_encrypt || account->default_sign))
+               COMPOSE_PRIVACY_WARNING();
 
        compose->updating = TRUE;
 
@@ -2393,6 +2402,9 @@ Compose *compose_reedit(MsgInfo *msginfo, gboolean batch)
                compose_activate_privacy_system(compose, account, FALSE);
        }
        compose_apply_folder_privacy_settings(compose, msginfo->folder);
+       if (privacy_system_can_sign(compose->privacy_system) == FALSE &&
+           (account->default_encrypt || account->default_sign))
+               COMPOSE_PRIVACY_WARNING();
 
        compose->targetinfo = procmsg_msginfo_copy(msginfo);
        compose->targetinfo->tags = g_slist_copy(msginfo->tags);
@@ -2520,7 +2532,6 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
        cm_return_val_if_fail(account != NULL, NULL);
 
        compose = compose_create(account, msginfo->folder, COMPOSE_REDIRECT, batch);
-       compose_apply_folder_privacy_settings(compose, msginfo->folder);
 
        compose->updating = TRUE;
 
@@ -2598,6 +2609,10 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
                gtk_widget_set_sensitive(compose->toolbar->linewrap_current_btn, FALSE);
        if (compose->toolbar->linewrap_all_btn)
                gtk_widget_set_sensitive(compose->toolbar->linewrap_all_btn, FALSE);
+       if (compose->toolbar->privacy_sign_btn)
+               gtk_widget_set_sensitive(compose->toolbar->privacy_sign_btn, FALSE);
+       if (compose->toolbar->privacy_encrypt_btn)
+               gtk_widget_set_sensitive(compose->toolbar->privacy_encrypt_btn, FALSE);
 
        compose->modified = FALSE;
        compose_set_title(compose);
@@ -2894,7 +2909,7 @@ static gint compose_parse_header(Compose *compose, MsgInfo *msginfo)
 
        cm_return_val_if_fail(msginfo != NULL, -1);
 
-       if ((fp = procmsg_open_message(msginfo)) == NULL) return -1;
+       if ((fp = procmsg_open_message(msginfo, FALSE)) == NULL) return -1;
        procheader_get_header_fields(fp, hentry);
        claws_fclose(fp);
 
@@ -3019,7 +3034,7 @@ static gint compose_parse_manual_headers(Compose *compose, MsgInfo *msginfo, Hea
 
        cm_return_val_if_fail(msginfo != NULL, -1);
 
-       if ((fp = procmsg_open_message(msginfo)) == NULL) return -1;
+       if ((fp = procmsg_open_message(msginfo, FALSE)) == NULL) return -1;
        procheader_get_header_fields(fp, entries);
        claws_fclose(fp);
 
@@ -5274,6 +5289,9 @@ static void _display_queue_error(ComposeQueueResult val)
                        alertpanel_error(_("Could not queue message for sending:\n\n"
                                                "Couldn't get recipient encryption key."));
                        break;
+               case COMPOSE_QUEUE_SIGNING_CANCELLED:
+                       debug_print("signing cancelled\n");
+                       break;
                default:
                        /* unhandled error */
                        debug_print("oops, unhandled compose_queue() return value %d\n",
@@ -5912,7 +5930,10 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action, gbool
                        compose->account, from_addr)) {
                        g_free(from_name);
                        g_free(from_addr);
-                       return COMPOSE_QUEUE_ERROR_SIGNING_FAILED;
+                       if (!privacy_peek_error())
+                               return COMPOSE_QUEUE_SIGNING_CANCELLED;
+                       else
+                               return COMPOSE_QUEUE_ERROR_SIGNING_FAILED;
        }
        g_free(from_name);
        g_free(from_addr);
@@ -7773,7 +7794,6 @@ static Compose *compose_create(PrefsAccount *account,
        window = gtkut_window_new(GTK_WINDOW_TOPLEVEL, "compose");
 
        gtk_window_set_resizable(GTK_WINDOW(window), TRUE);
-       gtk_window_set_type_hint(GTK_WINDOW(window), GDK_WINDOW_TYPE_HINT_DIALOG);
        gtk_widget_set_size_request(window, prefs_common.compose_width,
                                        prefs_common.compose_height);
 
@@ -10899,186 +10919,60 @@ static void entry_copy_clipboard(GtkWidget *entry)
                        gtk_clipboard_get(GDK_SELECTION_CLIPBOARD));
 }
 
-static void paste_text(Compose *compose, GtkWidget *entry,
-                      gboolean wrap, GdkAtom clip, GtkTextIter *insert_place,
-                      const gchar *contents)
-{
-       GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(entry));
-       GtkTextMark *mark_start = gtk_text_buffer_get_insert(buffer);
-       GtkTextIter start_iter, end_iter;
-       gint start, end;
-
-       if (contents == NULL)
-               return;
-
-       /* we shouldn't delete the selection when middle-click-pasting, or we
-        * can't mid-click-paste our own selection */
-       if (clip != GDK_SELECTION_PRIMARY) {
-               undo_paste_clipboard(GTK_TEXT_VIEW(compose->text), compose->undostruct);
-               gtk_text_buffer_delete_selection(buffer, FALSE, TRUE);
-       }
-
-       if (insert_place == NULL) {
-               /* if insert_place isn't specified, insert at the cursor.
-                * used for Ctrl-V pasting */
-               gtk_text_buffer_get_iter_at_mark(buffer, &start_iter, mark_start);
-               start = gtk_text_iter_get_offset(&start_iter);
-               gtk_text_buffer_insert(buffer, &start_iter, contents, strlen(contents));
-       } else {
-               /* if insert_place is specified, paste here.
-                * used for mid-click-pasting */
-               start = gtk_text_iter_get_offset(insert_place);
-               gtk_text_buffer_insert(buffer, insert_place, contents, strlen(contents));
-               if (prefs_common.primary_paste_unselects)
-                       gtk_text_buffer_select_range(buffer, insert_place, insert_place);
-       }
-
-       if (!wrap) {
-               /* paste unwrapped: mark the paste so it's not wrapped later */
-               end = start + strlen(contents);
-               gtk_text_buffer_get_iter_at_offset(buffer, &start_iter, start);
-               gtk_text_buffer_get_iter_at_offset(buffer, &end_iter, end);
-               gtk_text_buffer_apply_tag_by_name(buffer, "no_wrap", &start_iter, &end_iter);
-       } else if (wrap && clip == GDK_SELECTION_PRIMARY) {
-               /* rewrap paragraph now (after a mid-click-paste) */
-               mark_start = gtk_text_buffer_get_insert(buffer);
-               gtk_text_buffer_get_iter_at_mark(buffer, &start_iter, mark_start);
-               gtk_text_iter_backward_char(&start_iter);
-               compose_beautify_paragraph(compose, &start_iter, TRUE);
-       }
-       compose->modified = TRUE;
-}
-
-static void attach_uri_list(Compose *compose, GtkSelectionData *data)
-{
-       GList *list, *tmp;
-
-       list = uri_list_extract_filenames(
-               (const gchar *)gtk_selection_data_get_data(data));
-       for (tmp = list; tmp != NULL; tmp = tmp->next) {
-               gchar *utf8_filename = conv_filename_to_utf8((const gchar *)tmp->data);
-               compose_attach_append
-                       (compose, (const gchar *)tmp->data,
-                        utf8_filename, NULL, NULL);
-               g_free(utf8_filename);
-       }
-       if (list)
-               compose_changed_cb(NULL, compose);
-
-       list_free_strings_full(list);
-}
-
-int attach_image(Compose *compose, GtkSelectionData *data, const gchar *subtype)
-{
-       FILE *fp;
-       const guchar *contents;
-       gchar *file;
-       gchar *type;
-       size_t len;
-       int r;
-
-       cm_return_val_if_fail(data != NULL, -1);
-
-       contents = gtk_selection_data_get_data(data);
-       len = gtk_selection_data_get_length(data);
-
-       file = g_strconcat(get_tmp_file(), "-image.", subtype, NULL);
-
-       debug_print("writing image to %s\n", file);
-
-       if ((fp = claws_fopen(file, "wb")) == NULL) {
-               FILE_OP_ERROR(file, "claws_fopen");
-               return -1;
-       }
-
-       if (claws_fwrite(contents, 1, len, fp) != len) {
-               FILE_OP_ERROR(file, "claws_fwrite");
-               claws_fclose(fp);
-               claws_unlink(file);
-               return -1;
-       }
-
-       r = claws_safe_fclose(fp);
-
-       if (r == EOF) {
-               FILE_OP_ERROR(file, "claws_fclose");
-               claws_unlink(file);
-               return -1;
-       }
-
-       type = g_strconcat("image/", subtype, NULL);
-
-       compose_attach_append(compose, (const gchar *)file, 
-               (const gchar *)file, type, NULL);
-
-       alertpanel_notice(_("The pasted image has been attached as: \n%s"), file);
-
-       g_free(file);
-       g_free(type);
-
-       return 0;
-}
-
 static void entry_paste_clipboard(Compose *compose, GtkWidget *entry, 
-                                 gboolean wrap, GdkAtom clip, GtkTextIter *insert_place)
-{
-       if (GTK_IS_TEXT_VIEW(entry)) {
-               GdkAtom types = gdk_atom_intern ("TARGETS", FALSE);
-               GdkAtom *targets = NULL;
-               int n_targets = 0, i;
-               gboolean paste_done = FALSE;
-               GtkClipboard *clipboard = gtk_clipboard_get(clip);
-
-               GtkSelectionData *contents = gtk_clipboard_wait_for_contents(
-                                               clipboard, types);
-
-               if (contents != NULL) {
-                       gtk_selection_data_get_targets(contents, &targets, &n_targets);
-                       gtk_selection_data_free(contents);
-               }
-
-               for (i = 0; i < n_targets; i++) {
-                       GdkAtom atom = targets[i];
-                       gchar *atom_type = gdk_atom_name(atom);
-
-                       if (atom_type != NULL) {
-                               GtkSelectionData *data = gtk_clipboard_wait_for_contents(
-                                               clipboard, atom);
-                               debug_print("got contents of type %s\n", atom_type);
-                               if (!strcmp(atom_type, "text/plain")) {
-                                       /* let the default text handler handle it */
-                    break;
-                               } else if (!strcmp(atom_type, "text/uri-list")) {
-                                       attach_uri_list(compose, data);
-
-                                       paste_done = TRUE;
-                                       break;
-                               } else if (!strncmp(atom_type, "image/", strlen("image/"))) {
-                                       gchar *subtype = g_strdup((gchar *)(strstr(atom_type, "/")+1));
-                                       debug_print("image of type %s\n", subtype);
-
-                                       attach_image(compose, data, subtype);
-                                       g_free(subtype);
-
-                                       paste_done = TRUE;
-                                       break;
-                               }
-                       }
-               }
-               if (!paste_done) {
-                       gchar *def_text = gtk_clipboard_wait_for_text(clipboard);
-                       paste_text(compose, entry, wrap, clip,
-                                  insert_place, def_text);
-                       g_free(def_text);
+                                 gboolean wrap, GdkAtom clip, GtkTextIter *insert_place)
+{
+       if (GTK_IS_TEXT_VIEW(entry)) {
+               GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(entry));
+               GtkTextMark *mark_start = gtk_text_buffer_get_insert(buffer);
+               GtkTextIter start_iter, end_iter;
+               gint start, end;
+               gchar *contents = gtk_clipboard_wait_for_text(gtk_clipboard_get(clip));
+               if (contents == NULL)
+                       return;
+       
+               /* we shouldn't delete the selection when middle-click-pasting, or we
+                * can't mid-click-paste our own selection */
+               if (clip != GDK_SELECTION_PRIMARY) {
+                       undo_paste_clipboard(GTK_TEXT_VIEW(compose->text), compose->undostruct);
+                       gtk_text_buffer_delete_selection(buffer, FALSE, TRUE);
                }
-               g_free(targets);
-
-       } else if (GTK_IS_EDITABLE(entry)) {
+               
+               if (insert_place == NULL) {
+                       /* if insert_place isn't specified, insert at the cursor.
+                        * used for Ctrl-V pasting */
+                       gtk_text_buffer_get_iter_at_mark(buffer, &start_iter, mark_start);
+                       start = gtk_text_iter_get_offset(&start_iter);
+                       gtk_text_buffer_insert(buffer, &start_iter, contents, strlen(contents));
+               } else {
+                       /* if insert_place is specified, paste here.
+                        * used for mid-click-pasting */
+                       start = gtk_text_iter_get_offset(insert_place);
+                       gtk_text_buffer_insert(buffer, insert_place, contents, strlen(contents));
+                       if (prefs_common.primary_paste_unselects)
+                               gtk_text_buffer_select_range(buffer, insert_place, insert_place);
+               }
+               
+               if (!wrap) {
+                       /* paste unwrapped: mark the paste so it's not wrapped later */
+                       end = start + strlen(contents);
+                       gtk_text_buffer_get_iter_at_offset(buffer, &start_iter, start);
+                       gtk_text_buffer_get_iter_at_offset(buffer, &end_iter, end);
+                       gtk_text_buffer_apply_tag_by_name(buffer, "no_wrap", &start_iter, &end_iter);
+               } else if (wrap && clip == GDK_SELECTION_PRIMARY) {
+                       /* rewrap paragraph now (after a mid-click-paste) */
+                       mark_start = gtk_text_buffer_get_insert(buffer);
+                       gtk_text_buffer_get_iter_at_mark(buffer, &start_iter, mark_start);
+                       gtk_text_iter_backward_char(&start_iter);
+                       compose_beautify_paragraph(compose, &start_iter, TRUE);
+               }
+       } else if (GTK_IS_EDITABLE(entry))
                gtk_editable_paste_clipboard (GTK_EDITABLE(entry));
-               compose->modified = TRUE;
-       }
+       compose->modified = TRUE;
 }
-
 static void entry_allsel(GtkWidget *entry)
 {
        if (GTK_IS_EDITABLE(entry))
@@ -11736,13 +11630,25 @@ static void compose_attach_drag_received_cb (GtkWidget                *widget,
                                             gpointer            user_data)
 {
        Compose *compose = (Compose *)user_data;
+       GList *list, *tmp;
        GdkAtom type;
 
        type = gtk_selection_data_get_data_type(data);
        if ((gdk_atom_name(type) && !strcmp(gdk_atom_name(type), "text/uri-list"))
           && gtk_drag_get_source_widget(context) !=
                summary_get_main_widget(mainwindow_get_mainwindow()->summaryview)) {
-               attach_uri_list(compose, data);
+               list = uri_list_extract_filenames(
+                       (const gchar *)gtk_selection_data_get_data(data));
+               for (tmp = list; tmp != NULL; tmp = tmp->next) {
+                       gchar *utf8_filename = conv_filename_to_utf8((const gchar *)tmp->data);
+                       compose_attach_append
+                               (compose, (const gchar *)tmp->data,
+                                utf8_filename, NULL, NULL);
+                       g_free(utf8_filename);
+               }
+               if (list)
+                       compose_changed_cb(NULL, compose);
+               list_free_strings_full(list);
        } else if (gtk_drag_get_source_widget(context)
                   == summary_get_main_widget(mainwindow_get_mainwindow()->summaryview)) {
                /* comes from our summaryview */