X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fmaemo%2Fmodest-account-settings-dialog.c;h=c0d4d755dc0f49164875bcbeec82138a839c340e;hp=0e67eaa21bc8cba8f1665bcbc51aa09f5804f9af;hb=53dc7de1843cd26cd027d1f8653165f769c7e2ed;hpb=5e81b9393b137ad4c3163deffde5f24952fb1ef4 diff --git a/src/maemo/modest-account-settings-dialog.c b/src/maemo/modest-account-settings-dialog.c index 0e67eaa..c0d4d75 100644 --- a/src/maemo/modest-account-settings-dialog.c +++ b/src/maemo/modest-account-settings-dialog.c @@ -50,13 +50,16 @@ #include "modest-account-mgr.h" #include "modest-account-mgr-helpers.h" /* For modest_account_mgr_get_account_data(). */ #include "modest-runtime.h" /* For modest_runtime_get_account_mgr(). */ +#include "modest-protocol-info.h" #include "maemo/modest-connection-specific-smtp-window.h" #include "maemo/modest-signature-editor-dialog.h" +#include "maemo/modest-maemo-utils.h" #include "widgets/modest-ui-constants.h" #include #include #include +#include #include #include /* For strlen(). */ @@ -205,19 +208,18 @@ on_caption_combobox_changed (GtkComboBox *widget, gpointer user_data) /** This is a convenience function to create a caption containing a mandatory widget. * When the widget is edited, the enable_buttons() vfunc will be called. */ -static GtkWidget* create_caption_new_with_asterix(ModestAccountSettingsDialog *self, +static GtkWidget* create_caption_new_with_asterisk(ModestAccountSettingsDialog *self, GtkSizeGroup *group, const gchar *value, GtkWidget *control, GtkWidget *icon, HildonCaptionStatus flag) { - GtkWidget *caption = hildon_caption_new (group, value, control, icon, flag); + GtkWidget *caption = NULL; -/* The translated strings seem to already contain the *, - * but this code can be used if that is not true in future. - */ -#if 0 + /* Note: Previously, the translated strings already contained the "*", + * Comment out this code if they do again. + */ /* Add a * character to indicate mandatory fields, * as specified in our "Email UI Specification": */ if (flag == HILDON_CAPTION_MANDATORY) { @@ -227,7 +229,6 @@ static GtkWidget* create_caption_new_with_asterix(ModestAccountSettingsDialog *s } else caption = hildon_caption_new (group, value, control, icon, flag); -#endif /* Connect to the appropriate changed signal for the widget, * so we can ask for the prev/next buttons to be enabled/disabled appropriately: @@ -271,7 +272,7 @@ create_page_account_details (ModestAccountSettingsDialog *self) /* Do use auto-capitalization: */ hildon_gtk_entry_set_input_mode (GTK_ENTRY (self->entry_account_title), HILDON_GTK_INPUT_MODE_FULL | HILDON_GTK_INPUT_MODE_AUTOCAP); - GtkWidget *caption = create_caption_new_with_asterix (self, sizegroup, _("mcen_fi_account_title"), + GtkWidget *caption = create_caption_new_with_asterisk (self, sizegroup, _("mcen_fi_account_title"), self->entry_account_title, NULL, HILDON_CAPTION_MANDATORY); gtk_widget_show (self->entry_account_title); connect_for_modified (self, self->entry_account_title); @@ -304,7 +305,7 @@ create_page_account_details (ModestAccountSettingsDialog *self) /* The retrieve combobox: */ self->combo_retrieve = GTK_WIDGET (modest_retrieve_combo_box_new ()); - caption = create_caption_new_with_asterix (self, sizegroup, _("mcen_fi_advsetup_retrievetype"), + caption = create_caption_new_with_asterisk (self, sizegroup, _("mcen_fi_advsetup_retrievetype"), self->combo_retrieve, NULL, HILDON_CAPTION_MANDATORY); gtk_widget_show (self->combo_retrieve); connect_for_modified (self, self->combo_retrieve); @@ -313,7 +314,7 @@ create_page_account_details (ModestAccountSettingsDialog *self) /* The limit-retrieve combobox: */ self->combo_limit_retrieve = GTK_WIDGET (modest_limit_retrieve_combo_box_new ()); - caption = create_caption_new_with_asterix (self, sizegroup, _("mcen_fi_advsetup_limit_retrieve"), + caption = create_caption_new_with_asterisk (self, sizegroup, _("mcen_fi_advsetup_limit_retrieve"), self->combo_limit_retrieve, NULL, HILDON_CAPTION_MANDATORY); gtk_widget_show (self->combo_limit_retrieve); connect_for_modified (self, self->combo_limit_retrieve); @@ -323,12 +324,15 @@ create_page_account_details (ModestAccountSettingsDialog *self) /* The leave-messages widgets: */ if(!self->checkbox_leave_messages) self->checkbox_leave_messages = gtk_check_button_new (); - caption = create_caption_new_with_asterix (self, sizegroup, _("mcen_fi_advsetup_leave_on_server"), - self->checkbox_leave_messages, NULL, HILDON_CAPTION_MANDATORY); + if (!self->caption_leave_messages) { + self->caption_leave_messages = create_caption_new_with_asterisk (self, sizegroup, _("mcen_fi_advsetup_leave_on_server"), + self->checkbox_leave_messages, NULL, HILDON_CAPTION_MANDATORY); + } + gtk_widget_show (self->checkbox_leave_messages); connect_for_modified (self, self->checkbox_leave_messages); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); - gtk_widget_show (caption); + gtk_box_pack_start (GTK_BOX (box), self->caption_leave_messages, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_widget_show (self->caption_leave_messages); gtk_widget_show (GTK_WIDGET (box)); @@ -390,7 +394,7 @@ create_page_user_details (ModestAccountSettingsDialog *self) gtk_entry_set_max_length (GTK_ENTRY (self->entry_user_name), 64); modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (self->entry_user_name), on_entry_max, self); - GtkWidget *caption = create_caption_new_with_asterix (self, sizegroup, + GtkWidget *caption = create_caption_new_with_asterisk (self, sizegroup, _("mcen_li_emailsetup_name"), self->entry_user_name, NULL, HILDON_CAPTION_OPTIONAL); gtk_widget_show (self->entry_user_name); connect_for_modified (self, self->entry_user_name); @@ -410,7 +414,7 @@ create_page_user_details (ModestAccountSettingsDialog *self) self->entry_user_username = GTK_WIDGET (modest_validating_entry_new ()); /* Auto-capitalization is the default, so let's turn it off: */ hildon_gtk_entry_set_input_mode (GTK_ENTRY (self->entry_user_username), HILDON_GTK_INPUT_MODE_FULL); - caption = create_caption_new_with_asterix (self, sizegroup, _("mail_fi_username"), + caption = create_caption_new_with_asterisk (self, sizegroup, _("mail_fi_username"), self->entry_user_username, NULL, HILDON_CAPTION_MANDATORY); gtk_widget_show (self->entry_user_username); connect_for_modified (self, self->entry_user_username); @@ -435,7 +439,7 @@ create_page_user_details (ModestAccountSettingsDialog *self) HILDON_GTK_INPUT_MODE_FULL | HILDON_GTK_INPUT_MODE_INVISIBLE); gtk_entry_set_visibility (GTK_ENTRY (self->entry_user_password), FALSE); /* gtk_entry_set_invisible_char (GTK_ENTRY (self->entry_user_password), '*'); */ - caption = create_caption_new_with_asterix (self, sizegroup, + caption = create_caption_new_with_asterisk (self, sizegroup, _("mail_fi_password"), self->entry_user_password, NULL, HILDON_CAPTION_OPTIONAL); gtk_widget_show (self->entry_user_password); connect_for_modified (self, self->entry_user_password); @@ -446,7 +450,7 @@ create_page_user_details (ModestAccountSettingsDialog *self) self->entry_user_email = GTK_WIDGET (modest_validating_entry_new ()); /* Auto-capitalization is the default, so let's turn it off: */ hildon_gtk_entry_set_input_mode (GTK_ENTRY (self->entry_user_email), HILDON_GTK_INPUT_MODE_FULL); - caption = create_caption_new_with_asterix (self, sizegroup, + caption = create_caption_new_with_asterisk (self, sizegroup, _("mcen_li_emailsetup_email_address"), self->entry_user_email, NULL, HILDON_CAPTION_MANDATORY); gtk_entry_set_text (GTK_ENTRY (self->entry_user_email), EXAMPLE_EMAIL_ADDRESS); /* Default text. */ gtk_widget_show (self->entry_user_email); @@ -493,8 +497,14 @@ static void update_incoming_server_title (ModestAccountSettingsDialog *self, Mod * because the compiler does not know that the translated string will have a %s in it. * I do not see a way to avoid the warning while still using these Logical IDs. murrayc. */ gchar* incomingserver_title = g_strdup_printf(_("mcen_li_emailsetup_servertype"), type); - g_object_set (G_OBJECT (self->caption_incoming), "label", incomingserver_title, NULL); - g_free(incomingserver_title); + + /* This is a mandatory field, so add a *. This is usually done by + * create_caption_new_with_asterisk() but we can't use that here. */ + gchar *with_asterisk = g_strconcat (incomingserver_title, "*", NULL); + g_free (incomingserver_title); + + g_object_set (G_OBJECT (self->caption_incoming), "label", with_asterisk, NULL); + g_free(with_asterisk); } /** Change the caption title for the incoming server, @@ -529,7 +539,7 @@ static GtkWidget* create_page_incoming (ModestAccountSettingsDialog *self) /* The caption title will be updated in update_incoming_server_title(). * so this default text will never be seen: */ /* (Note: Changing the title seems pointless. murrayc) */ - self->caption_incoming = create_caption_new_with_asterix (self, sizegroup, + self->caption_incoming = create_caption_new_with_asterisk (self, sizegroup, "Incoming Server", self->entry_incomingserver, NULL, HILDON_CAPTION_MANDATORY); gtk_widget_show (self->entry_incomingserver); connect_for_modified (self, self->entry_incomingserver); @@ -677,7 +687,7 @@ static GtkWidget* create_page_outgoing (ModestAccountSettingsDialog *self) self->entry_outgoingserver = gtk_entry_new (); /* Auto-capitalization is the default, so let's turn it off: */ hildon_gtk_entry_set_input_mode (GTK_ENTRY (self->entry_outgoingserver), HILDON_GTK_INPUT_MODE_FULL); - GtkWidget *caption = create_caption_new_with_asterix (self, sizegroup, + GtkWidget *caption = create_caption_new_with_asterisk (self, sizegroup, _("mcen_li_emailsetup_smtp"), self->entry_outgoingserver, NULL, HILDON_CAPTION_OPTIONAL); gtk_widget_show (self->entry_outgoingserver); connect_for_modified (self, self->entry_outgoingserver); @@ -701,7 +711,7 @@ static GtkWidget* create_page_outgoing (ModestAccountSettingsDialog *self) self->entry_outgoing_username = GTK_WIDGET (modest_validating_entry_new ()); /* Auto-capitalization is the default, so let's turn it off: */ hildon_gtk_entry_set_input_mode (GTK_ENTRY (self->entry_outgoing_username), HILDON_GTK_INPUT_MODE_FULL); - self->caption_outgoing_username = create_caption_new_with_asterix (self, sizegroup, _("mail_fi_username"), + self->caption_outgoing_username = create_caption_new_with_asterisk (self, sizegroup, _("mail_fi_username"), self->entry_outgoing_username, NULL, HILDON_CAPTION_MANDATORY); gtk_widget_show (self->entry_outgoing_username); connect_for_modified (self, self->entry_outgoing_username); @@ -726,7 +736,7 @@ static GtkWidget* create_page_outgoing (ModestAccountSettingsDialog *self) HILDON_GTK_INPUT_MODE_FULL | HILDON_GTK_INPUT_MODE_INVISIBLE); gtk_entry_set_visibility (GTK_ENTRY (self->entry_outgoing_password), FALSE); /* gtk_entry_set_invisible_char (GTK_ENTRY (self->entry_outgoing_password), '*'); */ - self->caption_outgoing_password = create_caption_new_with_asterix (self, sizegroup, + self->caption_outgoing_password = create_caption_new_with_asterisk (self, sizegroup, _("mail_fi_password"), self->entry_outgoing_password, NULL, HILDON_CAPTION_OPTIONAL); gtk_widget_show (self->entry_outgoing_password); connect_for_modified (self, self->entry_outgoing_password); @@ -738,6 +748,7 @@ static GtkWidget* create_page_outgoing (ModestAccountSettingsDialog *self) * and modest_serversecurity_combo_box_set_active_serversecurity(). */ if (!self->combo_outgoing_security) + self->combo_outgoing_security = GTK_WIDGET (modest_serversecurity_combo_box_new ()); caption = hildon_caption_new (sizegroup, _("mcen_li_emailsetup_secure_connection"), self->combo_outgoing_security, NULL, HILDON_CAPTION_OPTIONAL); @@ -813,7 +824,7 @@ check_data (ModestAccountSettingsDialog *self) if (name_in_use) { /* Warn the user via a dialog: */ - show_error (GTK_WINDOW (self), _("mail_ib_account_name_already_existing")); + hildon_banner_show_information(NULL, NULL, _("mail_ib_account_name_already_existing")); return FALSE; } @@ -824,15 +835,73 @@ check_data (ModestAccountSettingsDialog *self) if ((!email_address) || (strlen(email_address) == 0)) return FALSE; - if (!modest_text_utils_validate_email_address (email_address)) { + if (!modest_text_utils_validate_email_address (email_address, NULL)) { /* Warn the user via a dialog: */ - show_error (GTK_WINDOW (self), _("mcen_ib_invalid_email")); + /*show_error (GTK_WINDOW (self), _("mcen_ib_invalid_email"));*/ + hildon_banner_show_information (NULL, NULL, _("mcen_ib_invalid_email")); - /* Return focus to the email address entry: */ - gtk_widget_grab_focus (self->entry_user_email); - + /* Return focus to the email address entry: */ + gtk_widget_grab_focus (self->entry_user_email); + gtk_editable_select_region (GTK_EDITABLE (self->entry_user_email), 0, -1); return FALSE; } + + /* Find a suitable authentication method when secure authentication is desired */ + const gchar* hostname = gtk_entry_get_text (GTK_ENTRY (self->entry_incomingserver)); + gint port_num = hildon_number_editor_get_value ( + HILDON_NUMBER_EDITOR (self->entry_incoming_port)); + const gchar* username = gtk_entry_get_text (GTK_ENTRY (self->entry_user_username)); + + const ModestConnectionProtocol protocol_security_incoming = modest_serversecurity_combo_box_get_active_serversecurity ( + MODEST_SERVERSECURITY_COMBO_BOX (self->combo_incoming_security)); + + /* If we use an encrypted protocol then there is no need to encrypt the password */ + if (!modest_protocol_info_is_secure(protocol_security_incoming)) + { + if (gtk_toggle_button_get_active ( + GTK_TOGGLE_BUTTON (self->checkbox_incoming_auth))) { + GError *error = NULL; + GList *list_auth_methods = + modest_maemo_utils_get_supported_secure_authentication_methods (self->incoming_protocol, + hostname, port_num, username, GTK_WINDOW (self), &error); + if (list_auth_methods) { + /* Use the first supported method. + * TODO: Should we prioritize them, to prefer a particular one? */ + GList* method; + for (method = list_auth_methods; method != NULL; method = g_list_next(method)) + { + ModestAuthProtocol proto = (ModestAuthProtocol)(GPOINTER_TO_INT(list_auth_methods->data)); + // Allow secure methods, e.g MD5 only + if (modest_protocol_info_auth_is_secure(proto)) + { + self->protocol_authentication_incoming = proto; + break; + } + } + g_list_free (list_auth_methods); + } + + if (list_auth_methods == NULL || + !modest_protocol_info_auth_is_secure(self->protocol_authentication_incoming)) + { + if(error == NULL || error->domain != modest_maemo_utils_get_supported_secure_authentication_error_quark() || + error->code != MODEST_MAEMO_UTILS_GET_SUPPORTED_SECURE_AUTHENTICATION_ERROR_CANCELED) + { + GtkWidget* error_dialog = gtk_message_dialog_new(GTK_WINDOW(self), + GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, + GTK_BUTTONS_OK, (error != NULL) ? error->message : _("Server does not support secure authentication!")); + gtk_dialog_run(GTK_DIALOG(error_dialog)); + gtk_widget_destroy(error_dialog); + } + + if(error != NULL) g_error_free(error); + /* This is a nasty hack. jschmid. */ + /* Don't let the dialog close */ + /*g_signal_stop_emission_by_name (dialog, "response");*/ + return FALSE; + } + } + } /* TODO: The UI Spec wants us to check that the servernames are valid, * but does not specify how. @@ -872,25 +941,28 @@ on_response (GtkDialog *wizard_dialog, if (prevent_response) { /* This is a nasty hack. murrayc. */ /* Don't let the dialog close */ - g_signal_stop_emission_by_name (wizard_dialog, "response"); + g_signal_stop_emission_by_name (wizard_dialog, "response"); return; } if (response_id == GTK_RESPONSE_OK) { - /* Try to save the changes: */ - const gboolean saved = save_configuration (self); - if (saved) { - /* Do not show the account-saved dialog if we are just saving this - * temporarily, because from the user's point of view it will not - * really be saved (saved + enabled) until later. - */ - const gboolean enabled = - modest_account_mgr_get_enabled (self->account_manager, self->account_name); - if (enabled) - show_ok (GTK_WINDOW (self), _("mcen_ib_advsetup_settings_saved")); + /* Try to save the changes if modified (NB #59251): */ + if (self->modified) + { + const gboolean saved = save_configuration (self); + if (saved) { + /* Do not show the account-saved dialog if we are just saving this + * temporarily, because from the user's point of view it will not + * really be saved (saved + enabled) until later. + */ + const gboolean enabled = + modest_account_mgr_get_enabled (self->account_manager, self->account_name); + if (enabled) + show_ok (GTK_WINDOW (self), _("mcen_ib_advsetup_settings_saved")); + } + else + show_error (GTK_WINDOW (self), _("mail_ib_setting_failed")); } - else - show_error (GTK_WINDOW (self), _("mail_ib_setting_failed")); } } @@ -901,9 +973,6 @@ modest_account_settings_dialog_init (ModestAccountSettingsDialog *self) * Each page of the notebook will be a page of the wizard: */ GtkNotebook *notebook = GTK_NOTEBOOK (gtk_notebook_new()); - - gtk_window_set_title (GTK_WINDOW (self), _("mcen_ti_emailsetup")); - /* Get the account manager object, * so we can check for existing accounts, * and create new accounts: */ @@ -911,6 +980,8 @@ modest_account_settings_dialog_init (ModestAccountSettingsDialog *self) g_assert (self->account_manager); g_object_ref (self->account_manager); + self->protocol_authentication_incoming = MODEST_PROTOCOL_AUTH_PASSWORD; + /* Create the common pages, */ self->page_account_details = create_page_account_details (self); @@ -1029,11 +1100,12 @@ void modest_account_settings_dialog_set_account_name (ModestAccountSettingsDialo /* Only show the leave-on-server checkbox for POP, * as per the UI spec: */ if (incoming_account->proto != MODEST_PROTOCOL_STORE_POP) { - gtk_widget_hide (dialog->checkbox_leave_messages); + gtk_widget_hide (dialog->caption_leave_messages); } else { - gtk_widget_show (dialog->checkbox_leave_messages); + gtk_widget_show (dialog->caption_leave_messages); } - + + update_incoming_server_security_choices (dialog, incoming_account->proto); if (incoming_account) { /* Remember this for later: */ dialog->incoming_protocol = incoming_account->proto; @@ -1050,21 +1122,34 @@ void modest_account_settings_dialog_set_account_name (ModestAccountSettingsDialo * If secure authentication is unchecked, allow sending username and password also as plain text. * If secure authentication is checked, require one of the secure methods during connection: SSL, TLS, CRAM-MD5 etc. * TODO: Do we need to discover which of these (SSL, TLS, CRAM-MD5) is supported? - */ - const ModestAuthProtocol secure_auth = modest_server_account_get_secure_auth( - dialog->account_manager, incoming_account->account_name); - gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON (dialog->checkbox_incoming_auth), - secure_auth != MODEST_PROTOCOL_AUTH_PASSWORD); - /* Note that MODEST_PROTOCOL_AUTH_PLAIN should probably never be used. */ - - update_incoming_server_title (dialog, incoming_account->proto); - update_incoming_server_security_choices (dialog, incoming_account->proto); - + */ const ModestConnectionProtocol security = modest_server_account_get_security ( dialog->account_manager, incoming_account->account_name); modest_serversecurity_combo_box_set_active_serversecurity ( MODEST_SERVERSECURITY_COMBO_BOX (dialog->combo_incoming_security), security); + /* Check if we have + - a secure protocol + OR + - use encrypted passwords + */ + const ModestAuthProtocol secure_auth = modest_server_account_get_secure_auth( + dialog->account_manager, incoming_account->account_name); + dialog->protocol_authentication_incoming = secure_auth; + if (modest_protocol_info_is_secure(security) || + modest_protocol_info_auth_is_secure(secure_auth)) + { + gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON (dialog->checkbox_incoming_auth), + TRUE); + } + else + { + gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON (dialog->checkbox_incoming_auth), + FALSE); + }; + + update_incoming_server_title (dialog, incoming_account->proto); + const gint port_num = modest_account_mgr_get_int (dialog->account_manager, incoming_account->account_name, MODEST_ACCOUNT_PORT, TRUE /* server account */); @@ -1122,8 +1207,30 @@ void modest_account_settings_dialog_set_account_name (ModestAccountSettingsDialo hildon_number_editor_set_value ( HILDON_NUMBER_EDITOR (dialog->entry_outgoing_port), port_num); } + + const gboolean has_specific = + modest_account_mgr_get_has_connection_specific_smtp ( + dialog->account_manager, + account_name); + gtk_toggle_button_set_active ( + GTK_TOGGLE_BUTTON (dialog->checkbox_outgoing_smtp_specific), + has_specific); } - + + /* Set window title according to account: */ + /* TODO: Is this the correct way to find a human-readable name for + * the protocol used? */ + const gchar* proto_str = modest_protocol_info_get_transport_store_protocol_name (dialog->incoming_protocol); + gchar *proto_name = g_utf8_strup(proto_str, -1); + gchar *account_title = modest_account_mgr_get_display_name(dialog->account_manager, account_name); + + gchar *title = g_strdup_printf(_("mcen_ti_account_settings"), proto_name, account_title); + g_free (proto_name); + g_free (account_title); + + gtk_window_set_title (GTK_WINDOW (dialog), title); + g_free (title); + /* account_data->is_enabled, */ /*account_data->is_default, */ @@ -1135,9 +1242,6 @@ void modest_account_settings_dialog_set_account_name (ModestAccountSettingsDialo dialog->modified = FALSE; } -static GList* get_supported_secure_authentication_methods (ModestTransportStoreProtocol proto, - const gchar* hostname, gint port, GtkWindow *parent_window); - static gboolean save_configuration (ModestAccountSettingsDialog *dialog) { @@ -1214,32 +1318,12 @@ save_configuration (ModestAccountSettingsDialog *dialog) * If secure authentication is unchecked, allow sending username and password also as plain text. * If secure authentication is checked, require one of the secure methods during connection: SSL, TLS, CRAM-MD5 etc. */ - ModestAuthProtocol protocol_authentication_incoming = - MODEST_PROTOCOL_AUTH_PASSWORD; - if (gtk_toggle_button_get_active ( - GTK_TOGGLE_BUTTON (dialog->checkbox_incoming_auth))) { - GList *list_auth_methods = - get_supported_secure_authentication_methods (dialog->incoming_protocol, - hostname, port_num, GTK_WINDOW (dialog)); - if (list_auth_methods) { - /* Use the first supported method. - * TODO: Should we prioritize them, to prefer a particular one? */ - protocol_authentication_incoming = - (ModestAuthProtocol)(GPOINTER_TO_INT(list_auth_methods->data)); - g_list_free (list_auth_methods); - } - else - g_warning ("%s: get_supported_secure_authentication_methods() returned NULL.\n", __FUNCTION__); - - } - - modest_server_account_set_secure_auth (dialog->account_manager, incoming_account_name, protocol_authentication_incoming); - const ModestConnectionProtocol protocol_security_incoming = modest_serversecurity_combo_box_get_active_serversecurity ( MODEST_SERVERSECURITY_COMBO_BOX (dialog->combo_incoming_security)); modest_server_account_set_security (dialog->account_manager, incoming_account_name, protocol_security_incoming); + modest_server_account_set_secure_auth (dialog->account_manager, incoming_account_name, dialog->protocol_authentication_incoming); g_free (incoming_account_name); @@ -1294,9 +1378,10 @@ save_configuration (ModestAccountSettingsDialog *dialog) } /* Save connection-specific SMTP server accounts: */ - if (dialog->specific_window) + if (dialog->specific_window) { return modest_connection_specific_smtp_window_save_server_accounts ( MODEST_CONNECTION_SPECIFIC_SMTP_WINDOW (dialog->specific_window), account_name); + } else return TRUE; } @@ -1375,173 +1460,7 @@ show_error (GtkWindow *parent_window, const gchar* text) static void show_ok (GtkWindow *parent_window, const gchar* text) { - GtkDialog *dialog = GTK_DIALOG (hildon_note_new_information (parent_window, text)); - /* - GtkDialog *dialog = GTK_DIALOG (gtk_message_dialog_new (parent_window, - (GtkDialogFlags)0, - GTK_MESSAGE_INFO, - GTK_BUTTONS_OK, - text )); - */ - - gtk_dialog_run (dialog); - gtk_widget_destroy (GTK_WIDGET (dialog)); -} - -/* TODO: Enable this when tinymail has the API: */ -#if 0 -typedef struct -{ - gboolean finished; - GList *result; -} ModestGetSupportedAuthInfo; - -static void on_camel_account_get_supported_secure_authentication_status ( - GObject *self, TnyStatus *status, gpointer user_data) -{ - printf ("DEBUG: %s.\n", __FUNCTION__); -} - -static void -on_camel_account_get_supported_secure_authentication ( - TnyCamelAccount *self, gboolean cancelled, - GList *auth_types, GError **err, - gpointer user_data) -{ - printf ("DEBUG: %s.\n", __FUNCTION__); - - ModestGetSupportedAuthInfo *info = (ModestGetSupportedAuthInfo*)user_data; - g_return_if_fail (info); - - if (!auth_types) { - printf ("DEBUG: %s: auth_types is NULL.\n", __FUNCTION__); - info->finished = TRUE; /* We are blocking, waiting for this. */ - return; - } - - ModestPairList* pairs = modest_protocol_info_get_protocol_auth_pair_list (); - - /* Get the enum value for the strings: */ - GList *result = NULL; - GList* iter = auth_types; - while (iter) { - const gchar *auth_name = (const gchar*)iter->data; - printf("DEBUG: %s: auth_name=%s\n", __FUNCTION__, auth_name); - ModestPair *matching = modest_pair_list_find_by_first_as_string (pairs, - auth_name); - if (matching) - g_list_append (result, GINT_TO_POINTER((ModestConnectionProtocol)matching->second)); - - iter = g_list_next (iter); - } - - g_list_free (auth_types); - - modest_pair_list_free (pairs); - - info->result = result; - info->finished = TRUE; /* We are blocking, waiting for this. */ -} -#endif - - -static GList* get_supported_secure_authentication_methods (ModestTransportStoreProtocol proto, - const gchar* hostname, gint port, GtkWindow *parent_window) -{ - return NULL; - -/* TODO: Enable this when tinymail has the API: */ -#if 0 - g_return_val_if_fail (proto != MODEST_PROTOCOL_TRANSPORT_STORE_UNKNOWN, NULL); - - /* - result = g_list_append (result, GINT_TO_POINTER (MODEST_PROTOCOL_AUTH_CRAMMD5)); - */ - - /* Create a TnyCamelAccount so we can use - * tny_camel_account_get_supported_secure_authentication(): */ - TnyAccount * tny_account = NULL; - switch (proto) { - case MODEST_PROTOCOL_TRANSPORT_SENDMAIL: - case MODEST_PROTOCOL_TRANSPORT_SMTP: - tny_account = TNY_ACCOUNT(tny_camel_transport_account_new ()); break; - case MODEST_PROTOCOL_STORE_POP: - tny_account = TNY_ACCOUNT(tny_camel_pop_store_account_new ()); break; - case MODEST_PROTOCOL_STORE_IMAP: - tny_account = TNY_ACCOUNT(tny_camel_imap_store_account_new ()); break; - case MODEST_PROTOCOL_STORE_MAILDIR: - case MODEST_PROTOCOL_STORE_MBOX: - tny_account = TNY_ACCOUNT(tny_camel_store_account_new()); break; - default: - tny_account = NULL; - } - - if (!tny_account) { - g_printerr ("%s could not create tny account.", __FUNCTION__); - return NULL; - } - - /* Set proto, so that the prepare_func() vfunc will work when we call - * set_session(): */ - /* TODO: Why isn't this done in account_new()? */ - tny_account_set_proto (tny_account, - modest_protocol_info_get_transport_store_protocol_name(proto)); - - /* Set the session for the account, so we can use it: */ - ModestTnyAccountStore *account_store = modest_runtime_get_account_store (); - TnySessionCamel *session = - modest_tny_account_store_get_session (TNY_ACCOUNT_STORE (account_store)); - g_return_val_if_fail (session, NULL); - tny_camel_account_set_session (TNY_CAMEL_ACCOUNT(tny_account), session); - - tny_account_set_hostname (tny_account, hostname); - - if(port > 0) - tny_account_set_port (tny_account, port); - - - /* Ask camel to ask the server, asynchronously: */ - ModestGetSupportedAuthInfo *info = g_slice_new (ModestGetSupportedAuthInfo); - info->finished = FALSE; - info->result = NULL; - - GtkDialog *dialog = GTK_DIALOG (hildon_note_new_information (parent_window, - _("Asking the server for supported secure authentication mechanisms."))); - gtk_dialog_run (dialog); - - printf ("DEBUG: %s: STARTING.\n", __FUNCTION__); - tny_camel_account_get_supported_secure_authentication ( - TNY_CAMEL_ACCOUNT (tny_account), - on_camel_account_get_supported_secure_authentication, - on_camel_account_get_supported_secure_authentication_status, - info); - - printf ("DEBUG: %s: AFTER STARTING.\n", __FUNCTION__); - - /* Block until the callback has been called, - * driving the main context, so that the (idle handler) callback can be - * called, and so that our dialog is clickable: */ - while (!(info->finished)) { - printf ("DEBUG: %s: finished is FALSE.\n", __FUNCTION__); - if (g_main_context_pending (NULL)) { - printf ("DEBUG: iterating\n"); - g_main_context_iteration (NULL, FALSE); - printf ("DEBUG: after iterating\n"); - } - } - - printf ("DEBUG: %s: FINISHED.\n", __FUNCTION__); - - gtk_widget_destroy (GTK_WIDGET (dialog)); - - GList *result = info->result; - g_slice_free (ModestGetSupportedAuthInfo, info); - info = NULL; - - return result; -#endif + /* Don't show a dialog but Banner (NB #59248) */ + hildon_banner_show_information(GTK_WIDGET( + gtk_widget_get_parent_window(GTK_WIDGET(parent_window))), NULL, text); } - - - -