+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:
+ (modest_msg_view_window_get_header),
+ (modest_msg_view_window_last_message_selected):
+ Added comments to explain the use of priv->header_model.
+ * src/modest-ui-dimming-rules.c:
+ (modest_ui_dimming_rules_on_open_msg),
+ (modest_ui_dimming_rules_on_mark_as_read_msg),
+ (modest_ui_dimming_rules_on_mark_as_unread_msg),
+ (modest_ui_dimming_rules_on_remove_attachments),
+ (_invalid_attach_selected), (_selected_msg_marked_as):
+ Renamed _marked_as_deleted() to _selected_message_is_marked_as_deleted(),
+ and changed the parameter to a ModestMainWindow to make it clearer that
+ it never uses any other kind of window.
+ Added _message_is_marked_as_deleted() for examining a ModestMsgViewWindow
+ instead of the selected message.
+ I notice that much of the dimming code and the ModestMsgViewWindow code
+ hard-codes the assumption that the viewed message is always selected in
+ the header view. This does not seem wise or simple.
+
+ (modest_ui_dimming_rules_on_delete_msg): For a ModestMsgViewWindow,
+ examine the actual message instead of whatever might be selected
+ in the header view. This stops the delete toolbar button and menu item
+ from being dimmed when opening a message view the search UI,
+ partially fixing projects.maemo.org bug NB#63811.
+
+ However, when actually deleting, I now get this warning:
+ "tny_header_set_flags: This is a header instance for a new message.
+ Only priority and attachment flags are supported"
+
+2007-07-30 Johannes Schmid <johannes.schmid@openismus.com>
+
+ * src/modest-ui-actions.c: (modest_ui_actions_on_send):
+
+ Show account wizard if no accounts are defined while sending
+ (automatic sending after the account wizard finished does not
+ work because it takes some time until the account manager
+ synchronizes everything. (projects.maemo.org NB#62792)
+
+2007-07-30 Murray Cumming <murrayc@murrayc.com>
+
+ * src/modest-tny-account-store.c: (on_account_removed),
+ (on_account_changed): Emit the ACCOUNT_UPDATE signal so that the
+ account is removed from the View menu. We have TnyAccountStore
+ and ModestTnyAccountStore signals that seem very similar, but this
+ seems to work for now.
+
+ * src/maemo/modest-main-window.c:
+ (on_show_account_action_activated): Prevent dereference
+ of a NULL if the accoutn does not really exist, though that
+ should not happen anymore.
+
+ This fixes projects.maemo.org bug NB#64606.
+
+2007-07-30 Murray Cumming <murrayc@murrayc.com>
+
+ * src/modest-ui-actions.c: (run_account_setup_wizard),
+ (modest_ui_actions_on_accounts): Revert a change that set these
+ to modal dialogs, freezing the UI when they show secondary dialogs.
+ This fixed projects.maemo.org bug NB#64519.
+
+2007-07-27 Murray Cumming <murrayc@murrayc.com>
+
+ * src/widgets/modest-header-view-render.c:
+ (_modest_header_view_compact_header_cell_data):
+ Some checks and initialization to try to deal with a crash,
+ probably ineffective.
+
2007-07-27 Murray Cumming <murrayc@murrayc.com>
* src/maemo/modest-maemo-utils.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>