X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=ChangeLog2;h=e7cbef7250249cd2f2c222bb7b399fd84f94708a;hp=0d8d88e74c00a69a2e9c973873fb1c4acfe71048;hb=7441d1df9d7eb43ff93e411e7b4b9e7d72d9d066;hpb=11742f63b8ad452284588aa2246b5ace9cf239e6 diff --git a/ChangeLog2 b/ChangeLog2 index 0d8d88e..e7cbef7 100644 --- a/ChangeLog2 +++ b/ChangeLog2 @@ -1,3 +1,530 @@ +2007-06-30 Johannes Schmid + + * src/maemo/modest-connection-specific-smtp-edit-window.c: + (on_value_changed), + (modest_connection_specific_smtp_edit_window_init): + + Fixed project.maemo.org bug NB#60479 by using "notify::value" instead + of "changed" for now + +2007-06-29 Murray Cumming + + * src/modest-tny-account.c: + (modest_tny_account_new_from_server_account): + Handle the TnyAccount::"connection-status-changed" signal, which tinymail + now uses to request a connection when it needs one, and which it uses + to tell us that something is wrong with the connection, so we can force + the accounts offline to prevent further problems. + + * src/modest-defs.h: + * src/modest-init.c: (init_default_settings): + * src/modest-mail-operation.c: + (modest_mail_operation_update_account): + * src/modest-main.c: (on_show): Do not try to connect explicitly on startup. + This should not be necessary anymore. Do not set the conf key that + determined this (it was always TRUE and not set anywhere else). + + * src/modest-ui-actions.h: Remove unused and unimplemented + modest_ui_actions_on_online_toggle_toggled(). + +2007-06-29 Murray Cumming + + * src/modest-mail-operation.c: Fixed a crash when canceling a mail + receive operation. + +2007-06-27 Murray Cumming + + * src/maemo/modest-maemo-utils.c: Replaced a busy wait while waiting + for supported secure authentication methods from server by + gtk_dialog_run(), saving CPU time and a potential race condition. + +2007-06-26 Armin Burgmeier + + * src/maemo/modest-maemo-utils.h: + * src/maemo/modest-maemo-utils.c: Added a + ModestMaemoUtilsGetSupportedSecureAuthenticationError with CANCELED as + only error code to tell the calling code when the user canceled the + operation. + + * src/maemo/modest-account-settings-dialog.h: + * src/maemo/modest-account-settings-dialog.c: + * src/maemo/easysetup/modest-easysetup-wizard.c: Do not show an error + message if the retrieval of secure authentication methods was canceled + by the user. Perform the check before saving anything to configuration + to not end up with half-committed configuration changes, do not show + a "Saving to configuration failed" note when authentication methods + could not be retrieved. + + * src/maemo/modest-main-window.c: Fixed a crash on startup when built + with old hildon. + +2007-06-26 Murray Cumming + + * src/dbus_api/modest-dbus-callbacks.c: (add_folders_to_list): + Only return the first level of folders under accounts, because the + osso-global-search UI spec doesn't want the rest. + This fixed projects.maemo.org bug NB#61811. + +2007-06-26 Murray Cumming + + * src/modest-tny-account-store.c: + * src/modest-tny-account-store.h: + Added modest_tny_account_is_memory_card_account(). + * src/maemo/modest-main-window.c: (create_details_widget): + Show the same information items for the memory card as we show for the + local-folders, to fix projects.maemo.org bug NB#59307. + +2007-06-26 Murray Cumming + + * src/modest-tny-account-store.c: (get_password): + Added some checks for NULL to maybe prevent crashes, though I think + it was Philip's last commit that actually prevented the crash here + when creating a first new account. + But the new account is still not usable (no messages are shown) + until a restart of the application. + +2007-06-26 Murray Cumming + + * src/maemo/modest-maemo-global-settings-dialog.c: + (idle_select_default_focus): + * src/modest-mail-operation.c: (idle_notify_progress), + (idle_notify_progress_once), (idle_notify_update_account_queue), + (set_last_updated_idle), (update_account_thread), + (notify_get_msgs_full), (get_msgs_full_destroyer), + (get_msgs_full_thread): + * src/widgets/modest-attachment-view.c: (get_size_idle_func): + * src/widgets/modest-header-view.c: + (idle_notify_headers_count_changed_destroy), + (idle_notify_headers_count_changed): + Use gdk_threads_enter() and gdk_threads_enter_leave() in + all idle handlers, because some other thread could be blocking + the GDK lock, so that it can do GUI stuff. + +2007-06-26 Murray Cumming + + * src/dbus_api/modest-dbus-callbacks.c: (on_idle_mail_to), + (on_idle_compose_mail), (on_compose_mail), (on_idle_open_message), + (on_idle_send_receive), (on_idle_open_default_inbox): + * src/maemo/modest-msg-edit-window.c: + (modest_msg_edit_window_attach_file_noninteractive): + Use gdk_threads_enter() and gdk_threads_leave() in all the idle handlers, + to prevent a hang. I don't quite understand why these are necessary, + beause we should already be in the main loop. I guess something outside + of the main loop is taking the lock. + + * tests/dbus_api/test_compose_mail.c: (main): Use file:// prefixes so + we are really using URIs, and use files that are more likely to exist. + +2007-06-26 Murray Cumming + + * tests/dbus_api/Makefile.am: Fix the build of the tests. This was + necessary because configure.ac now defines different variables. + +2007-06-26 Murray Cumming + + * libmodest-dbus-client/libmodest-dbus-client.c: + * libmodest-dbus-client/libmodest-dbus-client.h: + Removed libmodest_dbus_client_send_mail() because it is not used by anything. + libmodest_dbus_client_compose_mail() is what things actually use. + + * src/dbus_api/modest-dbus-api.h: + * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_handler): + * tests/dbus_api/Makefile.am: + * tests/dbus_api/test_send_mail.c: Removed the MODEST_DBUS_METHOD_SEND_MAIL + D-Bus method. + +2007-06-26 Murray Cumming + + * src/modest-init.c: (modest_init_local_folders): + For memory-card (or other non local-folders) maildir accounts, + always create an Archive folder, so that there is somewhere to put + messages. This should be less confusing, because the user does not need + to guess that the need to create a folder. + +2007-06-25 Johannes Schmid + + * src/maemo/easysetup/modest-wizard-dialog.c: (init): + Fixed icon size by using (50, 50) which is the same as defined by + newer hildon-libs + +2007-06-25 Armin Burgmeier + + * src/widgets/modest-window-mgr.c: + * src/modest-init.c: + * configure.ac: Correctly recognize hildon-notify presence and fixed + the build without it being installed. + +2007-06-25 Murray Cumming + + * libmodest-dbus-client/libmodest-dbus-client.c: + (libmodest_dbus_client_search): Fix the build. Sorry. + * src/modest-ui-actions.c: (modest_ui_actions_on_new_msg), + (reply_forward_cb): Initialize variables to fix the build. + + * src/modest-tny-account-store.c: + (modest_tny_account_store_get_tny_account_by): Fix the account URI + comparison (a silly typo) so that email search results can be opened + when there are more than one account. + +2007-06-25 Murray Cumming + + * libmodest-dbus-client/libmodest-dbus-client.c: + (libmodest_dbus_client_search), + (libmodest_dbus_client_get_folders): Use large timeouts (2 minutes) + instead of 1 second, because the search gets folders and messages from + the server, which can take a long time. When it times out, + osso-global-search will report no results. + + We need to specify whether we should search only cached information + (maybe with a user preference), and the osso-email-search-plugin + API should really be improved to allow that option, and to allow results + to be delivered asynchronously in batches. + +2007-06-25 Murray Cumming + + * src/maemo/easysetup/modest-wizard-dialog.c: (init): + Fix a typo to fix the osso 1.1. build. See the TODO comment though - + GTK_ICON_SIZE_DIALOG is probably the wrong size. + +2007-06-25 Murray Cumming + + * src/modest-search.c: (modest_search_folder): Ignore a seach folder + that is an empty string. D-Bus seems to give us these instead of NULL. + +2007-06-25 Johannes Schmid + + * src/dbus_api/modest-dbus-callbacks.c: + * src/maemo/easysetup/modest-easysetup-wizard.h: + * src/maemo/easysetup/modest-wizard-dialog.c: (init): + * src/maemo/modest-connection-specific-smtp-edit-window.c: + (modest_connection_specific_smtp_edit_window_init): + * src/maemo/modest-maemo-global-settings-dialog.c: + (on_range_error): + * src/maemo/modest-maemo-utils.c: + (modest_maemo_utils_setup_images_filechooser): + * src/maemo/modest-main-window.c: (create_details_widget): + * src/maemo/modest-msg-edit-window.c: + (modest_msg_edit_window_attach_file_noninteractive), + (modest_msg_edit_window_color_button_change), + (modest_msg_edit_window_set_zoom): + * src/modest-mail-operation.c: (update_account_thread): + * src/modest-tny-account-store.c: + * src/modest-tny-account.c: + * src/widgets/modest-global-settings-dialog.c: + * src/widgets/modest-window.h: + + Fixed build on bora! + +2007-06-25 Murray Cumming + + * src/dbus_api/modest-dbus-callbacks.c: (on_dbus_method_search): + * src/modest-search.c: (modest_search_folder), + (modest_search_all_accounts): Actually search only in the specified folder, + doing a simple match on the folder ID. + * src/modest-search.h: Added a folder field to the ModestSearch struct, + and added some documentation. + +2007-06-22 Armin Burgmeier + + * src/maemo/modest-maemo-utils.h: + * src/maemo/modest-maemo-utils.c: Added a GError parameter to + modest_maemo_utils_get_supported_secure_authentication_methods. + + * src/maemo/easysetup/modest-easysetup-wizard.c: + * src/maemo/modest-account-settings-dialog.c: Show correct error if + one occured, not just "Server does not support secure authentication". + +2007-06-22 Murray Cumming + + * src/modest-tny-account-store.c: (get_password): + Restored the call to tny_account_set_user() because Philip seems to have + stopped this from crashing (by not using the freed string). It still does + not work, but now we can test it. + +2007-06-22 Murray Cumming + + * src/maemo/modest-msg-edit-window.c: (init_window): + Comment out use of an uninitialzed GtkScrolledWindow variable, + to fix the build and stop a crash. Yet again. + +2007-06-22 Murray Cumming + + * src/modest-tny-account-store.c: (get_password): + Remove the call to tny_account_set_user() because this actually crashes. + As mentioned in the last commit, it is known to not work anyway. + +2007-06-22 Murray Cumming + + * src/modest-tny-account-store.c: (get_password): + Use tny_account_set_username() to tell tinymail that the username might + have changed too. But tinymail ignores this: see projects.maemo.org + bug #61463. + * src/modest-ui-actions.c: + (modest_ui_actions_on_password_requested): Remove the useless attempt to + reset the connection when changing the username, and improve the comments. + Save the changed username to gconf at least, so it is used when Modest is + restarted. + +2007-06-21 Murray Cumming + + * src/dbus_api/modest-dbus-callbacks.c: (add_folders_to_list): + Do not use a query, so that we get all folders, including for POP and + local-folders. They work as search criteria too. + This fixes projects.maemo.org bug NB#57740. + +2007-06-21 Murray Cumming + + * src/maemo/modest-platform.c: (modest_platform_on_new_msg): + Comment out the use of hildon_notification_set_sound() because it is + not declared, to fix the build. Warnings are serious, people. + +2007-06-21 Armin Burgmeier + + * src/maemo/modest-maemo-utils.h: + * src/maemo/modest-maemo-utils.c: Added a username parameter to the + modest_maemo_utils_get_supported_secure_authentication_methods() + to query authentication methods for a specific user. This should fix + projects.maemo.org bug NB#61369. + + * src/maemo/easysetup/modest-easysetup-wizard.c: + * src/maemo/modest-account-settings-dialog.c: Adjusted calls to that + function. + +2007-06-21 Murray Cumming + + * libmodest-dbus-client/libmodest-dbus-client.h: + * libmodest-dbus-client/libmodest-dbus-client.c: + (modest_dbus_message_iter_get_search_hit), + (libmodest_dbus_client_search): + Reorganised and commented this. + Removed modest_search_hit_free() from the public API because it + is not needed. + + Added libmodest_dbus_client_get_folders() and + modest_folder_result_list_free(). + + * src/dbus_api/modest-dbus-api.h: + * src/dbus_api/modest-dbus-callbacks.c: + (modest_dbus_req_filter): Commented. Moved most code to + on_dbus_method_search() and added on_dbus_method_get_folders(). + + * src/maemo/modest-platform.c: (modest_platform_init): Commented the use + of the extra D-Bus handler. + + * tests/dbus_api/Makefile.am: + * tests/dbus_api/test_get_folders.c: New test for + libmodest_dbus_client_get_folders(). This seems to return an empty list + at the moment. That could be the same error that causes the message list + to be empty sometimes. + This should help with projects.maemo.org bug NB#57740. + +2007-06-21 Armin Burgmeier + + * src/modest-mail-operation.c: Do not call + modest_mail_operation_queue_cancel_all() in _cancel() but do instead + notify the operation's end after having set the status to CANCELED for + the operation queue to listen on. This fixes a deadlock when there + were mail operations in the queue during its finalization. + + * src/modest-singletons.c: Reordered uninitialization of the account + manager to happen after the mail operation queue was uninitialized + because the mail operation's in the queue might still try to access + the account manager (see modest_mail_operation_notify_end()). This + fixes a potential crash when terminating the application. + +2007-06-21 Murray Cumming + + * src/modest-search.c: + (search_mime_part_and_child_parts): Moved some code here so we + can search recursively in child folders. + (modest_search_folder): Search in the message as a mime part, instead + of just its child mime parts. + (read_chunk), (search_mime_part_ogs): + Check how many bytes were read, to prevent an endless loop. + Searching in message bodies now works. This finishes the fix for + projects.maemo.org bug NB#61007. + +2007-06-20 Murray Cumming + + * src/modest-ui-actions.c: (modest_ui_actions_on_details): + Add a g_warning when the item is not a folder. + + * src/modest-ui-dimming-rules.c: + (modest_ui_dimming_rules_on_details): Dim the Email/Details menu item + when selected item is not a folder (for instance, an account), because + the UI spec only talks about folders. + I am not sure what the UI spec wants, so this might not totally fix + projects.maemo.org bug NB#60794. + +2007-06-20 Murray Cumming + + * src/maemo/easysetup/modest-easysetup-country-combo-box.c: + (load_from_file): Sort the list. The unsorted list would be more noticeable + in non-English locales. + + * src/maemo/modest-msg-edit-window.c: + (modest_msg_edit_window_attach_file): Initialize a variable to avoid a + warning and fix the build. + + * src/maemo/modest-progress-bar-widget.c: + (modest_progress_bar_widget_set_progress): Avoid a division by zero, to + avoid a g_warning, and probably to avoid a crash. + +2007-06-20 Murray Cumming + + * src/modest-search.c: (modest_search_folder): Check for a GError + when calling tny_folder_get_headers(), to show that this is not the cause + of later failure. + +2007-06-20 Murray Cumming + + * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_filter): + Actually set the body query instead of the subject query. + + * src/modest-search.c: (search_mime_part_ogs), + (search_mime_part_strcmp): Remove extra space from the text/* regex, + which would probably cause it to fail if we got this far. + + (modest_search_folder): Use simple #ifdefs instead of the function pointer, + to simplify the code. + Warn when tny_mime_part_get_parts() returns an empty list, which seems to + be causing search-in-body to fail. + +2007-06-20 Murray Cumming + + * libmodest-dbus-client/libmodest-dbus-client.h: Document the folder + struct item. + * src/dbus_api/modest-dbus-callbacks.c: (find_message_by_url): + Better debug message when the folder cannot be found. + This should help with fixing projects.maemo.org bug #61287. + * src/modest-search.c: (add_hit): Do not leak the furl string. + +2007-06-20 Murray Cumming + + * src/modest-ui-actions.c: + (modest_ui_actions_on_folder_selection_changed): + Do not special-case TNY_MERGE_FOLDER here, because + modest_tny_folder_get_account() already does that. This fixes + projects.maemo.org bug NB#60955 (The smtp connections dialog was not + shown when the Outbox was selected). + +2007-06-19 Murray Cumming,,, + + * libmodest-dbus-client/libmodest-dbus-client.c: + (_dbus_iter_get_string_or_null): Initialize the string variable in + case this helps with Dirk's crash. + +2007-06-19 Murray Cumming + + * src/dbus_api/modest-dbus-api.h: Add the attachments enum for send_mail, + though it is not implemented yet (this function is maybe not even used), + to avoid breaking ABI in future. + * libmodest-dbus-client/libmodest-dbus-client.c: + (get_attachments_string): Moved some code into this generic function. + (libmodest_dbus_client_send_mail): Corrected the documentation. This really + sends email. It is maybe not used. + * src/dbus_api/modest-dbus-callbacks.c: (on_idle_send_mail), + (on_send_mail): Use attachments. + + * src/widgets/modest-msg-edit-window.h: Improve the documentation. + * src/maemo/modest-msg-edit-window.c: + (modest_msg_edit_window_attach_file_noninteractive): Convert the URI to + a filename before using it, to fix projects.maemo.org bug NB#58233. + +2007-06-19 Murray Cumming + + * src/modest-ui-actions.c: (modest_ui_actions_on_copy): + Initialize the continue_download variable to fix the build, and probably + to fix randome behaviour. + +2007-06-18 Armin Burgmeier + + * src/modest-tny-send-queue.h: + * src/modest-tny-send-queue.c: Keep track of the status of messages in + the queue so we can later show it in the UI. Replaced + modest_tny_send_queue_msg_is_being_sent() by + modest_tny_send_queue_get_msg_status(). + + * src/modest-ui-dimming-rules.c: Adjust for the replaced function. + + * src/modest-tny-account.c: Changed a cast to gpointer* to one to + gpointer. This probably was not harmful, but just to be sure. + +2007-06-18 Murray Cumming + + * src/maemo/modest-platform.c: (modest_platform_activate_uri): + Fix the build, but leave the function broken on Bora, because + hildon_uri_action_get_service() does not exist. + osso_uri_action_get_service() does not exist either, it seems. + +2007-06-18 Murray Cumming + + * src/modest-tny-account.c: + (modest_tny_account_new_from_server_account): + Uncommented the url debug output again, because it is useful for + debugging, as I said in the comment. + 2007-06-18 Murray Cumming * configure.ac: Do not require hildon-notify for Maemo Bora. It is not @@ -2968,6 +3495,3 @@ Added modest_ui_actions_on_new_account() as a signal handler for the New Account menu item. It shows the easysetup wizard. * po/en_GB.po: Added additional needed logical IDs and translations. - -Started ChangeLog2 file, because I cannot work without a ChangeLog. murrayc. -Apparently ChangeLog is not used for some vague Nokia legal reasons.