2007-06-25 Johannes Schmid <johannes.schmid@openismus.com>
[modest] / ChangeLog2
index 5596139..551133e 100644 (file)
@@ -1,3 +1,238 @@
+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):
        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.