X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=ChangeLog2;h=f6b93dd96020218edb2e4c608d4d09e70be569df;hp=112175ff4ddcc4b36f5048546e3a13df04c60590;hb=1807d0ffbcc503686889d878c8a37b5c31f30b23;hpb=74cae6c0690b9f7a85bd3909407bad8cdc040cf3 diff --git a/ChangeLog2 b/ChangeLog2 index 112175f..f6b93dd 100644 --- a/ChangeLog2 +++ b/ChangeLog2 @@ -1,3 +1,355 @@ +2007-05-15 Murray Cumming + + * src/modest-protocol-info.c: + * src/modest-protocol-info.h: Adde enum-specific get functions, + instead of using a ModestProtocolType enum parameter. + + * src/gnome/modest-account-assistant.c: + * src/gnome/modest-store-widget.c: + * src/gnome/modest-store-widget.h: + * src/gnome/modest-transport-widget.c: + * src/gnome/modest-transport-widget.h: + * src/maemo/easysetup/modest-easysetup-servertype-combo-box.c: + * src/maemo/easysetup/modest-easysetup-servertype-combo-box.h: + * src/maemo/easysetup/modest-easysetup-wizard.c: + * src/maemo/modest-account-settings-dialog.c: + * src/maemo/modest-account-settings-dialog.h: + * src/maemo/modest-store-widget.c: + * src/maemo/modest-store-widget.h: + * src/maemo/modest-transport-widget.c: + * src/maemo/modest-transport-widget.h: + * src/modest-account-mgr-helpers.c: + * src/modest-account-mgr-helpers.h: + * src/modest-account-mgr.c: + * src/modest-account-mgr.h: + * src/modest-tny-account.c: + * src/modest-tny-folder.c: + * src/widgets/modest-account-view.c: + * src/widgets/modest-retrieve-combo-box.c: + * src/widgets/modest-retrieve-combo-box.h: + * src/widgets/modest-secureauth-combo-box.c: + * src/widgets/modest-serversecurity-combo-box.c: + * src/widgets/modest-serversecurity-combo-box.h: + * tests/check_account-mgr.c: + Use the specific functions, and use the re-renamed enum values for GNOME too. + +2007-05-15 Murray Cumming + + * src/modest-pair.h: + * src/modest-pair.c: Added modest_pair_list_find_by_first_as_string(), + so we can get the second based on the first. + + * src/maemo/modest-account-settings-dialog.h: + * src/maemo/modest-account-settings-dialog.c: + Added get_supported_secure_authentication_methods(), though it is mostly + commented-out for now until the necessary API is committed to tinymail. + (modest_account_settings_dialog_set_account_name), + (save_configuration): Request the supported authentication methods from the + server, so we can choose a working method. + + * src/modest-tny-account-store.h: + * src/modest-tny-account-store.c: + (modest_tny_account_store_get_session): Rename from + tny_account_store_get_session(). + +2007-05-14 Murray Cumming + + * src/modest-protocol-info.h: Split the ModestProtocol enum + into ModestProtocol, ModestSecureConnection, and ModestSecureAuthentication. + There was no need to have these unrelated values in one enum. + + * src/: many files: Adapted to changed enum. This makes the code a bit + clearer, and the compiler could catch some errors. + +2007-05-14 Murray Cumming + + * src/modest-tny-account-store.c: (modest_tny_account_store_alert): + Show the detailed internal camel error message when an error happens + during the configuration, so that our testers can give us more clues. + For instance, in projects.maemo.org bug NB#56910 . + +2007-05-14 Murray Cumming + + * src/modest-tny-account.c: + (modest_tny_account_new_from_server_account): Adapt to changed tinymail API: + change tny_account_set_mech() to tny_account_set_secure_auth_mech(). + +2007-05-14 Murray Cumming + + * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account): + * src/maemo/modest-account-settings-dialog.c: + (modest_account_settings_dialog_set_account_name), + (save_configuration): For the incoming secure authentication checkbox, + use PASSWORD for unchecked, and CRAM-MD5 for checked, after I reread + the UI specification. PLAIN does not seem to be supported for most IMAP + servers anyway, and I am not sure what it would mean. + However, we probably need to discover which of the secure-authentication + mechanisms are supported by the server, instead of hard-coding CRAM-MD5. + * src/modest-account-mgr-helpers.c: + (modest_account_mgr_get_server_account_data): + + * src/modest-tny-account.c: + (modest_tny_account_new_from_server_account): + Use tny_account_set_mech() to set secure-authentication methods, + with some special-casing for IMAP, based on my observations of how + evolution behaves. + + * src/modest-account-mgr-helpers.h: + * src/modest-account-mgr.c: + * src/modest-defs.h: Comment that the URI is only used for local folders. + +2007-05-11 Murray Cumming + + * src/maemo/modest-main-window.c: (on_account_update): Make the gchar* + parameter const, just to be neat. + * src/modest-text-utils.c: (modest_text_utils_inline): + End g_strconcat() with NULL, as its documentation says. This is almost + certainly the cause of my crash at startup on the device. Interestingly, + valgrind doesn't tell us about these errors. + +2007-05-11 Murray Cumming + + * scripts/build-packages: Added more precise comments about how to use + this script, though I still get this error: + dpkg-genchanges: error: cannot open .dsc file ../tinymail_1.0-svn1938.dsc: No such file or directory + error building tinymail + +2007-05-11 Murray Cumming + + * src/maemo/modest-main-window.c: (on_account_update): + Very slight cleanup, hoping to make it simpler to discover what causes + a crash here sometimes. + + * src/modest-tny-account.c: + (modest_tny_account_new_from_server_account): Set the port. Added + comments about the options and a TODO comment because I need to find out + how to specify the secure authentication method. + +2007-05-11 Marcus Bauer + + * libmodest-dbus-client/libmodest-dbus-client-1.0.pc.in: + changed CFlags paths form modest to libmodest + +2007-05-10 Murray Cumming + + * src/modest-account-mgr-helpers.c: + (modest_account_mgr_get_server_account_data), + (modest_account_mgr_free_server_account_data): + * src/modest-account-mgr-helpers.h: + * src/modest-account-mgr.c: + (modest_account_mgr_add_server_account): + * src/modest-defs.h: + * src/modest-tny-account.c: + (modest_tny_account_new_from_server_account): + Determine the options for tny_camel_account_add_option() here, + based on the settings, rather than storing them directly in gconf. + +2007-05-10 Murray Cumming + + * src/modest-tny-account-store.c: (modest_tny_account_store_alert): + Show a more friendly error dialog now that we expect (from the tinymail + documentation) to get the UNKNOWN_ALERT error. Add explanatory comments. + +2007-05-10 Murray Cumming + + * src/maemo/modest-main-window.c: (on_account_update): Check that + account_name is not NULL, to prevent a crash at startup that I am + seeing on the N800. I do wonder why it is NULL. + + * src/maemo/easysetup/modest-easysetup-wizard.c: + (create_page_user_details): + * src/maemo/modest-account-settings-dialog.c: + (create_page_user_details), (create_page_outgoing): + * src/maemo/modest-connection-specific-smtp-edit-window.c: + (modest_connection_specific_smtp_edit_window_init): + * src/modest-ui-actions.c: + (modest_ui_actions_on_password_requested): + Use HILDON_GTK_INPUT_MODE_INVISIBLE with password entries, which + might do something useful with the on-screen keyboard. + +2007-05-10 Murray Cumming + + * src/maemo/modest-main-window.c: (on_account_update): + Avoid adding a menu item to priv->accounts_popup if accounts_popup is + NULL, to avoid lots of warnings. However, it probably should not be NULL. + +2007-05-09 Murray Cumming + + * src/modest-tny-account-store.c: (on_account_changed): + Correct the signature of this signal handler. + +2007-05-09 Murray Cumming + + * src/modest-tny-account-store.c: (modest_tny_account_store_alert): + Show a HildonNote instead of a GtkDialog for Maemo. + +2007-05-09 Murray Cumming + + * src/widgets/modest-account-view.h: + * src/widgets/modest-account-view.c: (on_account_changed), + (on_account_removed). Added a boolean flag, set/unset by + modest_account_view_block_conf_updates(), + modest_account_view_unblock_conf_updates() to prevent unnecessary + updates, but this is not very useful because the gconf notifications are + so delays (maybe only on Maemo Bora). + So auto-updating is turned off, and these functions do an explicit + update when necessary. + However, something else is still doing too much work when adding/removing + accounts, probably in another part of the application. + + * src/maemo/modest-account-view-window.c: + (on_delete_button_clicked), (on_edit_button_clicked), + (on_new_button_clicked): Use the new functions. + +2007-05-09 Murray Cumming + + * src/modest-account-mgr-helpers.c: + (modest_account_mgr_set_first_account_as_default): + Sort the list of names alphabetically-by-title, so we choose the first one + based on that. + * src/widgets/modest-account-view.c: (init_view): Sort the TreeModel + alphabetically by the title. + + This fixes projects.maemo.org bug NB#56418 . + + * src/modest-account-mgr-priv.c: + (_modest_account_mgr_account_from_key): + * src/modest-account-mgr.c: (on_key_change): Initialize variables to + avoid a valgrind error, and possible random behaviour. + +2007-05-09 Murray Cumming + + * src/modest-tny-account-store.c: (modest_tny_account_store_alert): + Handle the new TNY_ACCOUNT_STORE_ERROR error domain. I will update the + tinymail documentation to say that this can be expected. + Handle the new TNY_ACCOUNT_STORE_ERROR_UNKNOWN_ALERT error code. It is not + pretty, and should not happen, but I would prefer to know when it does. + +2007-05-09 Murray Cumming + + * configure.ac: When using the older hildon version, probably on Bora, + require gnome-vfs-module-2.0, which is the old provider of gnome-vfs-mime.h, + to fix the build. + * src/widgets/modest-attachment-view.c: + (modest_attachment_view_set_part_default): Initialize variables, to fix the + build. + +2007-05-09 Marcus Bauer + + * src/dbus_api/modest-dbus-callbacks.c: (on_idle_open_message): + Implemented the prototype. Can be tested with + tests/dbus_api/test_open_message: + +2007-05-08 Murray Cumming + + * src/modest-tny-account-store.c: + (modest_tny_account_store_instance_init), (get_password): + * src/modest-ui-actions.c: + (modest_ui_actions_on_password_requested): + Do not store the password in gconf, because this function is called + for non-remembered passwords. Actually use the cached (in the hash map) + password instead of releasing it. + The password will now not be remembered if it is asked via the protected + password dialog, as per the UI spec. + +2007-05-08 Murray Cumming + + * src/modest-defs.h: + * src/modest-account-mgr-helpers.h: + * src/modest-account-mgr-helpers.c: + Added modest_server_account_get_username_has_succeeded(), + modest_server_account_set_username_has_succeeded(). + (modest_server_account_set_username), + Reset the has-succeeded flag if the username changes. + + * src/maemo/modest-account-settings-dialog.c: (save_configuration): + * src/maemo/modest-connection-specific-smtp-window.c: + (modest_connection_specific_smtp_window_save_server_accounts): + Use modest_server_account_set_username() instead of using + modest_conf_set_string() directly, so that the has-succeeded flag is always + reset. + + * src/modest-ui-actions.c: + (modest_ui_actions_on_password_requested): Dim the username entry if + the username has ever worked, as per the UI spec. + +2007-05-08 Murray Cumming + + * src/modest-tny-account-store.c: (get_password): + * src/modest-ui-actions.c: + (modest_ui_actions_on_password_requested): + Allow the username to be changed too, though there is more work to + be done on exactly how this behaves. + +2007-05-08 Murray Cumming + + * src/modest-account-mgr-helpers.h: + * src/modest-account-mgr-helpers.c: + Added modest_account_mgr_get_display_name(), + modest_server_account_get_username(), + modest_server_account_set_username(), + modest_server_account_set_password(), + modest_server_account_get_hostname() functions so we do not need to + use the conf API directly. + + * src/maemo/modest-main-window.c: (connect_signals): + Actually connect ot the ModestAccountStore::request-password signal, + so that we show the dialog when the password is requested by Tinymail, + for isntance if it is empty. + + * src/modest-marshal.list: + * src/modest-tny-account-store.c: + (modest_tny_account_store_class_init), (get_password), + (modest_tny_account_store_alert): + * src/modest-tny-account-store.h: + * src/modest-ui-actions.h: + * src/modest-ui-actions.c: + (modest_ui_actions_on_password_requested): + Change the signal parameters, so it is obvious that we are providing + the non human-readable server name, and receiving both the username + and password, though changing of the username is not yet implemented. + Change the dialog UI to match the Maemo UI specifications, with #idfefing + for the GNOME version. + + This should fix the projects.maemo.org bug NB#56209, though it does not + work on Bora, because the gconf_client_get() for the password fails + immediately after we save it with gconf_client_set(). + +2007-05-08 Murray Cumming + + * configure.ac: + * src/Makefile.am: Reverted the use of AM_CONDITIONAL(), because it + caused an undefined symbol error: + modest-platform.c:84: undefined reference to `modest_osso_cb_hw_state_handler + +2007-05-08 Murray Cumming + + * src/modest-tny-account-store.c: + (modest_tny_account_store_alert): Adapt to the changed tinymail API, + using a GError instead of a string message, so we can translate it in + Modest. + +2007-05-08 Murray Cumming + + * src/maemo/modest-address-book.c: + (modest_address_book_check_names): + * src/maemo/modest-msg-view-window.c: + (modest_msg_view_window_clipboard_owner_change): + * src/modest-init.c: (modest_init_init_ui): + * src/modest-mail-operation.c: (modest_mail_operation_send_mail), + (update_folders_cb), (modest_mail_operation_update_account): + * src/modest-ui-actions.c: (modest_ui_actions_on_msg_link_hover), + (modest_ui_actions_on_msg_attachment_clicked), + (modest_ui_actions_on_msg_recpt_activated): + * src/widgets/modest-header-view.c: (drag_data_get_cb): + * src/widgets/modest-msg-view.c: (modest_msg_view_search), + (modest_msg_view_search_next): + * src/widgets/modest-recpt-editor.c: + (modest_recpt_editor_on_button_release_event), + (modest_recpt_editor_on_key_press_event): Commented-out unnecessary calls to + g_message(). At this point we should only be seeing interesting errors on + stdout. Debugging messages could be #ifdefed-out if they are still useful. + 2007-05-07 Murray Cumming (modest_wizard_dialog_force_title_update):