* Close sort dialog after select sorting keys.
* Fixes NB#64682

pmo-trunk-r2994

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.

pmo-trunk-r2992

Added known issues and fixed bugs, week 33 pre release - 14th Aug.

pmo-trunk-r2990

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.

pmo-trunk-r2989

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.

pmo-trunk-r2983

Bugfix for 63283

pmo-trunk-r2982

* Fixed a bug related with filter_row function and
recent new boolean field 'empty' added to header-view
priv structure.

pmo-trunk-r2981

* Emission of progress-changed signal is not alowed, because hang on
folder_copy_asunc was fixed.
* Set the correct path for mcc_mapping file (Fixes NB#61072)
* Fixed some bugs on dimming rules, to allow copy/cut/paste
of folders.

pmo-trunk-r2980

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.

pmo-trunk-r2979

* Fixed some Gtk warnings at start.

pmo-trunk-r2974

* Fixed some bugs on dimming rules about copy/cur/paste folders.
* Check if folder is empty on POP accounts refreshing, due to
POP refresh operation does not emit notifications.
* Use filter_row function, called to hide marked as deleted
rows on header-view, to determine if folder is empty.

pmo-trunk-r2973

Javier: if I remove this one, tny_folder_copy_async does not hang anymore
g_signal_emit (G_OBJECT (self), signals[PROGRESS_CHANGED_SIGNAL], 0, state, NULL);
https://projects.maemo.org/bugzilla/show_bug.cgi?id=63060

pmo-trunk-r2972

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.

pmo-trunk-r2971

removed a g_debug

pmo-trunk-r2970

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

pmo-trunk-r2969

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.

pmo-trunk-r2968

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.

pmo-trunk-r2967

Added known issues and fixed bugs, week 33 pre release - 13th Aug.

pmo-trunk-r2966

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.

pmo-trunk-r2965

Added known issues and fixed bugs, week 33 pre release - 13th Aug.

pmo-trunk-r2964

* modest-header-view.c: Remove some commented and unsed code.
* modest-ui-actios.c: implemented some changes on
new_messages_arrieved callback, executed afte send&receive has
been completed, to check if folder is empty and new message
have arrived; this imlpies a change in main-view window style.
* Fixes: NB#57458

pmo-trunk-r2963

Changed the changelog to resolve some bugs issues, week 32 post release - 10th Aug.

pmo-trunk-r2962

Changed the changelog to resolve some bugs issues, week 32 post release - 10th Aug.

pmo-trunk-r2961

Added a few fixed bugs and release notes to the changelog, week 32 post release - 10th Aug.

pmo-trunk-r2960

modest-ui-actions.c: some alignment in the code
modest-ui-dimming-rules.c: some cleanup of unnecessary codes
modest-msg-view-window.c: partially fixes bug 60714, and have further
modifications. The concept is using signals of tree_model for any
change. If there is a change, dimming rules should be applied on
msg-view-window.

Not: There remains some bugs, which I do not know if registered on
bugzilla or not, but are near the same problems like 60714.
The tree_view (header_view) in main window might have new tree_model
due to some user actions and so 60714 bug become current again.

pmo-trunk-r2959

* Review fucntion 'has_retrieved_msgs' implementation
in order to use CACHED flag to determine wether one message
has been downloaded or not.
* Fixes: NB#62450

pmo-trunk-r2958

* Add a confirmation dialog to Paste operation.
* Fixes: NB#63124

pmo-trunk-r2955

* Fixed some warnings

pmo-trunk-r2954

* Fixed some warnings.

pmo-trunk-r2953

* Fixed some bug in new dimming rules approach.

pmo-trunk-r2950

* Set focus on folder-view at startup to show correctly
the main window title.
* Fixes: NB#61815

pmo-trunk-r2949

Added a few fixed bugs and release notes to the changelog, week 32 pre release - 8th Aug.

pmo-trunk-r2948

* Implement optimizations on DimmingRules management.
* Copy/Cut/Paste dimming rules of Editor are not working fine;
it does not take into account if something was selected or not.
These changes on src/maemo/modest-msg-edit-window.c try to
solve these problems, with no success at this moment.
* Fixes: NB#64553

pmo-trunk-r2947

Added fixed bugs and release notes to the changelog, week 32 pre release - 8th Aug.

pmo-trunk-r2946

* Check folder type in folder comparison.
* Fixes: NB#63055

pmo-trunk-r2945

* Call send-queue-flush when account is edited and changed,
and also at startup.

pmo-trunk-r2944

Fixing API changes

pmo-trunk-r2941

* fix some issues on progress-bar to show more pulses on
undetermined progress data.
* Define again a OPEN type mail-operation for cached messages.
* Fixes bug: NB#56201

pmo-trunk-r2940

* Set some g_return_if_fail because i notice some random
chrashed trying to add one account, when it gets the private
structure of ModestaccountView widget.

pmo-trunk-r2939

* Removed a g_return_if_fail because this function
is called from show_tollbar method with NULL in account
parameter.

pmo-trunk-r2938

* Fixed some bug on account_number_changed signal handler.
The fisrt parameter, account_name, was declared as string,
but account-remove, account-inserted, and account-changed
emmits a signal with a TnyAccount type parameter.

pmo-trunk-r2937

Added fixed bugs to the changelog, week 32 pre release - 6th Aug.

pmo-trunk-r2936

2007-08-06  Philip Van Hoof  <pvanhoof@gnome.org>

* GDK lock awareness

pmo-trunk-r2935

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)

pmo-trunk-r2934

* src/modest-ui-actions.c:
        * (modest_ui_actions_create_folder): now, if we push create
          folder in a pop folder, unable to create folder information
          banner is shown (fixes NB#64001).

pmo-trunk-r2933

* src/maemo/modest-account-settings-dialog.c:
* Now we show the "settings saved" or "settings" failed
  information banners again (we were parenting it with the
  dialog we were closing, so the dialog was not saved)
  (fixes NB#63748).

pmo-trunk-r2932

* src/maemo/modest-msg-view-window.c:
        * Now the removing attachments banner is shown after 2 seconds
          of progress of the operation, and not at the end of the
          operation (fixes NB#63026).

pmo-trunk-r2931

* src/modest-text-utils.c:
        * Now derived subject composes a string with no subject
          also for a "" subject.
* src/widgets/modest-attachment-view.c:
        * If subject is "", we also show no subject string
* src/modest-tny-msg.c:
        * If we are forwarding, and the mail is formatted in html,
          we forward the message as attachment (fixes NB#62261).
* src/modest-formatter.[ch]:
        * Fixed modest_formatter_attach, to attach the TnyMsg instead
          of a mime part.

pmo-trunk-r2930

Improvements in focus management of Check names operation
(fixes NB#59427).
* src/widgets/modest-recpt-editor.[ch]:
        * New method (modest_recpt_editor_has_focus). This is used
          to be able to know if a recipient editor text view is
          focused.
* src/maemo/modest-address-book.c:
        * Removed all recipient editor focus management. Now we rely
          on the window implementation.
* src/maemo/modest-msg-edit-window.c:
        * Now we set focus of the failing recipient editor in check
          names if it fails.
        * If check names does not fail, and focus is in a recipient
          field, it does not change the focus from them to to: field.

pmo-trunk-r2929

* src/widgets/modest-msg-view.c:
* (idle_readjust_scroll): hackish solution to force a
  scroll readjustment (setting 1.0 and 0.0 in the same handler
  and after showing the widget for the first time). This way
  we don't get a broken scroll with stylus (fixes NB#62820).

pmo-trunk-r2928

* Added some coments/mark in all gdk_threads_enter
to document what is their purpose on each funciton.

pmo-trunk-r2927

* fix NB#64931, cleaning up the details fields a bit.

pmo-trunk-r2926

removed a released bug from the changelog, week 31 post release - 3rd Aug.

pmo-trunk-r2925

Added new fixes to changelog, week 31 post release - 3rd Aug.

pmo-trunk-r2924

Remove of an false warning.

pmo-trunk-r2922

An ref/unref problem resolved, but there might be another one.

pmo-trunk-r2921

* check for a valid foldername

Fixes: NB#58923

pmo-trunk-r2920

* add a function to check for valid folder name;
  we do a comprehensive test, also disallowing names which are not
  valid on Windows filesystems (like AUX, or ':'); thus, it
  should also work on MMC, and also make sure that user-created local folders
  can be moved there.

pmo-trunk-r2919

fixed crash:modest-mail-operation.c:on_refresh_folder:priv==NULL

pmo-trunk-r2917

* src/maemo/modest-msg-view-window.c:
* Now we don't show an information banner on trying to view
  a purged attachment (fixes NB#63498).

pmo-trunk-r2916

* Move tny_folder_sync after remove operation and define it
on modest-ui-actions, when all messages have been removed.
* Review progress-bar widget to get more information when
undetermined data is come back from camel-operation.
* Move header_view_refilter operation, defined in
folder_monitor_update to main_window, which is an observer
of headers view. Sometimes, refilter operation was done before
model was updated, causing several crashes.

pmo-trunk-r2915

* src/widgets/modest-attachment-view.c:
        * Now get size thread worker is not anymore a mail operation,
          so the implementation is moved here. This should prevent
          some bad issues with getting uncancellable operations in
          mail operation queue, and then, causing hanges.
* src/modest-mail-operation.[ch]:
        * Now get mime part size is not anymore a mail operation, so
          this has been removed from here.

pmo-trunk-r2914

* optimize _modest_account_mgr_get_account_keyname, which was showing
  at the top of the startup profiles. now, no longer escape/allocate strings
  if that's not needed

pmo-trunk-r2913

* fix the checks a bit; don't try to top the window before checking it has been created

pmo-trunk-r2912

* let the window manager close only after the mail-operations-queue is empty,
  recheck every 3 secs, and close anyway after a given timeout (30 sec. now).

this should help with the bugs were mail was corrupted when sending it.

pmo-trunk-r2911

* when the window-manager is asked for the main_window, and none such
  exists, create one in place. the fix in modest_window_mgr is trivial,
  but it would crash immediately because of old signal handlers still
  be invoked; now, we clear all signal handlers when destroy a main_window

Fixes: NB#59752

pmo-trunk-r2910

* add modest-signal-mgr, some small utility functions for gobject to
  remember what g_signal_connects they did, and make sure they are all
  disconnected when you finalize the object; this was needed for a bugfix

pmo-trunk-r2909

* src/modest-tny-send-queue.c:
* Now it shows a "message sent" information banner on successing
  to send a message (fixes NB#63356).

pmo-trunk-r2908

Added new fixes to changelog, will be part of week 32 release-  week 31 - 2nd Aug.

pmo-trunk-r2907

* with the latest tinymail, there is no need to force a refresh
  of the folder model anymore, when the account name changes.

pmo-trunk-r2905

* src/modest-ui-actions.c:
        * (modest_ui_actions_xfer_messages_from_move_to): now we
          show an information banner on trying to move messages to a pop
          folder as this is not allowed (fixes NB#63285).

pmo-trunk-r2904

* src/modest-ui-actions.c:
* (modest_ui_actions_on_delete_folder): now we focus inbox
  folder on finishing the delete folder operation (fixes
  NB#63239).

pmo-trunk-r2903

* Review Move_to dimming rules to check for ROOT folders,
or INBOX or SENT, DRAFT and OUTBOX special folder.
* Fixes: NB#62939

pmo-trunk-r2902

* src/widgets/modest-recpt-editor.c:
* Modified the maximum number of lines, to 1024. Practically,
  it means the field has an "unlimited" number of lines
  (fixes NB#63209).

pmo-trunk-r2901

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.

pmo-trunk-r2900

* src/modest-text-utils.c:
        * (modest_text_utils_validate_email_address): a domain part
          that finishes with a dot (for example "a@b.") is not valid
          (fixes NB#63204).

pmo-trunk-r2899

* src/maemo/modest-msg-edit-window.c:
        * A missing else was breaking save to drafts deletion of
          old draft.
* src/modest-mail-operation.c:
        * Now save to drafts first adds the new message, and then
          removes the old one.
        * Now save to drafts folder sync is asynchronous.

pmo-trunk-r2898

* fix some of the tedious value checking, as specified in
  NB#59361 (which is fixed now)

pmo-trunk-r2896

* src/maemo/modest-msg-edit-window.c:
        * Added entry verification in subject field. In particular we
          discard the bullet special unicode character (fixes NB#62721).

pmo-trunk-r2895

* the real fix for NB#64809, the previous accidentaly
  fixed the same issue in the normal account setup, now we're
  fixing it for the connection-specific smtp servers as well.

also fixed the label for the Port number

pmo-trunk-r2894

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.

pmo-trunk-r2893

* validate the domain names
  Fixes: NB#64809

pmo-trunk-r2892

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.

pmo-trunk-r2891

* Use the correct translation string instead of a fixed one when
  trying to save several attachments (Fixes: NB#63764)

pmo-trunk-r2890

* port numbers should be in the range [1,65535], 0 is not valid.

Fixes: #NB64831

pmo-trunk-r2889

* more changes to the account management; now the following should work again:
  - changing account names (updates modest-account-view, and modest-folder-view)
  - expand root folders
  - change modest-account-mgr to only emit change signals once per 500ms
  - modest-account-mgr changes notification now only give the "parent account",
    not the server account, and not for every single key
  - accounts are changed, not recreated

pmo-trunk-r2888

* src/maemo/modest-progress-bar-widget.c:
        * Now we show the undetermined string for receiving and sending
          when we're getting only one message. This should fit the
          optional ammendment in UI specs (fixes NB#56182).

pmo-trunk-r2887

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

pmo-trunk-r2886

* forgot to commit this

pmo-trunk-r2885

* src/widgets/modest-msg-view.c:
        * Modified the html adjustment change handler to prevent some
          infinite recursions. In particular one happening on removing
          attachments.
* src/widgets/modest-attachment-view.c:
        * Add support for showing purged attached messages.
* src/modest-ui-dimming-rules.c:
        * Now we don't dim remove attachments for message attachments in
          view, as we now support this (fixes NB#64068).

pmo-trunk-r2884

* step 2 of the account fixes; for now, account changes
  are turned-off, as we get hangs when we get alert-popups

pmo-trunk-r2883

* src/widgets/modest-msg-edit-window.h,
  src/maemo/modest-msg-edit-window.c:
        * New method (modest_msg_edit_window_get_message_uid), that
          gets the message uid of the current draft or outbox message
          the mail was openend from.
* src/widgets/modest-window-mgr.c:
        * Modify all the message register methods, to add support for
          also registering edit windows (fixes NB#64714).

pmo-trunk-r2882

* src/modest-text-utils.[ch]:
        * New (modest_text_utils_get_email_address) function, that
          obtains the email address email@address from a full
          address Full Address <email@address>.
* src/maemo/modest-address-book.c:
        * (modest_address_book_add_address): now we add the email part
          of an address to the addressbook, instead of the full address
          (fixes NB#64733).

pmo-trunk-r2881

* added modest_tny_account_update_from_server_account_name, we need
  this function for the new account management. this is done by refactoring
  some code -- the functions here are much too long anyway, which makes
  maintenance / refactoring / fixing hard.

no functional changes yet.

pmo-trunk-r2880

* don't try to load some icons which are not present,
  and we're not using anyway. saves some runtime warnings

pmo-trunk-r2879

* Review dimming rules about mark as read/unread headers.
Now, rules considerd that all headers must be equal to dim
or not menu options.
* Fixes: NB#60676

pmo-trunk-r2878

* Show progress data of the first queued mail operation.
* Show a initial message avoiding show empty progress bar
while async operation does not send back progress information.
* Fixes: NB#60517

pmo-trunk-r2877

* src/maemo/modest-msg-edit-window.c:
        * We make the max image with even smaller, to avoid confusion
          having in one line a full image and a small character.
        * Now we correct the scroll position to show the cursor if
          the text view is focused and the cursor is not really visible.
          Calculation of cursor visibility is done manually, and after
          last text view size update. This way we finally get the cursor
          at its proper position on inserting big images (fixes
          NB#62829).

pmo-trunk-r2876

* src/maemo/easysetup/modest-easysetup-provider-combo-box.c:
* Set "Other..." option as the first in providers combobox.
* Use the logical id (fixes NB#64683).

pmo-trunk-r2875

* src/widgets/modest-recpt-editor.c:
        * Fix some recursion errors in validation methods
          (fixes NB#64721).

pmo-trunk-r2873