From 51f9daed7077c6b7f616c92ee86459a02f63bc16 Mon Sep 17 00:00:00 2001 From: Milan Crha Date: Tue, 13 Oct 2009 18:50:20 +0200 Subject: Bug #597123 - Composer's Send Options doesn't work --- composer/e-composer-actions.c | 32 --------------------- composer/e-composer-actions.h | 2 -- composer/e-composer-private.h | 1 - composer/e-msg-composer.c | 62 +--------------------------------------- composer/e-msg-composer.h | 2 -- composer/evolution-composer.ui | 2 +- composer/mail-composer.error.xml | 5 ---- 7 files changed, 2 insertions(+), 104 deletions(-) (limited to 'composer') diff --git a/composer/e-composer-actions.c b/composer/e-composer-actions.c index 93317c83f6..62372a7f1d 100644 --- a/composer/e-composer-actions.c +++ b/composer/e-composer-actions.c @@ -240,31 +240,6 @@ action_send_cb (GtkAction *action, e_msg_composer_send (composer); } -static void -action_send_options_cb (GtkAction *action, - EMsgComposer *composer) -{ - /* FIXME: KILL-BONOBO - should this be here when -no-undefined removed? */ - /* - EMEvent *event = em_event_peek (); - EMEventTargetComposer *target; - - target = em_event_target_new_composer ( - event, composer, EM_EVENT_COMPOSER_SEND_OPTION); - e_msg_composer_set_send_options (composer, FALSE); - - e_event_emit ( - (EEvent *) event, - "composer.selectsendoption", - (EEventTarget *) target); - */ - - if (!composer->priv->send_invoked) - e_error_run ( - GTK_WINDOW (composer), - "mail-composer:send-options-support", NULL); -} - static void action_new_message_cb (GtkAction *action, EMsgComposer *composer) @@ -353,13 +328,6 @@ static GtkActionEntry entries[] = { N_("Send this message"), G_CALLBACK (action_send_cb) }, - { "send-options", - NULL, - N_("_Send Options"), - NULL, - N_("Insert Send options"), - G_CALLBACK (action_send_options_cb) }, - { "new-message", "mail-message-new", N_("New _Message"), diff --git a/composer/e-composer-actions.h b/composer/e-composer-actions.h index 5d49cde158..18c75eb3be 100644 --- a/composer/e-composer-actions.h +++ b/composer/e-composer-actions.h @@ -47,8 +47,6 @@ E_COMPOSER_ACTION ((composer), "security-menu") #define E_COMPOSER_ACTION_SEND(composer) \ E_COMPOSER_ACTION ((composer), "send") -#define E_COMPOSER_ACTION_SEND_OPTIONS(composer) \ - E_COMPOSER_ACTION ((composer), "send-options") #define E_COMPOSER_ACTION_NEW_MESSAGE(composer) \ E_COMPOSER_ACTION ((composer), "new-message") #define E_COMPOSER_ACTION_SMIME_ENCRYPT(composer) \ diff --git a/composer/e-composer-private.h b/composer/e-composer-private.h index 97164e3513..7695df5179 100644 --- a/composer/e-composer-private.h +++ b/composer/e-composer-private.h @@ -115,7 +115,6 @@ struct _EMsgComposerPrivate { CamelMimeMessage *redirect; - gboolean send_invoked; gboolean is_from_message; }; diff --git a/composer/e-msg-composer.c b/composer/e-msg-composer.c index bc1ce20dd6..6f4aee2ff7 100644 --- a/composer/e-msg-composer.c +++ b/composer/e-msg-composer.c @@ -1353,7 +1353,6 @@ msg_composer_account_changed_cb (EMsgComposer *composer) ESignature *signature; EAccount *account; gboolean active; - gboolean sensitive; const gchar *uid; table = e_msg_composer_get_header_table (composer); @@ -1380,57 +1379,11 @@ msg_composer_account_changed_cb (EMsgComposer *composer) signature = uid ? e_get_signature_by_uid (uid) : NULL; e_composer_header_table_set_signature (table, signature); - /* XXX This should be done more generically. The composer - * should not know about particular account types. */ - sensitive = - (strstr (account->transport->url, "exchange") != NULL) || - (strstr (account->transport->url, "groupwise") != NULL); - gtk_action_set_sensitive (ACTION (SEND_OPTIONS), sensitive); - -exit: + exit: e_msg_composer_show_sig_file (composer); } -static void -msg_composer_account_list_changed_cb (EMsgComposer *composer) -{ - EComposerHeaderTable *table; - EAccountList *account_list; - EIterator *iterator; - gboolean visible = FALSE; - - /* Determine whether to show the "send-options" action by - * examining the account list for account types that support it. - * - * XXX I'd prefer a more general way of doing this. The composer - * should not know about particular account types. Perhaps - * add a "supports advanced send options" flag to EAccount. */ - - table = E_COMPOSER_HEADER_TABLE (composer->priv->header_table); - account_list = e_composer_header_table_get_account_list (table); - iterator = e_list_get_iterator (E_LIST (account_list)); - - while (!visible && e_iterator_is_valid (iterator)) { - EAccount *account; - const gchar *url; - - /* XXX EIterator misuses const. */ - account = (EAccount *) e_iterator_get (iterator); - e_iterator_next (iterator); - - if (!account->enabled) - continue; - - url = account->transport->url; - visible |= (strstr (url, "exchange") != NULL); - visible |= (strstr (url, "groupwise") != NULL); - } - - gtk_action_set_visible (ACTION (SEND_OPTIONS), visible); - g_object_unref (iterator); -} - struct _drop_data { EMsgComposer *composer; @@ -2137,9 +2090,6 @@ msg_composer_init (EMsgComposer *composer) g_signal_connect_swapped ( table, "notify::account", G_CALLBACK (msg_composer_account_changed_cb), composer); - g_signal_connect_swapped ( - table, "notify::account-list", - G_CALLBACK (msg_composer_account_list_changed_cb), composer); g_signal_connect_swapped ( table, "notify::destinations-bcc", G_CALLBACK (msg_composer_notify_header_cb), composer); @@ -2163,7 +2113,6 @@ msg_composer_init (EMsgComposer *composer) G_CALLBACK (msg_composer_notify_header_cb), composer); msg_composer_account_changed_cb (composer); - msg_composer_account_list_changed_cb (composer); /* Attachments */ @@ -3991,15 +3940,6 @@ e_msg_composer_get_attachment_view (EMsgComposer *composer) return E_ATTACHMENT_VIEW (composer->priv->attachment_paned); } -void -e_msg_composer_set_send_options (EMsgComposer *composer, - gboolean send_enable) -{ - g_return_if_fail (E_IS_MSG_COMPOSER (composer)); - - composer->priv->send_invoked = send_enable; -} - GList * e_load_spell_languages (void) { diff --git a/composer/e-msg-composer.h b/composer/e-msg-composer.h index f1e760ff47..ab30093294 100644 --- a/composer/e-msg-composer.h +++ b/composer/e-msg-composer.h @@ -145,8 +145,6 @@ EComposerHeaderTable * EAttachmentView * e_msg_composer_get_attachment_view (EMsgComposer *composer); -void e_msg_composer_set_send_options (EMsgComposer *composer, - gboolean send_enable); GByteArray * e_msg_composer_get_raw_message_text (EMsgComposer *composer); diff --git a/composer/evolution-composer.ui b/composer/evolution-composer.ui index 1e72d9eafc..f0e9c01de9 100644 --- a/composer/evolution-composer.ui +++ b/composer/evolution-composer.ui @@ -39,7 +39,7 @@ - + diff --git a/composer/mail-composer.error.xml b/composer/mail-composer.error.xml index a13fe34054..d622c80520 100644 --- a/composer/mail-composer.error.xml +++ b/composer/mail-composer.error.xml @@ -76,9 +76,4 @@ Please make sure that you have the correct version of gtkhtml and libgtkhtml installed. - - - <_primary>Send options not available. - <_secondary>Send options available only for Novell GroupWise and Microsoft Exchange accounts. - -- cgit v1.2.3