* src/widgets/modest-global-settings-dialog-priv.h
[modest] / ChangeLog2
index 71db947..e57cb4c 100644 (file)
@@ -1,3 +1,260 @@
+2007-08-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (on_dbus_method_get_folders): Avoid trying to get the account name 
+       when there are no accounts, to avoid critical g_warnings, fixing 
+       projects.maemo.org bug NB#65574.
+
+2007-08-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c: (modest_ui_actions_on_new_msg): 
+       Use modest_account_mgr_get_signature() instead of accessing gconf 
+       directly, to slightly simplify the code.
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_compose_mail):
+       Append the signature also when opening via other applications, 
+       fixing projects.maemo.org bug NB#65828.
+
+2007-08-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-send-queue.c: (modest_tny_send_queue_get_msg_id):
+       Check for NULLs to prevent a crash when opening an email that is 
+       an attachment in another email, fixing projects.maemo.org bug 
+       NB#66012.
+
+2007-08-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-dimming-rules.c:
+       (modest_ui_dimming_rules_on_view_window_move_to): 
+       Enable the Move To toolbar button for search results, fixing the 
+       last part of projects.maemo.org bug NB#63807.
+
+2007-08-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (check_and_offer_account_creation): Use gdk_thread_enter/leave(), 
+       because this is called from idle handlers, to fix hangs afte the 
+       wizard closes.
+       (find_message_by_url):  Added a TODO comment about a crash that 
+       happens when tinymail was built with DBC checks. 
+
+2007-08-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-search.c: (modest_search_folder):
+       * src/modest-search.h: Rename ModestSearch::before to start_date 
+       and after to end_date, using the same names as the libogs struct 
+       fields.
+       * src/dbus_api/modest-dbus-callbacks.c: (on_dbus_method_search):
+       The start_date and end_date were swapped in the check. This 
+       fixes date range searches, fixing projects.maemo.org bug NB#63407.
+
+2007-08-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-view-window.c:
+       Added (check_for_active_acount).
+       (on_delete_button_clicked), (on_edit_button_clicked):
+       When an account is doing something (such as refreshing), 
+       show the same warning for account deletion as for account 
+       editing, to fix projects.maemo.org bug NB#64588. 
+
+2007-08-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-msg-view-window.h:
+       * src/gnome/modest-msg-view-window.c:
+       * src/maemo/modest-msg-view-window.c:
+       Added modest_msg_view_window_construct().
+       (modest_msg_view_window_new_with_header_model),
+       Added modest_msg_view_window_new_for_search_result),
+       Renamed modest_msg_view_window_new() to 
+       modest_msg_view_window_new_for_attachment(),
+       and modest_msg_view_window_is_search_result(),
+       so we can explicitly know that the window is for an 
+       attachment or a search result or a message from the list.
+
+       (modest_msg_view_window_view_attachment):
+       * src/modest-ui-actions.c: (open_msg_cb): Use the renamed function.
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_open_message):
+       Use modest_msg_view_window_new_for_search_result().
+
+       * src/modest-ui-dimming-rules.c:
+       (modest_ui_dimming_rules_on_delete_msg): Enable the delete button 
+       for search results, fixing projects.maemo.org bug NB#63811.
+
+2007-08-16  Armin Burgmeier  <armin@openismus.com>
+
+       * src/modest-ui-actions.c: Show a banner instead of a dialog when
+       renaming a folder fails, to stay consistent with what happens when
+       creating a folder fails.
+
+       * src/maemo/easysetup/modest-easysetup-country-combo-box.c: Optimized
+       slightly country loading from the mmc_mappings file, fixed a memory
+       leak.
+
+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):
+       Handle TNY_ACCOUNT_ERROR_TRY_CONNECT_SERVICE_UNAVAILABLE in the same 
+       way as TNY_ACCOUNT_ERROR_TRY_CONNECT_HOST_LOOKUP_FAILED, showing an 
+       error dialog, instead of ignoring it.
+
 2007-08-01  Murray Cumming  <murrayc@murrayc.com>
 
        * src/maemo/modest-msg-view-window.c:
 2007-07-16  Armin Burgmeier  <armin@openismus.com>
 
        * src/modest-ui-actions.c: Added a button to create a folder to the
-       move-to dialog. This fixes projects.maemo.org bug NB#.
+       move-to dialog. This fixes projects.maemo.org bug NB#59863.
 
 2007-07-16  Johannes Schmid <johannes.schmid@openismus.com>