2007-07-02 Murray Cumming <murrayc@murrayc.com
[modest] / ChangeLog2
index 46e2c5f..32f311d 100644 (file)
@@ -1,3 +1,775 @@
+2007-07-02  Murray Cumming  <murrayc@murrayc.com
+
+       * src/modest-tny-account.c: Added util_run_in_main_thread_and_wait() 
+       which uses an idlen handler and mainloop to ensure that the function 
+       is called in the main thread.
+       (on_connection_status_changed): Use util_run_in_main_thread_and_wait() 
+       to make sure that we call attempt the connection in the mainloop only, 
+       which might solve some hangs.
+
+2007-07-02  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account.h:
+       * src/modest-tny-account.c:
+       (modest_tny_account_new_from_server_account),
+       (modest_tny_account_new_from_server_account_name),
+       (modest_tny_account_new_from_account): Added a TnyCamelSession parameter, 
+       because this must now be set before setting the proto. This is usd in 
+       modest_tny_account_new_from_server_account(). It avoids a (maybe harmless) 
+       warning in camel_session_get_service() caused by 
+       tny_camel_transport_account_prepare().
+       
+       * src/modest-tny-account-store.c: (get_server_accounts): Pass the new 
+       session parameter.
+
+2007-06-30  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com
+
+       * src/maemo/modest-main-window.c: (connect_signals):
+       Don't connect to delete_event twice. Maybe this was harmless.
+
+2007-06-29  Murray Cumming  <murrayc@murrayc.com
+
+       * libmodest-dbus-client/libmodest-dbus-client.c: Improved the documentation slightly.
+       * src/modest-mail-operation.c: (modest_mail_operation_xfer_folder):
+       Debug output to give some more clue when a folder move is not allowed.
+
+2007-06-28  Murray Cumming  <murrayc@murrayc.com
+
+       * src/modest-tny-account.c: (on_modest_file_system_info),
+       (modest_tny_account_get_mmc_account_name):
+       Always call the callback, without checking whether the name has changed.
+       * src/widgets/modest-folder-view.c: (strings_are_equal),
+       (on_model_foreach_set_name), (on_get_mmc_account_name),
+       (text_cell_data): Compare the name here, and use the name from 
+       the account rather than from the 
+       TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN because that will be wrong 
+       if the name was changed after the account was added to the model.
+       That's a tinymail bug but not one that I want to fix now.
+
+       This finally fixes projects.maemo.org bug NB#59301.
+
+2007-06-27  Armin Burgmeier  <armin@openismus.com>
+
+       * src/modest-mail-operation.c: Fixed a crash when canceling a mail
+       receive operation.
+
+2007-06-27  Murray Cumming  <murrayc@murrayc.com
+
+       * src/modest-tny-account.h:
+       * src/modest-tny-account.c: 
+       Added modest_tny_account_get_mmc_account_name(), to set the memory 
+       card name asynchronously for the given account and then notify.
+       (modest_tny_account_new_for_local_folders): Use this new function instead 
+       of using hildon_file_system_info_async_new() directly.
+       
+       * src/widgets/modest-folder-view.c: (text_cell_data):
+       Use modest_tny_account_get_mmc_account_name() to get the real name, calling 
+       gtk_tree_model_row_changed() when it is available.
+       
+       This should fix projects.maemo.org bug NB#59301.
+
+2007-06-27  Murray Cumming  <murrayc@murrayc.com
+
+       * src/modest-mail-operation.c: (update_account_thread),
+       (get_msgs_full_thread): Fix typo failures in the revert
+       to fix the build.
+
+2007-06-27  Murray Cumming  <murrayc@murrayc.com
+
+       * src/maemo/modest-maemo-global-settings-dialog.c:
+       (idle_select_default_focus):
+       * src/modest-mail-operation.c: (idle_notify_progress),
+       (idle_notify_progress_once), (set_last_updated_idle),
+       (update_account_thread), (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):
+       Reverted the additions of gdk_threads_enter/leave(), in case this 
+       helps with a startup hang on the device.
+
+2007-06-26  Armin Burgmeier  <armin@openismus.com>
+
+       * 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  <armin@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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 <johannes.schmid@openismus.com>
+
+       * 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  <armin@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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 <johannes.schmid@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <armin@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <armin@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <armin@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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,,,  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <armin@openismus.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * 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  <murrayc@murrayc.com>
+
+       * configure.ac: Do not require hildon-notify for Maemo Bora. It is not 
+       available.
+       * src/modest-init.c: (modest_init_init_ui): Do not call notify_init() 
+       for Mameo Bora, to fix the build.
+       
+2007-06-17  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * configure.ac:
+       Fixed build with hildon < 0.99
+       
+       * src/maemo/modest-account-view-window.c: (on_edit_button_clicked):
+       Check if the selected account is busy instead of checking all accounts
+       
+       * src/modest-account-mgr-priv.h:
+       * src/modest-account-mgr.c: (modest_account_mgr_class_init),
+       (modest_account_mgr_init), (compare_account_name),
+       (modest_account_mgr_set_account_busy),
+       (modest_account_mgr_account_is_busy):
+       * src/modest-account-mgr.h:
+       Added busy information to the account manager. It possible to
+       set an account to busy or not busy now.
+       
+       * src/modest-mail-operation.c: (modest_mail_operation_finalize),
+       (modest_mail_operation_update_account),
+       (modest_mail_operation_notify_end):
+               Set the account to busy/not busy depending on the operation state.
+
+       * src/widgets/modest-account-view.c:
+       (modest_account_view_finalize), (get_last_updated_string),
+       (update_account_view), (on_account_busy_changed), (init_view):
+       Show string "Refreshing..." if the account is currently busy.
+
+       All this fixes projects.maemo.org bug NB#59711
+
+2007-06-14  Armin Burgmeier  <armin@openismus.com>
+
+       * src/widgets/modest-attachments-view.h:
+       * src/widgets/modest-attachments-view.c: Added
+       modest_attachments_view_has_attachments() and fixed coordinate
+       transformation in get_att_view_at_coords(). This fixes attachment
+       selection.
+
+       * src/widgets/modest-msg-view.c: Make use of
+       modest_attachments_view_has_attachments() to decide whether to show
+       attachments in the header.
+
+2007-06-14  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/modest-connection-specific-smtp-edit-window.c:
+       (on_change), (on_combo_security_changed),
+       (modest_connection_specific_smtp_edit_window_init),
+       (modest_connection_specific_smtp_edit_window_set_connection),
+       (modest_connection_specific_smtp_edit_window_is_dirty):
+       * src/maemo/modest-connection-specific-smtp-edit-window.h:
+       * src/maemo/modest-connection-specific-smtp-window.c:
+       (on_button_edit):
+       
+       Show an comfirmation dialog for the smpt-edit-window if there are
+       unsaved changes and the user hits cancel.
+       (projects.maemo.org bug NB#60479)
+
+2007-06-14  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       * src/maemo/modest-account-settings-dialog.c: Show "invalid email
+       address" info in banner instead of node, and select all the text as
+       per UI spec. This fixes projects.maemo.org bug NB#60550.
+
+2007-06-14  set REAL_NAME environment variable  <set EMAIL_ADDRESS environment variable>
+
+       reviewed by: <delete if not using a buddy>
+
+       * src/maemo/modest-main-window.c: (_on_msg_count_changed):
+       * src/modest-ui-actions.c: (folder_refreshed_cb):
+
+2007-06-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-folder-view.c: (filter_row):
+       Explictly filter out the per-account outbox accounts. I don't know why 
+       these only sometimes show up, but now they should never show up in the UI.
+       This should fix projects.maemo.org bug NB#60160.
+
+2007-06-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-main-window.c: (set_toolbar_mode): Prevent a warning
+       when the toolbar does not exist yet.
+       * src/modest-widget-memory.c: (save_settings_header_view):
+       Added g_warnings() to help us to discover when we are trying to set 
+       an empty columns list in gconf.
+
+2007-06-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-init.h:
+       * src/modest-init.c: 
+       Added modest_init_get_default_header_view_column_ids().
+
+       * src/modest-widget-memory.c: (restore_settings_header_view):
+       Use the new function to get defaults, if the gconf key is empty, 
+       instead of not creating any columns. This seems to fix the 
+       empty right-hand pane problems.
+       
+       * src/widgets/modest-header-view.h: Improved documentation.
+
+2007-06-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
+       Revert the use of the display name as the account name, because this
+       is generally wrong and would break the code that follows.
+
+2007-06-13  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (check_for_supported_auth_methods), (on_before_next),
+       (create_account):
+       
+       More correct fix for projects.maemo.org bug NB# 58991
+       Do not longer accept AUTH_PASSWORD as secure method and choose
+       correct method when creating the account.
+
+2007-06-13  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
+       Fixed projects.maemo.org bug NB# 58991
+
+2007-06-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-init.c: (modest_init_init_core),
+       (modest_init_local_folders): Fixed compilation errors.
+
+2007-06-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * libmodest-dbus-client/libmodest-dbus-client-1.0.pc.in:
+       Remove unnecessary lib/include, though it is good in principle when needed.
+       
+       * src/modest-init.c: (modest_init_one_local_folder),
+       (modest_init_local_folders):
+       * src/modest-init.h: Initialize the .modest library for memory cards, 
+       instead of just doing nothing for them. This should allow us to create 
+       folders there, fixing projects.maemo.org bug NB#59297.
+
+2007-06-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * libmodest-dbus-client/libmodest-dbus-client.c:
+       (libmodest_dbus_client_send_mail), (libmodest_dbus_client_mail_to),
+       (libmodest_dbus_client_compose_mail),
+       (libmodest_dbus_client_open_message),
+       (libmodest_dbus_client_send_and_receive),
+       (libmodest_dbus_client_open_default_inbox):  Improved debug messages.
+       
+       * src/dbus_api/modest-dbus-callbacks.c: (find_message_by_url),
+       (on_idle_open_message), (modest_dbus_req_handler),
+       (search_result_to_message), (modest_dbus_req_filter):
+       * src/modest-search.c: (add_hit), (modest_search_folder),
+       (modest_search_all_accounts): Improved debug messages.
+       
+       * tests/dbus_api/test_open_message.c: (main): Improved comments.
+       
+       * src/modest-tny-account-store.c:
+       (modest_tny_account_store_get_tny_account_by): Initialize a variable to 
+       fix the build.
+
+2007-06-13  Murray Cumming  <murrayc@murrayc.com>
+
+
+2007-06-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (get_server_accounts):
+       Call modest_tny_local_folders_account_add_merged_outbox_folders() 
+       even when there are no accounts, so that the Outbox folder is always 
+       visible. This fixes projects.maemo.org bug NB#60158.
+
+2007-06-12   Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-main-window.c: (create_details_widget):
+       Use the mcen_va_never logical ID.
+
+2007-06-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-api.h:
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (on_idle_open_default_inbox), (on_open_default_inbox),
+       (modest_dbus_req_handler):
+       Added a new OpenDefaultInbox method.
+       
+       * libmodest-dbus-client/libmodest-dbus-client.h:
+       * libmodest-dbus-client/libmodest-dbus-client.c:
+       Added libmodest_dbus_client_open_default_inbox() to call the new 
+       dbus method. This should be useful to projects.maemo.org bug NB#57214,
+       though it should maybe be used by a new osso-email-interface function.
+       
+       * tests/dbus_api/Makefile.am:
+       * tests/dbus_api/test_open_default_inbox.c: Added a test for the new 
+       function.
+
+2007-06-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr.c: (on_key_change): Append to the list, not to 
+       NULL, so that the signals get all the keys. This should not actually 
+       change any behaviour because we currently update everything when anything 
+       changes.
+
+2007-06-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-folder-view.c: (text_cell_data),
+       (icon_cell_data), (filter_row), (on_selection_changed), (cmp_rows),
+       (find_inbox_iter): If tinymail tells us that the folder is 
+       a non-specific TNY_FOLDER_TYPE_NORMAL then do an extra check on the name.
+       This causes the inbox to be selected at startup, fixing part of 
+       projects.maemo.org bug NB#56305.
+
+       * src/modest-ui-dimming-rules.c:
+       (modest_ui_dimming_rules_on_tools_smtp_servers):
+       * src/modest-ui-dimming-rules.h: Added a callback function to dim 
+       the SMTP servers menu item.
+       * src/maemo/modest-main-window-ui-dimming.h: Used the function, so 
+       that the SMTP Servers menu item is dimmed when there are no accounts, 
+       as a future version of the UI spec will require. This fixes 
+       projects.maemo.org bug NB#56305.
+
 2007-06-11  Johannes Schmid <johannes.schmid@openismus.com>
 
        * src/maemo/modest-maemo-utils.c:
        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.