37d472663a37bc2e622212a395de554f217c144d
[modest] / ChangeLog2
1 2007-07-11  Murray Cumming  <murrayc@murrayc.com>
2
3         * src/dbus_api/modest-dbus-api.h: Added a define for a 
4         top-application D-Bus method.
5         * src/dbus_api/modest-dbus-callbacks.c: 
6         (on_top_application),
7         (modest_dbus_req_handler): Handle the top-application D-Bus 
8         method, which might be an undocumented D-Bus signal sent by 
9         hildon-desktop when it starts the application from the menu.
10         But I am not sure and I need to test it on the device, because 
11         my scratchbox does not have the application in the menu now.
12         This method shows the main window, so that it can be invisible 
13         when started, for instance, just for email searching from 
14         osso-global-search.
15
16         * src/maemo/modest-main-window.c: (restore_settings),
17         (modest_main_window_new): Do not show the window immediately 
18         upon creation. We show it later.
19         * src/modest-widget-memory.c: (save_settings_paned):
20         Try (unsuccessfully) to avoid saving the paned position when 
21         it is not even visible. 
22         (restore_settings_paned): Do not allow a silly paned position.
23         This seems necessary when not showing the window at first.
24
25         * tests/dbus_api/Makefile.am:
26         * tests/dbus_api/test_top_application.c: (main): Test for the 
27         new D-Bus method.
28
29 2007-07-10  Murray Cumming  <murrayc@murrayc.com>
30
31         * src/modest-ui-dimming-rules.c:
32         (modest_ui_dimming_rules_on_new_msg),
33         (modest_ui_dimming_rules_on_reply_msg),
34         (modest_ui_dimming_rules_on_delete_msg),
35         (modest_ui_dimming_rules_on_details),
36         (modest_ui_dimming_rules_on_view_window_move_to),
37         (modest_ui_dimming_rules_on_view_previous),
38         (modest_ui_dimming_rules_on_view_next),
39         (_invalid_folder_for_purge): Added checks before 
40         casting the window. This looks like an actual 
41         logic error in _invalid_folder_for_purge(), because it 
42         tries to use a ModestMainWindow as a message window.
43
44 2007-07-10  Murray Cumming  <murrayc@murrayc.com>
45
46         * src/maemo/modest-account-settings-dialog.c:
47         (show_error): Use the same implementation as in 
48         modest-easysetup-wizard.c:
49         (on_entry_invalid_character): The logical ID contains a %s. 
50         Guessed what it should be.
51         (on_entry_max), (check_data), (on_response): Adapt to 
52         changed show_error().
53         (create_page_account_details): Really call 
54         modest_validating_entry_set_func() on the correct widgets.
55         * src/maemo/easysetup/modest-easysetup-wizard.c:
56         (on_entry_invalid_character): Fix the logical ID here too.
57  (create_page_account_details),
58         (create_page_user_details):
59         Really call 
60         modest_validating_entry_set_func() on the correct widgets.
61
62         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
63         Do not try to use the account if it is NULL.
64
65 2007-07-10  Johannes Schmid <johannes.schmid@openismus.com>
66
67         * src/modest-ui-actions.c: (create_move_to_dialog),
68         (modest_ui_actions_on_main_window_move_to),
69         (modest_ui_actions_on_msg_view_window_move_to):
70         * src/widgets/modest-folder-view.c: (modest_folder_view_init),
71         (filter_row), (modest_folder_view_show_non_move_folders):
72         * src/widgets/modest-folder-view.h:
73
74         Hide Sent & Outbox folder in "move to"-dialog. Does not work for
75         the Sent folder yet as it does not report the correct folder type.
76
77 2007-07-10  Johannes Schmid <johannes.schmid@openismus.com>
78
79         * src/maemo/modest-msg-edit-window.c: (init_window):
80         Fixed bora build
81
82 2007-07-10   Murray Cumming  <murrayc@murrayc.com>
83
84         * src/maemo/easysetup/modest-easysetup-wizard.c: (show_error):
85         Take a parent widget instead of a parent window, and 
86         implement this with hildon_banner_show_information() instead.
87         Non-Maemo builds can just ifdef that there.
88         This fixed projects.maemo.org bug NB#62766.
89
90         (on_entry_invalid_character): Use show_error() again instead 
91         of using hildon directly, and use the _CS() macro to use the correct 
92         translation domain.
93         (create_account), (check_for_supported_auth_methods), (on_entry_max):
94         Pass a widget instead od a window to show_error().
95
96 2007-07-10  Murray Cumming  <murrayc@murrayc.com>
97
98         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
99         Adapt to the changed tinymail API, using the new TnyAccount* parameter.
100         This allows us to show the server name in the error message, fixing 
101         projects.maemo.org bug NB#62904.
102         Also, use the appropriate logical ID message for the account protocol.
103
104 2007-07-10  Murray Cumming  <murrayc@murrayc.com>
105
106         * src/modest-tny-account-store.c: (on_idle_wrong_password):
107         A compilation fix, though the build wasn't broken for me.
108
109 2007-07-09  Murray Cumming  <murrayc@murrayc.com>
110
111         * src/modest-ui-actions.c: (modest_ui_actions_on_send_receive):
112         Do not try to unref a NULL folder store, to avoid a runtime warning.
113         This seems to happen sometimes when changing accounts settings.
114
115 2007-07-09  Murray Cumming  <murrayc@murrayc.com>
116
117         * src/modest-tny-account-store.c:
118         (modest_tny_account_store_instance_init),
119         (modest_tny_account_store_finalize),
120         (on_account_settings_hide), (on_idle_wrong_password):
121         Use a GHashTable to ensure that we only show one account settings 
122         dialog per modest account. This avoids us having a dialog open for 
123         both the store and transport accounts, or due to repeated get_password() 
124         calls that shouldn't happen anyway.
125
126 2007-07-09  Murray Cumming  <murrayc@murrayc.com>
127
128         * src/maemo/modest-maemo-utils.c: (on_hide),
129         (modest_maemo_show_dialog_and_forget): Handle the response, because 
130         the window is not even hidden by default.
131
132         * src/maemo/modest-account-view-window.c: (on_edit_button_clicked):
133         * src/modest-ui-actions.c: (modest_ui_actions_on_accounts):
134         Use modest_maemo_show_dialog_and_forget() instead of gtk_dialog_run() 
135         which seems to prevent some modality problems.
136
137         * src/maemo/modest-main-window.c: (on_sendqueue_error_happened):
138         Ignore user cancellation errors.
139
140         * src/modest-account-mgr-helpers.h:
141         * src/modest-account-mgr-helpers.c:
142         Added modest_server_account_get_password() and 
143         modest_server_account_get_has_password() to avoid direct use of 
144         conf enums.
145
146         * src/maemo/modest-account-settings-dialog.h:
147         * src/maemo/modest-account-settings-dialog.c:
148         (modest_account_settings_dialog_init): Store the notebook so we can 
149         use it later.
150         Added modest_account_settings_dialog_switch_to_user_info().
151
152         * src/modest-tny-account-store.c: 
153         (get_password): When the password in the account settings is wrong (we think), 
154         show the relevant page of the account settings dialog, in the mainloop.
155         However, we currently get many of these at once because cancel does not 
156         seem to cancel.
157         (forget_password): Do not forget the password 
158         from the account settings, because this causes side-effects when 
159         tinymail seems to call it at strange time, maybe because of how we 
160         are cancelling get_password().
161         (modest_tny_account_store_alert): Ignore user cancellations.
162
163 2007-07-09  Murray Cumming  <murrayc@murrayc.com>
164
165         * src/maemo/modest-maemo-utils.h:
166         * src/maemo/modest-maemo-utils.c: 
167         Added modest_maemo_show_information_note_and_forget() for use instead of 
168         gtk_dialog_run() when the response is not needed, to avoid an extra 
169         mainloop which could cause a hang.
170
171         * src/maemo/modest-main-window.c: (on_sendqueue_error_happened):
172         Use the logical IDs for the error message, though they are not 
173         specific enough. Note that I used modest_maemo_show_information_note_and_forget() 
174         because gtk_dialog_run() caused a hang here (the window did not close).
175         This fixes projects.maemo.org bug NB#62646.
176
177 2007-07-09  Murray Cumming  <murrayc@murrayc.com>
178
179         * src/modest-account-mgr-helpers.h:
180         * src/modest-account-mgr-helpers.c:
181         Added modest_account_mgr_get_store_protocol().
182
183         * src/maemo/modest-account-view-window.c:
184         (on_delete_button_clicked): Show different warning text depending on 
185         the protocol (IMAP or POP) as per the UI spec. Use a HildonNote instead 
186         of a GtkDialog, which fixes the truncation, fixing projects.maemo.org bug 
187         NB#59037.
188
189 2007-07-09  Murray Cumming  <murrayc@murrayc.com>
190
191         * src/maemo/modest-msg-edit-window.c:
192         (modest_msg_edit_window_attach_file): Set the file chooser dialog title 
193         as per the UI spec. This fixes proejcts.maemo.org bug NB#59056.
194
195 2007-07-07  Armin Burgmeier  <armin@openismus.com>
196
197         * src/maemo/easysetup/modest-easysetup-wizard.c: Translate
198         ckdg_ib_maximum_characters_reached from correct package.
199
200         * src/modest-ui-actions.c: Show an error message when a folder could
201         not be created. Tinymail does not give us specific error codes about
202         what has gone wrong, so we assume that a folder with that name
203         already exists, though there might be other reasons.
204
205 2007-07-06  Murray Cumming <murrayc@murrayc.com>
206
207         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
208         Use the new TNY_ACCOUNT_ERROR_TRY_CONNECT_CERTIFICATE error code to 
209         give a more exact error message that can be partly translated.
210
211 2007-07-05  Murray Cumming <murrayc@murrayc.com>
212
213         * src/maemo/modest-address-book.c:
214         (select_email_addrs_for_contact): Specify a default window size for the 
215         select-email-address dialog, so that it is probably big enough for the 
216         title. This should fix projects.maemo.org bug NB#62277.
217
218 2007-07-05  Murray Cumming <murrayc@murrayc.com>
219
220         * src/maemo/easysetup/modest-easysetup-wizard.c:
221         (create_page_complete_easysetup): Use 
222         gtk_widget_set_size() request to ensure that the text wraps 
223         even in osso 1.1.
224
225 2007-07-05  Johannes Schmid <johannes.schmid@openismus.com>
226
227         * src/maemo/easysetup/modest-easysetup-wizard.c:
228         (on_entry_invalid_character), (create_page_account_details),
229         (create_page_user_details):
230         Use new API of ModestValidatingEntry. But for some reason the banner is
231         not shown. I will have to investigate that further.
232
233         * src/widgets/modest-validating-entry.c: (on_insert_text),
234         (modest_validating_entry_set_func):
235         * src/widgets/modest-validating-entry.h:
236         Added a new callback function to be able to react to prevented 
237         characters whitespaces to e.g show a banner.
238
239 2007-07-05  Armin Burgmeier  <armin@openismus.com>
240
241         * src/maemo/modest-account-settings-dialog.c: Use the notify::value
242         notification of HildonNumberEditor instead of remembering the port
243         number. Thanks to Johannes for pointing this out.
244
245 2007-07-05  Johannes Schmid <johannes.schmid@openismus.com>
246
247         * src/maemo/easysetup/modest-easysetup-wizard.c:
248         Check if there are any user changes before showing a confirmation
249         dialog (projects.maemo.org NB#60946)
250
251 2007-07-05  Armin Burgmeier  <armin@openismus.com>
252
253         * src/maemo/modest-account-settings-dialog.c: Remember custom port
254         number at the beginning of the dialog and set the modified flag at the
255         end if the port number has changed. This is necessary because
256         HildonNumberEditor does not support notification when the value has
257         changed.
258
259 2007-07-05  Murray Cumming <murrayc@murrayc.com>
260
261         * src/modest-ui-actions.c:
262         (modest_ui_actions_on_main_window_move_to): Check whether the source folder needs 
263         a connection before asking anything else. Correct the check for the destination 
264         folder so that it succeeds when online.
265         This fixed projects.maemo.org bug NB#61645.
266
267 2007-07-05  Murray Cumming <murrayc@murrayc.com>
268
269         * src/modest-ui-actions.c: (modest_ui_actions_on_send):
270         Call modest_platform_connect_and_wait() to ask for a connection if necessary, 
271         to fix projects.maemo.org bug NB#62439.
272
273 2007-07-05  Murray Cumming <murrayc@murrayc.com>
274
275         * src/modest-tny-account.c: (on_connection_status_changed):
276         Really do not use tny_device_force_offline() here. See the previous comment.
277
278 2007-07-04  Murray Cumming <murrayc@murrayc.com>
279
280         * src/modest-tny-account.c: (on_connection_status_changed):
281         Do not use tny_device_force_offline() when tinymail says that something 
282         went wrong with the connection. It seems to report this unnecessarily.
283
284 2007-07-04  Murray Cumming <murrayc@murrayc.com>
285
286         * src/maemo/modest-connection-specific-smtp-window.c:
287         (modest_connection_specific_smtp_window_init):
288         Use gtk_widget_set_size_request(), because gtk_label_set_max_width_chars()
289         does not seem to have any effect in osso 1.1 (though it works in 
290         Maemo Bora). This fixes bug projects.maemo.org NB#59212.
291
292 2007-07-04  Johannes Schmid <johannes.schmid@openismus.com>
293
294         * src/maemo/modest-connection-specific-smtp-window.c:
295         Fix UI specification problems with projects.meamo.org NB#60479
296
297 2007-07-04   Murray Cumming <murrayc@murrayc.com>
298
299         * src/modest-ui-actions.c: (open_msg_cb): Add a printf() to provide a 
300         clue when this has failed, because the error dialog is not always shown 
301         (at least in earlier version).
302         
303         (modest_ui_actions_get_msgs_full_error_handler): prinf() the actual 
304         GError message, because the generic error dialog is not helpful.
305         I am getting
306         'This message is not currently available (can't let a new connection go online)'
307         when trying to open an IMAP message.
308         
309         (_modest_ui_actions_open): Offer a connection when some messages would 
310         have to be downloaded and we are offline.
311
312 2007-07-04  Johannes Schmid <johannes.schmid@openismus.com>
313
314         * src/widgets/modest-folder-view.c:
315         Don't check for MERGE type anymore as we use the correct type
316         now.
317         NOTE: For the last two changes you need to update your tinymail copy
318         because of API changes!
319
320 2007-07-04  Johannes Schmid <johannes.schmid@openismus.com>
321
322         * src/modest-tny-local-folders-account.c:
323         Set the folder type for the TnyMergeFolder to TNY_MERGE_FOLDER_OUTBOX
324         (see projects.maemo.org NB#61580)
325
326 2007-07-04  Murray Cumming <murrayc@murrayc.com>
327
328         * src/modest-tny-account.c: (on_connection_status_changed):
329         Commented out the call to modest_platform_connect_and_wait() because this 
330         seems to always hang, and I would like to know if this is the only thing 
331         that causes a hang when showing the connection dialog.
332         It is not very useful anyway.
333
334 2007-07-04  Murray Cumming <murrayc@murrayc.com>
335
336         * src/maemo/modest-msg-view-window.c:
337         (modest_msg_view_window_select_next_message):
338         
339         * src/modest-platform.h:
340         * src/maemo/modest-platform.c: 
341         Added modest_platform_connect_and_wait(), which calls 
342         tny_maemo_conic_device_connect() always in the main thread, by calling it 
343         in an idle handler and waiting for the idle handler to run. It should be 
344         in the main thread because it uses GTK+ UI code. It just returns TRUE 
345         if the device is already online.
346         
347         There is some nasty code to ensure that subsequent calls to 
348         modest_platform_connect_and_wait() just wait for the first one to finish, 
349         by checking repeatedly in another idle handler. This is necessary, 
350         because other threads (or even the main thread, via another idle handler) 
351         can call modest_platform_connect_and_wait() while 
352         modest_platform_connect_and_wait() is running.
353         
354         (modest_platform_connect_and_wait_if_network_account),
355         (modest_platform_connect_and_wait_if_network_folderstore):
356         Convenience functions that calls modest_platform_connect_and_wait() if 
357         the account or folder might want network access.
358         
359         * src/modest-mail-operation.c:
360         (modest_mail_operation_update_account):
361         * src/modest-ui-actions.c: (download_uncached_messages),
362         (modest_ui_actions_on_rename_folder), (delete_folder),
363         (modest_ui_actions_on_main_window_move_to),
364         (modest_ui_actions_on_msg_view_window_move_to):
365         * src/widgets/modest-folder-view.c:
366         (drag_and_drop_from_folder_view):
367         Offer a connection if we are offline.
368         
369         * src/modest-tny-account.c: (on_connection_status_changed): 
370         Document this as only handling errors during network operations in progress,
371         and replace the idle code with a simple call to 
372         modest_platform_connect_and_wait(), which does this instead.
373
374 2007-07-03  Murray Cumming <murrayc@murrayc.com>
375
376         * src/modest-tny-account.c:
377         (modest_tny_account_get_special_folder): (util_on_idle),
378         Handle the GError so we can see when tny_folder_store_get_folders() 
379         fails, as it currently does while something else is connecting.
380         
381         Added util_run_in_main_thread_and_wait() 
382         which uses an idlen handler and mainloop to ensure that the function 
383         is called in the main thread.
384         (on_connection_status_changed): Use util_run_in_main_thread_and_wait() 
385         to make sure that we call attempt the connection in the mainloop only, 
386         which might solve some hangs.
387         
388 2007-07-03  Murray Cumming <murrayc@murrayc.com>
389
390         (modest_tny_account_store_get_tny_account_by): Do not allow accounts to 
391         be found by their display name. This is never useful.
392         * src/modest-tny-account-store.c: (on_account_removed): Find the account 
393         by its ID, instead of trying to use its ID as its display name.
394         
395         * src/modest-tny-account-store.h:
396         * src/modest-tny-account.c:
397         (modest_tny_account_get_special_folder): Add a g_warning() when 
398         tny_folder_store_get_folders() returns an empty list for the maildir.
399         
400         * src/modest-tny-send-queue.c: (modest_tny_send_queue_get_outbox):
401         Add a g_warning() when this returns NULL, because that it not allowed 
402         by tinymail.
403
404 2007-07-02  Armin Burgmeier  <armin@openismus.com>
405
406         * src/maemo/easysetup/modest-easysetup-wizard.c: Added a confirmation
407         note when trying to cancel the wizard. This fixes projects.maemo.org
408         bug NB#60946.
409
410 2007-07-02  Murray Cumming  <murrayc@murrayc.com>
411
412         * src/modest-tny-account.h:
413         * src/modest-tny-account.c:
414         (modest_tny_account_new_from_server_account),
415         (modest_tny_account_new_from_server_account_name),
416         (modest_tny_account_new_from_account): Added a TnyCamelSession parameter, 
417         because this must now be set before setting the proto. This is usd in 
418         modest_tny_account_new_from_server_account(). It avoids a (maybe harmless) 
419         warning in camel_session_get_service() caused by 
420         tny_camel_transport_account_prepare().
421         
422         * src/modest-tny-account-store.c: (get_server_accounts): Pass the new 
423         session parameter.
424
425 2007-06-30  Johannes Schmid <johannes.schmid@openismus.com>
426
427         * src/maemo/modest-connection-specific-smtp-edit-window.c:
428         (on_value_changed),
429         (modest_connection_specific_smtp_edit_window_init):
430         
431         Fixed project.maemo.org bug NB#60479 by using "notify::value" instead 
432         of "changed" for now
433
434 2007-06-29  Murray Cumming  <murrayc@murrayc.com>
435
436         * src/modest-tny-account.c: 
437         (modest_tny_account_new_from_server_account):
438         Handle the TnyAccount::"connection-status-changed" signal, which tinymail 
439         now uses to request a connection when it needs one, and which it uses 
440         to tell us that something is wrong with the connection, so we can force 
441         the accounts offline to prevent further problems.
442         
443         * src/modest-defs.h:
444         * src/modest-init.c: (init_default_settings):
445         * src/modest-mail-operation.c:
446         (modest_mail_operation_update_account):
447         * src/modest-main.c: (on_show): Do not try to connect explicitly on startup.
448         This should not be necessary anymore. Do not set the conf key that 
449         determined this (it was always TRUE and not set anywhere else).
450         
451         * src/modest-ui-actions.h: Remove unused and unimplemented 
452         modest_ui_actions_on_online_toggle_toggled().
453
454 2007-06-29  Murray Cumming  <murrayc@murrayc.com
455
456         * src/maemo/modest-main-window.c: (connect_signals):
457         Don't connect to delete_event twice. Maybe this was harmless.
458
459 2007-06-29  Murray Cumming  <murrayc@murrayc.com
460
461         * libmodest-dbus-client/libmodest-dbus-client.c: Improved the documentation slightly.
462         * src/modest-mail-operation.c: (modest_mail_operation_xfer_folder):
463         Debug output to give some more clue when a folder move is not allowed.
464
465 2007-06-28  Murray Cumming  <murrayc@murrayc.com
466
467         * src/modest-tny-account.c: (on_modest_file_system_info),
468         (modest_tny_account_get_mmc_account_name):
469         Always call the callback, without checking whether the name has changed.
470         * src/widgets/modest-folder-view.c: (strings_are_equal),
471         (on_model_foreach_set_name), (on_get_mmc_account_name),
472         (text_cell_data): Compare the name here, and use the name from 
473         the account rather than from the 
474         TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN because that will be wrong 
475         if the name was changed after the account was added to the model.
476         That's a tinymail bug but not one that I want to fix now.
477
478         This finally fixes projects.maemo.org bug NB#59301.
479
480 2007-06-27  Armin Burgmeier  <armin@openismus.com>
481
482         * src/modest-mail-operation.c: Fixed a crash when canceling a mail
483         receive operation.
484
485 2007-06-27  Murray Cumming  <murrayc@murrayc.com
486
487         * src/modest-tny-account.h:
488         * src/modest-tny-account.c: 
489         Added modest_tny_account_get_mmc_account_name(), to set the memory 
490         card name asynchronously for the given account and then notify.
491         (modest_tny_account_new_for_local_folders): Use this new function instead 
492         of using hildon_file_system_info_async_new() directly.
493         
494         * src/widgets/modest-folder-view.c: (text_cell_data):
495         Use modest_tny_account_get_mmc_account_name() to get the real name, calling 
496         gtk_tree_model_row_changed() when it is available.
497         
498         This should fix projects.maemo.org bug NB#59301.
499
500 2007-06-27  Murray Cumming  <murrayc@murrayc.com
501
502         * src/modest-mail-operation.c: (update_account_thread),
503         (get_msgs_full_thread): Fix typo failures in the revert
504         to fix the build.
505
506 2007-06-27  Murray Cumming  <murrayc@murrayc.com
507
508         * src/maemo/modest-maemo-global-settings-dialog.c:
509         (idle_select_default_focus):
510         * src/modest-mail-operation.c: (idle_notify_progress),
511         (idle_notify_progress_once), (set_last_updated_idle),
512         (update_account_thread), (get_msgs_full_thread):
513         * src/widgets/modest-attachment-view.c: (get_size_idle_func):
514         * src/widgets/modest-header-view.c:
515         (idle_notify_headers_count_changed_destroy),
516         (idle_notify_headers_count_changed):
517         Reverted the additions of gdk_threads_enter/leave(), in case this 
518         helps with a startup hang on the device.
519
520 2007-06-26  Armin Burgmeier  <armin@openismus.com>
521
522         * src/maemo/modest-maemo-utils.c: Replaced a busy wait while waiting
523         for supported secure authentication methods from server by
524         gtk_dialog_run(), saving CPU time and a potential race condition.
525
526 2007-06-26  Armin Burgmeier  <armin@openismus.com>
527
528         * src/maemo/modest-maemo-utils.h: 
529         * src/maemo/modest-maemo-utils.c: Added a
530         ModestMaemoUtilsGetSupportedSecureAuthenticationError with CANCELED as
531         only error code to tell the calling code when the user canceled the
532         operation.
533
534         * src/maemo/modest-account-settings-dialog.h:
535         * src/maemo/modest-account-settings-dialog.c:
536         * src/maemo/easysetup/modest-easysetup-wizard.c: Do not show an error
537         message if the retrieval of secure authentication methods was canceled
538         by the user. Perform the check before saving anything to configuration
539         to not end up with half-committed configuration changes, do not show
540         a "Saving to configuration failed" note when authentication methods
541         could not be retrieved.
542
543         * src/maemo/modest-main-window.c: Fixed a crash on startup when built
544         with old hildon.
545
546 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
547
548         * src/dbus_api/modest-dbus-callbacks.c: (add_folders_to_list):
549         Only return the first level of folders under accounts, because the 
550         osso-global-search UI spec doesn't want the rest.
551         This fixed projects.maemo.org bug NB#61811.
552
553 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
554
555         * src/modest-tny-account-store.c:
556         * src/modest-tny-account-store.h:
557         Added modest_tny_account_is_memory_card_account().
558         * src/maemo/modest-main-window.c: (create_details_widget):
559         Show the same information items for the memory card as we show for the 
560         local-folders, to fix projects.maemo.org bug NB#59307.
561
562 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
563
564         * src/modest-tny-account-store.c: (get_password):
565         Added some checks for NULL to maybe prevent crashes, though I think 
566         it was Philip's last commit that actually prevented the crash here 
567         when creating a first new account.
568         But the new account is still not usable (no messages are shown) 
569         until a restart of the application.
570
571 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
572
573         * src/maemo/modest-maemo-global-settings-dialog.c:
574         (idle_select_default_focus):
575         * src/modest-mail-operation.c: (idle_notify_progress),
576         (idle_notify_progress_once), (idle_notify_update_account_queue),
577         (set_last_updated_idle), (update_account_thread),
578         (notify_get_msgs_full), (get_msgs_full_destroyer),
579         (get_msgs_full_thread):
580         * src/widgets/modest-attachment-view.c: (get_size_idle_func):
581         * src/widgets/modest-header-view.c:
582         (idle_notify_headers_count_changed_destroy),
583         (idle_notify_headers_count_changed):
584         Use gdk_threads_enter() and gdk_threads_enter_leave() in 
585         all idle handlers, because some other thread could be blocking 
586         the GDK lock, so that it can do GUI stuff.
587
588 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
589
590         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_mail_to),
591         (on_idle_compose_mail), (on_compose_mail), (on_idle_open_message),
592         (on_idle_send_receive), (on_idle_open_default_inbox):
593         * src/maemo/modest-msg-edit-window.c:
594         (modest_msg_edit_window_attach_file_noninteractive):
595         Use gdk_threads_enter() and gdk_threads_leave() in all the idle handlers, 
596         to prevent a hang. I don't quite understand why these are necessary, 
597         beause we should already be in the main loop. I guess something outside 
598         of the main loop is taking the lock.
599         
600         * tests/dbus_api/test_compose_mail.c: (main): Use file:// prefixes so 
601         we are really using URIs, and use files that are more likely to exist.
602
603 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
604
605         * tests/dbus_api/Makefile.am: Fix the build of the tests. This was 
606         necessary because configure.ac now defines different variables.
607
608 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
609
610         * libmodest-dbus-client/libmodest-dbus-client.c:
611         * libmodest-dbus-client/libmodest-dbus-client.h:
612         Removed libmodest_dbus_client_send_mail() because it is not used by anything.
613         libmodest_dbus_client_compose_mail() is what things actually use.
614         
615         * src/dbus_api/modest-dbus-api.h:
616         * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_handler):
617         * tests/dbus_api/Makefile.am:
618         * tests/dbus_api/test_send_mail.c: Removed the MODEST_DBUS_METHOD_SEND_MAIL
619         D-Bus method.
620
621 2007-06-26  Murray Cumming  <murrayc@murrayc.com>
622
623         * src/modest-init.c: (modest_init_local_folders):
624         For memory-card (or other non local-folders) maildir accounts, 
625         always create an Archive folder, so that there is somewhere to put 
626         messages. This should be less confusing, because the user does not need 
627         to guess that the need to create a folder.
628
629 2007-06-25  Johannes Schmid <johannes.schmid@openismus.com>
630
631         * src/maemo/easysetup/modest-wizard-dialog.c: (init):
632         Fixed icon size by using (50, 50) which is the same as defined by
633         newer hildon-libs
634
635 2007-06-25  Armin Burgmeier  <armin@openismus.com>
636
637         * src/widgets/modest-window-mgr.c:
638         * src/modest-init.c:
639         * configure.ac: Correctly recognize hildon-notify presence and fixed
640         the build without it being installed.
641
642 2007-06-25  Murray Cumming  <murrayc@murrayc.com>
643
644         * libmodest-dbus-client/libmodest-dbus-client.c:
645         (libmodest_dbus_client_search): Fix the build. Sorry.
646         * src/modest-ui-actions.c: (modest_ui_actions_on_new_msg),
647         (reply_forward_cb): Initialize variables to fix the build.
648         
649         * src/modest-tny-account-store.c:
650         (modest_tny_account_store_get_tny_account_by): Fix the account URI 
651         comparison (a silly typo) so that email search results can be opened 
652         when there are more than one account.
653
654 2007-06-25  Murray Cumming  <murrayc@murrayc.com>
655
656         * libmodest-dbus-client/libmodest-dbus-client.c:
657         (libmodest_dbus_client_search),
658         (libmodest_dbus_client_get_folders): Use large timeouts (2 minutes) 
659         instead of 1 second, because the search gets folders and messages from 
660         the server, which can take a long time. When it times out, 
661         osso-global-search will report no results.
662         
663         We need to specify whether we should search only cached information 
664         (maybe with a user preference), and the osso-email-search-plugin
665         API should really be improved to allow that option, and to allow results 
666         to be delivered asynchronously in batches.
667
668 2007-06-25  Murray Cumming  <murrayc@murrayc.com>
669
670         * src/maemo/easysetup/modest-wizard-dialog.c: (init):
671         Fix a typo to fix the osso 1.1. build. See the TODO comment though -
672         GTK_ICON_SIZE_DIALOG is probably the wrong size.
673
674 2007-06-25  Murray Cumming  <murrayc@murrayc.com>
675
676         * src/modest-search.c: (modest_search_folder): Ignore a seach folder 
677         that is an empty string. D-Bus seems to give us these instead of NULL.
678
679 2007-06-25  Johannes Schmid <johannes.schmid@openismus.com>
680
681         * src/dbus_api/modest-dbus-callbacks.c:
682         * src/maemo/easysetup/modest-easysetup-wizard.h:
683         * src/maemo/easysetup/modest-wizard-dialog.c: (init):
684         * src/maemo/modest-connection-specific-smtp-edit-window.c:
685         (modest_connection_specific_smtp_edit_window_init):
686         * src/maemo/modest-maemo-global-settings-dialog.c:
687         (on_range_error):
688         * src/maemo/modest-maemo-utils.c:
689         (modest_maemo_utils_setup_images_filechooser):
690         * src/maemo/modest-main-window.c: (create_details_widget):
691         * src/maemo/modest-msg-edit-window.c:
692         (modest_msg_edit_window_attach_file_noninteractive),
693         (modest_msg_edit_window_color_button_change),
694         (modest_msg_edit_window_set_zoom):
695         * src/modest-mail-operation.c: (update_account_thread):
696         * src/modest-tny-account-store.c:
697         * src/modest-tny-account.c:
698         * src/widgets/modest-global-settings-dialog.c:
699         * src/widgets/modest-window.h:
700         
701         Fixed build on bora!
702
703 2007-06-25  Murray Cumming  <murrayc@murrayc.com>
704
705         * src/dbus_api/modest-dbus-callbacks.c: (on_dbus_method_search):
706         * src/modest-search.c: (modest_search_folder),
707         (modest_search_all_accounts): Actually search only in the specified folder, 
708         doing a simple match on the folder ID.
709         * src/modest-search.h: Added a folder field to the ModestSearch struct, 
710         and added some documentation.
711
712 2007-06-22  Armin Burgmeier  <armin@openismus.com>
713
714         * src/maemo/modest-maemo-utils.h:
715         * src/maemo/modest-maemo-utils.c: Added a GError parameter to
716         modest_maemo_utils_get_supported_secure_authentication_methods.
717
718         * src/maemo/easysetup/modest-easysetup-wizard.c:
719         * src/maemo/modest-account-settings-dialog.c: Show correct error if
720         one occured, not just "Server does not support secure authentication".
721
722 2007-06-22  Murray Cumming  <murrayc@murrayc.com>
723
724         * src/modest-tny-account-store.c: (get_password):
725         Restored the call to tny_account_set_user() because Philip seems to have 
726         stopped this from crashing (by not using the freed string). It still does 
727         not work, but now we can test it.
728
729 2007-06-22  Murray Cumming  <murrayc@murrayc.com>
730
731         * src/maemo/modest-msg-edit-window.c: (init_window):
732         Comment out use of an uninitialzed GtkScrolledWindow variable, 
733         to fix the build and stop a crash. Yet again.
734
735 2007-06-22  Murray Cumming  <murrayc@murrayc.com>
736
737         * src/modest-tny-account-store.c: (get_password):
738         Remove the call to tny_account_set_user() because this actually crashes.
739         As mentioned in the last commit, it is known to not work anyway.
740
741 2007-06-22  Murray Cumming  <murrayc@murrayc.com>
742
743         * src/modest-tny-account-store.c: (get_password):
744         Use tny_account_set_username() to tell tinymail that the username might 
745         have changed too. But tinymail ignores this: see projects.maemo.org 
746         bug #61463.
747         * src/modest-ui-actions.c:
748         (modest_ui_actions_on_password_requested): Remove the useless attempt to 
749         reset the connection when changing the username, and improve the comments.
750         Save the changed username to gconf at least, so it is used when Modest is 
751         restarted.
752
753 2007-06-21  Murray Cumming  <murrayc@murrayc.com>
754
755         * src/dbus_api/modest-dbus-callbacks.c: (add_folders_to_list):
756         Do not use a query, so that we get all folders, including for POP and 
757         local-folders. They work as search criteria too.
758         This fixes projects.maemo.org bug NB#57740.
759
760 2007-06-21  Murray Cumming  <murrayc@murrayc.com>
761
762         * src/maemo/modest-platform.c: (modest_platform_on_new_msg):
763         Comment out the use of hildon_notification_set_sound() because it is 
764         not declared, to fix the build. Warnings are serious, people.
765
766 2007-06-21  Armin Burgmeier  <armin@openismus.com>
767
768         * src/maemo/modest-maemo-utils.h:
769         * src/maemo/modest-maemo-utils.c: Added a username parameter to the
770         modest_maemo_utils_get_supported_secure_authentication_methods()
771         to query authentication methods for a specific user. This should fix
772         projects.maemo.org bug NB#61369.
773
774         * src/maemo/easysetup/modest-easysetup-wizard.c:
775         * src/maemo/modest-account-settings-dialog.c: Adjusted calls to that
776         function.
777
778 2007-06-21  Murray Cumming  <murrayc@murrayc.com>
779
780         * libmodest-dbus-client/libmodest-dbus-client.h:
781         * libmodest-dbus-client/libmodest-dbus-client.c:
782         (modest_dbus_message_iter_get_search_hit),
783         (libmodest_dbus_client_search):
784         Reorganised and commented this.
785         Removed modest_search_hit_free() from the public API because it 
786         is not needed.
787         
788         Added libmodest_dbus_client_get_folders() and 
789         modest_folder_result_list_free().
790         
791         * src/dbus_api/modest-dbus-api.h:
792         * src/dbus_api/modest-dbus-callbacks.c: 
793         (modest_dbus_req_filter): Commented. Moved most code to 
794         on_dbus_method_search() and added on_dbus_method_get_folders().
795
796         * src/maemo/modest-platform.c: (modest_platform_init): Commented the use 
797         of the extra D-Bus handler.
798         
799         * tests/dbus_api/Makefile.am:
800         * tests/dbus_api/test_get_folders.c: New test for 
801         libmodest_dbus_client_get_folders(). This seems to return an empty list 
802         at the moment. That could be the same error that causes the message list 
803         to be empty sometimes.
804         This should help with projects.maemo.org bug NB#57740.
805
806 2007-06-21  Armin Burgmeier  <armin@openismus.com>
807
808         * src/modest-mail-operation.c: Do not call
809         modest_mail_operation_queue_cancel_all() in _cancel() but do instead
810         notify the operation's end after having set the status to CANCELED for
811         the operation queue to listen on. This fixes a deadlock when there
812         were mail operations in the queue during its finalization.
813
814         * src/modest-singletons.c: Reordered uninitialization of the account
815         manager to happen after the mail operation queue was uninitialized
816         because the mail operation's in the queue might still try to access
817         the account manager (see modest_mail_operation_notify_end()). This
818         fixes a potential crash when terminating the application.
819
820 2007-06-21  Murray Cumming  <murrayc@murrayc.com>
821
822         * src/modest-search.c: 
823         (search_mime_part_and_child_parts): Moved some code here so we 
824         can search recursively in child folders.
825         (modest_search_folder): Search in the message as a mime part, instead 
826         of just its child mime parts.
827         (read_chunk), (search_mime_part_ogs):
828         Check how many bytes were read, to prevent an endless loop.
829         Searching in message bodies now works. This finishes the fix for 
830         projects.maemo.org bug NB#61007.
831
832 2007-06-20  Murray Cumming  <murrayc@murrayc.com>
833
834         * src/modest-ui-actions.c: (modest_ui_actions_on_details):
835         Add a g_warning when the item is not a folder.
836         
837         * src/modest-ui-dimming-rules.c:
838         (modest_ui_dimming_rules_on_details): Dim the Email/Details menu item
839         when selected item is not a folder (for instance, an account), because 
840         the UI spec only talks about folders.
841         I am not sure what the UI spec wants, so this might not totally fix 
842         projects.maemo.org bug NB#60794.
843         
844 2007-06-20  Murray Cumming  <murrayc@murrayc.com>
845
846         * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
847         (load_from_file): Sort the list. The unsorted list would be more noticeable 
848         in non-English locales.
849         
850         * src/maemo/modest-msg-edit-window.c:
851         (modest_msg_edit_window_attach_file): Initialize a variable to avoid a
852         warning and fix the build.
853         
854         * src/maemo/modest-progress-bar-widget.c:
855         (modest_progress_bar_widget_set_progress): Avoid a division by zero, to 
856         avoid a g_warning, and probably to avoid a crash.
857
858 2007-06-20  Murray Cumming  <murrayc@murrayc.com>
859
860         * src/modest-search.c: (modest_search_folder): Check for a GError 
861         when calling tny_folder_get_headers(), to show that this is not the cause 
862         of later failure.
863
864 2007-06-20  Murray Cumming  <murrayc@murrayc.com>
865
866         * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_filter):
867         Actually set the body query instead of the subject query.
868         
869         * src/modest-search.c: (search_mime_part_ogs),
870         (search_mime_part_strcmp): Remove extra space from the text/* regex,
871         which would probably cause it to fail if we got this far.
872                 
873         (modest_search_folder): Use simple #ifdefs instead of the function pointer, 
874         to simplify the code.
875         Warn when tny_mime_part_get_parts() returns an empty list, which seems to 
876         be causing search-in-body to fail.
877
878 2007-06-20  Murray Cumming  <murrayc@murrayc.com>
879
880         * libmodest-dbus-client/libmodest-dbus-client.h: Document the folder 
881         struct item.
882         * src/dbus_api/modest-dbus-callbacks.c: (find_message_by_url):
883         Better debug message when the folder cannot be found.
884         This should help with fixing projects.maemo.org bug #61287.
885         * src/modest-search.c: (add_hit): Do not leak the furl string.
886
887 2007-06-20  Murray Cumming  <murrayc@murrayc.com>
888
889         * src/modest-ui-actions.c:
890         (modest_ui_actions_on_folder_selection_changed): 
891         Do not special-case TNY_MERGE_FOLDER here, because 
892         modest_tny_folder_get_account() already does that. This fixes 
893         projects.maemo.org bug NB#60955 (The smtp connections dialog was not 
894         shown when the Outbox was selected).
895
896 2007-06-19  Murray Cumming,,,  <murrayc@murrayc.com>
897
898         * libmodest-dbus-client/libmodest-dbus-client.c:
899         (_dbus_iter_get_string_or_null): Initialize the string variable in 
900         case this helps with Dirk's crash.
901
902 2007-06-19  Murray Cumming  <murrayc@murrayc.com>
903
904         * src/dbus_api/modest-dbus-api.h: Add the attachments enum for send_mail, 
905         though it is not implemented yet (this function is maybe not even used),
906         to avoid breaking ABI in future.
907         * libmodest-dbus-client/libmodest-dbus-client.c:
908         (get_attachments_string): Moved some code into this generic function.
909         (libmodest_dbus_client_send_mail): Corrected the documentation. This really 
910         sends email. It is maybe not used.
911         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_send_mail),
912         (on_send_mail): Use attachments.
913         
914         * src/widgets/modest-msg-edit-window.h: Improve the documentation.
915         * src/maemo/modest-msg-edit-window.c:
916         (modest_msg_edit_window_attach_file_noninteractive): Convert the URI to 
917         a filename before using it, to fix projects.maemo.org bug NB#58233.
918
919 2007-06-19  Murray Cumming  <murrayc@murrayc.com>
920
921         * src/modest-ui-actions.c: (modest_ui_actions_on_copy):
922         Initialize the continue_download variable to fix the build, and probably
923         to fix randome behaviour.
924
925 2007-06-18  Armin Burgmeier  <armin@openismus.com>
926
927         * src/modest-tny-send-queue.h:
928         * src/modest-tny-send-queue.c: Keep track of the status of messages in
929         the queue so we can later show it in the UI. Replaced
930         modest_tny_send_queue_msg_is_being_sent() by
931         modest_tny_send_queue_get_msg_status().
932
933         * src/modest-ui-dimming-rules.c: Adjust for the replaced function.
934
935         * src/modest-tny-account.c: Changed a cast to gpointer* to one to
936         gpointer. This probably was not harmful, but just to be sure.
937
938 2007-06-18  Murray Cumming  <murrayc@murrayc.com>
939
940         * src/maemo/modest-platform.c: (modest_platform_activate_uri):
941         Fix the build, but leave the function broken on Bora, because
942         hildon_uri_action_get_service() does not exist. 
943         osso_uri_action_get_service() does not exist either, it seems.
944
945 2007-06-18  Murray Cumming  <murrayc@murrayc.com>
946
947         * src/modest-tny-account.c:
948         (modest_tny_account_new_from_server_account):
949         Uncommented the url debug output again, because it is useful for 
950         debugging, as I said in the comment.
951
952 2007-06-18  Murray Cumming  <murrayc@murrayc.com>
953
954         * configure.ac: Do not require hildon-notify for Maemo Bora. It is not 
955         available.
956         * src/modest-init.c: (modest_init_init_ui): Do not call notify_init() 
957         for Mameo Bora, to fix the build.
958         
959 2007-06-17  Johannes Schmid <johannes.schmid@openismus.com>
960
961         * configure.ac:
962         Fixed build with hildon < 0.99
963         
964         * src/maemo/modest-account-view-window.c: (on_edit_button_clicked):
965         Check if the selected account is busy instead of checking all accounts
966         
967         * src/modest-account-mgr-priv.h:
968         * src/modest-account-mgr.c: (modest_account_mgr_class_init),
969         (modest_account_mgr_init), (compare_account_name),
970         (modest_account_mgr_set_account_busy),
971         (modest_account_mgr_account_is_busy):
972         * src/modest-account-mgr.h:
973         Added busy information to the account manager. It possible to
974         set an account to busy or not busy now.
975         
976         * src/modest-mail-operation.c: (modest_mail_operation_finalize),
977         (modest_mail_operation_update_account),
978         (modest_mail_operation_notify_end):
979                 Set the account to busy/not busy depending on the operation state.
980
981         * src/widgets/modest-account-view.c:
982         (modest_account_view_finalize), (get_last_updated_string),
983         (update_account_view), (on_account_busy_changed), (init_view):
984         Show string "Refreshing..." if the account is currently busy.
985
986         All this fixes projects.maemo.org bug NB#59711
987
988 2007-06-14  Armin Burgmeier  <armin@openismus.com>
989
990         * src/widgets/modest-attachments-view.h:
991         * src/widgets/modest-attachments-view.c: Added
992         modest_attachments_view_has_attachments() and fixed coordinate
993         transformation in get_att_view_at_coords(). This fixes attachment
994         selection.
995
996         * src/widgets/modest-msg-view.c: Make use of
997         modest_attachments_view_has_attachments() to decide whether to show
998         attachments in the header.
999
1000 2007-06-14  Johannes Schmid <johannes.schmid@openismus.com>
1001
1002         * src/maemo/modest-connection-specific-smtp-edit-window.c:
1003         (on_change), (on_combo_security_changed),
1004         (modest_connection_specific_smtp_edit_window_init),
1005         (modest_connection_specific_smtp_edit_window_set_connection),
1006         (modest_connection_specific_smtp_edit_window_is_dirty):
1007         * src/maemo/modest-connection-specific-smtp-edit-window.h:
1008         * src/maemo/modest-connection-specific-smtp-window.c:
1009         (on_button_edit):
1010         
1011         Show an comfirmation dialog for the smpt-edit-window if there are
1012         unsaved changes and the user hits cancel.
1013         (projects.maemo.org bug NB#60479)
1014
1015 2007-06-14  Armin Burgmeier  <armin@openismus.com>
1016
1017         * src/maemo/easysetup/modest-easysetup-wizard.c:
1018         * src/maemo/modest-account-settings-dialog.c: Show "invalid email
1019         address" info in banner instead of node, and select all the text as
1020         per UI spec. This fixes projects.maemo.org bug NB#60550.
1021
1022 2007-06-14  set REAL_NAME environment variable  <set EMAIL_ADDRESS environment variable>
1023
1024         reviewed by: <delete if not using a buddy>
1025
1026         * src/maemo/modest-main-window.c: (_on_msg_count_changed):
1027         * src/modest-ui-actions.c: (folder_refreshed_cb):
1028
1029 2007-06-14  Murray Cumming  <murrayc@murrayc.com>
1030
1031         * src/widgets/modest-folder-view.c: (filter_row):
1032         Explictly filter out the per-account outbox accounts. I don't know why 
1033         these only sometimes show up, but now they should never show up in the UI.
1034         This should fix projects.maemo.org bug NB#60160.
1035
1036 2007-06-14  Murray Cumming  <murrayc@murrayc.com>
1037
1038         * src/maemo/modest-main-window.c: (set_toolbar_mode): Prevent a warning
1039         when the toolbar does not exist yet.
1040         * src/modest-widget-memory.c: (save_settings_header_view):
1041         Added g_warnings() to help us to discover when we are trying to set 
1042         an empty columns list in gconf.
1043
1044 2007-06-14  Murray Cumming  <murrayc@murrayc.com>
1045
1046         * src/modest-init.h:
1047         * src/modest-init.c: 
1048         Added modest_init_get_default_header_view_column_ids().
1049
1050         * src/modest-widget-memory.c: (restore_settings_header_view):
1051         Use the new function to get defaults, if the gconf key is empty, 
1052         instead of not creating any columns. This seems to fix the 
1053         empty right-hand pane problems.
1054         
1055         * src/widgets/modest-header-view.h: Improved documentation.
1056
1057 2007-06-13  Murray Cumming  <murrayc@murrayc.com>
1058
1059         * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
1060         Revert the use of the display name as the account name, because this
1061         is generally wrong and would break the code that follows.
1062
1063 2007-06-13  Johannes Schmid <johannes.schmid@openismus.com>
1064
1065         * src/maemo/easysetup/modest-easysetup-wizard.c:
1066         (check_for_supported_auth_methods), (on_before_next),
1067         (create_account):
1068         
1069         More correct fix for projects.maemo.org bug NB# 58991
1070         Do not longer accept AUTH_PASSWORD as secure method and choose
1071         correct method when creating the account.
1072
1073 2007-06-13  Johannes Schmid <johannes.schmid@openismus.com>
1074
1075         * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
1076         Fixed projects.maemo.org bug NB# 58991
1077
1078 2007-06-13  Murray Cumming  <murrayc@murrayc.com>
1079
1080         * src/modest-init.c: (modest_init_init_core),
1081         (modest_init_local_folders): Fixed compilation errors.
1082
1083 2007-06-13  Murray Cumming  <murrayc@murrayc.com>
1084
1085         * libmodest-dbus-client/libmodest-dbus-client-1.0.pc.in:
1086         Remove unnecessary lib/include, though it is good in principle when needed.
1087         
1088         * src/modest-init.c: (modest_init_one_local_folder),
1089         (modest_init_local_folders):
1090         * src/modest-init.h: Initialize the .modest library for memory cards, 
1091         instead of just doing nothing for them. This should allow us to create 
1092         folders there, fixing projects.maemo.org bug NB#59297.
1093
1094 2007-06-13  Murray Cumming  <murrayc@murrayc.com>
1095
1096         * libmodest-dbus-client/libmodest-dbus-client.c:
1097         (libmodest_dbus_client_send_mail), (libmodest_dbus_client_mail_to),
1098         (libmodest_dbus_client_compose_mail),
1099         (libmodest_dbus_client_open_message),
1100         (libmodest_dbus_client_send_and_receive),
1101         (libmodest_dbus_client_open_default_inbox):  Improved debug messages.
1102         
1103         * src/dbus_api/modest-dbus-callbacks.c: (find_message_by_url),
1104         (on_idle_open_message), (modest_dbus_req_handler),
1105         (search_result_to_message), (modest_dbus_req_filter):
1106         * src/modest-search.c: (add_hit), (modest_search_folder),
1107         (modest_search_all_accounts): Improved debug messages.
1108         
1109         * tests/dbus_api/test_open_message.c: (main): Improved comments.
1110         
1111         * src/modest-tny-account-store.c:
1112         (modest_tny_account_store_get_tny_account_by): Initialize a variable to 
1113         fix the build.
1114
1115 2007-06-13  Murray Cumming  <murrayc@murrayc.com>
1116
1117
1118 2007-06-12  Murray Cumming  <murrayc@murrayc.com>
1119
1120         * src/modest-tny-account-store.c: (get_server_accounts):
1121         Call modest_tny_local_folders_account_add_merged_outbox_folders() 
1122         even when there are no accounts, so that the Outbox folder is always 
1123         visible. This fixes projects.maemo.org bug NB#60158.
1124
1125 2007-06-12   Murray Cumming  <murrayc@murrayc.com>
1126
1127         * src/maemo/modest-main-window.c: (create_details_widget):
1128         Use the mcen_va_never logical ID.
1129
1130 2007-06-12  Murray Cumming  <murrayc@murrayc.com>
1131
1132         * src/dbus_api/modest-dbus-api.h:
1133         * src/dbus_api/modest-dbus-callbacks.c:
1134         (on_idle_open_default_inbox), (on_open_default_inbox),
1135         (modest_dbus_req_handler):
1136         Added a new OpenDefaultInbox method.
1137         
1138         * libmodest-dbus-client/libmodest-dbus-client.h:
1139         * libmodest-dbus-client/libmodest-dbus-client.c:
1140         Added libmodest_dbus_client_open_default_inbox() to call the new 
1141         dbus method. This should be useful to projects.maemo.org bug NB#57214,
1142         though it should maybe be used by a new osso-email-interface function.
1143         
1144         * tests/dbus_api/Makefile.am:
1145         * tests/dbus_api/test_open_default_inbox.c: Added a test for the new 
1146         function.
1147
1148 2007-06-12  Murray Cumming  <murrayc@murrayc.com>
1149
1150         * src/modest-account-mgr.c: (on_key_change): Append to the list, not to 
1151         NULL, so that the signals get all the keys. This should not actually 
1152         change any behaviour because we currently update everything when anything 
1153         changes.
1154
1155 2007-06-12  Murray Cumming  <murrayc@murrayc.com>
1156
1157         * src/widgets/modest-folder-view.c: (text_cell_data),
1158         (icon_cell_data), (filter_row), (on_selection_changed), (cmp_rows),
1159         (find_inbox_iter): If tinymail tells us that the folder is 
1160         a non-specific TNY_FOLDER_TYPE_NORMAL then do an extra check on the name.
1161         This causes the inbox to be selected at startup, fixing part of 
1162         projects.maemo.org bug NB#56305.
1163
1164         * src/modest-ui-dimming-rules.c:
1165         (modest_ui_dimming_rules_on_tools_smtp_servers):
1166         * src/modest-ui-dimming-rules.h: Added a callback function to dim 
1167         the SMTP servers menu item.
1168         * src/maemo/modest-main-window-ui-dimming.h: Used the function, so 
1169         that the SMTP Servers menu item is dimmed when there are no accounts, 
1170         as a future version of the UI spec will require. This fixes 
1171         projects.maemo.org bug NB#56305.
1172
1173 2007-06-11  Johannes Schmid <johannes.schmid@openismus.com>
1174
1175         * src/maemo/modest-maemo-utils.c:
1176         (modest_maemo_utils_get_supported_secure_authentication_methods):
1177         Added label to dialog instead of unreadable title
1178         
1179         * src/maemo/modest-main-window.c: (sync_accounts_cb),
1180         (modest_main_window_new):
1181         Fixed projects.maemo.org bug NB#58576 by moving previous fix out of
1182         dead code.
1183         
1184         * src/modest-account-mgr.c: (modest_account_mgr_has_accounts):
1185         * src/modest-account-mgr.h:
1186         Added method to avoid code duplication
1187         
1188         * src/modest-ui-actions.c: (modest_ui_actions_on_accounts),
1189         (modest_ui_actions_on_send_receive):
1190         Use modest_account_mgr_has_accounts
1191
1192 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1193
1194         * src/maemo/easysetup/modest-presets.c:
1195         (modest_presets_get_info_server_security): Do not report secure auth 
1196         if there is none.
1197         * src/modest-protocol-info.c: Use the defines for the gconf key names 
1198         for secure authentication. This was not working because we were writing 
1199         Password and reading password, and the check was case sensitive.
1200         With this, gmail now works for receiving and sending when created via 
1201         easysetup.
1202
1203 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1204
1205         * src/maemo/easysetup/modest-presets.c:
1206         (modest_presets_get_info_server_security): Correct the key name for 
1207         SecureSmtp, because it seems to be case-sensitive.
1208         * src/maemo/easysetup/modest-wizard-dialog.c: (create_title):
1209         Do not try to get the 0th notebook page if there are 0 pages, to 
1210         avoid a g_warning().
1211
1212 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1213
1214         * src/maemo/modest-platform.c: (modest_platform_on_new_msg):
1215         Prevent use of the new notification feature for now, because it currently 
1216         shows one dialog for each new email, which is annoying.
1217
1218 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1219
1220         * src/maemo/easysetup/modest-easysetup-provider-combo-box.c:
1221         (easysetup_provider_combo_box_fill): Ask for the providers for mmc=0 
1222         when no country was selected, so that we show the global ones.
1223         * src/maemo/easysetup/modest-presets.c:
1224         (modest_presets_get_providers): Return only the global providers, not 
1225         all providers when mmc=0 was requested.
1226
1227 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1228
1229         * src/maemo/easysetup/modest-easysetup-provider-combo-box.c:
1230         (easysetup_provider_combo_box_fill): Remove debug code that showed the 
1231         provider ID in the combo.
1232         
1233         * src/maemo/easysetup/modest-wizard-dialog.c: (create_title):
1234         Do not show the page numbers, because it would be confusing when we add 
1235         the 2 extra pages for the custom setup, when the user chooses the "Other"
1236         provider. This fixes projects.maemo.org bug NB#59028
1237         
1238         * src/modest-init.c: (modest_init_init_core),
1239         (modest_init_local_folders):
1240         * src/modest-init.h:
1241         * src/modest-tny-account.c:
1242         (modest_tny_account_new_for_local_folders): Do not create default folders 
1243         for the memory card. This should fix projects.maemo.org bug NB#59328.
1244
1245 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1246
1247         * src/modest-account-mgr.c: (modest_account_mgr_finalize):
1248         Free the changed_conf_keys string items. They are already freed in the 
1249         idle handler if that runs first.
1250
1251 2007-06-11  Murray Cumming  <murrayc@murrayc.com>
1252
1253         * src/modest-ui-actions.c:
1254         (modest_ui_actions_on_password_requested): Check that the server name 
1255         is not NULL, because I just saw a password dialog with (null) as the 
1256         server name.
1257
1258         * src/modest-tny-folder.[h|c]: Added 
1259         modest_tny_folder_is_memory_card_folder().
1260         * src/modest-ui-dimming-rules.c:
1261         (modest_ui_dimming_rules_on_new_folder): Do not dim the new folder context 
1262         menu item for memory cards, as per the UI spec (section 7.4). This should 
1263         fix projects.maemo.org bug NB#59297.
1264
1265 2007-06-10  Johannes Schmid <johannes.schmid@openismus.com>
1266
1267         * src/maemo/modest-main-window.c: (sync_accounts_cb):
1268         Fixed projects.maemo.org bug NB#58576 (not yet activated as there
1269         is another problem with the code, see Murray's comment at line 772)
1270
1271 2007-06-08  Murray Cumming  <murrayc@murrayc.com>
1272
1273         * src/maemo/easysetup/modest-easysetup-country-combo-box.h:
1274         * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
1275         (on_model_foreach_release), (easysetup_country_combo_box_finalize),
1276         (load_from_file), (easysetup_country_combo_box_init),
1277         (easysetup_country_combo_box_get_active_country_ids),
1278         (on_model_foreach_select_id): Store a GSList* of IDs in the model row, 
1279         instead of just one ID, to avoid duplicate country names in the UI.
1280
1281         * src/maemo/easysetup/modest-easysetup-provider-combo-box.h:
1282         * src/maemo/easysetup/modest-easysetup-provider-combo-box.c:
1283         (easysetup_provider_combo_box_fill): Adapt to use a list rather than 
1284         a single ID, avoiding duplicate providers.
1285         * src/maemo/easysetup/modest-easysetup-wizard.c:
1286         (on_combo_account_country): Adapt to use a list.
1287
1288 2007-06-08  Murray Cumming  <murrayc@murrayc.com>
1289
1290         * src/modest-account-mgr-priv.h: Added a timeout id and a GSList of 
1291         keys changed in the last X seconds.
1292         
1293         * src/modest-account-mgr.h:
1294         * src/modest-account-mgr.c: (on_timeout_notify_changes),
1295         (on_key_change), (modest_account_mgr_class_init),
1296         (modest_account_mgr_init), (modest_account_mgr_finalize):
1297         As the keys are changed, save them in a GSList* and signal that GSList* 
1298         further every X seconds, to batch changes, to allow us to do just one 
1299         update per batch. This should fix bug projects.maemo.org bug NB#56472 and 
1300         bug NB#56941 and other (UI-blocking) delays when changing account settings.
1301         
1302         * src/modest-tny-account-store.c: (on_account_changed): Adapted to 
1303         changed signal signature.
1304         
1305         * src/modest-conf.c: (modest_conf_set_list),
1306         (modest_conf_on_change): Reenable notification.
1307         
1308         (modest_conf_maemo_fake_on_change): Comment out the hack that 
1309         notified immediately instead of asynchronously.
1310         
1311         * src/widgets/modest-account-view.h: 
1312         * src/widgets/modest-account-view.c:
1313         (update_account_view): Select the previously-selected account ID, 
1314         if possible. This fixes projects.maemo.org bug NB#56933.
1315         
1316         (on_account_changed), (on_account_removed),
1317         (modest_account_view_select_account):
1318         
1319         Remove the block/unblock functions 
1320         because they were not useful.
1321         
1322         * src/maemo/modest-account-view-window.c:
1323         (on_delete_button_clicked), (on_edit_button_clicked),
1324         (on_new_button_clicked): Remove the use of the non-working block/unblock.
1325
1326 2007-06-08  Murray Cumming  <murrayc@murrayc.com>
1327
1328         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
1329         Improve the error dialog to show Incorrect Account Settings, which seems 
1330         clearer, and say that the internal error message (to be removed from the 
1331         dialog later) is often misleading.
1332         I have confirmed that projects.maemo.org bug NB#58543 is no longer a problem.
1333         
1334         * src/modest-tny-account.c:
1335         (modest_tny_account_new_from_server_account): Uncomment the printf of the 
1336         URL because this is so useful for debugging on the device.
1337         
1338         * src/widgets/modest-serversecurity-combo-box.c:
1339         (modest_serversecurity_combo_box_fill): Correct a logical ID.
1340
1341 2007-06-08  Murray Cumming  <murrayc@murrayc.com>
1342
1343         * src/modest-mail-operation.c:
1344         (modest_mail_operation_send_new_mail):
1345         Added a comment about a nasty but apparently harmless message. Details 
1346         added to the wiki g_warnings() page.
1347
1348 2007-06-08  Murray Cumming  <murrayc@murrayc.com>
1349
1350         * src/modest-tny-send-queue.c: (modest_tny_send_queue_add),
1351         (modest_tny_send_queue_class_init):
1352         Do not use the priv struct because it is empty, to avoid g_warnings 
1353         about a non-existant priv struct.
1354         
1355         * src/modest-tny-account-store.c:
1356         (modest_tny_account_store_get_server_account),
1357         (modest_tny_account_store_get_transport_account_for_open_connection
1358         ): Do not try to get transport server accounts for local folders, 
1359         because they don't have any. This avoids a g_warning().
1360         
1361         * src/maemo/modest-main-window.c:
1362         (modest_main_window_notify_send_receive_initied),
1363         (on_configuration_key_changed):
1364         * src/maemo/modest-msg-edit-window.c: (on_delete_event):
1365         * src/modest-defs.h:
1366         * src/modest-mail-operation.c:
1367         (modest_mail_operation_update_account):
1368         * src/modest-tny-account-store.h:
1369         * src/modest-tny-account.c:
1370         (modest_tny_account_get_special_folder),
1371         (modest_tny_account_new_for_local_folders),
1372         (modest_tny_account_new_for_per_account_local_outbox_folder):
1373         * src/modest-tny-folder.c: (modest_tny_folder_is_local_folder):
1374         * src/modest-tny-local-folders-account.c:
1375         (add_account_folders_to_merged_folder):
1376         * src/modest-ui-actions.c: (modest_ui_actions_on_new_msg),
1377         (reply_forward_cb), (modest_ui_actions_on_save_to_drafts),
1378         (modest_ui_actions_on_send):
1379         * src/widgets/modest-folder-view.c: (update_model):
1380         Rename MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID to 
1381         MODEST_LOCAL_FOLDERS_ACCOUNT_ID and rename 
1382         modest_tny_account_store_get_tny_account_by_account() to 
1383         modest_tny_account_store_get_server_account() to make the code clearer.
1384
1385 2007-06-07  Marcus Bauer  <marcusb@openismus.com>
1386
1387         * src/dbus_api/modest-dbus-api.h:
1388         changed MODEST_DBUS_NAME in order to get dbus auto-activation to work
1389         * src/maemo/modest-msg-edit-window.c:
1390         (modest_msg_edit_window_attach_file_noninteractive):
1391         added g_warning in case the file to be attached does not exist
1392         * tests/dbus_api/test_compose_mail.c: (main):
1393         removed obsolete comment, added generic files as test case
1394
1395 2007-06-07  Murray Cumming  <murrayc@murrayc.com>
1396
1397         * src/modest-tny-account-store.c: (get_server_accounts): 
1398         Create the local folders account only once, not once for each outbox, and 
1399         not only when there are some outboxes.
1400
1401 2007-06-07  Murray Cumming  <murrayc@murrayc.com>
1402
1403         * src/widgets/modest-folder-view.c: (update_model): Comment out some 
1404         code that is causing a crash at startup. Sergio says that he is fixing this.
1405
1406 2007-06-07  Johannes Schmid <johannes.schmid@openismus.com>
1407
1408         * src/maemo/easysetup/modest-easysetup-wizard.c: (on_before_next):
1409         * src/maemo/modest-account-settings-dialog.c:
1410         (create_page_outgoing),
1411         (modest_account_settings_dialog_set_account_name),
1412         (save_configuration):
1413         - Fixed handling of the "Use secure authentication" toggle to comply with
1414         the ui specification. See also projects.maemo.org bug NB#58400
1415         
1416         - Restore the Secure connection setting combo correctly when loading
1417         the settings dialog
1418         
1419         * src/maemo/modest-maemo-utils.c:
1420         (on_camel_account_get_supported_secure_authentication),
1421         (modest_maemo_utils_get_supported_secure_authentication_methods):
1422         Fixed a crasher if operation is canceled
1423         
1424         * src/modest-protocol-info.c: (modest_protocol_info_is_secure),
1425         (modest_protocol_info_auth_is_secure):
1426         * src/modest-protocol-info.h:
1427         
1428         Added two new methods to determine if a protocol/auth method is secure/encrypted in 
1429         the sense of the "Use secure authentication" toggle.
1430
1431 2007-06-07  Murray Cumming  <murrayc@murrayc.com>
1432
1433         * src/modest-tny-account-store.h:
1434         * src/modest-tny-account-store.c:
1435         Added modest_tny_account_store_get_local_folders_account().
1436         
1437         * src/modest-tny-folder.h:
1438         * src/modest-tny-folder.c:
1439         (modest_tny_folder_get_account):
1440         Added this replacement for tny_folder_get_account() which returns 
1441         the local-folders account for TnyMergeFolders, to avoid the 
1442         warning about the unimplemented get_account_func() vfunc().
1443         
1444         (modest_tny_folder_get_rules),
1445         * src/modest-ui-actions.c:
1446         (modest_ui_actions_on_folder_selection_changed):
1447         * src/modest-mail-operation.c:
1448         (modest_mail_operation_remove_folder),
1449         (modest_mail_operation_xfer_folder),
1450         (modest_mail_operation_rename_folder),
1451         (modest_mail_operation_get_msg),
1452         (modest_mail_operation_get_msgs_full),
1453         (modest_mail_operation_remove_msg),
1454         (modest_mail_operation_xfer_msgs),
1455         (modest_mail_operation_refresh_folder):
1456         Use modest_tny_folder_get_account() instead of tny_folder_get_account().
1457         
1458 2007-06-07  Murray Cumming  <murrayc@murrayc.com>
1459
1460         * src/modest-account-mgr-helpers.c: (get_security_for_conf_string),
1461         (modest_server_account_set_security): 
1462         * src/modest-defs.h: Add comments explaining that our 
1463         UI spec means TLS when it says Normal, so these do not look like errors.
1464         
1465         * src/modest-tny-account.c:
1466         (modest_tny_account_new_from_server_account): Use the new tinymail 
1467         'wrapped' option for regular SSL, and use the new tinymail 'tls' option 
1468         for TLS. This should fix projects.maemo.org bug NB#58460.
1469
1470 2007-06-06  Murray Cumming  <murrayc@murrayc.com>
1471
1472         * src/maemo/easysetup/modest-easysetup-wizard.c:
1473         (create_account): Correct the position of the gconf sanity check warning.
1474         (on_button_edit_advanced_settings): Show the dialog immediately, instead 
1475         of using the g_idle_add() hack, because this does not seem to be necessary 
1476         on osso 1.1. It might even work on Maemo Bora if the problems were actually 
1477         caused by something other than maemo's gconf.
1478         This fixes projects.maemo.org bug #59022.
1479
1480 2007-06-06  Murray Cumming  <murrayc@murrayc.com>
1481
1482         * src/modest-account-mgr-helpers.c:
1483         (modest_account_mgr_get_default_account): Do not check that the account 
1484         exists if there is no default account (NULL), to fix a runtime warning.
1485
1486 2007-06-06  Murray Cumming  <murrayc@murrayc.com>
1487
1488         * src/widgets/modest-folder-view.c: (update_model):
1489         Do not unref the treemodel before we have finished with it.
1490         This prevents a valgrind warning and an occasional crash at startup.
1491         
1492 2007-06-06  Marcus Bauer  <marcusb@openismus.com>
1493
1494         * libmodest-dbus-client/libmodest-dbus-client.h:
1495         added libmodest_dbus_client_compose_mail()
1496         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_compose_mail),
1497         (on_compose_mail):
1498         added functions needed for SendAsMail() functionality
1499         * src/maemo/modest-msg-edit-window.c:
1500         (modest_msg_edit_window_attach_file_noninteractive):
1501         non interactively attaching files to message edit window
1502         * src/widgets/modest-msg-edit-window.h:
1503         * tests/dbus_api/Makefile.am:
1504         added test_compose_mail.c
1505         * tests/dbus_api/test_compose_mail.c: (main):
1506
1507 2007-06-05  Johannes Schmid <johannes.schmid@openismus.com>
1508
1509         * src/modest-account-view.c:
1510         Fixed bug projects.maemo.org bug NB#59307
1511
1512 2007-06-05  Johannes Schmid <johannes.schmid@openismus.com>
1513
1514         * src/maemo/easysetup/modest-easysetup-wizard.c:
1515         (get_serverport_incoming), (on_before_next), (create_account):
1516         * src/maemo/modest-account-settings-dialog.c:
1517         (modest_account_settings_dialog_set_account_name):
1518         * src/maemo/modest-maemo-utils.c:
1519         (on_camel_account_get_supported_secure_authentication_status),
1520         (on_camel_account_get_supported_secure_authentication),
1521         (on_secure_auth_cancel),
1522         (modest_maemo_utils_get_supported_secure_authentication_methods):
1523
1524         Implemented security authentication for settings and easysetup.
1525
1526 2007-06-05  Murray Cumming  <murrayc@murrayc.com>
1527
1528         * src/maemo/modest-main-window.c: (create_details_widget):
1529         Show the total size for memory card accounts too, instead of showing 
1530         the name again. Fixes bug projects.maemo.org bug NB#59307.
1531
1532 2007-06-05  Murray Cumming  <murrayc@murrayc.com>
1533
1534         * src/Makefile.am:
1535         * src/modest-tny-simple-folder-store.c:
1536         * src/modest-tny-simple-folder-store.h: Removed.
1537         
1538         * src/modest-tny-local-folders-account.h
1539         * src/modest-tny-local-folders-account.c: Added a derived TnyAccount 
1540         for the local-folders maildir, with an extra TnyMergedFolder containing 
1541         the folders of the per-account outbox store accounts.
1542         * src/modest-tny-account-store.c: (get_server_accounts):
1543         Add the outboxes to the special local folders account after creating it.
1544                 
1545         * src/modest-tny-account.c:
1546         (modest_tny_account_new_for_local_folders): Create the derived 
1547         account when creating the main local-folders account.
1548         
1549         * src/maemo/modest-main-window.c: (create_details_widget),
1550         (modest_main_window_set_contents_style):
1551         Adapted to the use of the derived account instead of the folder store.
1552         
1553         * src/modest-ui-actions.c:
1554         (modest_ui_actions_on_folder_selection_changed):
1555         * src/modest-ui-dimming-rules.c:
1556         (modest_ui_dimming_rules_on_new_folder):
1557         * src/widgets/modest-folder-view.c: (text_cell_data),
1558         (icon_cell_data), (filter_row), (update_model),
1559         (get_cmp_rows_type_pos):
1560         
1561
1562         * src/modest-tny-send-queue.c: (modest_tny_send_queue_new): 
1563         Do not get the priv struct, because no priv struct has been set, 
1564         to avoid a runtime warning.
1565         
1566         * src/maemo/modest-platform.c: (modest_platform_show_help): Replace 
1567         #ifelse with #else, to fix a warning, to fix the build.
1568         
1569         
1570 2007-06-05  Christian Kellner  <ckellner@openismus.com> 
1571
1572         * libmodest-dbus-client/libmodest-dbus-client.c:
1573         Fix small typo in doc (why do you always see them
1574         AFTER committing stuff?)
1575
1576
1577 2007-06-05  Christian Kellner  <ckellner@openismus.com> 
1578
1579         * libmodest-dbus-client/libmodest-dbus-client.c:
1580         (libmodest_dbus_client_search):
1581         Initialize hit list with NULL. Also set folder list 
1582         to "" if NULL was supplied. 
1583         Add documentation.
1584
1585 2007-06-03  Christian Kellner  <ckellner@openismus.com> 
1586
1587         * libmodest-dbus-client/libmodest-dbus-client.c:
1588         (modest_search_hit_list_free):
1589         Make it save to call it with NULL.
1590
1591         * src/dbus_api/modest-dbus-callbacks.c:
1592         * src/modest-search.c:
1593         Return full ModestSearchHits as the search result list not TnyHeader
1594         since can't be around withouth the correspoding TnyFolder.
1595
1596 2007-06-03  Johannes Schmid <johannes.schmid@openismus.com>
1597
1598         * src/maemo/easysetup/modest-easysetup-wizard.c: (on_before_next):
1599         Added currently commented code to check for secure auth
1600         
1601         * src/maemo/modest-account-settings-dialog.c: (on_response),
1602         (save_configuration), (show_ok):
1603         Fixed projects.maemo.org bugs NB #59248 and #59251
1604         
1605         * src/maemo/modest-maemo-utils.c:
1606         (on_camel_account_get_supported_secure_authentication_status),
1607         (on_camel_account_get_supported_secure_authentication),
1608         (modest_maemo_utils_get_supported_secure_authentication_methods):
1609         * src/maemo/modest-maemo-utils.h:
1610         Moved get_supported_secure_authentication() method here to be able
1611         to use it from easysetup wizard.
1612         
1613         * src/modest-pair.c: (on_pair_compare_as_string):
1614         Should compare string with first part of pair and not
1615         strcmp pairs which will crash.
1616         
1617         * src/modest-protocol-info.c:
1618         (modest_protocol_info_protocol_is_store):
1619         * src/modest-protocol-info.h:
1620         Added modest_protocol_info_get_auth_protocol_name()
1621
1622 2007-06-03  Armin Burgmeier  <armin@openismus.com>
1623
1624         * src/modest-mail-operation.c:
1625         (modest_mail_operation_get_msgs_full): Set priv->account also when
1626         only retrieving a single message. This avoids a crash when canceling
1627         the operation.
1628
1629         (modest_mail_operation_notify_end): Do not explicitely remove the
1630         operation from the queue but only emit the progress_changed signal.
1631         The queue itself listens to that signal to remove the operation when
1632         it is finished. This reduces coupling and removes a potential deadlock
1633         when modest_mail_operation_notify_end() is called from the queue
1634         itself. If that is a problem performance-wise, I would suggest to
1635         implement a status-changed signal or something.
1636
1637         * src/modest-mail-operation-queue.c: Make sure to never call a
1638         function from another file while the queue is locked. This (hopefully)
1639         fixes another deadlock when exiting the application while retrieving
1640         mail.
1641
1642 2007-06-03  Johannes Schmid <johannes.schmid@openismus.com>
1643
1644         * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_filter):
1645         Fixed build if libogs is not available
1646         
1647 2007-06-03  Christian Kellner  <ckellner@openismus.com>
1648
1649         * libmodest-dbus-client/libmodest-dbus-client.h:
1650         * libmodest-dbus-client/libmodest-dbus-client.c:
1651         (libmodest_dbus_client_delete_message):
1652         Implement deletion of messages (Client side).
1653
1654         (modest_search_hit_free),
1655         (modest_search_hit_list_free):
1656         Utility functions to free the result hits.
1657
1658         * src/dbus_api/modest-dbus-api.h:
1659         * src/dbus_api/modest-dbus-callbacks.c:
1660         (on_delete_message),
1661         (modest_dbus_req_handler):
1662         Implement deletion of messages (Server side).
1663         
1664         * tests/dbus_api/test_search.c: (main):
1665         Free the hits.
1666
1667         * tests/dbus_api/Makefile.am:
1668         * tests/dbus_api/test_delete_message:
1669         Add little test for the message deletion logic
1670
1671 2007-06-03  Armin Burgmeier  <armin@openismus.com>
1672
1673         * src/modest-mail-operation-queue.c: Fixed a deadlock in finalize()
1674         when the operation queue was not empty. projects.maemo.org bug
1675         NB#58976.
1676
1677 2007-06-03  Armin Burgmeier  <armin@openismus.com>
1678
1679         * src/widgets/modest-account-view.c
1680         (on_account_default_toggled): Toggle default account column
1681         explicitely instead of waiting for gconf notification (which is
1682         currently ignored). This should fix projects.maemo.org bug NB#56910
1683         for now. It should probably be removed when gconf notifications are
1684         re-enabled.
1685
1686 2007-06-03  Christian Kellner  <ckellner@openismus.com>
1687
1688         * libmodest-dbus-client/libmodest-dbus-client.c:
1689         Removed debugging spew.
1690
1691         * src/dbus_api/modest-dbus-callbacks.c:
1692         Added debugging info.
1693
1694         * src/modest-search.c: 
1695         (modest_search_all_accounts):
1696         Use tny_account_store_get_accounts () on the account store
1697         to get all accounts which is the way to get all accounts.
1698
1699         * tests/dbus_api/test_search.c: (main):
1700         Make it possible to supply the search term on the command line.
1701
1702 2007-06-03  Christian Kellner  <ckellner@openismus.com>
1703
1704         * libmodest-dbus-client/libmodest-dbus-client.c:
1705         * libmodest-dbus-client/libmodest-dbus-client.h:
1706         Implement searching for mesages over raw dbus. (Client side)
1707         
1708         * src/dbus_api/Makefile.am:
1709         * src/dbus_api/modest-dbus-api.h:
1710         * src/dbus_api/modest-dbus-callbacks.c:
1711         * src/dbus_api/modest-dbus-callbacks.h:
1712         Search framework (Server side). Also implement on_idle_open_message().
1713
1714         * src/maemo/modest-platform.c: (modest_platform_init):
1715         Prepare for raw dbus/osso rpc interaction.
1716
1717         * src/modest-search.h:
1718         Changed minsize from unspecific guint to guint32.
1719
1720         * src/modest-search.c: 
1721         (add_header):
1722         Added strange hack which magically made tnymail work.
1723         
1724         (modest_search_folder),
1725         (modest_search_account),
1726         (modest_search_all_accounts):
1727         Implement helper functions to search all accounts, specific account
1728         and folder.
1729
1730         * tests/dbus_api/Makefile.am:
1731         * tests/dbus_api/test_open_message.c:
1732         Read url to open from commandline.
1733
1734         * tests/dbus_api/test_search.c:
1735         Implement small test for dbus search stuff
1736
1737 2007-06-02  Armin Burgmeier  <armin@openismus.com>
1738
1739         * src/maemo/modest-account-settings-dialog.c: Changed window title to
1740         include account type and account title. This fixes projects.maemo.org
1741         bug NB#59000.
1742
1743         * src/maemo/modest-account-view-window.c: Set a title for the window.
1744
1745 2007-06-02  Armin Burgmeier  <armin@openismus.com>
1746
1747         * src/maemo/easysetup/modest-easysetup-wizard.c: Always prefill
1748         incoming and outgoing server name based on email address and server
1749         types as long as the names have not been edited manually. This fixes
1750         projects.maemo.org bug NB#58671.
1751
1752 2007-06-02  Armin Burgmeier  <armin@openismus.com>
1753
1754         * src/maemo/easysetup/modest-easysetup-wizard.c: Added a note that the
1755         server type cannot be changed in future to the incoming details page
1756         when creating an account. This fixes projects.maemo.org bug NB#59029.
1757
1758 2007-06-02  Armin Burgmeier  <armin@openismus.com>
1759
1760         * src/maemo/easysetup/modest-easysetup-wizard.c: Made labels wrapping
1761         and assigned max width chars so that they wrap instead of being
1762         truncated. Fixes projects.maemo.org bug NB#56126.
1763
1764 2007-06-02  Armin Burgmeier  <armin@openismus.com>
1765
1766         * src/maemo/modest-connection-specific-smtp-window.c: Add an
1767         introductory note to the connection specific smtp servers dialog, as
1768         section 8.12 of the email UI specification requires. This fixes
1769         projects.maemo.org bug NB#59212.
1770
1771 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1772
1773         * src/maemo/modest-account-view-window.c: (on_edit_button_clicked):
1774         Check for running mail operations, and offer to cancel them, as per the UI 
1775         spec. This fixes projects.maemo.org bug NB#59041 .
1776         However, I suspect that tinymail may still use the network connection 
1777         in ways that we cannot detect or cancel. But it should at least adapt to 
1778         changes fairly well.
1779
1780 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1781
1782         * src/modest-tny-account-store.c: (recreate_all_accounts),
1783         (get_cached_accounts), (get_server_accounts): Do more sensible 
1784         ref-counting of the temporarily-cached outbox accounts list. This seems 
1785         to fix a crash when entering the password when the password was wrong.
1786
1787 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1788
1789         * src/gnome/modest-main-window.c: (connect_signals):
1790         * src/modest-tny-account-store.c: (get_password),
1791         (modest_tny_account_store_alert): Added comments and clues and 
1792         commented-out code about how we currently fail to deal with connection 
1793         errors and wrong passwords, and how we should.
1794
1795 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1796
1797         * src/widgets/modest-retrieve-combo-box.c:
1798         (modest_retrieve_combo_box_fill): Correc the choices for POP and IMAP.
1799         This fixes projects.maemo.org bug NB#58472 .
1800
1801 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1802
1803         * src/maemo/easysetup/modest-easysetup-wizard.c:
1804         (create_page_complete_easysetup): 
1805         (create_page_complete_custom): Correct the last commit, 
1806         putting the correct sentences in the correct pages.
1807
1808 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1809
1810         * src/maemo/easysetup/modest-easysetup-wizard.c:
1811         (create_page_complete_custom): Show the second sentence of text 
1812         (mcen_ia_easysetup_complete). This fixes projects.maemo.org bug 
1813         NB#59023 .
1814
1815 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1816
1817         * src/modest-account-mgr.c: (modest_account_mgr_add_account):
1818         Set the leave-on-server conf key by default. This fixes 
1819         projects.maemo.org bug NB#59258 .
1820
1821 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1822
1823         * src/maemo/modest-account-view-window.c:
1824         (on_delete_button_clicked): Use the new emev_nc_delete_mailboximap 
1825         logical ID for the confirmation messages. Fixed projects.maemo.org bug 
1826         NB#59037 .
1827
1828 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1829
1830         * src/maemo/easysetup/modest-easysetup-wizard.c:
1831         (create_caption_new_with_asterisk): 
1832         * src/maemo/modest-account-settings-dialog.c:
1833         (create_caption_new_with_asterisk):
1834         Uncomment the code to add the asterisk, because they are no longer in the 
1835         translated text in the .po file. Fixes projects.maemo.org bug NB#59202 .
1836         (update_incoming_server_title): Add an asterisk here too.
1837
1838 2007-06-01  Murray Cumming  <murrayc@murrayc.com>
1839
1840         * src/maemo/modest-maemo-global-settings-dialog.c:
1841         (current_connection): Check that we are even connected before trying to 
1842         get the current connection name, to prevent a warning.
1843         
1844         * src/maemo/modest-platform.c: (modest_platform_connect_and_wait):
1845         Change the comments because tny_maemo_conic_device_connect() is now meant to 
1846         block.
1847         * src/modest-mail-operation.c:
1848         (modest_mail_operation_update_account): Call modest_platform_connect_and_wait() to 
1849         make sure we have a connection and fail if we can't connect.
1850
1851 2007-05-31  Murray Cumming  <murrayc@murrayc.com>
1852
1853         * src/maemo/modest-main-window.c:
1854         (on_account_store_connecting_finished): Do not try to get the 
1855         connection ID if we are not really connected. I am surprised that 
1856         this signal handler is called when we are not connected.
1857
1858 2007-05-31  Murray Cumming  <murrayc@murrayc.com>
1859
1860         * src/modest-error.h: 
1861         * src/modest-ui-actions.c:
1862         (modest_ui_actions_get_msgs_full_error_handler):
1863         Renamed an enum, added comments, added an enum 
1864         for retrieve-limit errors.
1865         
1866         * src/modest-mail-operation.c: (foreach_add_item),
1867         (internal_folder_observer_update), (update_account_thread),
1868         (modest_mail_operation_get_msgs_full):
1869         (modest_mail_operation_update_account),
1870         Added comments about how this works, and some code that should 
1871         (but does not) check the retrieve-limit. See the TODO comment.
1872
1873 2007-05-31  Murray Cumming  <murrayc@murrayc.com>
1874
1875         * src/modest-ui-actions.c: (modest_ui_actions_on_help):
1876         Initialize help_id to fix the build. Warnings are good.
1877
1878 2007-05-31  Murray Cumming  <murrayc@murrayc.com>
1879
1880         * src/widgets/modest-hbox-cell-renderer.c:
1881         (modest_hbox_cell_renderer_append):
1882         * src/widgets/modest-vbox-cell-renderer.c:
1883         (modest_vbox_cell_renderer_append):
1884         Fix the last commit by doing an extra ref.
1885
1886 2007-05-31  Murray Cumming  <murrayc@murrayc.com>
1887
1888         * src/widgets/modest-hbox-cell-renderer.c:
1889         (modest_hbox_cell_renderer_append):
1890         * src/widgets/modest-vbox-cell-renderer.c:
1891         (modest_vbox_cell_renderer_append):
1892         For glib < 2.10, use gtk_object_sink() instead of g_object_ref_sink(), 
1893         to fix the build with Maemo Bora.
1894         However, there seems to be a remaining crash, probably due to reference 
1895         counting errors.
1896
1897 2007-05-30  Murray Cumming  <murrayc@murrayc.com>
1898
1899         * src/modest-tny-account.c:
1900         (modest_tny_account_new_for_local_folders):
1901         Use hildon_file_system_info_async_new() to get a better name for 
1902         the memory card account, setting it as the account name.
1903         The timing seems to work, but this is probably 
1904         not the best place to do this. See the TODO comment.
1905         * src/maemo/modest-main-window.c: (create_details_widget):
1906         * src/widgets/modest-folder-view.c: (text_cell_data):
1907         Use the tny account name instead of hardcoding a name.
1908
1909 2007-05-30  Christian Kellner  <ckellner@openismus.com>
1910
1911         * src/modest-search.c: (modest_search):
1912         Correctly set part_search_func even if we have already parsed
1913         the query and thus a valid OgsTextSearcher object.
1914
1915 2007-05-30  Christian Kellner  <ckellner@openismus.com>
1916
1917         * configure.ac:
1918         Search and check for libogs.
1919
1920         * src/Makefile.am:
1921         Incooparte the ogs build env.
1922
1923         * src/modest-mime-part-search-stream.c:
1924         * src/modest-mime-part-search-stream.h:
1925         Removed since we do the real searching now in modest-search.c.
1926
1927         * src/modest-search.c:
1928         * src/modest-search.h:
1929         Implement support for ogs. We use that if available and requested
1930         and fall back to modest_text_utils_utf8_strcmp (). The search logic
1931         should now handle the corner case when the search term is spread
1932         across multiple reads.
1933         Also the logic in modest_search () was corrected to search all
1934         requested fields until one is found.
1935
1936 2007-05-30  Murray Cumming  <murrayc@murrayc.com>
1937
1938         * src/modest-defs.h: Define MODEST_MCC1_VOLUMEPATH and MODEST_MCC1_VOLUMEPATH_URI.
1939         
1940         * src/modest-init.h:
1941         * src/modest-init.c: Made modest_init_local_folders() take a
1942         path and be public so we can iniitialize maildir folders on 
1943         arbitrary volumes.
1944
1945         * src/modest-local-folder-info.h:
1946         * src/modest-local-folder-info.c:
1947         (modest_local_folder_info_get_maildir_path): Add a path parameter.
1948
1949         * src/modest-tny-account.c:
1950         (modest_tny_account_new_for_local_folders): Take a path parameter, though NULL 
1951         still means the standard local-folders path on the local volume.
1952         
1953         * src/modest-tny-account-store.c:
1954         (modest_tny_account_store_instance_init): Connect to GnomeVfsVolumeMonitor 
1955         signal so we can detect mount/unmount of the MMC1 card, and do an update of 
1956         all accounts when that happens.
1957         (get_server_accounts): If the MMC1 card is mounted, create a store account for 
1958         this too, so it shows up in the treeview.
1959
1960 2007-05-28  Murray Cumming  <murrayc@murrayc.com>
1961
1962         * src/maemo/modest-maemo-global-settings-dialog.c:
1963         (current_connection):
1964         * src/widgets/modest-global-settings-dialog.c:
1965         (current_connection_default):
1966         Use the enum values instead of 0, 1 and -1,
1967         to make the code clearer.
1968         
1969         * src/modest-account-mgr-helpers.c:
1970         (modest_account_mgr_get_connection_specific_smtp): Make this code 
1971         more robust.
1972         
1973         * src/modest-runtime.h:
1974         * src/modest-runtime.c:
1975         Added modest_runtime_remove_all_send_queues().
1976         * src/maemo/modest-main-window.c:
1977         (on_account_store_connecting_finished): Recreate the send queues, 
1978         using the appropriate transport accounts for this new connection.
1979         Some futher adapting might be needed if the old send queues are not 
1980         automatically stopped at this point.
1981
1982         * src/modest-tny-account-store.c: (get_server_accounts):
1983         Create the connection-specific transport accounts, so that they can 
1984         be found later by their ID.
1985         
1986         * src/modest-tny-account.h:
1987         * src/modest-tny-account.c:
1988         Added modest_tny_account_new_from_server_account_name().
1989         modest_tny_account_new_from_server_account(): Do not create the 
1990         send queues here, because they will be 
1991         recreated anyway when the connection changes for the first time.
1992
1993 2007-05-28  Murray Cumming  <murrayc@murrayc.com>
1994
1995         * src/modest-local-folder-info.h:
1996         * src/modest-local-folder-info.c:
1997         (modest_per_account_local_outbox_folder_info_get_maildir_path),
1998         (modest_per_account_local_outbox_folder_info_get_maildir_path_to_ou
1999         tbox_folder): Take a modest account name instead of a server TnyAccount,
2000         so we can have per-modest-account rather than per-transport-account 
2001         outboxes.
2002         We must make sure that we change or recreate the exisitng send queues 
2003         when the connection changes for connection-specific transport accounts.
2004         
2005         * src/modest-tny-account-store.c:
2006         (create_per_account_local_outbox_folders): Create outboxed per modest 
2007         account, not per transport account.
2008         
2009         * src/modest-tny-account.h:
2010         * src/modest-tny-account.c:
2011         (modest_tny_account_get_special_folder): For outboxes, get the 
2012         parent modest account name. The account is ignored for other folders.
2013         (modest_tny_account_new_for_per_account_local_outbox_folder): Take a 
2014         modest account name rather than a server TnyAccount.
2015
2016 2007-05-25  Murray Cumming  <murrayc@murrayc.com>
2017
2018         * src/modest-tny-account.c:
2019         (modest_tny_account_new_from_server_account): 
2020         Call modest_runtime_get_send_queue() immediately after creating transport 
2021         accounts, so that they start trying to send email from the outbox as soon 
2022         as possible at startup. This is probably what we want.
2023         
2024         * src/modest-mail-operation.c: (update_account_thread):
2025         * src/modest-runtime.c: (modest_runtime_get_send_queue):
2026         * src/modest-tny-send-queue.c: (modest_tny_send_queue_try_to_send):
2027         * src/modest-tny-send-queue.h:
2028         * src/modest-ui-actions.c: (modest_ui_actions_do_send_receive),
2029         (modest_ui_actions_on_send):
2030         Added comments about how sending works, though it needs some more work.
2031
2032 2007-05-25  Murray Cumming  <murrayc@murrayc.com>
2033
2034         * src/modest-tny-account.c:
2035         (modest_tny_account_get_special_folder): Add TODO because this fails when 
2036         the network is busy.
2037         (modest_tny_folder_store_get_local_size):  Fix an incorrect type check.
2038         * src/modest-ui-actions.c: (modest_ui_actions_on_smtp_servers): 
2039         Do not try to show this window when there is no active account. The UI 
2040         specs does not tell us what to do in this case.
2041
2042 2007-05-24  Murray Cumming  <murrayc@murrayc.com>
2043
2044         * Makefile.am:
2045         * po/POTFILES.in:
2046         * src/maemo/Makefile.am:
2047         * src/widgets/Makefile.am: Some dist fixes.
2048
2049 2007-05-24  Murray Cumming  <murrayc@murrayc.com>
2050
2051         * src/maemo/modest-connection-specific-smtp-window.c:
2052         (on_button_edit): Make the code clearer, to prevent a double free.
2053
2054 2007-05-24   Murray Cumming  <murrayc@murrayc.com>
2055
2056         * src/maemo/modest-account-settings-dialog.c:
2057         (modest_account_settings_dialog_set_account_name),
2058         Check the connection-specific smtp servers checkbox if there are any.
2059         
2060         * src/maemo/modest-connection-specific-smtp-edit-window.c:
2061         (modest_connection_specific_smtp_edit_window_init): 
2062         Start with a suitable port number.
2063         
2064         * src/modest-account-mgr-helpers.h:
2065         * src/modest-account-mgr-helpers.c: Added 
2066         modest_account_mgr_get_has_connection_specific_smtp().
2067         
2068         (modest_account_mgr_set_connection_specific_smtp),
2069         (modest_account_mgr_remove_connection_specific_smtp),
2070         (modest_account_mgr_get_has_connection_specific_smtp),
2071         (modest_account_mgr_get_connection_specific_smtp): Set/Get the list of 
2072         connection-specific smtp server accounts in the account, rather than 
2073         the server account conf dir.
2074         
2075         This fixes bug projects.maemo.org bug NB#58263 .
2076
2077         * src/modest-tny-account-store.c:
2078         (create_per_account_local_outbox_folders): Do not call get_server_accounts() 
2079         if there are no server accounts yet, to prevent an infinite loop when there 
2080         are no accounts. Document that one must be called before the other.
2081
2082 2007-05-24  Murray Cumming  <murrayc@murrayc.com>
2083
2084         * src/modest-account-mgr-helpers.c:
2085         (modest_account_mgr_get_default_account): Make sure that at least 
2086         one account is always the default, if there are any enabled accounts. 
2087         This is just a sanity check that shouldn't be necessary.
2088         
2089         * src/modest-account-mgr-priv.c:
2090         (_modest_account_mgr_account_from_key): Unescape the conf key to 
2091         get the real account name. For instance, this is an issue if a space 
2092         was in the account name, which can happen because the account name 
2093         is generated from the display name.
2094         (_modest_account_mgr_get_account_keyname): Always escape the 
2095         account name.
2096         * src/modest-account-mgr.c: (modest_account_mgr_account_names): 
2097         Unescape the conf keys to get the real account names.
2098         
2099         For instance, previously the is-default radio button in the accounts 
2100         list was not checked for any accounts at first. This fixes that,
2101         and might fix several similar bugs.
2102
2103 2007-05-23  Murray Cumming  <murrayc@murrayc.com>
2104
2105         * src/modest-ui-actions.c: (set_active_account_from_tny_account):
2106         Removed some debug printfs.
2107
2108 2007-05-23  Murray Cumming  <murrayc@murrayc.com>
2109
2110         Modest on-disk outbox directories are now here, for instance:
2111         $HOME/.modest/outboxes/<account-id-1>/outbox
2112         $HOME/.modest/outboxes/<account-id-2>/outbox
2113         instead of here:
2114         $HOME/.modest/local-folders/outbox
2115         so we can have a separate outbox for each account.
2116         But they are shown as one outbox in the GtkTreeView, by using a 
2117         TnyMergeFolder in a ModestTnySimpleFolderStore.
2118         
2119         * src/Makefile.am:
2120         * src/modest-tny-simple-folder-store.c:
2121         * src/modest-tny-simple-folder-store.h:
2122         Added ModestTnySimpleFolderStore, used to contain folders from 
2123         other folder stores, such as other accounts.
2124         
2125         * src/modest-tny-outbox-account.h:
2126         * src/modest-tny-outbox-account.c:
2127         Added ModestTnyOutboxAccount, derived from TnyCamelStoreAccount, 
2128         used for the per-account local outbox folders.
2129         
2130         * src/modest-defs.h: Rename MODEST_LOCAL_FOLDERS_ACCOUNT_ID to 
2131         MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID, to make it clearer that we 
2132         only use this (internally) now for the non-outbox local folders.
2133
2134         * src/modest-init.h:
2135         * src/modest-init.c: Added modest_init_one_local_folder(), for 
2136         setting up the directory structure for each local folder.
2137
2138         * src/modest-local-folder-info.c:
2139         (modest_per_account_local_outbox_folder_info_get_maildir_path),
2140         (modest_per_account_local_outbox_folder_info_get_maildir_path_to_ou
2141         tbox_folder):
2142         * src/modest-local-folder-info.h:
2143         * src/modest-tny-account-store.c: (on_account_removed),
2144         (on_account_changed), (create_per_account_local_outbox_folders),
2145         (get_server_accounts), (modest_tny_account_store_get_accounts),
2146         (modest_tny_account_store_get_tny_account_by_account),
2147         (modest_tny_folder_store_is_virtual_local_folders):
2148
2149         * src/modest-tny-account.h:
2150         * src/modest-tny-account.c:
2151         Added modest_tny_account_new_for_per_account_local_outbox_folder(), 
2152         to create an on-disk per-account local outbox folder for each 
2153         transport account.
2154         (create_per_account_local_outbox_folders): Create a local outbox folder 
2155         for each transport account.
2156         (modest_tny_account_new_for_local_folders): Do not create an outbox folder 
2157         for all local folders.
2158         
2159         Added modest_tny_account_get/set_parent_modest_account_name_for_server_account() 
2160         instead of using g_object_get/set() directly. It is clearer.
2161         
2162         (modest_tny_folder_store_get_message_count),
2163         (modest_tny_folder_store_get_local_size
2164         (modest_tny_folder_store_get_folder_count): Renamed from modest_tny_account*, 
2165         because we now use these with non-acount folder stores.
2166         
2167         (modest_tny_account_get_special_folder),
2168         (modest_tny_account_new_from_server_account),
2169         (modest_tny_account_new_from_account):
2170         (recurse_folders): Adjust to the new outbox system.
2171         
2172         * src/modest-tny-folder.h:
2173         * src/modest-tny-folder.c: 
2174         (modest_tny_folder_is_local_folder),
2175         (modest_tny_folder_get_local_folder_type),
2176         (modest_tny_folder_is_outbox_for_account): Adjust for the new outbox system.
2177         
2178         * src/widgets/modest-main-window.h:
2179         * src/maemo/modest-main-window.c: (create_details_widget),
2180         (modest_main_window_set_contents_style): Show details for any folder store, 
2181         not just accounts.
2182         
2183         (modest_tny_folder_get_rules): 
2184         Remove the const. C does not support constness enough for this.
2185         
2186         * src/modest-ui-actions.c:
2187         update_model(): Use a ModestTnySimpleListStore and a TnyMergeFolder to 
2188         make all outboxes appear as one, in the usual local-folders tree node.
2189         
2190         (set_active_account_from_tny_account),
2191         (modest_ui_actions_on_folder_selection_changed):
2192         * src/widgets/modest-folder-view.c: (text_cell_data),
2193         (icon_cell_data), (filter_row),
2194         (add_account_folders_to_merged_folder),
2195         (add_account_folders_to_simple_folder_store), (),
2196         (get_cmp_rows_type_pos), (cmp_rows),
2197         (modest_folder_view_set_account_id_of_visible_server_account):
2198         Adjust to the new outbox system, using generic folder stores and folders 
2199         in the GtkTreeModel.
2200
2201 2007-05-23  Christian Kellner  <ckellner@openismus.com>
2202
2203         * src/modest-mail-operation.c:
2204         (modest_mail_operation_update_account):
2205         Use info->max_size instead of unintialized local var [small
2206         build fix]. (Added TODO to review that change since I don't
2207         know the code)
2208
2209 2007-05-23  Marcus Bauer  <marcusb@openismus.com>
2210
2211         * libmodest-dbus-client/libmodest-dbus-client.c:
2212         (libmodest_dbus_client_compose_mail): function added
2213         * src/dbus_api/modest-dbus-api.h:
2214         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_compose_mail),
2215         (on_compose_mail): Implementation of the SendAsMail functionality
2216
2217 2007-05-22 Murray Cumming  <murrayc@murrayc.com>
2218
2219         * src/modest-account-mgr.c:
2220         (modest_account_mgr_add_server_account): Set the secure-connection 
2221         conf key for transport accounts as well as store accounts. I don't know 
2222         why the if was there. For instance, this makes sure that we use SSL 
2223         for the gmail preset from the easysetup wizard.
2224
2225 2007-05-22 Murray Cumming  <murrayc@murrayc.com>
2226
2227         * src/gnome/modest-main-window.c:
2228         (on_account_store_connecting_finished), (connect_signals):
2229         * src/maemo/modest-main-window.c:
2230         (on_account_store_connecting_finished), (connect_signals),
2231         (modest_main_window_new):
2232         Handle TnyAccountStore::connecting-finished instead of 
2233         TnyDevice::connection-changed, because we cannot actually perform 
2234         operations on accounts until TnyAccountStore has itself responded to 
2235         the TnyDevice::connection-changed signal. This prevents a GError from 
2236         tinymail.
2237         Do not try to update at startup, because we (at least on scratchbox) 
2238         already do an update at startup when the device goes online. 
2239         This prevents a GError from tinymail.
2240         * src/modest-ui-actions.c: (modest_ui_actions_do_send_receive): 
2241         Added TODO about stopping simultaneous operations.
2242
2243 2007-05-21  Murray Cumming  <murrayc@murrayc.com>
2244
2245         * src/modest-account-mgr.c: (modest_account_mgr_add_account):
2246         * src/modest-mail-operation-queue.c:
2247         (modest_mail_operation_queue_remove):
2248         * src/modest-tny-account-store.c: When GErrors happen, give more clues 
2249         about where they happened.
2250
2251 2007-05-21  Murray Cumming  <murrayc@murrayc.com>
2252
2253         * src/modest-ui-actions.c:
2254         (modest_ui_actions_on_password_requested): Make sure that the remember 
2255         output parameter is always TRUE.
2256
2257 2007-05-21  Murray Cumming  <murrayc@murrayc.com>
2258
2259         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
2260         Show a OK/Cancel dialog if the error is a question, though we still do not 
2261         know anything more about what we are asking the user.
2262         * src/widgets/modest-folder-view.c: (on_drag_data_received):
2263         Fix an uninitialized variable (helper). This is a logic error, however.
2264
2265 2007-05-21  Christian Kellner  <ckellner@openismus.com>
2266         
2267         * src/modest-search.h:
2268         Correct bit shifting in the ModestSearchFlags enum.
2269
2270 2007-05-21  Murray Cumming  <murrayc@murrayc.com>
2271
2272         * src/modest-tny-account.c:
2273         (modest_tny_account_new_from_server_account): Do not use the lsub and 
2274         check-all hard-coded IMAP options when using POP.
2275
2276 2007-05-21  Murray Cumming  <murrayc@murrayc.com>
2277
2278         * src/modest-tny-account.c:
2279         (modest_tny_account_new_from_server_account): 
2280         Use NULL for the secure-auth method for POP when Password auth has been 
2281         specified by the user, as we already do for IMAP. "auth=Login" causes 
2282         an error from camel or the server.
2283
2284 2007-05-17  Murray Cumming  <murrayc@murrayc.com>
2285
2286         * src/maemo/modest-account-settings-dialog.c:
2287         (create_page_account_details),
2288         (modest_account_settings_dialog_set_account_name):
2289         * src/maemo/modest-account-settings-dialog.h: Hide the whole caption 
2290         for the leave-on-server checkbox, rathr than just the checkbox, when 
2291         the protocol is not POP.
2292
2293 2007-05-17  Murray Cumming  <murrayc@murrayc.com>
2294
2295         * src/widgets/modest-account-view.c:
2296         * src/widgets/modest-account-view.h:
2297         Added modest_account_view_select_account() which will be useful later 
2298         to asynchronously re-select the edited account.
2299
2300 2007-05-17  Murray Cumming  <murrayc@murrayc.com>
2301
2302         * src/maemo/modest-account-settings-dialog.c:
2303         (modest_account_settings_dialog_set_account_name): 
2304         Automatically choose the port numbers based on the secure authentication 
2305         methods, if they have not been chosen before.
2306         This fixes projects.maemo.org bug NB#56273 .
2307
2308 2007-05-17  Murray Cumming  <murrayc@murrayc.com>
2309
2310         * src/maemo/modest-main-window-ui.h:
2311         * src/modest-ui-actions.c:
2312         * src/modest-ui-actions.h:
2313         Connect the Tools/SMTP Servers menu item to 
2314         modest_ui_actions_on_smtp_servers(), which shows the connection-specific 
2315         SMTP servers dialog, as in the UI specification and saves changes when it 
2316         is closed. This fixed  projects.maemo.org bug NB#56305 .
2317
2318 2007-05-17  Murray Cumming  <murrayc@murrayc.com>
2319
2320         * src/maemo/modest-connection-specific-smtp-window.c:
2321         (modest_connection_specific_smtp_window_init):
2322         Use gtk_window_set_default_size() to make this window big enough, so 
2323         we see more of the GtkTreeView. This fixes projects.maemo.org bug NB#55497 .
2324
2325 2007-05-17  Murray Cumming  <murrayc@murrayc.com>
2326
2327         * src/maemo/modest-connection-specific-smtp-edit-window.h:
2328         * src/maemo/modest-connection-specific-smtp-edit-window.c:
2329         (modest_connection_specific_smtp_edit_window_init),
2330         Do not try to connect to a combobox signal before the combobox exists.
2331         This fixes the cras in projects.maemo.org bug NB#56288 .
2332         (modest_connection_specific_smtp_edit_window_get_settings):
2333         Remove the server_account_name parameter which was being checked for NULL 
2334         though it was not used.
2335
2336         * src/maemo/modest-connection-specific-smtp-window.c:
2337         (modest_connection_specific_smtp_window_fill_with_connections),
2338         (on_button_edit), (modest_connection_specific_smtp_window_init):
2339         Correct the number of treemodel columns to prevent a warning.
2340         Added DEBUG_WITHOUT_LIBCONIC #define, for debugging on scratchbox, where 
2341         libconic does not work.
2342
2343 2007-05-16  Murray Cumming  <murrayc@murrayc.com>
2344
2345         * src/modest-tny-account.c:
2346         (modest_tny_account_new_from_server_account):
2347         Explicitly use ANONYMOUS secure authentication, instead of "PLAIN" for 
2348         SMTP when "None" was chosen by the user. However, with my SMTP server that 
2349         does not support ANONYMOUS, the emails do not leave the Outbox and I see 
2350         no error dialog.
2351
2352 2007-05-16  Murray Cumming  <murrayc@murrayc.com>
2353
2354         * src/modest-pair.h: Improve the documentation for modest_combo_box_new() 
2355         to explain the memory management issue with the ID in the ModestPair.
2356         
2357         * src/gnome/modest-account-assistant.c: (add_receiving_page),
2358         (add_sending_page), (modest_account_assistant_finalize):
2359         * src/gnome/modest-gnome-global-settings-dialog.c:
2360         (create_updating_page), (create_composing_page):
2361         * src/gnome/modest-msg-edit-window.c: (get_transports),
2362         (init_window), (modest_msg_edit_window_finalize),
2363         (modest_msg_edit_window_get_msg_data):
2364         * src/gnome/modest-store-widget.c: (imap_pop_configuration):
2365         * src/gnome/modest-transport-widget.c:
2366         (modest_transport_widget_finalize), (smtp_configuration):
2367         * src/maemo/modest-maemo-global-settings-dialog.c:
2368         (create_updating_page), (create_composing_page):
2369         * src/maemo/modest-msg-edit-window.c: (get_transports),
2370         (init_window), (modest_msg_edit_window_finalize),
2371         (modest_msg_edit_window_get_msg_data),
2372         (modest_msg_edit_window_free_msg_data):
2373         * src/maemo/modest-store-widget.c: (imap_pop_configuration),
2374         (modest_store_widget_finalize):
2375         * src/maemo/modest-transport-widget.c:
2376         (modest_transport_widget_finalize), (smtp_configuration):
2377         * src/modest-main.c: (send_mail):
2378         * src/widgets/modest-combo-box.h:
2379         * src/widgets/modest-global-settings-dialog-priv.h:
2380         * src/widgets/modest-global-settings-dialog.c:
2381         (modest_global_settings_dialog_finalize):
2382         * src/widgets/modest-msg-edit-window.h:
2383         Whenever using modest_combo_box_new(), keep the ModestPairList alive
2384         as long as the combo box, to prevent use of freed memory for the ID.
2385
2386 2007-05-15  Murray Cumming  <murrayc@murrayc.com>
2387
2388         * src/modest-tny-account-store.c: (get_server_accounts): Renamed from 
2389         get_accounts(), to make this clearer. Do not check for the enabled gconf
2390         key, because we do not use this for server accounts (at least, not yet).
2391         This allows us to send email again.
2392
2393 2007-05-15  Murray Cumming  <murrayc@murrayc.com>
2394
2395         * src/maemo/modest-platform.c:
2396         (on_modest_conf_update_interval_changed), (modest_platform_init):
2397         Only respond to this particular key change. All these strcmp()s cannot be 
2398         efficient.
2399
2400 2007-05-15  Murray Cumming  <murrayc@murrayc.com>
2401
2402         * src/maemo/modest-maemo-global-settings-dialog.c:
2403         (current_connection): Prevent a crash in scratchbox when 
2404         ny_maemo_conic_device_get_current_iap_id() returns NULL.
2405         * src/maemo/modest-platform.c:
2406         (on_modest_conf_update_interval_changed), (modest_platform_init):
2407         Get the update interval from gconf and reset it when the gconf key changes.
2408
2409 2007-05-15  Murray Cumming  <murrayc@murrayc.com>
2410
2411         * src/modest-protocol-info.c:
2412         * src/modest-protocol-info.h: Adde enum-specific get functions, 
2413         instead of using a ModestProtocolType enum parameter.
2414         
2415         * src/gnome/modest-account-assistant.c:
2416         * src/gnome/modest-store-widget.c:
2417         * src/gnome/modest-store-widget.h:
2418         * src/gnome/modest-transport-widget.c:
2419         * src/gnome/modest-transport-widget.h:
2420         * src/maemo/easysetup/modest-easysetup-servertype-combo-box.c:
2421         * src/maemo/easysetup/modest-easysetup-servertype-combo-box.h:
2422         * src/maemo/easysetup/modest-easysetup-wizard.c:
2423         * src/maemo/modest-account-settings-dialog.c:
2424         * src/maemo/modest-account-settings-dialog.h:
2425         * src/maemo/modest-store-widget.c:
2426         * src/maemo/modest-store-widget.h:
2427         * src/maemo/modest-transport-widget.c:
2428         * src/maemo/modest-transport-widget.h:
2429         * src/modest-account-mgr-helpers.c:
2430         * src/modest-account-mgr-helpers.h:
2431         * src/modest-account-mgr.c:
2432         * src/modest-account-mgr.h:
2433         * src/modest-tny-account.c:
2434         * src/modest-tny-folder.c: 
2435         * src/widgets/modest-account-view.c:
2436         * src/widgets/modest-retrieve-combo-box.c:
2437         * src/widgets/modest-retrieve-combo-box.h:
2438         * src/widgets/modest-secureauth-combo-box.c:
2439         * src/widgets/modest-serversecurity-combo-box.c:
2440         * src/widgets/modest-serversecurity-combo-box.h:
2441         * tests/check_account-mgr.c: 
2442         Use the specific functions, and use the re-renamed enum values for GNOME too.
2443
2444 2007-05-15  Murray Cumming  <murrayc@murrayc.com>
2445
2446         * src/modest-pair.h: 
2447         * src/modest-pair.c: Added modest_pair_list_find_by_first_as_string(),
2448         so we can get the second based on the first.
2449
2450         * src/maemo/modest-account-settings-dialog.h: 
2451         * src/maemo/modest-account-settings-dialog.c:
2452         Added get_supported_secure_authentication_methods(), though it is mostly 
2453         commented-out for now until the necessary API is committed to tinymail.
2454         (modest_account_settings_dialog_set_account_name),
2455         (save_configuration): Request the supported authentication methods from the 
2456         server, so we can choose a working method.
2457         
2458         * src/modest-tny-account-store.h:
2459         * src/modest-tny-account-store.c:
2460         (modest_tny_account_store_get_session): Rename from 
2461         tny_account_store_get_session().
2462
2463 2007-05-14  Murray Cumming  <murrayc@murrayc.com>
2464
2465         * src/modest-protocol-info.h: Split the ModestProtocol enum 
2466         into ModestProtocol, ModestSecureConnection, and ModestSecureAuthentication.
2467         There was no need to have these unrelated values in one enum.
2468         
2469         * src/: many files: Adapted to changed enum. This makes the code a bit
2470         clearer, and the compiler could catch some errors.
2471
2472 2007-05-14  Murray Cumming  <murrayc@murrayc.com>
2473
2474         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
2475         Show the detailed internal camel error message when an error happens 
2476         during the configuration, so that our testers can give us more clues.
2477         For instance, in projects.maemo.org bug NB#56910 .
2478
2479 2007-05-14  Murray Cumming  <murrayc@murrayc.com>
2480
2481         * src/modest-tny-account.c:
2482         (modest_tny_account_new_from_server_account): Adapt to changed tinymail API: 
2483         change tny_account_set_mech() to tny_account_set_secure_auth_mech().
2484
2485 2007-05-14  Murray Cumming  <murrayc@murrayc.com>
2486
2487         * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
2488         * src/maemo/modest-account-settings-dialog.c:
2489         (modest_account_settings_dialog_set_account_name),
2490         (save_configuration): For the incoming secure authentication checkbox, 
2491         use PASSWORD for unchecked, and CRAM-MD5 for checked, after I reread 
2492         the UI specification. PLAIN does not seem to be supported for most IMAP 
2493         servers anyway, and I am not sure what it would mean.
2494         However, we probably need to discover which of the secure-authentication 
2495         mechanisms are supported by the server, instead of hard-coding CRAM-MD5.
2496         * src/modest-account-mgr-helpers.c:
2497         (modest_account_mgr_get_server_account_data):
2498
2499         * src/modest-tny-account.c:
2500         (modest_tny_account_new_from_server_account):
2501         Use tny_account_set_mech() to set secure-authentication methods, 
2502         with some special-casing for IMAP, based on my observations of how 
2503         evolution behaves.
2504         
2505         * src/modest-account-mgr-helpers.h:
2506         * src/modest-account-mgr.c:
2507         * src/modest-defs.h: Comment that the URI is only used for local folders.
2508
2509 2007-05-11  Murray Cumming  <murrayc@murrayc.com>
2510
2511         * src/maemo/modest-main-window.c: (on_account_update): Make the gchar* 
2512         parameter const, just to be neat.
2513         * src/modest-text-utils.c: (modest_text_utils_inline):
2514         End g_strconcat() with NULL, as its documentation says. This is almost 
2515         certainly the cause of my crash at startup on the device. Interestingly, 
2516         valgrind doesn't tell us about these errors.
2517
2518 2007-05-11  Murray Cumming  <murrayc@murrayc.com>
2519
2520         * scripts/build-packages: Added more precise comments about how to use 
2521         this script, though I still get this error:
2522         dpkg-genchanges: error: cannot open .dsc file ../tinymail_1.0-svn1938.dsc: No such file or directory
2523         error building tinymail
2524         
2525 2007-05-11  Murray Cumming  <murrayc@murrayc.com>
2526
2527         * src/maemo/modest-main-window.c: (on_account_update):
2528         Very slight cleanup, hoping to make it simpler to discover what causes 
2529         a crash here sometimes.
2530         
2531         * src/modest-tny-account.c:
2532         (modest_tny_account_new_from_server_account): Set the port. Added 
2533         comments about the options and a TODO comment because I need to find out 
2534         how to specify the secure authentication method.
2535
2536 2007-05-11  Marcus Bauer  <marcusb@openismus.com>
2537
2538         * libmodest-dbus-client/libmodest-dbus-client-1.0.pc.in:
2539         changed CFlags paths form modest to libmodest
2540
2541 2007-05-10  Murray Cumming  <murrayc@murrayc.com>
2542
2543         * src/modest-account-mgr-helpers.c:
2544         (modest_account_mgr_get_server_account_data),
2545         (modest_account_mgr_free_server_account_data):
2546         * src/modest-account-mgr-helpers.h:
2547         * src/modest-account-mgr.c:
2548         (modest_account_mgr_add_server_account):
2549         * src/modest-defs.h:
2550         * src/modest-tny-account.c:
2551         (modest_tny_account_new_from_server_account):
2552         Determine the options for tny_camel_account_add_option() here, 
2553         based on the settings, rather than storing them directly in gconf.
2554
2555 2007-05-10  Murray Cumming  <murrayc@murrayc.com>
2556
2557         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
2558         Show a more friendly error dialog now that we expect (from the tinymail 
2559         documentation) to get the UNKNOWN_ALERT error. Add explanatory comments.
2560
2561 2007-05-10  Murray Cumming  <murrayc@murrayc.com>
2562
2563         * src/maemo/modest-main-window.c: (on_account_update): Check that 
2564         account_name is not NULL, to prevent a crash at startup that I am 
2565         seeing on the N800. I do wonder why it is NULL.
2566         
2567         * src/maemo/easysetup/modest-easysetup-wizard.c:
2568         (create_page_user_details):
2569         * src/maemo/modest-account-settings-dialog.c:
2570         (create_page_user_details), (create_page_outgoing):
2571         * src/maemo/modest-connection-specific-smtp-edit-window.c:
2572         (modest_connection_specific_smtp_edit_window_init):
2573         * src/modest-ui-actions.c:
2574         (modest_ui_actions_on_password_requested):
2575         Use HILDON_GTK_INPUT_MODE_INVISIBLE with password entries, which 
2576         might do something useful with the on-screen keyboard.
2577
2578 2007-05-10  Murray Cumming  <murrayc@murrayc.com>
2579
2580         * src/maemo/modest-main-window.c: (on_account_update):
2581         Avoid adding a menu item to priv->accounts_popup if accounts_popup is 
2582         NULL, to avoid lots of warnings. However, it probably should not be NULL.
2583
2584 2007-05-09  Murray Cumming  <murrayc@murrayc.com>
2585
2586         * src/modest-tny-account-store.c: (on_account_changed): 
2587         Correct the signature of this signal handler.
2588
2589 2007-05-09  Murray Cumming  <murrayc@murrayc.com>
2590
2591         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
2592         Show a HildonNote instead of a GtkDialog for Maemo.
2593
2594 2007-05-09  Murray Cumming  <murrayc@murrayc.com>
2595
2596         * src/widgets/modest-account-view.h:
2597         * src/widgets/modest-account-view.c: (on_account_changed),
2598         (on_account_removed). Added a boolean flag, set/unset by 
2599         modest_account_view_block_conf_updates(), 
2600         modest_account_view_unblock_conf_updates() to prevent unnecessary 
2601         updates, but this is not very useful because the gconf notifications are 
2602         so delays (maybe only on Maemo Bora).
2603         So auto-updating is turned off, and these functions do an explicit 
2604         update when necessary.
2605         However, something else is still doing too much work when adding/removing 
2606         accounts, probably in another part of the application.
2607         
2608         * src/maemo/modest-account-view-window.c:
2609         (on_delete_button_clicked), (on_edit_button_clicked),
2610         (on_new_button_clicked): Use the new functions.
2611
2612 2007-05-09  Murray Cumming  <murrayc@murrayc.com>
2613
2614         * src/modest-account-mgr-helpers.c:
2615         (modest_account_mgr_set_first_account_as_default):
2616         Sort the list of names alphabetically-by-title, so we choose the first one 
2617         based on that.
2618         * src/widgets/modest-account-view.c: (init_view): Sort the TreeModel 
2619         alphabetically by the title.
2620         
2621         This fixes projects.maemo.org bug NB#56418 .
2622         
2623         * src/modest-account-mgr-priv.c:
2624         (_modest_account_mgr_account_from_key):
2625         * src/modest-account-mgr.c: (on_key_change): Initialize variables to 
2626         avoid a valgrind error, and possible random behaviour.
2627
2628 2007-05-09  Murray Cumming  <murrayc@murrayc.com>
2629
2630         * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
2631         Handle the new TNY_ACCOUNT_STORE_ERROR error domain. I will update the 
2632         tinymail documentation to say that this can be expected.
2633         Handle the new TNY_ACCOUNT_STORE_ERROR_UNKNOWN_ALERT error code. It is not 
2634         pretty, and should not happen, but I would prefer to know when it does.
2635
2636 2007-05-09  Murray Cumming  <murrayc@murrayc.com>
2637
2638         * configure.ac: When using the older hildon version, probably on Bora, 
2639         require gnome-vfs-module-2.0, which is the old provider of gnome-vfs-mime.h,
2640         to fix the build.
2641         * src/widgets/modest-attachment-view.c:
2642         (modest_attachment_view_set_part_default): Initialize variables, to fix the 
2643         build.
2644
2645 2007-05-09  Marcus Bauer  <marcusb@openismus.com>
2646
2647         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_open_message):
2648         Implemented the prototype. Can be tested with
2649         tests/dbus_api/test_open_message: 
2650
2651 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2652
2653         * src/modest-tny-account-store.c:
2654         (modest_tny_account_store_instance_init), (get_password):
2655         * src/modest-ui-actions.c:
2656         (modest_ui_actions_on_password_requested):
2657         Do not store the password in gconf, because this function is called 
2658         for non-remembered passwords. Actually use the cached (in the hash map) 
2659         password instead of releasing it.
2660         The password will now not be remembered if it is asked via the protected 
2661         password dialog, as per the UI spec.
2662
2663 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2664
2665         * src/modest-defs.h:
2666         * src/modest-account-mgr-helpers.h:
2667         * src/modest-account-mgr-helpers.c:
2668         Added modest_server_account_get_username_has_succeeded(),
2669         modest_server_account_set_username_has_succeeded().
2670         (modest_server_account_set_username),
2671         Reset the has-succeeded flag if the username changes.
2672         
2673         * src/maemo/modest-account-settings-dialog.c: (save_configuration):
2674         * src/maemo/modest-connection-specific-smtp-window.c:
2675         (modest_connection_specific_smtp_window_save_server_accounts):
2676         Use modest_server_account_set_username() instead of using 
2677         modest_conf_set_string() directly, so that the has-succeeded flag is always 
2678         reset.
2679         
2680         * src/modest-ui-actions.c:
2681         (modest_ui_actions_on_password_requested): Dim the username entry if 
2682         the username has ever worked, as per the UI spec.
2683
2684 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2685
2686         * src/modest-tny-account-store.c: (get_password):
2687         * src/modest-ui-actions.c:
2688         (modest_ui_actions_on_password_requested):
2689         Allow the username to be changed too, though there is more work to 
2690         be done on exactly how this behaves.
2691
2692 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2693
2694         * src/modest-account-mgr-helpers.h:
2695         * src/modest-account-mgr-helpers.c:
2696         Added modest_account_mgr_get_display_name(),
2697         modest_server_account_get_username(),
2698         modest_server_account_set_username(),
2699         modest_server_account_set_password(),
2700         modest_server_account_get_hostname() functions so we do not need to 
2701         use the conf API directly.
2702         
2703         * src/maemo/modest-main-window.c: (connect_signals):
2704         Actually connect ot the ModestAccountStore::request-password signal, 
2705         so that we show the dialog when the password is requested by Tinymail, 
2706         for isntance if it is empty.
2707
2708         * src/modest-marshal.list:
2709         * src/modest-tny-account-store.c:
2710         (modest_tny_account_store_class_init), (get_password),
2711         (modest_tny_account_store_alert):
2712         * src/modest-tny-account-store.h:
2713         * src/modest-ui-actions.h:
2714         * src/modest-ui-actions.c:
2715         (modest_ui_actions_on_password_requested):
2716         Change the signal parameters, so it is obvious that we are providing 
2717         the non human-readable server name, and receiving both the username 
2718         and password, though changing of the username is not yet implemented.
2719         Change the dialog UI to match the Maemo UI specifications, with #idfefing 
2720         for the GNOME version.
2721         
2722         This should fix the projects.maemo.org bug NB#56209, though it does not 
2723         work on Bora, because the gconf_client_get() for the password fails 
2724         immediately after we save it with gconf_client_set().
2725
2726 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2727
2728         * configure.ac:
2729         * src/Makefile.am: Reverted the use of AM_CONDITIONAL(), because it 
2730         caused an undefined symbol error:
2731         modest-platform.c:84: undefined reference to `modest_osso_cb_hw_state_handler
2732
2733 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2734
2735         * src/modest-tny-account-store.c:
2736         (modest_tny_account_store_alert): Adapt to the changed tinymail API, 
2737         using a GError instead of a string message, so we can translate it in 
2738         Modest.
2739
2740 2007-05-08  Murray Cumming  <murrayc@murrayc.com>
2741
2742         * src/maemo/modest-address-book.c:
2743         (modest_address_book_check_names):
2744         * src/maemo/modest-msg-view-window.c:
2745         (modest_msg_view_window_clipboard_owner_change):
2746         * src/modest-init.c: (modest_init_init_ui):
2747         * src/modest-mail-operation.c: (modest_mail_operation_send_mail),
2748         (update_folders_cb), (modest_mail_operation_update_account):
2749         * src/modest-ui-actions.c: (modest_ui_actions_on_msg_link_hover),
2750         (modest_ui_actions_on_msg_attachment_clicked),
2751         (modest_ui_actions_on_msg_recpt_activated):
2752         * src/widgets/modest-header-view.c: (drag_data_get_cb):
2753         * src/widgets/modest-msg-view.c: (modest_msg_view_search),
2754         (modest_msg_view_search_next):
2755         * src/widgets/modest-recpt-editor.c:
2756         (modest_recpt_editor_on_button_release_event),
2757         (modest_recpt_editor_on_key_press_event): Commented-out unnecessary calls to 
2758         g_message(). At this point we should only be seeing interesting errors on 
2759         stdout. Debugging messages could be #ifdefed-out if they are still useful.
2760
2761 2007-05-07  Murray Cumming  <murrayc@murrayc.com>
2762         
2763         (modest_wizard_dialog_force_title_update):
2764         * src/maemo/easysetup/modest-wizard-dialog.h:
2765         * src/maemo/easysetup/modest-wizard-dialog.c: (set_property):
2766         For GTK+ 2.10, when present, connect to the GtkNotebook signals so we 
2767         can update the title when appropriate. Otherwise the title is wrong if the 
2768         pages are added after adding the notebook to the dialog.
2769         This must be a problem in HildonWizardDialog too.
2770         Added modest_wizard_dialog_force_title_update() for GTK+ 2.6.
2771         
2772         * src/maemo/easysetup/modest-easysetup-wizard.c:
2773         (create_subsequent_customsetup_pages),
2774         (create_subsequent_easysetup_pages): 
2775         Call modest_wizard_dialog_force_title_update() so that the title is 
2776         correct even with GTK+ < 2.10.
2777         This fixes projects.maemo.org bug NB#56145 .
2778         
2779 2007-05-07  Murray Cumming  <murrayc@murrayc.com>
2780
2781         * src/maemo/modest-platform.c:
2782         (modest_platform_set_update_interval): Use the ALARM_EVENT_NO_DIALOG 
2783         flag (awful lack of namespacing in this API) to prevent a useless %s dialog 
2784         from being shown.
2785
2786 2007-05-07  Murray Cumming  <murrayc@murrayc.com>
2787
2788         * src/widgets/modest-window.h:  
2789         * src/widgets/modest-window.c: Added a save_state_func vfunc, 
2790         and modest_window_save_state(), which invokes it.
2791
2792         * src/widgets/modest-window-mgr.h:
2793         * src/widgets/modest-window-mgr.c:
2794         Added modest_window_mgr_save_state_for_all_windows(), which calls 
2795         the save_state vfunc on all registered windows, if implemented.
2796         
2797         * src/gnome/modest-msg-edit-window.c:
2798         * src/gnome/modest-msg-view-window.c:
2799         * src/maemo/modest-msg-edit-window.c:
2800         * src/maemo/modest-msg-view-window.c:
2801         * src/maemo/modest-main-window.c: 
2802         Specify the existing save_settings functions as implementations of 
2803         ModestWindow::save_state_func().
2804
2805         * src/maemo/modest-osso-state-saving.c: modest_osso_save_state():
2806         Call modest_window_mgr_save_state_for_all_windows(). This happens before 
2807         enabling hibernation, for instance.
2808         
2809         We may need to do the same thing for restoring settings.
2810
2811
2812 2007-05-07  Murray Cumming  <murrayc@murrayc.com>
2813
2814         * src/widgets/modest-window-mgr.c: 
2815         * src/widgets/modest-window-mgr.h:
2816         Added modest_window_mgr_prevent_hibernation_while_window_is_shown() and
2817         modest_window_mgr_get_hibernation_is_prevented().
2818         
2819         * src/maemo/modest-main-window.c:
2820         (on_hildon_program_is_topmost_notify): Prevent hibernation 
2821         (possible when the application goes to the background in the WM), 
2822         if modest_window_mgr_get_hibernation_is_prevented() is TRUE.
2823         
2824         * src/maemo/easysetup/modest-easysetup-wizard.c:
2825         (modest_easysetup_wizard_dialog_init):
2826         * src/maemo/modest-account-settings-dialog.c:
2827         (modest_account_settings_dialog_init):
2828         * src/maemo/modest-connection-specific-smtp-edit-window.c:
2829         (modest_connection_specific_smtp_edit_window_init):
2830         * src/maemo/modest-connection-specific-smtp-window.c:
2831         (modest_connection_specific_smtp_window_init):
2832         * src/maemo/modest-signature-editor-dialog.c:
2833         (modest_signature_editor_dialog_init):
2834         Call modest_window_mgr_prevent_hibernation_while_window_is_shown() to 
2835         prevent hibernation while any of these account settings dialogs are open.
2836
2837 2007-05-04  Murray Cumming  <murrayc@murrayc.com>
2838
2839         * src/maemo/Makefile.am:
2840         * src/maemo/modest-osso-state-saving.c: 
2841         * src/maemo/modest-osso-state-saving.h: Added 
2842         modest_osso_load_state() and modest_osso_save_state() though they do 
2843         nothing now. They should iterate through all the open windows and load/save 
2844         their state.
2845         * src/modest-ui-actions.c: (modest_ui_actions_on_quit): For Mameo, 
2846         save state when closing.
2847         
2848         * src/maemo/modest-main-window.c:
2849         (on_hildon_program_is_topmost_notify), (modest_main_window_new):
2850         Add a notification handler for the HildonProgram::is-topmost property, 
2851         so we can allow hibernation when the application goes into the background.
2852         But we still need to detect when the accounts setup windows are open so 
2853         we can stop hibernation when they are open.
2854         To do this, I also added the main window the HildonProgram. This might have 
2855         other side-effects/benefits, and should maybe be done for other windows.
2856
2857         * src/maemo/modest-osso-autosave-callbacks.h:
2858         * src/maemo/modest-osso-autosave-callbacks.c:
2859         (modest_on_osso_application_autosave): Add a libosso auto-save callback, 
2860         though it does not do anything yet, and we might never need this to do 
2861         anything. For instance, we already save drafts.
2862         
2863         * src/maemo/modest-platform.c: (modest_platform_init):
2864         Specify the libosso auto-save callback.
2865
2866 2007-05-03  Murray Cumming  <murrayc@murrayc.com>
2867
2868         * src/maemo/modest-platform.c:
2869         (modest_platform_set_update_interval):
2870         * src/modest-defs.h: Store the alarmd cookie ID in gconf, because it 
2871         aparently stays valid between application instances, so we can use this 
2872         to remove and reset it later. According to a maemo-developers email from 
2873         David Weinehall.
2874
2875 2007-05-03  Murray Cumming  <murrayc@murrayc.com>
2876
2877         * configure.ac: Depend on libalarm for Maemo.
2878
2879         * src/dbus_api/modest-dbus-api.h:
2880         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_send_receive),
2881         (on_send_receive), (modest_dbus_req_handler):
2882         Handle a SEND_RECEIVE D-Bus method.
2883         
2884         * libmodest-dbus-client/libmodest-dbus-client.c:
2885         (libmodest_dbus_client_send_and_receive):
2886         * libmodest-dbus-client/libmodest-dbus-client.h:
2887         Add C convenience functions for calling the new D-Bus method.
2888         
2889         * src/modest-platform.h:
2890         * src/gnome/modest-platform.c:
2891         * src/maemo/modest-platform.c: 
2892         Added modest_platform_set_update_interval(), which uses the 
2893         alarmd API on Maemo.
2894         (modest_platform_init),
2895         Call modest_platform_set_update_interval(), using a hard-coded 
2896         interval for now (until the global settings dialog is implemented), 
2897         though I am not sure that this makes sense yet.
2898
2899 2007-05-02  Murray Cumming  <murrayc@murrayc.com>
2900
2901         * src/dbus_api/modest-dbus-callbacks.c:
2902         (modest_osso_cb_hw_state_handler):
2903         * src/dbus_api/modest-dbus-callbacks.h:
2904         * src/maemo/modest-platform.c: (modest_platform_init):
2905         Registered (empty) callbacks for the osso hardware state D-Bus signals, 
2906         in case this has some effect on the ability for Maemo to ping the 
2907         application.
2908
2909 2007-05-02  Murray Cumming  <murrayc@murrayc.com>
2910
2911         * src/modest-text-utils.c: (modest_text_utils_validate_recipient):
2912         Comment out an if() that checks a gchar for < 0, causing a build-breaking 
2913         warning. It is probably a logic error, but I need to fix the build.
2914
2915 2007-05-02  Murray Cumming  <murrayc@murrayc.com>
2916
2917         * src/modest-account-mgr.h:
2918         * src/modest-account-mgr.c: 
2919         (modest_account_mgr_account_names): Add a gboolean enabled_only 
2920         parameter, so we can get lists of onlt the enabled accounts.
2921         
2922         (modest_account_mgr_add_account),,
2923         (modest_account_mgr_account_with_display_name_exists):
2924         * src/maemo/modest-msg-edit-window.c: (get_transports):
2925         * src/modest-account-mgr-helpers.c:
2926         (modest_account_mgr_set_first_account_as_default):
2927         * src/modest-init.c: (init_default_account_maybe):
2928         * src/modest-tny-account-store.c: (get_accounts):
2929         * src/modest-ui-actions.c: (modest_ui_actions_on_accounts),
2930         (do_send_receive_auto), (modest_ui_actions_on_send_receive):
2931         * tests/check_account-mgr.c: (START_TEST):
2932         Provide the extra parameter, as appropriate.
2933         
2934         * src/widgets/modest-account-view.c: (update_account_view):
2935         Do not show disabled accounts (meaning that there is no way to 
2936         enable/disable accounts in the UI (the feature is not in our UI 
2937         specification), so we can use this internally only to mark unfinished 
2938         or temporary account data.
2939         
2940         * src/maemo/easysetup/modest-easysetup-wizard.c:
2941         (create_account): Add boolean enable parameter, so we can specify FALSE 
2942         to create the temporary account.
2943         (on_button_edit_advanced_settings): Create the temporary account as disabled.
2944         (on_before_next): When finishing, when there is a temporary account, just 
2945         set it as enabled.
2946         Also, use a timeout to delay the showing of the dialog until gconf is likely 
2947         to return correct information, due to a maemo gconf bug that is fixed in 
2948         osso 1.1, but not yet in Bora.
2949         However, the dialog stays on screen after it is destroyed.
2950         
2951         * src/maemo/modest-account-settings-dialog.c: on_response(): Do not check 
2952         for invalid data when cancelling. Use a hildon note instead of a dialog 
2953         to complain about invalid data. Do not show the account-saved note if the 
2954         account is disabled (a temporary account that will not really be saved for use 
2955         until later.)
2956
2957 2007-04-30  Murray Cumming  <murrayc@murrayc.com>
2958
2959         * src/modest-account-mgr-helpers.h:
2960         * src/modest-account-mgr-helpers.c:
2961         Added modest_account_mgr_unset_default_account(),
2962         Added modest_account_mgr_set_first_account_as_default().
2963
2964         * src/modest-account-mgr.c: (modest_account_mgr_remove_account): 
2965         If it was the default account, unset the default account name.
2966         
2967         * src/maemo/modest-account-view-window.c:
2968         (on_delete_button_clicked): If it was the default account, 
2969         set the first remaining account as the default instead.
2970         
2971         * src/maemo/modest-main-window.c: (on_account_update): Do not try to use 
2972         a NULL account or a NULL default account, to prevent a crash. This should 
2973         not happen now anyway. Bug #55343 in projects.maemo.org/bugzilla.
2974         
2975
2976 2007-04-30  Murray Cumming  <murrayc@murrayc.com>
2977
2978         * src/widgets/modest-recpt-editor.c:
2979         (modest_recpt_editor_on_key_press_event): Put an #ifdef around the 
2980         use of gtk_text_buffer_get_has_selection() (from GTK+ 2.10), and add a 
2981         TODO comment for this. This fixes the build.
2982         
2983         * src/modest-ui-actions.c: Add do_send_receive(), though it might not 
2984         be the best place for it. This will in future update all auto-update accounts, 
2985         rather than just the current/default account, when that gconf key exists.
2986         (modest_ui_actions_on_send_receive): Offer the settings dialog (or wizard) 
2987         if there are no accounts, before calling do_send_receive().
2988         * src/modest-ui-actions.h: Specify the callback for the Send/Recieve menu item.
2989         
2990         * src/gnome/modest-main-window.c: (on_online_toggle_toggled):
2991         * src/maemo/modest-main-window-ui.h:
2992         * src/maemo/modest-main-window.c: (on_connection_changed),
2993         (sync_accounts_cb): Use do_send_receive() instead of calling the signal handler 
2994         directly, becaue the signal handler does more.
2995
2996 2007-04-30  Murray Cumming  <murrayc@murrayc.com>
2997
2998         * src/modest-account-mgr-helpers.c:
2999         (modest_account_mgr_get_account_data): Add a more helpful error message, and comment, 
3000         for the case that the account does not exist.
3001         
3002         * src/widgets/modest-folder-view.c: (filter_row): Use the tinymail account ID 
3003         (equivalent to the modest account name) rather than the tinymail account name 
3004         (equivalent to the modest account title - human readable) to get the account data.
3005         Check for null account data to prevent the crash, and add a TODO comment saying that 
3006         this needs fixing because it is using the server account name instead of the account name.
3007
3008 2007-04-29  Murray Cumming  <murrayc@murrayc.com>
3009
3010         * src/maemo/easysetup/modest-easysetup-wizard.c: (show_error):
3011         * src/maemo/modest-account-settings-dialog.c: (show_error),
3012         (show_ok):
3013         Use hildon_note_new_information() for informative and error messages, 
3014         which seems to be expected for Maemo applications (no documentation that I know of 
3015         says this). I am surprised that there is no distinction between info and error 
3016         dialogs.
3017
3018 2007-04-29  Murray Cumming  <murrayc@murrayc.com>
3019
3020         * src/maemo/modest-maemo-ui-constants.h: Add a MODEST_MARGIN_NONE constant, so 
3021         it is easy to find where this is used.
3022         
3023         * src/maemo/easysetup/modest-easysetup-wizard.c:
3024         (create_page_welcome), (create_page_account_details),
3025         (create_page_user_details), (create_page_complete_easysetup),
3026         (create_page_custom_incoming), (create_page_custom_outgoing),
3027         (create_page_complete_custom):
3028         * src/maemo/modest-account-settings-dialog.c:
3029         (create_page_account_details), (create_page_user_details),
3030         (create_page_incoming), (create_page_outgoing):
3031         * src/maemo/modest-account-view-window.c: (window_vbox_new):
3032         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3033         (modest_connection_specific_smtp_edit_window_init):
3034         * src/maemo/modest-connection-specific-smtp-window.c:
3035         (modest_connection_specific_smtp_window_init):
3036         * src/maemo/modest-signature-editor-dialog.c:
3037         (modest_signature_editor_dialog_init): Use the correct padding/spacing/borders 
3038         and scrolling policy as per the Email Application Layout Guide, if I have 
3039         understood it.
3040
3041 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3042
3043         * src/modest-tny-account-store.c: (modest_tny_account_store_init): 
3044         Remove the add_transport_account_func and add_store_account_func vfunc implementations, 
3045         which were empty, because these vfuncs have been removed from TnyAccountStore.
3046         This fixes the build.
3047
3048 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3049
3050         * src/maemo/modest-main-window.c:
3051         (modest_main_window_show_toolbar): Check that a toolbar item is not NULL before 
3052         showing/hiding it. Added TODO because it probably should not be NULL.
3053         * src/modest-init.c: (modest_init_init_ui): Use a g_message() instead of a g_warning() 
3054         because unnecessary g_warnings() make degugging difficult.
3055
3056 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3057
3058         * src/maemo/modest-msg-edit-window.c:
3059         (modest_msg_edit_window_select_color),
3060         (modest_msg_edit_window_select_background_color): 2 const corrections and 
3061         moving 2 #endifs before closing brackets, to fix the build.
3062
3063 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3064
3065         * src/maemo/easysetup/modest-easysetup-wizard.h:
3066         * src/maemo/easysetup/modest-easysetup-wizard.c:
3067         (modest_easysetup_wizard_dialog_finalize),
3068         (on_button_edit_advanced_settings), (create_page_complete_custom),
3069         (on_response), (create_account):
3070         Implement the Advanced Settings edit button, by saving the account information, 
3071         for the Advanded Settings dialog to use directly from gconf, and removing it 
3072         if Finish is never clicked. There is still some UI strangeness, so this is not finished.
3073         Bug #5533 in the projects.maemo.org bugzilla.
3074
3075 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3076
3077         * src/maemo/modest-msg-view-window.c:
3078         (modest_msg_view_window_show_toolbar): Prevent hide/show of a NULL toolbar widget, 
3079         when opening a message window.
3080
3081 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3082
3083         * src/maemo/easysetup/modest-easysetup-wizard.c:
3084         (create_page_custom_outgoing): Correct the position of the horizontal separator to 
3085         match the UI spec.
3086         
3087         * src/maemo/modest-main-window.c: (set_toolbar_mode): Check that widgets are not NULL 
3088         before showing/hiding them, to avoid a crash when clicking on folders when there are no 
3089         gconf settings. I probably added more checks than necessary, but that is safer.
3090         
3091         * src/modest-ui-actions.c: (modest_ui_actions_on_move_to): Initialize the mail_op 
3092         variable to fix the build, but this seems to be used when it is still NULL.
3093
3094 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3095
3096         * src/modest-platform.h:
3097         * src/gnome/modest-platform.c: (modest_platform_connect_and_wait):
3098         * src/maemo/modest-platform.c: (modest_platform_connect_and_wait):
3099         * src/modest-main.c: Implement this function so we can avoid using 
3100         maemo-specific API from cross-platform code.
3101         (main):
3102         * src/modest-tny-account-store.c:
3103         (get_smtp_specific_transport_account_for_open_connection):
3104         * src/modest-ui-actions.c: (check_for_connection),
3105         (modest_ui_actions_on_item_not_found):
3106         Use modest_platform_connect_and_wait() and put #idefs around other 
3107         maemo-specific code, because only maemo currently has a way to 
3108         identify connection names.
3109
3110 2007-04-27  Murray Cumming  <murrayc@murrayc.com>
3111
3112         * src/modest-tny-account-store.h:
3113         * src/modest-tny-account-store.c:
3114         Added modest_tny_account_store_get_transport_account_for_open_connection(), which respects 
3115         the connection-specific SMTP server settings in the configuration.
3116         
3117         * src/dbus_api/modest-dbus-callbacks.c: (on_idle_send_mail),
3118         (on_idle_mail_to):
3119         * src/maemo/modest-main-window.c: (on_connection_changed),
3120         (connect_signals):
3121         * src/modest-main.c: (main), (start_ui), (send_mail):
3122         Use tny_maemo_conic_device_connect() instead of tny_maemo_conic_force_online(), when 
3123         that is what is intended.
3124         
3125         * src/modest-ui-actions.c: (action_send), (action_receive),
3126         (modest_ui_actions_on_item_not_found), (modest_ui_actions_on_send):
3127         Use modest_tny_account_store_get_transport_account_for_open_connection() instead of 
3128         modest_tny_account_store_get_tny_account_by_account() so that the connection-specific 
3129         SMTP server is used when it is specified.
3130         
3131         (modest_ui_actions_on_send_receive): 
3132         Check that a connection is open before proceeding.
3133         Receive and then send, instead of vice-versa, as per the specification.
3134
3135 2007-04-26  Murray Cumming  <murrayc@murrayc.com>
3136
3137         * src/modest-mail-operation.c:
3138         (modest_mail_operation_remove_folder): Intialize the parent variable, to fix the build, 
3139         though I guess it should be something other than NULL.
3140
3141 2007-04-26  Murray Cumming  <murrayc@murrayc.com>
3142
3143         * src/maemo/easysetup/modest-easysetup-wizard.c:
3144         (create_page_custom_incoming): Add the checkbox to the caption instead of adding the 
3145         combo twice, to avoid an g_warning and to make it visible again.
3146         * src/maemo/modest-main-window.c: (on_account_update): Check whether the popup_menu is 
3147         attached before detaching it, to avoid a g_warning, and add comment about the implicit 
3148         (already intended) dereference when detaching, because we then recreate the menu.
3149         But I am not sure why the warning was happening, and it is possible that this is a memory 
3150         leak.
3151
3152 2007-04-25  Murray Cumming  <murrayc@murrayc.com>
3153
3154         * src/dbus_api/modest-dbus-callbacks.c: Added uri_unescape(),
3155         and uri_parse_mailto() utility functions.
3156         (on_idle_mail_to): Parse, unescape, and use the subject, cc, bcc, and body items in 
3157         the mailto URI.
3158         * tests/dbus_api/test_mail_to.c: (main): Add escaped spaces in the URI to test this.
3159
3160 2007-04-24  Murray Cumming  <murrayc@murrayc.com>
3161
3162         * src/dbus_api/modest-dbus-api.h:
3163         * src/dbus_api/modest-dbus-callbacks.c:
3164         Use an idle callback to execute the modest code in the application's own thread.
3165         Remove the helloworld example method.
3166         Handle mail-to and open-message D-Bus methods, though the mail-to format needs to be 
3167         parsed, and the open-message method is not implemented because this is not yet implemented in 
3168         modest itself.
3169         * libmodest-dbus-client/libmodest-dbus-client.c:
3170         (libmodest_dbus_client_send_mail), (libmodest_dbus_client_mail_to),
3171         (libmodest_dbus_client_open_message):
3172         * libmodest-dbus-client/libmodest-dbus-client.h: New functions for the new methods.
3173         
3174         * src/modest-tny-msg.c: (modest_tny_msg_new), (add_body_part): Handle NULLs for subject and 
3175         body without crashing.
3176         
3177         * tests/dbus_api/Makefile.am:
3178         * tests/dbus_api/test_mail_to.c:
3179         * tests/dbus_api/test_open_message.c: New tests for the new methods.
3180
3181 2007-04-24  Murray Cumming  <murrayc@murrayc.com>
3182
3183         * libmodest-dbus-client/libmodest-dbus-client-1.0.pc.in:
3184         Added file to fix the build. Sorry.
3185         
3186         * src/dbus_api/Makefile.am:
3187         * src/dbus_api/modest-dbus-api.h:
3188         * src/dbus_api/modest-dbus-callbacks.h:
3189         * src/dbus_api/modest-dbus-callbacks.c: (on_send_mail),
3190         (modest_dbus_req_handler):
3191         Put constants in a separate header so it can be reused by the 
3192         client library, without duplication
3193         
3194         * libmodest-dbus-client/Makefile.am:
3195         * libmodest-dbus-client/libmodest-dbus-client.c:
3196         (libmodest_dbus_client_call_helloworld),
3197         (libmodfest_dbus_client_send_mail),
3198         (libmodfest_dbus_client_mailto),
3199         (libmodfest_dbus_client_open_message):
3200         * libmodest-dbus-client/libmodest-dbus-client.h:
3201         (Partly) Implement a send_mail D-Bus method.
3202         
3203         * tests/dbus_api/Makefile.am:
3204         * tests/dbus_api/test_send_mail.c: (main): Added a test for 
3205         the send_email D-Bus method.
3206
3207 2007-04-24  Murray Cumming  <murrayc@murrayc.com>
3208
3209         * Makefile.am:
3210         * configure.ac:
3211         * libmodest-dbus-client/Makefile.am:
3212         * libmodest-dbus-client/libmodest-dbus-client.c:
3213         * libmodest-dbus-client/libmodest-dbus-client.h:
3214         * tests/dbus_api/Makefile.am:
3215         * tests/dbus_api/test_hello.c: (main): Added directory structure for a C convenience library 
3216         for using the modest D-Bus API, and used that from the test. It is still just a helloworld.
3217
3218 2007-04-24  Murray Cumming  <murrayc@murrayc.com>
3219
3220         * src/maemo/easysetup/modest-easysetup-wizard.c:
3221         (create_page_account_details):
3222         * src/maemo/modest-account-settings-dialog.c:
3223         (create_page_account_details): Explicitly set auto-capitalization mode for the title 
3224         widgets, because this might not be the default in future versions of the Maemo GTK+. 
3225         However, I can not not get capitalization to work even in a simple test case.
3226
3227 2007-04-24  Murray Cumming  <murrayc@murrayc.com>
3228
3229         * src/maemo/easysetup/modest-easysetup-wizard.c:
3230         (create_page_custom_incoming), (create_page_custom_outgoing):
3231         * src/maemo/modest-account-settings-dialog.c:
3232         (create_page_account_details), (create_page_incoming),
3233         (create_page_outgoing): Use a separate label in a caption for GtkCheckButtons, 
3234         instead of using gtk_check_button_with_label(), so they are aligned like the other 
3235         widgets, as per the UI spec.
3236
3237 2007-04-24  Murray Cumming  <murrayc@murrayc.com>
3238
3239         * src/maemo/easysetup/modest-easysetup-wizard.c:
3240         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3241         * src/widgets/modest-validating-entry.c:
3242         * src/widgets/modest-validating-entry.h: Rename the validating widget so it has a 
3243         Modest prefix instead of an EasySetup prefix.
3244
3245 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3246
3247         * src/widgets/modest-validating-entry.h:
3248         * src/widgets/modest-validating-entry.c: (on_insert_text),
3249         Added easysetup_validating_entry_set_max_func(), used to set a callback 
3250         to call when the max number of characters is reached. GtkEntry has a max-length 
3251         already but is silent about it.
3252
3253         * src/maemo/easysetup/modest-easysetup-wizard.c: (on_entry_max),
3254         (create_page_account_details), (create_page_user_details),
3255         (create_account):
3256         * src/maemo/modest-account-settings-dialog.c: (on_entry_max),
3257         (create_page_account_details), (create_page_user_details),
3258         (create_page_outgoing):
3259         * src/modest-ui-actions.c: (modest_ui_actions_on_details): 
3260         Use easysetup_validating_entry_set_max_func() to show the warning dialog, as per 
3261         the UI spec, when the user tries to enter more than the max number of characters.
3262
3263 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3264
3265         * src/maemo/easysetup/modest-easysetup-wizard.c:
3266         (create_page_user_details), (create_page_custom_incoming),
3267         (create_page_custom_outgoing):
3268         * src/maemo/modest-account-settings-dialog.c:
3269         (create_page_user_details), (create_page_incoming),
3270         (create_page_outgoing):
3271         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3272         (modest_connection_specific_smtp_edit_window_init):
3273         Use hildon_gtk_entry_set_input_mode() to turn off auto-capitalization where it is 
3274         not appropriate. Bad Maemo-specific defaults!.
3275
3276 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3277
3278         * src/maemo/modest-signature-editor-dialog.c: (enable_widgets),
3279         (on_toggle_button_changed), (modest_signature_editor_dialog_init),
3280         (modest_signature_editor_dialog_set_settings): 
3281         Disable the label and textview when the checkbox is disabled, and use the 
3282         specified (UI spec) default signature text.
3283
3284 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3285
3286         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3287         (modest_connection_specific_smtp_edit_window_init): Use spacing constants.
3288         
3289         * src/modest-defs.h:
3290         * src/modest-account-mgr-helpers.h:
3291         * src/modest-account-mgr-helpers.c:
3292         Added modest_account_mgr_get/set_signature(), using new gconf keys.
3293         
3294         * src/maemo/Makefile.am:
3295         * src/maemo/modest-signature-editor-dialog.c:
3296         * src/maemo/modest-signature-editor-dialog.h: New dialog for editing signatures, 
3297         as per the UI spec.
3298         
3299         * src/maemo/modest-account-settings-dialog.h:
3300         * src/maemo/modest-account-settings-dialog.c:
3301         (modest_account_settings_dialog_finalize), (on_button_signature),
3302         (save_configuration): Use the new dialog when the Edit button is pressed, and 
3303         save the changes if necessary, as per the UI spec.
3304
3305 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3306
3307         * src/maemo/Makefile.am:
3308         * src/maemo/modest-maemo-ui-constants.h: New header with some defines for widget spacing, 
3309         based on maemo-develoers email from Dirk.
3310         
3311         * src/maemo/easysetup/modest-easysetup-wizard.c:
3312         (create_page_welcome), (create_page_account_details),
3313         (create_page_user_details), (create_page_complete_easysetup),
3314         (create_page_custom_incoming), (create_page_custom_outgoing),
3315         (create_page_complete_custom),
3316         (modest_easysetup_wizard_dialog_init):
3317         * src/maemo/modest-account-settings-dialog.c:
3318         (create_page_account_details), (create_page_user_details),
3319         (create_page_incoming), (create_page_outgoing),
3320         (modest_account_settings_dialog_init):
3321         * src/maemo/modest-connection-specific-smtp-window.c:
3322         (modest_connection_specific_smtp_window_init): Used the smallest of the spacing constants.
3323
3324 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3325
3326         * src/maemo/easysetup/modest-easysetup-wizard.c:
3327         (create_page_account_details), (create_page_custom_outgoing),
3328         (create_page_complete_custom):
3329         * src/maemo/modest-account-settings-dialog.c:
3330         (create_page_user_details), (create_page_incoming),
3331         (create_page_outgoing):
3332         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3333         (modest_connection_specific_smtp_edit_window_init): 
3334         Use the correct (UI spec) port min/max, and use horizontal separators, 
3335         as per the UI spec.
3336
3337 2007-04-23  Murray Cumming  <murrayc@murrayc.com>
3338
3339         * src/maemo/easysetup/modest-easysetup-wizard.c:
3340         (set_default_custom_servernames):
3341         * src/maemo/modest-account-settings-dialog.c:
3342         (create_page_incoming), (on_combo_outgoing_security_changed),
3343         (on_combo_incoming_security_changed), (create_page_outgoing),
3344         (modest_account_settings_dialog_set_account_name),
3345         (save_configuration):
3346         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3347         (on_combo_security_changed),
3348         (modest_connection_specific_smtp_edit_window_init),
3349         (modest_connection_specific_smtp_edit_window_set_connection),
3350         (modest_connection_specific_smtp_edit_window_get_settings): 
3351         Use HildonNumberEditor instead of GtkEntry for the port numbers so it has the - and + buttons,
3352         as per the UI spec.
3353
3354 2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3355
3356         * po/en_GB.po:
3357         * src/maemo/easysetup/modest-easysetup-wizard.c:
3358         (create_page_custom_outgoing), (create_page_complete_custom):
3359         * src/maemo/modest-account-settings-dialog.c:
3360         (create_page_user_details), (create_page_outgoing):
3361         * src/maemo/modest-account-view-window.c: (button_box_new):
3362         Use changed logical IDs as per the most recent UI spec:
3363         mcen_bd_emailsetup_edit -> mcen_bd_edit
3364         mcen_bd_emailsetup_delete -> mcen_bd_delete
3365         mcen_bd_emailsetup_close -> mcen_bd_close
3366
3367 2007-04-20  2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3368
3369         * configure.ac:
3370         * src/Makefile.am:
3371         * src/dbus_api/Makefile.am:
3372         * src/dbus_api/modest-dbus-callbacks.c:
3373         * src/dbus_api/modest-dbus-callbacks.h:
3374         * src/maemo/modest-platform.c: (modest_platform_init):
3375         Added the beginnings of a D-Bus API. At the moment it just has a HelloWorld 
3376         method.
3377         * tests/Makefile.am:
3378         * tests/dbus_api/Makefile.am:
3379         * tests/dbus_api/test_hello.c: (main): Very simple test of the D-Bus API.
3380
3381 2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3382
3383         * src/maemo/modest-account-settings-dialog.c:
3384         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3385         Removed unnecessary includes.
3386
3387 2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3388
3389         * src/maemo/easysetup/modest-easysetup-wizard.c:
3390         * src/maemo/modest-account-settings-dialog.c:
3391         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3392         * src/widgets/Makefile.am:
3393         * src/widgets/modest-easysetup-secureauth-combo-box.c:
3394         * src/widgets/modest-easysetup-secureauth-combo-box.h:
3395         * src/widgets/modest-easysetup-serversecurity-combo-box.c:
3396         * src/widgets/modest-easysetup-serversecurity-combo-box.h:
3397         * src/widgets/modest-secureauth-combo-box.c:
3398         * src/widgets/modest-secureauth-combo-box.h:
3399         * src/widgets/modest-serversecurity-combo-box.c:
3400         * src/widgets/modest-serversecurity-combo-box.h: Renamed the files and 
3401         made the GObject names more consistent with the rest of the Modest code.
3402
3403 2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3404
3405         * src/maemo/easysetup/Makefile.am:
3406         * src/maemo/easysetup/modest-easysetup-secureauth-combo-box.c:
3407         * src/maemo/easysetup/modest-easysetup-secureauth-combo-box.h:
3408         * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.c:
3409         * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.h:
3410         * src/maemo/easysetup/modest-easysetup-wizard.c:
3411         * src/maemo/easysetup/modest-validating-entry.c:
3412         * src/maemo/easysetup/modest-validating-entry.h:
3413         * src/maemo/modest-account-settings-dialog.c:
3414         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3415         * src/widgets/Makefile.am: Moved these widgets from maemo/easysetup into 
3416         widgets/ because they are used outside of easysetup too. After checking in, 
3417         svn will allow me to rename them too.
3418
3419 2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3420
3421         * src/gnome/modest-msg-edit-window.c:
3422         (modest_msg_edit_window_set_priority_flags): Added an empty implementation to fix a 
3423         linker error, to fix the build on the GNOME platform.
3424         modest_msg_edit_window_set_zoom(), modest_msg_edit_window_get_zoom(), 
3425         modest_msg_edit_window_zoom_plus(), modest_msg_edit_window_zoom_minus():
3426         ifdefed out to avoid a warning.
3427
3428 2007-04-20  Murray Cumming  <murrayc@murrayc.com>
3429
3430         * src/widgets/modest-header-view.c: (on_focus_in): Use GTK_CHECK_VERSION to #ifdef around the 
3431         use of gtk_tree_view_get_visible_range(), to fix the build on Mameo. This code probably needs 
3432         some alternative for GTK+ 2.6.
3433
3434 2007-04-19  Murray Cumming  <murrayc@murrayc.com>
3435
3436         * src/maemo/modest-address-book.c: Added include to avoid a warning.
3437
3438         * po/en_GB.po: Added missing mcen_bd_emailsetup_close string, 
3439         specified for EmailAccountsDialog in UI spec.
3440         
3441         * src/maemo/easysetup/modest-easysetup-wizard.c:
3442         * src/maemo/easysetup/modest-easysetup-wizard.h:
3443         * src/maemo/modest-account-settings-dialog.c:
3444         * src/maemo/modest-account-settings-dialog.h:
3445         Reuse a member window widget for the connection-specific SMTP server accounts, 
3446         so we can tell it to create/save the accounts only if we actually save changes.
3447
3448         * src/maemo/modest-connection-specific-smtp-edit-window.h:      
3449         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3450         (modest_connection_specific_smtp_edit_window_get_settings): 
3451         Return a struct instance for temporarily remembering the entered data.
3452
3453         * src/maemo/modest-connection-specific-smtp-window.h:
3454         * src/maemo/modest-connection-specific-smtp-window.c:
3455         (modest_connection_specific_smtp_window_finalize),
3456         (modest_connection_specific_smtp_window_fill_with_connections),
3457         (on_button_edit),
3458         (modest_connection_specific_smtp_window_save_server_accounts),
3459         (update_model_server_names): Do not actually save data until asked, 
3460         when the caller calls modest_connection_specific_smtp_window_save_server_accounts().
3461         
3462         (on_selection_changed),
3463         (modest_connection_specific_smtp_window_init): Disable the edit 
3464         button when nothing is selected.
3465         
3466         * src/modest-account-mgr-helpers.c:
3467         * src/modest-account-mgr-helpers.h: 
3468         Put modest_account_mgr_free_server_account_data() in the header now that I use the struct 
3469         elsewhere.
3470
3471 2007-04-19  Murray Cumming  <murrayc@murrayc.com>
3472
3473         * src/modest-account-mgr-helpers.h:
3474         * src/modest-account-mgr-helpers.c:
3475         (modest_account_mgr_get_unused_account_name),
3476         (modest_account_mgr_get_unused_account_display_name): Added these 
3477         utility functions, to avoid code duplication.
3478
3479         * src/maemo/easysetup/modest-easysetup-wizard.c:
3480         (create_page_account_details), (create_account):
3481         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3482         (on_combo_security_changed),
3483         (modest_connection_specific_smtp_edit_window_init),
3484         (modest_connection_specific_smtp_edit_window_save_settings):
3485         * src/maemo/modest-connection-specific-smtp-edit-window.h:
3486         * src/maemo/modest-connection-specific-smtp-window.c:
3487         (modest_connection_specific_smtp_window_finalize),
3488         (modest_connection_specific_smtp_window_fill_with_connections),
3489         (on_button_edit), (modest_connection_specific_smtp_window_init):
3490         * src/maemo/modest-connection-specific-smtp-window.h:
3491         Implementing saving of connection-specific server accounts.
3492         
3493         But I need to refactor this so we can save the data for each 
3494         possible connection-specific account and only actually create the server accounts 
3495         when the easysetup finished, or the OK button is pressed on the account settings 
3496         dialog.
3497
3498 2007-04-18  Murray Cumming  <murrayc@murrayc.com>
3499
3500         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3501         (on_combo_security_changed): 
3502         (modest_connection_specific_smtp_edit_window_init):
3503         Auto-fill the port number as in the account settings.
3504         
3505         (modest_connection_specific_smtp_edit_window_set_connection):
3506         * src/maemo/modest-connection-specific-smtp-edit-window.h:
3507         * src/maemo/modest-connection-specific-smtp-window.c:
3508         (fill_with_connections), (on_button_edit),
3509         (modest_connection_specific_smtp_window_init): 
3510         Turn the edit window into a dialog to simplify the code, and 
3511         set a WM hint as in a GtkDialog, because only a popup can be on top of 
3512         another window in Maemo, and popups usually have no WM decoration.
3513         
3514         * src/modest-defs.h:
3515         * src/modest-account-mgr-helpers.c:
3516         (modest_account_mgr_set_connection_specific_smtp),
3517         (modest_account_mgr_remove_connection_specific_smtp),
3518         (modest_account_mgr_get_connection_specific_smtp):
3519         * src/modest-account-mgr-helpers.h: Some functions to write the 
3520         connection-specific SMTP server information. Not yet used.
3521
3522         
3523         * src/widgets/modest-folder-view.c: (text_cell_data):
3524         Fix a warning about an uninitialized variable.
3525         (on_configuration_key_changed): Put an #ifdef around the use 
3526         of gtk_tree_view_column_queue_resize() because it is new in GTK+ 2.10.
3527
3528 2007-04-18  Murray Cumming  <murrayc@murrayc.com>
3529
3530         * src/maemo/modest-account-settings-dialog.h:
3531         * src/maemo/modest-account-settings-dialog.c:
3532         Added connect_for_modified() utility function to mark the new modified boolean.
3533         
3534         (create_page_account_details), (create_page_user_details),
3535         (create_page_incoming), (create_page_outgoing), (on_response),
3536         (modest_account_settings_dialog_init),
3537         (modest_account_settings_dialog_set_account_name):
3538         Only warn about unsaved changes when there are actually unsaved changes.
3539         
3540         * src/modest-account-mgr-helpers.c:
3541         * src/modest-account-mgr-helpers.h: Put the port, secure-auth, and security information 
3542         into the ModestServerAccountData struct. I do not use it yet, but someone might expect 
3543         to get the data from there in future.
3544
3545 2007-04-18  Murray Cumming  <murrayc@murrayc.com>
3546
3547         * src/modest-defs.h:
3548
3549         * src/modest-account-mgr-helpers.h:
3550         * src/modest-account-mgr-helpers.c:
3551         Store the security account setting in a single conf key instead of using 
3552         the options list, so that the list is left for just the to-be-removed camel hack, 
3553         and because gconf_client_set_list() is failing for some reason.
3554
3555         * src/modest-account-mgr.c:
3556         (modest_account_mgr_add_server_account):
3557         * src/maemo/modest-account-settings-dialog.c:
3558         (modest_account_settings_dialog_set_account_name),
3559         (save_configuration): Use the renamed functions.
3560
3561
3562 2007-04-18  Murray Cumming  <murrayc@murrayc.com>
3563
3564         * src/widgets/Makefile.am:
3565         * src/widgets/modest-limit-retrieve-combo-box.c:
3566         * src/widgets/modest-limit-retrieve-combo-box.h:
3567         * src/widgets/modest-retrieve-combo-box.c:
3568         * src/widgets/modest-retrieve-combo-box.h: Added new widgets for use in 
3569         the account settings dialog.
3570         
3571         * src/modest-defs.h:
3572         * src/modest-account-mgr-helpers.h:
3573         * src/modest-account-mgr-helpers.c:
3574         (modest_server_account_get_option_secure_auth),
3575         (modest_server_account_set_option_secure_auth):
3576         * src/modest-account-mgr.c:
3577         (modest_account_mgr_add_server_account):
3578         Use the existing AUTH_MECH conf key for the secure-auth setting, 
3579         instead of putting it in the list options, because this was actually used already.
3580         
3581         * src/maemo/modest-account-settings-dialog.c:
3582         (create_page_account_details),
3583         (modest_account_settings_dialog_set_account_name),
3584         (save_configuration):
3585         * src/maemo/modest-account-settings-dialog.h:
3586         Add the retrieve and retrieve-limit combo boxes, as per the UI spec.
3587         Hide the leave-messages checkbox for non-POP accounts, as per the UI spec.
3588
3589 2007-04-18  Murray Cumming  <murrayc@murrayc.com>
3590
3591         * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.h:
3592         * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.c:
3593         Added easysetup_serversecurity_combo_box_get_active_serversecurity_port()
3594
3595         * src/modest-defs.h:
3596         * src/maemo/modest-account-settings-dialog.c:
3597         (create_page_incoming), (on_combo_outgoing_security_changed),
3598         (on_combo_incoming_security_changed), (create_page_outgoing),
3599         (modest_account_settings_dialog_set_account_name),
3600         (save_configuration): Load/Save the port number, and change it when the 
3601         security combo changes, to suitable defaults.
3602
3603 2007-04-17  Murray Cumming  <murrayc@murrayc.com>
3604
3605         * src/modest-account-mgr.c:
3606         (modest_account_mgr_add_server_account): Set the security and secure-auth 
3607         options, and add a comment saying that the other camel-specific options should be 
3608         removed in future.
3609
3610 2007-04-17  Murray Cumming  <murrayc@murrayc.com>
3611
3612         * src/maemo/modest-account-settings-dialog.c: (check_data): Always return something, 
3613         found by valgrind.
3614         * src/modest-tny-account-store.c: (account_list_free),
3615         (on_account_changed): Add a TODO comment about a valgrind error, caused by using 
3616         cursor->data after unreffing it. But there is some strange stuff there so it might not be 
3617         easy to fix.
3618
3619 2007-04-17  Murray Cumming  <murrayc@murrayc.com>
3620
3621         * src/modest-protocol-info.c:
3622         * src/modest-protocol-info.h: Added a MODEST_PROTOCOL_AUTH_CRAMMD5 enum value, because it 
3623         is required by our UI spec. Maybe something in our implementation will use this eventually.
3624         * src/maemo/easysetup/modest-easysetup-secureauth-combo-box.c: 
3625         (easysetup_secureauth_combo_box_fill): Use the CRAM MD5 enum value 
3626         instead of the previous 0 placeholder.
3627         
3628         * src/modest-defs.h: Added gconf list pair key names and value strings for the security and 
3629         secure-auth settings.
3630         * src/modest-account-mgr-helpers.h:
3631         * src/modest-account-mgr-helpers.c:
3632         (modest_server_account_data_get_option_secure_auth),
3633         (modest_server_account_set_option_secure_auth),
3634         (modest_server_account_data_get_option_security),
3635         (modest_server_account_set_option_security): Added this API for getting and setting the security 
3636         and secure-auth settings.
3637         
3638         * src/maemo/modest-account-settings-dialog.h:
3639         * src/maemo/modest-account-settings-dialog.c:
3640         (modest_account_settings_dialog_finalize), (check_data),
3641         (on_response): Store the original title as well as the name, to avoid unnececessary warnings 
3642         about changing titles.
3643         (modest_account_settings_dialog_set_account_name): Load the security and secure-auth settings.
3644         (save_configuration): Save the security and secure-auth settings.
3645
3646         * src/modest-conf.c: (modest_conf_set_list): Get the list after setting it, to show that this 
3647         is failing sometimes (though reporting success). I fear I may need to debug gconf to fix this.
3648
3649 2007-04-17  Murray Cumming  <murrayc@murrayc.com>
3650
3651         * src/modest-defs.h: 
3652         * src/modest-account-mgr.c:
3653         (modest_account_mgr_add_server_account):
3654         Added and used defines for the options key and value pair strings, 
3655         that are used so far, though they do not seem to correspond to anything in our 
3656         UI specs or ModestProtocol enum.
3657         
3658         * src/modest-account-mgr-helpers.h:
3659         * src/modest-account-mgr-helpers.c:
3660         (compare_option_strings_for_name),
3661         (modest_server_account_data_get_option_value),
3662         (modest_server_account_data_get_option_bool),
3663         (modest_account_mgr_get_server_account_option):
3664         Added helper functions for parsing the options GSList.
3665
3666 2007-04-16  Murray Cumming  <murrayc@murrayc.com>
3667
3668         * src/modest-account-mgr-helpers.c: Clarified the documentation to say that the initial 
3669         display name is based on the supplied account name.
3670         * src/modest-account-mgr.c:
3671         * src/modest-account-mgr.h: Added modest_account_mgr_account_with_display_name_exists(), 
3672         with an inefficient, but good-enough, implementation.
3673         
3674         * src/maemo/easysetup/modest-easysetup-wizard.c:
3675         (create_page_account_details), (on_before_next), (create_account): 
3676         Check for existing display names rather than (non-user-visible) account names, and just 
3677         create and set a unique account name at the end.
3678         
3679         * src/maemo/modest-account-settings-dialog.c:
3680         (modest_account_settings_dialog_init),
3681         (modest_account_settings_dialog_set_account_name),
3682         (create_page_incoming): Show the display name, instead of the account name.
3683         (on_response): Do some (not all) extra checks, as in the UI spec.
3684         (save_configuration): Save the display name if it was changed.
3685
3686 2007-04-16  Murray Cumming  <murrayc@murrayc.com>
3687
3688         * po/en_GB.po: Added needed logical ID and translation.
3689         * src/widgets/modest-account-view.c: (on_account_default_toggled),
3690         (init_view): Made the is-default column clicking work.
3691
3692 2007-04-16  Murray Cumming  <murrayc@murrayc.com>
3693
3694         * configure.ac:
3695         * src/maemo/easysetup/Makefile.am:
3696         * src/maemo/easysetup/modest-easysetup-wizard.c:
3697         (modest_easysetup_wizard_dialog_init):
3698         * src/maemo/easysetup/provider-data-test.keyfile:
3699         * src/maemo/easysetup/provider-data.keyfile: 
3700         Moved the provider data file to the new modest-providers-data package and used it from there.
3701
3702 2007-04-16  Murray Cumming  <murrayc@murrayc.com>
3703
3704         * src/modest-local-folder-info.c:
3705         (modest_local_folder_info_get_type_display_name): Use _() to actually get the translated string. 
3706         N_() was already used on the static strings, but this just marks it for translation without 
3707         using the translation.
3708
3709 2007-04-06  Murray Cumming  <murrayc@murrayc.com>
3710
3711         * src/maemo/modest-account-settings-dialog.c:
3712         (create_page_incoming),
3713         (modest_account_settings_dialog_set_account_name): 
3714         Added comments about the need for new API in ModestAccountMgr to handle the authentication 
3715         and secure-connection stuff that is stored in the options GSList* with hard-coded names.
3716         I will do this if nobody else gets there first.
3717
3718 2007-04-06  Murray Cumming  <murrayc@murrayc.com>
3719
3720         * src/maemo/modest-account-settings-dialog.c: (save_configuration): 
3721         Saved much of the incoming and outgoing data too, though I cannot yet see how to 
3722         save the security stuff.
3723
3724 2007-04-06  Murray Cumming  <murrayc@murrayc.com>
3725
3726         * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account): 
3727         Set the email address and user fullname, which I now see must be 
3728         done separately.
3729         
3730         * src/maemo/modest-account-settings-dialog.h:
3731         * src/maemo/modest-account-settings-dialog.c:
3732          (on_response),
3733         (modest_account_settings_dialog_set_account_name): Show the email address, fullname, 
3734         leave-on-server and authentication.
3735         (save_configuration): Save most of the non-server-account specific stuff, but I don't see 
3736         how to do the rest.
3737
3738 2007-04-06  Murray Cumming  <murrayc@murrayc.com>
3739
3740
3741         * src/maemo/modest-account-settings-dialog.c:
3742         * src/maemo/modest-account-settings-dialog.h:
3743         Most (but not all) of the widgets are now present as specified by the UI spec.
3744         No changes are yet saved, and some details are not properly shown yet.
3745         
3746         * po/en_GB.po: Added translations for logical IDs used by the Account Settings dialog.
3747
3748 2007-04-06  Murray Cumming  <murrayc@murrayc.com>
3749
3750         * src/maemo/Makefile.am:
3751         * src/maemo/modest-account-settings-dialog.c:
3752         * src/maemo/modest-account-settings-dialog.h: The beginnings of the dialog to edit 
3753         an existing account. Doesn't do much yet.
3754         
3755         * src/maemo/modest-account-view-window.c: (on_edit_button_clicked): 
3756         Open the accounts settings dialog when Edit is clicked.
3757
3758 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
3759
3760         * src/maemo/easysetup/Makefile.am:
3761         * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
3762         (load_from_file): 
3763         * src/maemo/easysetup/modest-easysetup-wizard.c:
3764         (modest_easysetup_wizard_dialog_init):
3765         Try to get the source directory from the build system,
3766         instead of hard-coding "./" into the source code, when opening our local 
3767         copies of the configuration files.
3768         But this still fails when opening the wizard from the Accounts window, 
3769         maybe because the working directory seems to change at runtime.
3770         So the wizard will fail the second time unless you copy the files into their 
3771         proper locations - see the stdout warnings for the paths.
3772         
3773         (create_page_account_details),(on_before_next): 
3774         Check for existing non-server accounts instead of server accounts, 
3775         so the default account name is useful again, and to prevent trying to 
3776         recreate accounts.
3777         
3778         * src/maemo/modest-account-view-window.c: (on_new_button_clicked): 
3779         Show the wizard when New is clicked.
3780         
3781         * src/widgets/modest-account-view.c: (on_account_default_toggled),
3782         (init_view): Show the column headers, and show the columns that are 
3783         specified in the UI specs. However, something in Maemo's GTK+ prevents 
3784         us from handling the GtkCellRendererToggle::toggled signal, so we cannot 
3785         change the default account. This is despite unsetting the special properties.
3786
3787 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
3788
3789         * docs/reference/modest-design.sgml: Corrected spelling mistake.
3790         * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account): 
3791         Create the 2 server accounts and then create the account, using the names of the 2 server accounts.
3792         This seems to be how this should be used. If it is, then I will update the AccountMgr documentation 
3793         to make that clearer.
3794         * src/modest-account-mgr.h: Corrected the modest_account_mgr_search_server_account() documentation.
3795
3796 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
3797
3798         * src/maemo/modest-account-view-window.c:
3799         (modest_account_view_window_init),
3800         (modest_account_view_window_new):
3801         Use the same buttons as specified in the 
3802         UI spec. Add the widgets to the vbox and action_area instead of just assigning them to 
3803         the existing widgets, which never had a chance of working.
3804         Put the ModestAccountView treeview in a scrolled window.
3805         Do not repeat the initialization in both _init() and _new().
3806         
3807         * src/widgets/modest-account-view.c: (update_account_view):
3808         Add a warning about modest_account_mgr_account_names() returning NULL, which is why 
3809         this widget and window shows no accounts.
3810         
3811         * po/en_GB.po: Added translations needed for the dialog button logical IDs.
3812         
3813 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
3814
3815         * src/maemo/easysetup/modest-easysetup-wizard.c:
3816         (set_default_custom_servernames): Avoid a dereference of a destroyed widget.
3817         (create_account): Do a sanity check, showing that modest_account_mgr_account_names() 
3818         returns NULL after modest_account_mgr_add_server_account() returned TRUE, 
3819         which seems wrong.
3820         
3821         * src/maemo/modest-main-window-ui.h: Specify the callback for the Accounts menu item.
3822         * src/modest-ui-actions.c: (modest_ui_actions_on_accounts): Open the wizard if no 
3823         accounts exist yet, as in the UI spec.
3824         
3825         * src/modest-account-mgr.h: Add TODO comment about a possible memory leak.
3826
3827 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
3828
3829         * src/maemo/Makefile.am:
3830         * src/maemo/modest-connection-specific-smtp-edit-window.c:
3831         * src/maemo/modest-connection-specific-smtp-edit-window.h: 
3832         New window for editing connection-specific smtp details.
3833         Not fully implemented.
3834
3835         * src/maemo/modest-connection-specific-smtp-window.c:
3836         (modest_connection_specific_smtp_window_finalize) :
3837         unref the tree model.
3838         (on_button_edit): Show the edit window.
3839
3840 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
3841
3842         * src/Makefile.am: Change sequence of sub-libraries, which fixed an undefined symbol 
3843         error for me.
3844         
3845         * src/maemo/Makefile.am:
3846         * src/maemo/modest-connection-specific-smtp-window.c:
3847         * src/maemo/modest-connection-specific-smtp-window.h:
3848         New window for showing the list of connections, for connection-specific 
3849         SMTP servers. Not fully implemented, because it needs an enhancement to TnyMaemoDevice.
3850         * src/maemo/easysetup/modest-easysetup-wizard.c:
3851         (create_page_custom_outgoing): Show the new window when the button is clicked.
3852
3853 2007-04-04  Murray Cumming  <murrayc@murrayc.com>
3854
3855         * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
3856         * src/maemo/easysetup/modest-presets.c: (modest_presets_new):
3857         * src/maemo/modest-msg-edit-window.c:
3858         (modest_msg_edit_window_insert_image):
3859         * src/modest-conf.c: (modest_conf_key_escape):
3860         * src/modest-mail-operation.c:
3861         (modest_mail_operation_send_new_mail):
3862         * src/modest-ui-actions.c: (modest_ui_actions_on_accounts),
3863         (modest_ui_actions_on_new_account):
3864         * src/widgets/modest-attachment-view.c: (update_filename_request):
3865         * src/widgets/modest-mail-header-view.c: 
3866         Corrected warnings that were breaking the build, by commenting out unused variables, 
3867         and a function, and by including a header.
3868
3869 2007-04-04  Murray Cumming  <murrayc@murrayc.com>
3870
3871         * configure.ac: Define MODEST_PLATFORM_MAEMO and MODEST_PLATFORM_GNOME in config.h, 
3872         so we can #ifdef around platform-specific code when necessary.
3873         * src/modest-ui-actions.c: (modest_ui_actions_on_new_account): Use MODEST_PLATFORM_MAEMO 
3874         to allow this to build for the GNOME platform too.
3875
3876 2007-04-04  Murray Cumming  <murrayc@murrayc.com>
3877
3878         * src/maemo/easysetup/modest-easysetup-wizard.c:
3879         Added util_increment_name() and used it in create_page_account_details() 
3880         to make sure that the default account name is always a new name.
3881         
3882         (modest_easysetup_wizard_dialog_init), 
3883         (set_default_custom_servernames): Some minor memory management fixes.
3884
3885 2007-04-04  Murray Cumming  <murrayc@murrayc.com>
3886
3887         * Makefile.am:
3888         * configure.ac:
3889         * src/modest-runtime.c: (init_i18n): Restored the translation 
3890         infrastructure, so that GETTEXT_PACKAGE and MODEST_LOCALE_DIR 
3891         are defined in config.h, and the translations are installed.
3892         
3893         * src/maemo/easysetup/: Include config.h so that GETTEXT_PACKAGE 
3894         is defined for translation.
3895
3896 2007-04-03  Murray Cumming  <murrayc@murrayc.com>
3897
3898         * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
3899         (load_from_file):
3900         * src/maemo/easysetup/modest-easysetup-wizard.c:
3901         (modest_easysetup_wizard_dialog_init):
3902         * src/maemo/easysetup/modest-presets.c: (modest_presets_new): 
3903         Use realistic paths (though they should use $prefix generically) to the 
3904         mcc_mapping and provider-data.keyfile files which might work when they 
3905         are installed by the future version of the operator-wizard package, 
3906         and then fallback to paths in our source code, with an explanatory 
3907         warning.
3908
3909 2007-04-03  Murray Cumming  <murrayc@murrayc.com>
3910
3911         * configure.ac:
3912         * src/Makefile.am:
3913         * src/maemo/Makefile.am:
3914         * src/maemo/easysetup/: Added files from the previously-exernal 
3915         osso-modest-easysetup.
3916         * src/maemo/modest-main-window-ui.h:
3917         * src/modest-ui-actions.c: 
3918         * src/modest-ui-actions.h:
3919         Added modest_ui_actions_on_new_account() as a signal handler for the 
3920         New Account menu item. It shows the easysetup wizard.
3921         * po/en_GB.po: Added additional needed logical IDs and translations.