1 2007-04-19 Murray Cumming <murrayc@murrayc.com>
3 * src/maemo/modest-address-book.c: Added include to avoid a warning.
5 * po/en_GB.po: Added missing mcen_bd_emailsetup_close string,
6 specified for EmailAccountsDialog in UI spec.
8 * src/maemo/easysetup/modest-easysetup-wizard.c:
9 * src/maemo/easysetup/modest-easysetup-wizard.h:
10 * src/maemo/modest-account-settings-dialog.c:
11 * src/maemo/modest-account-settings-dialog.h:
12 Reuse a member window widget for the connection-specific SMTP server accounts,
13 so we can tell it to create/save the accounts only if we actually save changes.
15 * src/maemo/modest-connection-specific-smtp-edit-window.h:
16 * src/maemo/modest-connection-specific-smtp-edit-window.c:
17 (modest_connection_specific_smtp_edit_window_get_settings):
18 Return a struct instance for temporarily remembering the entered data.
20 * src/maemo/modest-connection-specific-smtp-window.h:
21 * src/maemo/modest-connection-specific-smtp-window.c:
22 (modest_connection_specific_smtp_window_finalize),
23 (modest_connection_specific_smtp_window_fill_with_connections),
25 (modest_connection_specific_smtp_window_save_server_accounts),
26 (update_model_server_names): Do not actually save data until asked,
27 when the caller calls modest_connection_specific_smtp_window_save_server_accounts().
29 (on_selection_changed),
30 (modest_connection_specific_smtp_window_init): Disable the edit
31 button when nothing is selected.
33 * src/modest-account-mgr-helpers.c:
34 * src/modest-account-mgr-helpers.h:
35 Put modest_account_mgr_free_server_account_data() in the header now that I use the struct
38 2007-04-19 Murray Cumming <murrayc@murrayc.com>
40 * src/modest-account-mgr-helpers.h:
41 * src/modest-account-mgr-helpers.c:
42 (modest_account_mgr_get_unused_account_name),
43 (modest_account_mgr_get_unused_account_display_name): Added these
44 utility functions, to avoid code duplication.
46 * src/maemo/easysetup/modest-easysetup-wizard.c:
47 (create_page_account_details), (create_account):
48 * src/maemo/modest-connection-specific-smtp-edit-window.c:
49 (on_combo_security_changed),
50 (modest_connection_specific_smtp_edit_window_init),
51 (modest_connection_specific_smtp_edit_window_save_settings):
52 * src/maemo/modest-connection-specific-smtp-edit-window.h:
53 * src/maemo/modest-connection-specific-smtp-window.c:
54 (modest_connection_specific_smtp_window_finalize),
55 (modest_connection_specific_smtp_window_fill_with_connections),
56 (on_button_edit), (modest_connection_specific_smtp_window_init):
57 * src/maemo/modest-connection-specific-smtp-window.h:
58 Implementing saving of connection-specific server accounts.
60 But I need to refactor this so we can save the data for each
61 possible connection-specific account and only actually create the server accounts
62 when the easysetup finished, or the OK button is pressed on the account settings
65 2007-04-18 Murray Cumming <murrayc@murrayc.com>
67 * src/maemo/modest-connection-specific-smtp-edit-window.c:
68 (on_combo_security_changed):
69 (modest_connection_specific_smtp_edit_window_init):
70 Auto-fill the port number as in the account settings.
72 (modest_connection_specific_smtp_edit_window_set_connection):
73 * src/maemo/modest-connection-specific-smtp-edit-window.h:
74 * src/maemo/modest-connection-specific-smtp-window.c:
75 (fill_with_connections), (on_button_edit),
76 (modest_connection_specific_smtp_window_init):
77 Turn the edit window into a dialog to simplify the code, and
78 set a WM hint as in a GtkDialog, because only a popup can be on top of
79 another window in Maemo, and popups usually have no WM decoration.
82 * src/modest-account-mgr-helpers.c:
83 (modest_account_mgr_set_connection_specific_smtp),
84 (modest_account_mgr_remove_connection_specific_smtp),
85 (modest_account_mgr_get_connection_specific_smtp):
86 * src/modest-account-mgr-helpers.h: Some functions to write the
87 connection-specific SMTP server information. Not yet used.
90 * src/widgets/modest-folder-view.c: (text_cell_data):
91 Fix a warning about an uninitialized variable.
92 (on_configuration_key_changed): Put an #ifdef around the use
93 of gtk_tree_view_column_queue_resize() because it is new in GTK+ 2.10.
95 2007-04-18 Murray Cumming <murrayc@murrayc.com>
97 * src/maemo/modest-account-settings-dialog.h:
98 * src/maemo/modest-account-settings-dialog.c:
99 Added connect_for_modified() utility function to mark the new modified boolean.
101 (create_page_account_details), (create_page_user_details),
102 (create_page_incoming), (create_page_outgoing), (on_response),
103 (modest_account_settings_dialog_init),
104 (modest_account_settings_dialog_set_account_name):
105 Only warn about unsaved changes when there are actually unsaved changes.
107 * src/modest-account-mgr-helpers.c:
108 * src/modest-account-mgr-helpers.h: Put the port, secure-auth, and security information
109 into the ModestServerAccountData struct. I do not use it yet, but someone might expect
110 to get the data from there in future.
112 2007-04-18 Murray Cumming <murrayc@murrayc.com>
116 * src/modest-account-mgr-helpers.h:
117 * src/modest-account-mgr-helpers.c:
118 Store the security account setting in a single conf key instead of using
119 the options list, so that the list is left for just the to-be-removed camel hack,
120 and because gconf_client_set_list() is failing for some reason.
122 * src/modest-account-mgr.c:
123 (modest_account_mgr_add_server_account):
124 * src/maemo/modest-account-settings-dialog.c:
125 (modest_account_settings_dialog_set_account_name),
126 (save_configuration): Use the renamed functions.
129 2007-04-18 Murray Cumming <murrayc@murrayc.com>
131 * src/widgets/Makefile.am:
132 * src/widgets/modest-limit-retrieve-combo-box.c:
133 * src/widgets/modest-limit-retrieve-combo-box.h:
134 * src/widgets/modest-retrieve-combo-box.c:
135 * src/widgets/modest-retrieve-combo-box.h: Added new widgets for use in
136 the account settings dialog.
139 * src/modest-account-mgr-helpers.h:
140 * src/modest-account-mgr-helpers.c:
141 (modest_server_account_get_option_secure_auth),
142 (modest_server_account_set_option_secure_auth):
143 * src/modest-account-mgr.c:
144 (modest_account_mgr_add_server_account):
145 Use the existing AUTH_MECH conf key for the secure-auth setting,
146 instead of putting it in the list options, because this was actually used already.
148 * src/maemo/modest-account-settings-dialog.c:
149 (create_page_account_details),
150 (modest_account_settings_dialog_set_account_name),
151 (save_configuration):
152 * src/maemo/modest-account-settings-dialog.h:
153 Add the retrieve and retrieve-limit combo boxes, as per the UI spec.
154 Hide the leave-messages checkbox for non-POP accounts, as per the UI spec.
156 2007-04-18 Murray Cumming <murrayc@murrayc.com>
158 * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.h:
159 * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.c:
160 Added easysetup_serversecurity_combo_box_get_active_serversecurity_port()
163 * src/maemo/modest-account-settings-dialog.c:
164 (create_page_incoming), (on_combo_outgoing_security_changed),
165 (on_combo_incoming_security_changed), (create_page_outgoing),
166 (modest_account_settings_dialog_set_account_name),
167 (save_configuration): Load/Save the port number, and change it when the
168 security combo changes, to suitable defaults.
170 2007-04-17 Murray Cumming <murrayc@murrayc.com>
172 * src/modest-account-mgr.c:
173 (modest_account_mgr_add_server_account): Set the security and secure-auth
174 options, and add a comment saying that the other camel-specific options should be
177 2007-04-17 Murray Cumming <murrayc@murrayc.com>
179 * src/maemo/modest-account-settings-dialog.c: (check_data): Always return something,
181 * src/modest-tny-account-store.c: (account_list_free),
182 (on_account_changed): Add a TODO comment about a valgrind error, caused by using
183 cursor->data after unreffing it. But there is some strange stuff there so it might not be
186 2007-04-17 Murray Cumming <murrayc@murrayc.com>
188 * src/modest-protocol-info.c:
189 * src/modest-protocol-info.h: Added a MODEST_PROTOCOL_AUTH_CRAMMD5 enum value, because it
190 is required by our UI spec. Maybe something in our implementation will use this eventually.
191 * src/maemo/easysetup/modest-easysetup-secureauth-combo-box.c:
192 (easysetup_secureauth_combo_box_fill): Use the CRAM MD5 enum value
193 instead of the previous 0 placeholder.
195 * src/modest-defs.h: Added gconf list pair key names and value strings for the security and
196 secure-auth settings.
197 * src/modest-account-mgr-helpers.h:
198 * src/modest-account-mgr-helpers.c:
199 (modest_server_account_data_get_option_secure_auth),
200 (modest_server_account_set_option_secure_auth),
201 (modest_server_account_data_get_option_security),
202 (modest_server_account_set_option_security): Added this API for getting and setting the security
203 and secure-auth settings.
205 * src/maemo/modest-account-settings-dialog.h:
206 * src/maemo/modest-account-settings-dialog.c:
207 (modest_account_settings_dialog_finalize), (check_data),
208 (on_response): Store the original title as well as the name, to avoid unnececessary warnings
209 about changing titles.
210 (modest_account_settings_dialog_set_account_name): Load the security and secure-auth settings.
211 (save_configuration): Save the security and secure-auth settings.
213 * src/modest-conf.c: (modest_conf_set_list): Get the list after setting it, to show that this
214 is failing sometimes (though reporting success). I fear I may need to debug gconf to fix this.
216 2007-04-17 Murray Cumming <murrayc@murrayc.com>
219 * src/modest-account-mgr.c:
220 (modest_account_mgr_add_server_account):
221 Added and used defines for the options key and value pair strings,
222 that are used so far, though they do not seem to correspond to anything in our
223 UI specs or ModestProtocol enum.
225 * src/modest-account-mgr-helpers.h:
226 * src/modest-account-mgr-helpers.c:
227 (compare_option_strings_for_name),
228 (modest_server_account_data_get_option_value),
229 (modest_server_account_data_get_option_bool),
230 (modest_account_mgr_get_server_account_option):
231 Added helper functions for parsing the options GSList.
233 2007-04-16 Murray Cumming <murrayc@murrayc.com>
235 * src/modest-account-mgr-helpers.c: Clarified the documentation to say that the initial
236 display name is based on the supplied account name.
237 * src/modest-account-mgr.c:
238 * src/modest-account-mgr.h: Added modest_account_mgr_account_with_display_name_exists(),
239 with an inefficient, but good-enough, implementation.
241 * src/maemo/easysetup/modest-easysetup-wizard.c:
242 (create_page_account_details), (on_before_next), (create_account):
243 Check for existing display names rather than (non-user-visible) account names, and just
244 create and set a unique account name at the end.
246 * src/maemo/modest-account-settings-dialog.c:
247 (modest_account_settings_dialog_init),
248 (modest_account_settings_dialog_set_account_name),
249 (create_page_incoming): Show the display name, instead of the account name.
250 (on_response): Do some (not all) extra checks, as in the UI spec.
251 (save_configuration): Save the display name if it was changed.
253 2007-04-16 Murray Cumming <murrayc@murrayc.com>
255 * po/en_GB.po: Added needed logical ID and translation.
256 * src/widgets/modest-account-view.c: (on_account_default_toggled),
257 (init_view): Made the is-default column clicking work.
259 2007-04-16 Murray Cumming <murrayc@murrayc.com>
262 * src/maemo/easysetup/Makefile.am:
263 * src/maemo/easysetup/modest-easysetup-wizard.c:
264 (modest_easysetup_wizard_dialog_init):
265 * src/maemo/easysetup/provider-data-test.keyfile:
266 * src/maemo/easysetup/provider-data.keyfile:
267 Moved the provider data file to the new modest-providers-data package and used it from there.
269 2007-04-16 Murray Cumming <murrayc@murrayc.com>
271 * src/modest-local-folder-info.c:
272 (modest_local_folder_info_get_type_display_name): Use _() to actually get the translated string.
273 N_() was already used on the static strings, but this just marks it for translation without
274 using the translation.
276 2007-04-06 Murray Cumming <murrayc@murrayc.com>
278 * src/maemo/modest-account-settings-dialog.c:
279 (create_page_incoming),
280 (modest_account_settings_dialog_set_account_name):
281 Added comments about the need for new API in ModestAccountMgr to handle the authentication
282 and secure-connection stuff that is stored in the options GSList* with hard-coded names.
283 I will do this if nobody else gets there first.
285 2007-04-06 Murray Cumming <murrayc@murrayc.com>
287 * src/maemo/modest-account-settings-dialog.c: (save_configuration):
288 Saved much of the incoming and outgoing data too, though I cannot yet see how to
289 save the security stuff.
291 2007-04-06 Murray Cumming <murrayc@murrayc.com>
293 * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
294 Set the email address and user fullname, which I now see must be
297 * src/maemo/modest-account-settings-dialog.h:
298 * src/maemo/modest-account-settings-dialog.c:
300 (modest_account_settings_dialog_set_account_name): Show the email address, fullname,
301 leave-on-server and authentication.
302 (save_configuration): Save most of the non-server-account specific stuff, but I don't see
305 2007-04-06 Murray Cumming <murrayc@murrayc.com>
308 * src/maemo/modest-account-settings-dialog.c:
309 * src/maemo/modest-account-settings-dialog.h:
310 Most (but not all) of the widgets are now present as specified by the UI spec.
311 No changes are yet saved, and some details are not properly shown yet.
313 * po/en_GB.po: Added translations for logical IDs used by the Account Settings dialog.
315 2007-04-06 Murray Cumming <murrayc@murrayc.com>
317 * src/maemo/Makefile.am:
318 * src/maemo/modest-account-settings-dialog.c:
319 * src/maemo/modest-account-settings-dialog.h: The beginnings of the dialog to edit
320 an existing account. Doesn't do much yet.
322 * src/maemo/modest-account-view-window.c: (on_edit_button_clicked):
323 Open the accounts settings dialog when Edit is clicked.
325 2007-04-05 Murray Cumming <murrayc@murrayc.com>
327 * src/maemo/easysetup/Makefile.am:
328 * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
330 * src/maemo/easysetup/modest-easysetup-wizard.c:
331 (modest_easysetup_wizard_dialog_init):
332 Try to get the source directory from the build system,
333 instead of hard-coding "./" into the source code, when opening our local
334 copies of the configuration files.
335 But this still fails when opening the wizard from the Accounts window,
336 maybe because the working directory seems to change at runtime.
337 So the wizard will fail the second time unless you copy the files into their
338 proper locations - see the stdout warnings for the paths.
340 (create_page_account_details),(on_before_next):
341 Check for existing non-server accounts instead of server accounts,
342 so the default account name is useful again, and to prevent trying to
345 * src/maemo/modest-account-view-window.c: (on_new_button_clicked):
346 Show the wizard when New is clicked.
348 * src/widgets/modest-account-view.c: (on_account_default_toggled),
349 (init_view): Show the column headers, and show the columns that are
350 specified in the UI specs. However, something in Maemo's GTK+ prevents
351 us from handling the GtkCellRendererToggle::toggled signal, so we cannot
352 change the default account. This is despite unsetting the special properties.
354 2007-04-05 Murray Cumming <murrayc@murrayc.com>
356 * docs/reference/modest-design.sgml: Corrected spelling mistake.
357 * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
358 Create the 2 server accounts and then create the account, using the names of the 2 server accounts.
359 This seems to be how this should be used. If it is, then I will update the AccountMgr documentation
360 to make that clearer.
361 * src/modest-account-mgr.h: Corrected the modest_account_mgr_search_server_account() documentation.
363 2007-04-05 Murray Cumming <murrayc@murrayc.com>
365 * src/maemo/modest-account-view-window.c:
366 (modest_account_view_window_init),
367 (modest_account_view_window_new):
368 Use the same buttons as specified in the
369 UI spec. Add the widgets to the vbox and action_area instead of just assigning them to
370 the existing widgets, which never had a chance of working.
371 Put the ModestAccountView treeview in a scrolled window.
372 Do not repeat the initialization in both _init() and _new().
374 * src/widgets/modest-account-view.c: (update_account_view):
375 Add a warning about modest_account_mgr_account_names() returning NULL, which is why
376 this widget and window shows no accounts.
378 * po/en_GB.po: Added translations needed for the dialog button logical IDs.
380 2007-04-05 Murray Cumming <murrayc@murrayc.com>
382 * src/maemo/easysetup/modest-easysetup-wizard.c:
383 (set_default_custom_servernames): Avoid a dereference of a destroyed widget.
384 (create_account): Do a sanity check, showing that modest_account_mgr_account_names()
385 returns NULL after modest_account_mgr_add_server_account() returned TRUE,
388 * src/maemo/modest-main-window-ui.h: Specify the callback for the Accounts menu item.
389 * src/modest-ui-actions.c: (modest_ui_actions_on_accounts): Open the wizard if no
390 accounts exist yet, as in the UI spec.
392 * src/modest-account-mgr.h: Add TODO comment about a possible memory leak.
394 2007-04-05 Murray Cumming <murrayc@murrayc.com>
396 * src/maemo/Makefile.am:
397 * src/maemo/modest-connection-specific-smtp-edit-window.c:
398 * src/maemo/modest-connection-specific-smtp-edit-window.h:
399 New window for editing connection-specific smtp details.
400 Not fully implemented.
402 * src/maemo/modest-connection-specific-smtp-window.c:
403 (modest_connection_specific_smtp_window_finalize) :
404 unref the tree model.
405 (on_button_edit): Show the edit window.
407 2007-04-05 Murray Cumming <murrayc@murrayc.com>
409 * src/Makefile.am: Change sequence of sub-libraries, which fixed an undefined symbol
412 * src/maemo/Makefile.am:
413 * src/maemo/modest-connection-specific-smtp-window.c:
414 * src/maemo/modest-connection-specific-smtp-window.h:
415 New window for showing the list of connections, for connection-specific
416 SMTP servers. Not fully implemented, because it needs an enhancement to TnyMaemoDevice.
417 * src/maemo/easysetup/modest-easysetup-wizard.c:
418 (create_page_custom_outgoing): Show the new window when the button is clicked.
420 2007-04-04 Murray Cumming <murrayc@murrayc.com>
422 * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
423 * src/maemo/easysetup/modest-presets.c: (modest_presets_new):
424 * src/maemo/modest-msg-edit-window.c:
425 (modest_msg_edit_window_insert_image):
426 * src/modest-conf.c: (modest_conf_key_escape):
427 * src/modest-mail-operation.c:
428 (modest_mail_operation_send_new_mail):
429 * src/modest-ui-actions.c: (modest_ui_actions_on_accounts),
430 (modest_ui_actions_on_new_account):
431 * src/widgets/modest-attachment-view.c: (update_filename_request):
432 * src/widgets/modest-mail-header-view.c:
433 Corrected warnings that were breaking the build, by commenting out unused variables,
434 and a function, and by including a header.
436 2007-04-04 Murray Cumming <murrayc@murrayc.com>
438 * configure.ac: Define MODEST_PLATFORM_MAEMO and MODEST_PLATFORM_GNOME in config.h,
439 so we can #ifdef around platform-specific code when necessary.
440 * src/modest-ui-actions.c: (modest_ui_actions_on_new_account): Use MODEST_PLATFORM_MAEMO
441 to allow this to build for the GNOME platform too.
443 2007-04-04 Murray Cumming <murrayc@murrayc.com>
445 * src/maemo/easysetup/modest-easysetup-wizard.c:
446 Added util_increment_name() and used it in create_page_account_details()
447 to make sure that the default account name is always a new name.
449 (modest_easysetup_wizard_dialog_init),
450 (set_default_custom_servernames): Some minor memory management fixes.
452 2007-04-04 Murray Cumming <murrayc@murrayc.com>
456 * src/modest-runtime.c: (init_i18n): Restored the translation
457 infrastructure, so that GETTEXT_PACKAGE and MODEST_LOCALE_DIR
458 are defined in config.h, and the translations are installed.
460 * src/maemo/easysetup/: Include config.h so that GETTEXT_PACKAGE
461 is defined for translation.
463 2007-04-03 Murray Cumming <murrayc@murrayc.com>
465 * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
467 * src/maemo/easysetup/modest-easysetup-wizard.c:
468 (modest_easysetup_wizard_dialog_init):
469 * src/maemo/easysetup/modest-presets.c: (modest_presets_new):
470 Use realistic paths (though they should use $prefix generically) to the
471 mcc_mapping and provider-data.keyfile files which might work when they
472 are installed by the future version of the operator-wizard package,
473 and then fallback to paths in our source code, with an explanatory
476 2007-04-03 Murray Cumming <murrayc@murrayc.com>
480 * src/maemo/Makefile.am:
481 * src/maemo/easysetup/: Added files from the previously-exernal
482 osso-modest-easysetup.
483 * src/maemo/modest-main-window-ui.h:
484 * src/modest-ui-actions.c:
485 * src/modest-ui-actions.h:
486 Added modest_ui_actions_on_new_account() as a signal handler for the
487 New Account menu item. It shows the easysetup wizard.
488 * po/en_GB.po: Added additional needed logical IDs and translations.
490 Started ChangeLog2 file, because I cannot work without a ChangeLog. murrayc.
491 Apparently ChangeLog is not used for some vague Nokia legal reasons.