2007-08-16 Murray Cumming <murrayc@murrayc.com>
[modest] / ChangeLog2
index f33214b..9f822b4 100644 (file)
@@ -1,3 +1,162 @@
+2007-08-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (volume_path_is_mounted):
+       Check the GnomeVFSVolume activation path instead of the device path, 
+       so that the memory card is really really shown in the UI.
+
+2007-08-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (get_entered_account_title), (on_before_next), (entry_is_empty),
+       (create_account):
+       * src/maemo/modest-account-settings-dialog.c:
+       (get_entered_account_title), (on_button_signature), (check_data),
+       (save_configuration), (entry_is_empty):
+       Strip the account title of whitespace at the start and end before 
+       using it or checking it. This means that an all-whitespace title 
+       is not allowed, fixing projects.maemo.org bug NB#64977.
+
+2007-08-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.h:
+       * src/modest-ui-actions.c: Renamed modest_run_account_setup_wizard() 
+       to modest_run_account_setup_wizard() and made it public.
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (check_and_offer_account_creation), (on_idle_mail_to),
+       (on_idle_compose_mail), (on_idle_open_default_inbox):
+       Offer account creation if necessary, fixing projects.maemo.org 
+       bug NB#63713.
+
+2007-08-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c: (run_account_setup_wizard):
+       Return a gboolean to indicate success.
+       (modest_ui_actions_on_new_msg), (open_msg_cb), (reply_forward),
+       (modest_ui_actions_on_send): When offering account creation, only 
+       give up (return) if account creation failed. Otherwise continue 
+       with the original request. This fixed projects.maemo.org bug 
+       NB#64317. 
+
+2007-08-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-recpt-editor.c:
+       (modest_recpt_editor_instance_init): Turn off hildon auto-capitalization,
+       to fix projects.maemo.org bug NB#63789.
+
+2007-08-15  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: Added volume_path_is_mounted().
+       (modest_tny_account_store_new): Properly detect the memory card at 
+       startup, and add an account for it.
+
+2007-08-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_new):
+       Commented out the MMC code again because it mistakenly detects 
+       the card even when it isn't inserted.
+
+2007-08-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_new):
+       Renabled the call to add_mmc_account() because I do not believe 
+       it could be the cause of the mentioned hang (in scratchbox, where 
+       there is no memory card), and because I have tested that it works 
+       on the device.
+       
+2007-08-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_new):
+       Comment out part of the last commit because it apparently causes 
+       a hang.
+
+2007-08-14  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: Added the add_mmc_account() 
+       function to save some copy/pasting.
+       (modest_tny_account_store_new): If the memory card is mounted, 
+       create an account for it at the start, instead of waiting for the 
+       user to unmount it and mount it.
+
+2007-08-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-account-view.c: (on_account_changed):
+       Do not try to use the human-readable TnyAccount name instead of the 
+       ID of the modest account to get the transport account. And unref 
+       the transport account. This prevents a g_warning(), and a possible 
+       ref leak.
+
+2007-08-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-platform.c:
+       (modest_platform_set_update_interval): Handle the new 
+       alarmd error to warn when alarmd is not running, as is the case in  
+       scratchbox for me now because 
+       "Module /usr/lib/alarmd/libretu.so initialization failed"
+
+2007-08-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-header-view-render.c:
+       (_modest_header_view_compact_header_cell_data):
+       NULL some strings after freeing them, trying to deal with 
+       the double deletion found in projects.maemo.org bug 64183, 
+       probably having no effect.
+
+2007-08-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-settings-dialog.c:
+       (create_page_incoming), (create_page_outgoing): Use a define 
+       for the min and max port numbers.
+
+       (check_hildon_number_editor_and_warn_value_not_in_range),
+       (check_data): Do an extra check for an invalid port number, as needed 
+       in projects.maemo.org bug #64275, when the number is cleared, but this 
+       does not work due to bugs.maemo.org bug #1806.
+
+2007-08-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_filter):
+       Comment out the debug warning about the unexpected D-Bus methods. 
+       They are just annoying now.
+       * src/maemo/modest-platform.c:
+       (modest_platform_set_update_interval): Correct the D-Bus path, instead 
+       of using a file path, though it did not seem to break things before. 
+       It might be needed now:
+       Use the ALARM_EVENT_ACTIVATION flag so that emails are updated even when 
+       modest is not already running. This is the behaviour of the old client, and     
+       Mox says it is what we should do for modest.
+       This closes projects.maemo.org bug NB#64327. 
+
+2007-08-04  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/modest-text-utils.c:
+       Fixed invalid jump due to incorrect string/pointer
+       operation. 
+       (See https://projects.maemo.org/trac/email/wiki/ValgrindErrors
+       for details)
+
+2007-08-01  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       Added modest_account_mgr_get_first_account_name(), 
+       using code from modest_account_mgr_set_first_account_as_default().
+
+       * src/maemo/modest-main-window.c: Added set_at_least_one_account_visible().
+       (account_number_changed): Call set_at_least_one_account_visible() so that 
+       the first account is visible when first added, without the user having to 
+       use the View menu.
+       I do wonder how this worked before.
+
+2007-08-01  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (show_password_warning_only),
+       (show_wrong_password), (request_password_and_wait), (get_password),
+       (modest_tny_account_store_alert): Remove the (incomplete anyway) 
+       attempts to show UI only in the main context, which used g_idle_add() 
+       and mainloops, because tinymail now guarantees that it only calls these 
+       callbacks in the mainloop with the gdk lock. This makes everything much 
+       simpler.
+
 2007-08-01  Murray Cumming  <murrayc@murrayc.com>
 
        * src/modest-tny-account-store.c: (modest_tny_account_store_alert):