2007-05-03 Murray Cumming <murrayc@murrayc.com>
[modest] / ChangeLog2
index eda8440..296f1e9 100644 (file)
@@ -1,3 +1,176 @@
+2007-05-03  Murray Cumming  <murrayc@murrayc.com>
+
+       * configure.ac: Depend on libalarm for Maemo.
+
+       * src/dbus_api/modest-dbus-api.h:
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_send_receive),
+       (on_send_receive), (modest_dbus_req_handler):
+       Handle a SEND_RECEIVE D-Bus method.
+       
+       * libmodest-dbus-client/libmodest-dbus-client.c:
+       (libmodest_dbus_client_send_and_receive):
+       * libmodest-dbus-client/libmodest-dbus-client.h:
+       Add C convenience functions for calling the new D-Bus method.
+       
+       * src/modest-platform.h:
+       * src/gnome/modest-platform.c:
+       * src/maemo/modest-platform.c: 
+       Added modest_platform_set_update_interval(), which uses the 
+       alarmd API on Maemo.
+       (modest_platform_init),
+       Call modest_platform_set_update_interval(), using a hard-coded 
+       interval for now (until the global settings dialog is implemented), 
+       though I am not sure that this makes sense yet.
+
+2007-05-02  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (modest_osso_cb_hw_state_handler):
+       * src/dbus_api/modest-dbus-callbacks.h:
+       * src/maemo/modest-platform.c: (modest_platform_init):
+       Registered (empty) callbacks for the osso hardware state D-Bus signals, 
+       in case this has some effect on the ability for Maemo to ping the 
+       application.
+
+2007-05-02  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-text-utils.c: (modest_text_utils_validate_recipient):
+       Comment out an if() that checks a gchar for < 0, causing a build-breaking 
+       warning. It is probably a logic error, but I need to fix the build.
+
+2007-05-02  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr.h:
+       * src/modest-account-mgr.c: 
+       (modest_account_mgr_account_names): Add a gboolean enabled_only 
+       parameter, so we can get lists of onlt the enabled accounts.
+       
+       (modest_account_mgr_add_account),,
+       (modest_account_mgr_account_with_display_name_exists):
+       * src/maemo/modest-msg-edit-window.c: (get_transports):
+       * src/modest-account-mgr-helpers.c:
+       (modest_account_mgr_set_first_account_as_default):
+       * src/modest-init.c: (init_default_account_maybe):
+       * src/modest-tny-account-store.c: (get_accounts):
+       * src/modest-ui-actions.c: (modest_ui_actions_on_accounts),
+       (do_send_receive_auto), (modest_ui_actions_on_send_receive):
+       * tests/check_account-mgr.c: (START_TEST):
+       Provide the extra parameter, as appropriate.
+       
+       * src/widgets/modest-account-view.c: (update_account_view):
+       Do not show disabled accounts (meaning that there is no way to 
+       enable/disable accounts in the UI (the feature is not in our UI 
+       specification), so we can use this internally only to mark unfinished 
+       or temporary account data.
+       
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (create_account): Add boolean enable parameter, so we can specify FALSE 
+       to create the temporary account.
+       (on_button_edit_advanced_settings): Create the temporary account as disabled.
+       (on_before_next): When finishing, when there is a temporary account, just 
+       set it as enabled.
+       Also, use a timeout to delay the showing of the dialog until gconf is likely 
+       to return correct information, due to a maemo gconf bug that is fixed in 
+       osso 1.1, but not yet in Bora.
+       However, the dialog stays on screen after it is destroyed.
+       
+       * src/maemo/modest-account-settings-dialog.c: on_response(): Do not check 
+       for invalid data when cancelling. Use a hildon note instead of a dialog 
+       to complain about invalid data. Do not show the account-saved note if the 
+       account is disabled (a temporary account that will not really be saved for use 
+       until later.)
+
+2007-04-30  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       Added modest_account_mgr_unset_default_account(),
+       Added modest_account_mgr_set_first_account_as_default().
+
+       * src/modest-account-mgr.c: (modest_account_mgr_remove_account): 
+       If it was the default account, unset the default account name.
+       
+       * src/maemo/modest-account-view-window.c:
+       (on_delete_button_clicked): If it was the default account, 
+       set the first remaining account as the default instead.
+       
+       * src/maemo/modest-main-window.c: (on_account_update): Do not try to use 
+       a NULL account or a NULL default account, to prevent a crash. This should 
+       not happen now anyway. Bug #55343 in projects.maemo.org/bugzilla.
+       
+
+2007-04-30  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-recpt-editor.c:
+       (modest_recpt_editor_on_key_press_event): Put an #ifdef around the 
+       use of gtk_text_buffer_get_has_selection() (from GTK+ 2.10), and add a 
+       TODO comment for this. This fixes the build.
+       
+       * src/modest-ui-actions.c: Add do_send_receive(), though it might not 
+       be the best place for it. This will in future update all auto-update accounts, 
+       rather than just the current/default account, when that gconf key exists.
+       (modest_ui_actions_on_send_receive): Offer the settings dialog (or wizard) 
+       if there are no accounts, before calling do_send_receive().
+       * src/modest-ui-actions.h: Specify the callback for the Send/Recieve menu item.
+       
+       * src/gnome/modest-main-window.c: (on_online_toggle_toggled):
+       * src/maemo/modest-main-window-ui.h:
+       * src/maemo/modest-main-window.c: (on_connection_changed),
+       (sync_accounts_cb): Use do_send_receive() instead of calling the signal handler 
+       directly, becaue the signal handler does more.
+
+2007-04-30  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-helpers.c:
+       (modest_account_mgr_get_account_data): Add a more helpful error message, and comment, 
+       for the case that the account does not exist.
+       
+       * src/widgets/modest-folder-view.c: (filter_row): Use the tinymail account ID 
+       (equivalent to the modest account name) rather than the tinymail account name 
+       (equivalent to the modest account title - human readable) to get the account data.
+       Check for null account data to prevent the crash, and add a TODO comment saying that 
+       this needs fixing because it is using the server account name instead of the account name.
+
+2007-04-29  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (show_error):
+       * src/maemo/modest-account-settings-dialog.c: (show_error),
+       (show_ok):
+       Use hildon_note_new_information() for informative and error messages, 
+       which seems to be expected for Maemo applications (no documentation that I know of 
+       says this). I am surprised that there is no distinction between info and error 
+       dialogs.
+
+2007-04-29  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-ui-constants.h: Add a MODEST_MARGIN_NONE constant, so 
+       it is easy to find where this is used.
+       
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (create_page_welcome), (create_page_account_details),
+       (create_page_user_details), (create_page_complete_easysetup),
+       (create_page_custom_incoming), (create_page_custom_outgoing),
+       (create_page_complete_custom):
+       * src/maemo/modest-account-settings-dialog.c:
+       (create_page_account_details), (create_page_user_details),
+       (create_page_incoming), (create_page_outgoing):
+       * src/maemo/modest-account-view-window.c: (window_vbox_new):
+       * src/maemo/modest-connection-specific-smtp-edit-window.c:
+       (modest_connection_specific_smtp_edit_window_init):
+       * src/maemo/modest-connection-specific-smtp-window.c:
+       (modest_connection_specific_smtp_window_init):
+       * src/maemo/modest-signature-editor-dialog.c:
+       (modest_signature_editor_dialog_init): Use the correct padding/spacing/borders 
+       and scrolling policy as per the Email Application Layout Guide, if I have 
+       understood it.
+
+2007-04-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_init): 
+       Remove the add_transport_account_func and add_store_account_func vfunc implementations, 
+       which were empty, because these vfuncs have been removed from TnyAccountStore.
+       This fixes the build.
+
 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
 
        * src/maemo/modest-main-window.c: