* Some fixes
[modest] / ChangeLog2
index fa51b9c..48d6fa3 100644 (file)
@@ -1,3 +1,130 @@
+2007-05-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-pair.h: Improve the documentation for modest_combo_box_new() 
+       to explain the memory management issue with the ID in the ModestPair.
+       
+       * src/gnome/modest-account-assistant.c: (add_receiving_page),
+       (add_sending_page), (modest_account_assistant_finalize):
+       * src/gnome/modest-gnome-global-settings-dialog.c:
+       (create_updating_page), (create_composing_page):
+       * src/gnome/modest-msg-edit-window.c: (get_transports),
+       (init_window), (modest_msg_edit_window_finalize),
+       (modest_msg_edit_window_get_msg_data):
+       * src/gnome/modest-store-widget.c: (imap_pop_configuration):
+       * src/gnome/modest-transport-widget.c:
+       (modest_transport_widget_finalize), (smtp_configuration):
+       * src/maemo/modest-maemo-global-settings-dialog.c:
+       (create_updating_page), (create_composing_page):
+       * src/maemo/modest-msg-edit-window.c: (get_transports),
+       (init_window), (modest_msg_edit_window_finalize),
+       (modest_msg_edit_window_get_msg_data),
+       (modest_msg_edit_window_free_msg_data):
+       * src/maemo/modest-store-widget.c: (imap_pop_configuration),
+       (modest_store_widget_finalize):
+       * src/maemo/modest-transport-widget.c:
+       (modest_transport_widget_finalize), (smtp_configuration):
+       * src/modest-main.c: (send_mail):
+       * src/widgets/modest-combo-box.h:
+       * src/widgets/modest-global-settings-dialog-priv.h:
+       * src/widgets/modest-global-settings-dialog.c:
+       (modest_global_settings_dialog_finalize):
+       * src/widgets/modest-msg-edit-window.h:
+       Whenever using modest_combo_box_new(), keep the ModestPairList alive
+       as long as the combo box, to prevent use of freed memory for the ID.
+
+2007-05-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (get_server_accounts): Renamed from 
+       get_accounts(), to make this clearer. Do not check for the enabled gconf
+       key, because we do not use this for server accounts (at least, not yet).
+       This allows us to send email again.
+
+2007-05-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-platform.c:
+       (on_modest_conf_update_interval_changed), (modest_platform_init):
+       Only respond to this particular key change. All these strcmp()s cannot be 
+       efficient.
+
+2007-05-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-global-settings-dialog.c:
+       (current_connection): Prevent a crash in scratchbox when 
+       ny_maemo_conic_device_get_current_iap_id() returns NULL.
+       * src/maemo/modest-platform.c:
+       (on_modest_conf_update_interval_changed), (modest_platform_init):
+       Get the update interval from gconf and reset it when the gconf key changes.
+
+2007-05-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-protocol-info.c:
+       * src/modest-protocol-info.h: Adde enum-specific get functions, 
+       instead of using a ModestProtocolType enum parameter.
+       
+       * src/gnome/modest-account-assistant.c:
+       * src/gnome/modest-store-widget.c:
+       * src/gnome/modest-store-widget.h:
+       * src/gnome/modest-transport-widget.c:
+       * src/gnome/modest-transport-widget.h:
+       * src/maemo/easysetup/modest-easysetup-servertype-combo-box.c:
+       * src/maemo/easysetup/modest-easysetup-servertype-combo-box.h:
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       * src/maemo/modest-account-settings-dialog.c:
+       * src/maemo/modest-account-settings-dialog.h:
+       * src/maemo/modest-store-widget.c:
+       * src/maemo/modest-store-widget.h:
+       * src/maemo/modest-transport-widget.c:
+       * src/maemo/modest-transport-widget.h:
+       * src/modest-account-mgr-helpers.c:
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr.c:
+       * src/modest-account-mgr.h:
+       * src/modest-tny-account.c:
+       * src/modest-tny-folder.c: 
+       * src/widgets/modest-account-view.c:
+       * src/widgets/modest-retrieve-combo-box.c:
+       * src/widgets/modest-retrieve-combo-box.h:
+       * src/widgets/modest-secureauth-combo-box.c:
+       * src/widgets/modest-serversecurity-combo-box.c:
+       * src/widgets/modest-serversecurity-combo-box.h:
+       * tests/check_account-mgr.c: 
+       Use the specific functions, and use the re-renamed enum values for GNOME too.
+
+2007-05-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-pair.h: 
+       * src/modest-pair.c: Added modest_pair_list_find_by_first_as_string(),
+       so we can get the second based on the first.
+
+       * src/maemo/modest-account-settings-dialog.h: 
+       * src/maemo/modest-account-settings-dialog.c:
+       Added get_supported_secure_authentication_methods(), though it is mostly 
+       commented-out for now until the necessary API is committed to tinymail.
+       (modest_account_settings_dialog_set_account_name),
+       (save_configuration): Request the supported authentication methods from the 
+       server, so we can choose a working method.
+       
+       * src/modest-tny-account-store.h:
+       * src/modest-tny-account-store.c:
+       (modest_tny_account_store_get_session): Rename from 
+       tny_account_store_get_session().
+
+2007-05-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-protocol-info.h: Split the ModestProtocol enum 
+       into ModestProtocol, ModestSecureConnection, and ModestSecureAuthentication.
+       There was no need to have these unrelated values in one enum.
+       
+       * src/: many files: Adapted to changed enum. This makes the code a bit
+       clearer, and the compiler could catch some errors.
+
+2007-05-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Show the detailed internal camel error message when an error happens 
+       during the configuration, so that our testers can give us more clues.
+       For instance, in projects.maemo.org bug NB#56910 .
+
 2007-05-14  Murray Cumming  <murrayc@murrayc.com>
 
        * src/modest-tny-account.c: