* modest-main.c:
- return 0 iff there are no errors
* modest-ui-actions:
- clear the queue when closing down
* modest-runtime.h:
- remove unimplemented modest_runtime_init_ui
* modest-tny-account-store:
- debugging messages

pmo-trunk-r2515

* don't set the window icon explicitly; maemo seems to
  ignore it, and as we could not find the icon anyway,
  this saves a runtime warning

pmo-trunk-r2514

* Signal handlers are executed in the main lock, so no need to block

pmo-trunk-r2512

2007-06-30  Johannes Schmid <johannes.schmid@openismus.com>

* src/maemo/modest-connection-specific-smtp-edit-window.c:
(on_value_changed),
(modest_connection_specific_smtp_edit_window_init):

Fixed project.maemo.org bug NB#60479 by using \"notify::value\" instead
of \"changed\" for now

pmo-trunk-r2511

* remove unneeded stuff from modest-main (the batch mode stuff which wasn't working anyway
* cleanup the initialization, which should make it more clear what happens
* don't do separate ui/core init

pmo-trunk-r2510

* leak fix; unref the icon

pmo-trunk-r2509

* leak fixes

pmo-trunk-r2508

* Add a error_handler for send_receive operation.
* Review dimming rules of 'Cancel sending' command.
* Fixes: NB#61952

pmo-trunk-r2507

* Review dimming rules of 'delete' and 'delete folder'
* Fixes: NB#61254

pmo-trunk-r2506

2007-06-29  Murray Cumming  <murrayc@murrayc.com

* src/modest-tny-account.c:
(modest_tny_account_new_from_server_account):
Handle the TnyAccount::connection-status-changed signal, which tinymail
now uses to request a connection when it needs one, and which it uses
to tell us that something is wrong with the connection, so we can force
the accounts offline to prevent further problems.

* src/modest-defs.h:
* src/modest-init.c: (init_default_settings):
* src/modest-mail-operation.c:
(modest_mail_operation_update_account):
* src/modest-main.c: (on_show): Do not try to connect explicitly on startup.
This should not be necessary anymore. Do not set the conf key that
determined this (it was always TRUE and not set anywhere else).

* src/modest-ui-actions.h: Remove unused and unimplemented
modest_ui_actions_on_online_toggle_toggled().

pmo-trunk-r2505

* Fixed a compilation warning

pmo-trunk-r2504

* fix a memleak in the msg edit window
* fix a memleak when registering the update_interval for the alarm daemon

pmo-trunk-r2503

* src/widgets/modest-header-view-render.c:
        * Added some implementation work to retrieve the send status of
          mails in outbox. get_status_string() obtains the proper string
          for a status, get_status_of_uid() should retrieve the current
          status trying in all send queues, the cell renderer has a
          specific path for the case of outbox.
        * As currently it does not get the proper status from the
          send queue, we set always "waiting".
* src/widgets/modest-header-view-priv.h:
        * Added an enum of the types of compact cell renderer.
* src/widgets/modest-header-view.c:
        * Pass the proper mode to the compact cell renderer (incoming,
          outgoing, outbox).

pmo-trunk-r2502

* Use an user handler for 'key-press-event' signal on
ModestWindow class.
* Fixes: NB#61828, NB#61825

pmo-trunk-r2501

Updated the changelog with the fixes made on week26 29th june.

pmo-trunk-r2500

Fixed the bug 61219, changed from showing dialog to showing banner if the account already exists.

pmo-trunk-r2499

* Fixed bug in select_next function.

pmo-trunk-r2498

* fix logical error, don't call
  search_mime_part_and_child_parts
   if msg == NULL. This fixes a crash if msg happens to be NULL

pmo-trunk-r2497

* src/maemo/modest-msg-edit-window.c:
        * Now the font size toolbutton always get the default font face
          (it was getting the default font, but then it was replaced
          with Serif on first font size change).

pmo-trunk-r2496

* src/maemo/modest-msg-edit-window.c:
        * Fixed some typo errors in set_format_state function.
        * Use set_attribute instead of set_format in font size change
          handler.

pmo-trunk-r2495

2007-06-29  Murray Cumming  <murrayc@murrayc-desktop>

* src/maemo/modest-main-window.c: (connect_signals):
Don't connect to delete_event twice. Maybe this was harmless.

pmo-trunk-r2494

2007-06-29  Murray Cumming  <murrayc@murrayc.com

* libmodest-dbus-client/libmodest-dbus-client.c: Improved the documentation slightly.
* src/modest-mail-operation.c: (modest_mail_operation_xfer_folder):
Debug output to give some more clue when a folder move is not allowed.

pmo-trunk-r2493

* update the various places where the modest-window-mgr was used to register
  the header *before* the message is opened (see previous patch).
* also update the dimming rules, so opened messages will  have the delete
  button in the main window dimmed, as well, as when selecting multiple of
  which one or more are opened already.

pmo-trunk-r2492

* update the modest-window-manager; together with the next patch, this
  fixes NB#57297
- we do this by adding modest_window_mgr_register_header,
modest_window_mgr_find_registered_header. in the next patch, these will
be called *before* actually opening the message, so it will be impossible
to open the same message several times. the problem used to be that
the windows were only registered after the message was retrieved, so
plenty time to have doubles, esp. on a slow connection
- also fix a bug where really the registered *window* is returned,
instead of the GList* (casting hid this bug), this enables us to popup
a msg-view-window if it was already opened for the activated message

pmo-trunk-r2491

* Reverted a do not wanted change

pmo-trunk-r2490

* Do "pulse" when done=0 and total=0

pmo-trunk-r2489

* Do not block signal emissions into idles callbacks, because signal handlers are already executed into the main lock

pmo-trunk-r2488

* Fixes some warning about uninitialized valrables.

pmo-trunk-r2487

small change for Fixed bug 58983, changed the labels from the previous commit.

pmo-trunk-r2486

Fixed bug 58983, changed the order and labels for delete account.

pmo-trunk-r2485

* Review 'move to' callback from viewer window with pop
accounts.
* Fixes: NB#61929

pmo-trunk-r2484

* Fixes: NB#61955

pmo-trunk-r2483

* Fixes NB#62026, folders can now be moved to the root local account
* Added some checking to cmp_rows of the folders view
* Removed some code of the mail operations that didn't allow a folder to be moved to an account
* The UI actions allows now a move to to an account, if it's the local account

pmo-trunk-r2482

* Fixes NB#61826, the outbox folder is shown as Outbox in the folder details dialog
* Fixed some warnings

pmo-trunk-r2481

* Fixes NB#61948, the accounts could not be cut nor copied

pmo-trunk-r2480

* Fixes NB#58923, do not allow '/' in folder names

pmo-trunk-r2479

Work to complete implementation of purge attachments operation
(fixes NB#61240).
* src/widget/modest-msg-view.[ch]:
        * Added API function to get all the attachments in
          the view, and not only the selected ones.
* src/widgets/modest-attachment-view.c:
        * Support for showing purged files improved (still pending
          a proper logical id).
* src/widgets/modest-msg-view-window.[ch]:
* src/modest-ui-actions.[ch]:
        * Added support for purging attachments of a message from the
          main window, with lots of exception conditions (trying to
          remove messages with all attachments already purged, etc).
        * Added confirmation dialog for removing attachments from
          main window.
* src/maemo/modest-main-window-ui.h:
        * Attach remove attachments action in main window.
* src/widgets/modest-msg-view-window.[ch]:
        * Now you cannot save or view a purged attachment.
        * Now you can choose in ..._remove_attachments to remove
          only selected attachments or all the available attachments.
        * Skip already removed attachments
        * More informationbanners for purge operation.
        * Now the message is correctly reloaded after being purged.
* src/widgets/modest-ui-dimming-rules.[ch]:
        * Added a lot of dimming rules for removing attachments both
          from main window and msg view window.

pmo-trunk-r2478

updated the changelog with fixes.

pmo-trunk-r2477

Fix for the bug 59012, changed appedn to prepend.

pmo-trunk-r2476

* Fixed some diomming rules.
* Set flags DELETED just after tny_folder_remove_msg
* Add a filter in heder view to hide deleted rows

pmo-trunk-r2475

2007-06-28  Murray Cumming  <murrayc@murrayc.com

pmo-trunk-r2474

Some debug output to help me with testing.

pmo-trunk-r2473

* Commented some code that causes a hang. It reopens a bug, but it's a normal one and Modest won't hang

pmo-trunk-r2472

* set all accounts offline when finalizing the modest-tny-account-store;
  this should help with the problem where the modest window is closed,
  but it's still running, which make restarting modest impossible.

pmo-trunk-r2471

* Wrap the gtk_main between gdk_threads_enter/leave code as documentation says

pmo-trunk-r2470

* Fixed an issue whith signal handlers when unregistering windows
* Make a more fine-grained gdk threads locking
* Added msg view window registration in the dbus callbacks
* Removed an uneeded gdk_threads_enter/leave

pmo-trunk-r2469

* Signal handlers are executed in the main lock, so no need to lock it again

pmo-trunk-r2468

2007-06-27  Armin Burgmeier  <armin@openismus.com>

* src/modest-mail-operation.c: Fixed a crash when canceling a mail
receive operation.

pmo-trunk-r2467

* fixed some bugs on dimming rules.

pmo-trunk-r2466

* Review dimming rules of paste operation.
* Review dimming rules of viewer window toolbar.
* Fixes: NB#60441, NB#61044

pmo-trunk-r2465

2007-06-27  Murray Cumming  <murrayc@murrayc.com

* src/modest-tny-account.h:
* src/modest-tny-account.c:
Added modest_tny_account_get_mmc_account_name(), to set the memory
card name asynchronously for the given account and then notify.
(modest_tny_account_new_for_local_folders): Use this new function instead
of using hildon_file_system_info_async_new() directly.

* src/widgets/modest-folder-view.c: (text_cell_data):
Use modest_tny_account_get_mmc_account_name() to get the real name, calling
gtk_tree_model_row_changed() when it is available.

This should fix projects.maemo.org bug NB#59301.

pmo-trunk-r2464

* Connect an insensitive-press signal handler to
OK button of new folder dialog.
* Add a new dimming rules to check folder depth, only
for Context Sensitive Menu of folder view.
* Fixes NB#60458, NB#61835

pmo-trunk-r2463

Commented out bugfix for setting new accounts online - regression problem

pmo-trunk-r2462

updated the changelog with the fixes on 27.june

pmo-trunk-r2461

Reverted again the gdk_threads_enter/leave sentences:
* src/widgets/modest-attachment-view.c
* src/widgets/modest-header-view.c
* src/maemo/modest-maemo-global-settings-dialog.c
* src/modest-mail-operation.c

Misc changes (mainly work to implement attachment purging):
* src/widgets/modest-msg-view.c:
        * Added find toolbar scroll fix, disabled as we're waiting
          for the new gtkhtml gtk_html_get_selection_area API.
        * Now we don't remove the attached part on removing attachments
          from view.
* src/widgets/modest-attachment-view.c:
        * Added support for showing purged files as purged. The icon is
          the default document one, and the text is a currently non
          translated string.
        * Now size is only shown for file attachments (if they're not
          purged).
* src/maemo/modest-msg-view-window.c:
        * Use the new Tinymail mime part purge API. It purges really the
          attachments. The UI is a work in progress still, but the
          cache seems to get updated as expected. This implementation
          replaces the old delete_attachments tinymail API based
          implementation.
* src/ui/modest-ui-dimming-rules.c:
        * Added some dimming rules for remove attachments in message view.
          It's not complete yet.
        * Now "move to" operation does not check if the source folder is
          writeable (fixes NB#61860).

pmo-trunk-r2460

Fixes: NB#61843

pmo-trunk-r2458

2007-06-27  Murray Cumming  <murrayc@murrayc.com

* src/modest-mail-operation.c: (update_account_thread),
(get_msgs_full_thread): Fix typo failures in the revert
to fix the build.

pmo-trunk-r2457

2007-06-27  Murray Cumming  <murrayc@murrayc.com

* src/maemo/modest-maemo-global-settings-dialog.c:
(idle_select_default_focus):
* src/modest-mail-operation.c: (idle_notify_progress),
(idle_notify_progress_once), (set_last_updated_idle),
(update_account_thread), (get_msgs_full_thread):
* src/widgets/modest-attachment-view.c: (get_size_idle_func):
* src/widgets/modest-header-view.c:
(idle_notify_headers_count_changed_destroy),
(idle_notify_headers_count_changed):
Reverted the additions of gdk_threads_enter/leave(), in case this
helps with a startup hang on the device.

pmo-trunk-r2456

* Check main window toolbar dimming rules after
open some message (delete toolbar must dimmed).

pmo-trunk-r2455

* Fixes: NB#61846

pmo-trunk-r2454

* Deleted some invalid unreferences, tinymail does take care about the objects passed to callbacks

pmo-trunk-r2453

* Fixes NB#61225 the application no longer hangs due to threading locks
* Modified modest_mail_operation_notify_end, now it receives a second argument that specifies if the signal emission should be blocked or NOT, we need to do this if we're calling it from a different thread
* Removed an invalid double-lock in the mail operations
* Added an empty error handler for the rename operation, no specs for the error message?

pmo-trunk-r2452

* leak fix

pmo-trunk-r2451

* Fixed a memory leak in modest folder view
* Fixes NB#61607, the renaming was not working because we're getting the wrong response from the dialog

pmo-trunk-r2450

* leak fix

pmo-trunk-r2449

2007-06-26  Armin Burgmeier  <armin@openismus.com>

* src/maemo/modest-maemo-utils.c: Replaced a busy wait while waiting
for supported secure authentication methods from server by
gtk_dialog_run(), saving CPU time and a potential race condition.

pmo-trunk-r2448

2007-06-26  Armin Burgmeier  <armin@openismus.com>

* src/maemo/modest-maemo-utils.h:
* src/maemo/modest-maemo-utils.c: Added a
ModestMaemoUtilsGetSupportedSecureAuthenticationError with CANCELED as
only error code to tell the calling code when the user canceled the
operation.

* src/maemo/modest-account-settings-dialog.h:
* src/maemo/modest-account-settings-dialog.c:
* src/maemo/easysetup/modest-easysetup-wizard.c: Do not show an error
message if the retrieval of secure authentication methods was canceled
by the user. Perform the check before saving anything to configuration
to not end up with half-committed configuration changes, do not show
a "Saving to configuration failed" note when authentication methods
could not be retrieved.

* src/maemo/modest-main-window.c: Fixed a crash on startup when built
with old hildon.

pmo-trunk-r2447

* Add some notifications on dimming rules.

pmo-trunk-r2446

* libmodest-dbus-client depends on modest

pmo-trunk-r2445

* fix leak (gtk_tree_row_reference_get_path needs a free)

pmo-trunk-r2444

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* src/dbus_api/modest-dbus-callbacks.c: (add_folders_to_list):
Only return the first level of folders under accounts, because the
osso-global-search UI spec doesn't want the rest.
This fixed projects.maemo.org bug NB#61811.

pmo-trunk-r2443

* Add new dimming rules for copy,cut and paste operations.
* Add dimming rules for undo operations.
* Implement undo operation for copy and cut operations.
* Fixes: NB#59325

pmo-trunk-r2442

* some other apps (sketch) send attachment list that start with ','.
  strip that one, to prevent a warnings later.

pmo-trunk-r2441

* leak fix (modest-init)
* modest-main: sometimes, 'ui' was a not a valid gobject, which caused glib warnings

pmo-trunk-r2440

Bugfix when creating an account

pmo-trunk-r2438

* replace the unnecessary fill_server_account_cache_if_need with
  direct calls to modest_tny_account_store_get_accounts. No need
  to make modest-tny-account-store even uglier.

(as always, murray is right)

pmo-trunk-r2437

* Added a new reference to priv->draft_msg of edit window
to manages its own reference, becasue sometines, this object
was unreference and application crashed when get_msg method of
this window was called.

pmo-trunk-r2436

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* src/modest-tny-account-store.c:
* src/modest-tny-account-store.h:
Added modest_tny_account_is_memory_card_account().
* src/maemo/modest-main-window.c: (create_details_widget):
Show the same information items for the memory card as we show for the
local-folders, to fix projects.maemo.org bug NB#59307.

pmo-trunk-r2435

* do lazy initialization of the server accounts; this avoids
  a chicken-egg problem at startup where modest_runtime_get_account_store would
  be needed before initialized

pmo-trunk-r2434

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* src/modest-tny-account-store.c: (get_password):
Added some checks for NULL to maybe prevent crashes, though I think
it was Philip's last commit that actually prevented the crash here
when creating a first new account.
But the new account is still not usable (no messages are shown)
until a restart of the application.

pmo-trunk-r2433

Bugfix when setting the account to online

pmo-trunk-r2432

* Check headers flags (CACHED) to determine if some message
was already retreived, and sets a different id in get_message
mail operation (OPEN). This new id will be used by progress
bar widget to show a different message.
* Fixes: NB#56201

pmo-trunk-r2431

* expunge deleted messages when switching folders or closing;
  otherwise it's not possible to delete at all...

this should fix NB#61769, NB#61764

pmo-trunk-r2430

Updated the changelog with the fixes, june, 26th, 16:50.

pmo-trunk-r2429

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* src/maemo/modest-maemo-global-settings-dialog.c:
(idle_select_default_focus):
* src/modest-mail-operation.c: (idle_notify_progress),
(idle_notify_progress_once), (idle_notify_update_account_queue),
(set_last_updated_idle), (update_account_thread),
(notify_get_msgs_full), (get_msgs_full_destroyer),
(get_msgs_full_thread):
* src/widgets/modest-attachment-view.c: (get_size_idle_func):
* src/widgets/modest-header-view.c:
(idle_notify_headers_count_changed_destroy),
(idle_notify_headers_count_changed):
Use gdk_threads_enter() and gdk_threads_enter_leave() in
all idle handlers, because some other thread could be blocking
the GDK lock, so that it can do GUI stuff.

pmo-trunk-r2428

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* src/dbus_api/modest-dbus-callbacks.c: (on_idle_mail_to),
(on_idle_compose_mail), (on_compose_mail), (on_idle_open_message),
(on_idle_send_receive), (on_idle_open_default_inbox):
* src/maemo/modest-msg-edit-window.c:
(modest_msg_edit_window_attach_file_noninteractive):
Use gdk_threads_enter() and gdk_threads_leave() in all the idle handlers,
to prevent a hang. I don't quite understand why these are necessary,
beause we should already be in the main loop. I guess something outside
of the main loop is taking the lock.

* tests/dbus_api/test_compose_mail.c: (main): Use file:// prefixes so
we are really using URIs, and use files that are more likely to exist.

pmo-trunk-r2427

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* tests/dbus_api/Makefile.am: Fix the build of the tests. This was
        necessary because configure.ac now defines different variables.

pmo-trunk-r2426

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* libmodest-dbus-client/libmodest-dbus-client.c:
        * libmodest-dbus-client/libmodest-dbus-client.h:
        Removed libmodest_dbus_client_send_mail() because it is not used by anything.
        libmodest_dbus_client_compose_mail() is what things actually use.

        * src/dbus_api/modest-dbus-api.h:
        * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_handler):
        * tests/dbus_api/Makefile.am:
        * tests/dbus_api/test_send_mail.c: Removed the MODEST_DBUS_METHOD_SEND_MAIL
        D-Bus method.

pmo-trunk-r2424

* Added an extra check that was causing a crash when the folder returns no stats

pmo-trunk-r2423

* src/maemo/modest-msg-edit-window.c:
        * Added gtk_text_iter_forward_search_insensitive to implement
          case insensitive search of strings using the find toolbar.
        * Added proper information banners for no matches, search
          complete and empty find.
        * Rework the find in page, to work similar to the upcoming
          find in page new implementation of viewer.

pmo-trunk-r2422

Changed the configure.ac file. corrected the helplib variable used instead of mimelib.

pmo-trunk-r2421

2007-06-26  Murray Cumming  <murrayc@murrayc.com>

* src/modest-init.c: (modest_init_local_folders):
        For memory-card (or other non local-folders) maildir accounts,
        always create an Archive folder, so that there is somewhere to put
        messages. This should be less confusing, because the user does not need
        to guess that the need to create a folder.

pmo-trunk-r2420

* Add a new check of new folder name string length in
'entry-changed' signal handler.
* Fixes: NB#61576

pmo-trunk-r2419

Compilation error

pmo-trunk-r2418

* call osso_abook_init before doing anything with the addressbook.
  this fixes NB#57415

pmo-trunk-r2417

Bugfix for 61408

pmo-trunk-r2416

* Added an attribute sent to the msg edit window that specifies that the mail has scheduled to be sent
* Modified the window destruction code in the window manager. Now checks that the message has not been sent before trying to save it to drafsts
* Some code reorganization in the window manager

pmo-trunk-r2415

* Added new dimming rules for Attachment menu option in
viewer window.
* Fixes: NB#61148

pmo-trunk-r2414

2007-06-25  Johannes Schmid <johannes.schmid@openismus.com>

* src/maemo/easysetup/modest-wizard-dialog.c: (init):
Fixed icon size by using (50, 50) which is the same as defined by
newer hildon-libs

pmo-trunk-r2413

* sync the folders when when we're moving to another one.
  this fixes NB#57773

pmo-trunk-r2412