2007-06-10 Johannes Schmid <johannes.schmid@openismus.com>
[modest] / ChangeLog2
index 970a287..aab92e6 100644 (file)
@@ -1,3 +1,144 @@
+2007-06-10  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/modest-main-window.c: (sync_accounts_cb):
+       Fixed projects.maemo.org bug NB#58576 (not yet activated as there
+       is another problem with the code, see Murray's comment at line 772)
+
+2007-06-08  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-country-combo-box.h:
+       * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
+       (on_model_foreach_release), (easysetup_country_combo_box_finalize),
+       (load_from_file), (easysetup_country_combo_box_init),
+       (easysetup_country_combo_box_get_active_country_ids),
+       (on_model_foreach_select_id): Store a GSList* of IDs in the model row, 
+       instead of just one ID, to avoid duplicate country names in the UI.
+
+       * src/maemo/easysetup/modest-easysetup-provider-combo-box.h:
+       * src/maemo/easysetup/modest-easysetup-provider-combo-box.c:
+       (easysetup_provider_combo_box_fill): Adapt to use a list rather than 
+       a single ID, avoiding duplicate providers.
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (on_combo_account_country): Adapt to use a list.
+
+2007-06-08  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-priv.h: Added a timeout id and a GSList of 
+       keys changed in the last X seconds.
+       
+       * src/modest-account-mgr.h:
+       * src/modest-account-mgr.c: (on_timeout_notify_changes),
+       (on_key_change), (modest_account_mgr_class_init),
+       (modest_account_mgr_init), (modest_account_mgr_finalize):
+       As the keys are changed, save them in a GSList* and signal that GSList* 
+       further every X seconds, to batch changes, to allow us to do just one 
+       update per batch. This should fix bug projects.maemo.org bug NB#56472 and 
+       bug NB#56941 and other (UI-blocking) delays when changing account settings.
+       
+       * src/modest-tny-account-store.c: (on_account_changed): Adapted to 
+       changed signal signature.
+       
+       * src/modest-conf.c: (modest_conf_set_list),
+       (modest_conf_on_change): Reenable notification.
+       
+       (modest_conf_maemo_fake_on_change): Comment out the hack that 
+       notified immediately instead of asynchronously.
+       
+       * src/widgets/modest-account-view.h: 
+       * src/widgets/modest-account-view.c:
+       (update_account_view): Select the previously-selected account ID, 
+       if possible. This fixes projects.maemo.org bug NB#56933.
+       
+       (on_account_changed), (on_account_removed),
+       (modest_account_view_select_account):
+       
+       Remove the block/unblock functions 
+       because they were not useful.
+       
+       * src/maemo/modest-account-view-window.c:
+       (on_delete_button_clicked), (on_edit_button_clicked),
+       (on_new_button_clicked): Remove the use of the non-working block/unblock.
+
+2007-06-08  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Improve the error dialog to show Incorrect Account Settings, which seems 
+       clearer, and say that the internal error message (to be removed from the 
+       dialog later) is often misleading.
+       I have confirmed that projects.maemo.org bug NB#58543 is no longer a problem.
+       
+       * src/modest-tny-account.c:
+       (modest_tny_account_new_from_server_account): Uncomment the printf of the 
+       URL because this is so useful for debugging on the device.
+       
+       * src/widgets/modest-serversecurity-combo-box.c:
+       (modest_serversecurity_combo_box_fill): Correct a logical ID.
+
+2007-06-08  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-mail-operation.c:
+       (modest_mail_operation_send_new_mail):
+       Added a comment about a nasty but apparently harmless message. Details 
+       added to the wiki g_warnings() page.
+
+2007-06-08  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-send-queue.c: (modest_tny_send_queue_add),
+       (modest_tny_send_queue_class_init):
+       Do not use the priv struct because it is empty, to avoid g_warnings 
+       about a non-existant priv struct.
+       
+       * src/modest-tny-account-store.c:
+       (modest_tny_account_store_get_server_account),
+       (modest_tny_account_store_get_transport_account_for_open_connection
+       ): Do not try to get transport server accounts for local folders, 
+       because they don't have any. This avoids a g_warning().
+       
+       * src/maemo/modest-main-window.c:
+       (modest_main_window_notify_send_receive_initied),
+       (on_configuration_key_changed):
+       * src/maemo/modest-msg-edit-window.c: (on_delete_event):
+       * src/modest-defs.h:
+       * src/modest-mail-operation.c:
+       (modest_mail_operation_update_account):
+       * src/modest-tny-account-store.h:
+       * src/modest-tny-account.c:
+       (modest_tny_account_get_special_folder),
+       (modest_tny_account_new_for_local_folders),
+       (modest_tny_account_new_for_per_account_local_outbox_folder):
+       * src/modest-tny-folder.c: (modest_tny_folder_is_local_folder):
+       * src/modest-tny-local-folders-account.c:
+       (add_account_folders_to_merged_folder):
+       * src/modest-ui-actions.c: (modest_ui_actions_on_new_msg),
+       (reply_forward_cb), (modest_ui_actions_on_save_to_drafts),
+       (modest_ui_actions_on_send):
+       * src/widgets/modest-folder-view.c: (update_model):
+       Rename MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID to 
+       MODEST_LOCAL_FOLDERS_ACCOUNT_ID and rename 
+       modest_tny_account_store_get_tny_account_by_account() to 
+       modest_tny_account_store_get_server_account() to make the code clearer.
+
+2007-06-07  Marcus Bauer  <marcusb@openismus.com>
+
+       * src/dbus_api/modest-dbus-api.h:
+       changed MODEST_DBUS_NAME in order to get dbus auto-activation to work
+       * src/maemo/modest-msg-edit-window.c:
+       (modest_msg_edit_window_attach_file_noninteractive):
+       added g_warning in case the file to be attached does not exist
+       * tests/dbus_api/test_compose_mail.c: (main):
+       removed obsolete comment, added generic files as test case
+
+2007-06-07  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (get_server_accounts): 
+       Create the local folders account only once, not once for each outbox, and 
+       not only when there are some outboxes.
+
+2007-06-07  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-folder-view.c: (update_model): Comment out some 
+       code that is causing a crash at startup. Sergio says that he is fixing this.
+
 2007-06-07  Johannes Schmid <johannes.schmid@openismus.com>
 
        * src/maemo/easysetup/modest-easysetup-wizard.c: (on_before_next):