2007-04-18 Murray Cumming <murrayc@murrayc.com>
[modest] / ChangeLog2
index 182b86d..0b229bc 100644 (file)
@@ -1,3 +1,237 @@
+2007-04-18  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/Makefile.am:
+       * src/widgets/modest-limit-retrieve-combo-box.c:
+       * src/widgets/modest-limit-retrieve-combo-box.h:
+       * src/widgets/modest-retrieve-combo-box.c:
+       * src/widgets/modest-retrieve-combo-box.h: Added new widgets for use in 
+       the account settings dialog.
+       
+       * src/modest-defs.h:
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       (modest_server_account_get_option_secure_auth),
+       (modest_server_account_set_option_secure_auth):
+       * src/modest-account-mgr.c:
+       (modest_account_mgr_add_server_account):
+       Use the existing AUTH_MECH conf key for the secure-auth setting, 
+       instead of putting it in the list options, because this was actually used already.
+       
+       * src/maemo/modest-account-settings-dialog.c:
+       (create_page_account_details),
+       (modest_account_settings_dialog_set_account_name),
+       (save_configuration):
+       * src/maemo/modest-account-settings-dialog.h:
+       Add the retrieve and retrieve-limit combo boxes, as per the UI spec.
+       Hide the leave-messages checkbox for non-POP accounts, as per the UI spec.
+
+2007-04-18  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.h:
+       * src/maemo/easysetup/modest-easysetup-serversecurity-combo-box.c:
+       Added easysetup_serversecurity_combo_box_get_active_serversecurity_port()
+
+       * src/modest-defs.h:
+       * src/maemo/modest-account-settings-dialog.c:
+       (create_page_incoming), (on_combo_outgoing_security_changed),
+       (on_combo_incoming_security_changed), (create_page_outgoing),
+       (modest_account_settings_dialog_set_account_name),
+       (save_configuration): Load/Save the port number, and change it when the 
+       security combo changes, to suitable defaults.
+
+2007-04-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr.c:
+       (modest_account_mgr_add_server_account): Set the security and secure-auth 
+       options, and add a comment saying that the other camel-specific options should be 
+       removed in future.
+
+2007-04-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-settings-dialog.c: (check_data): Always return something, 
+       found by valgrind.
+       * src/modest-tny-account-store.c: (account_list_free),
+       (on_account_changed): Add a TODO comment about a valgrind error, caused by using 
+       cursor->data after unreffing it. But there is some strange stuff there so it might not be 
+       easy to fix.
+
+2007-04-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-protocol-info.c:
+       * src/modest-protocol-info.h: Added a MODEST_PROTOCOL_AUTH_CRAMMD5 enum value, because it 
+       is required by our UI spec. Maybe something in our implementation will use this eventually.
+       * src/maemo/easysetup/modest-easysetup-secureauth-combo-box.c: 
+       (easysetup_secureauth_combo_box_fill): Use the CRAM MD5 enum value 
+       instead of the previous 0 placeholder.
+       
+       * src/modest-defs.h: Added gconf list pair key names and value strings for the security and 
+       secure-auth settings.
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       (modest_server_account_data_get_option_secure_auth),
+       (modest_server_account_set_option_secure_auth),
+       (modest_server_account_data_get_option_security),
+       (modest_server_account_set_option_security): Added this API for getting and setting the security 
+       and secure-auth settings.
+       
+       * src/maemo/modest-account-settings-dialog.h:
+       * src/maemo/modest-account-settings-dialog.c:
+       (modest_account_settings_dialog_finalize), (check_data),
+       (on_response): Store the original title as well as the name, to avoid unnececessary warnings 
+       about changing titles.
+       (modest_account_settings_dialog_set_account_name): Load the security and secure-auth settings.
+       (save_configuration): Save the security and secure-auth settings.
+
+       * src/modest-conf.c: (modest_conf_set_list): Get the list after setting it, to show that this 
+       is failing sometimes (though reporting success). I fear I may need to debug gconf to fix this.
+
+2007-04-17  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-defs.h: 
+       * src/modest-account-mgr.c:
+       (modest_account_mgr_add_server_account):
+       Added and used defines for the options key and value pair strings, 
+       that are used so far, though they do not seem to correspond to anything in our 
+       UI specs or ModestProtocol enum.
+       
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       (compare_option_strings_for_name),
+       (modest_server_account_data_get_option_value),
+       (modest_server_account_data_get_option_bool),
+       (modest_account_mgr_get_server_account_option):
+       Added helper functions for parsing the options GSList.
+
+2007-04-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-helpers.c: Clarified the documentation to say that the initial 
+       display name is based on the supplied account name.
+       * src/modest-account-mgr.c:
+       * src/modest-account-mgr.h: Added modest_account_mgr_account_with_display_name_exists(), 
+       with an inefficient, but good-enough, implementation.
+       
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (create_page_account_details), (on_before_next), (create_account): 
+       Check for existing display names rather than (non-user-visible) account names, and just 
+       create and set a unique account name at the end.
+       
+       * src/maemo/modest-account-settings-dialog.c:
+       (modest_account_settings_dialog_init),
+       (modest_account_settings_dialog_set_account_name),
+       (create_page_incoming): Show the display name, instead of the account name.
+       (on_response): Do some (not all) extra checks, as in the UI spec.
+       (save_configuration): Save the display name if it was changed.
+
+2007-04-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * po/en_GB.po: Added needed logical ID and translation.
+       * src/widgets/modest-account-view.c: (on_account_default_toggled),
+       (init_view): Made the is-default column clicking work.
+
+2007-04-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * configure.ac:
+       * src/maemo/easysetup/Makefile.am:
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (modest_easysetup_wizard_dialog_init):
+       * src/maemo/easysetup/provider-data-test.keyfile:
+       * src/maemo/easysetup/provider-data.keyfile: 
+       Moved the provider data file to the new modest-providers-data package and used it from there.
+
+2007-04-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-local-folder-info.c:
+       (modest_local_folder_info_get_type_display_name): Use _() to actually get the translated string. 
+       N_() was already used on the static strings, but this just marks it for translation without 
+       using the translation.
+
+2007-04-06  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-settings-dialog.c:
+       (create_page_incoming),
+       (modest_account_settings_dialog_set_account_name): 
+       Added comments about the need for new API in ModestAccountMgr to handle the authentication 
+       and secure-connection stuff that is stored in the options GSList* with hard-coded names.
+       I will do this if nobody else gets there first.
+
+2007-04-06  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-settings-dialog.c: (save_configuration): 
+       Saved much of the incoming and outgoing data too, though I cannot yet see how to 
+       save the security stuff.
+
+2007-04-06  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account): 
+       Set the email address and user fullname, which I now see must be 
+       done separately.
+       
+       * src/maemo/modest-account-settings-dialog.h:
+       * src/maemo/modest-account-settings-dialog.c:
+        (on_response),
+       (modest_account_settings_dialog_set_account_name): Show the email address, fullname, 
+       leave-on-server and authentication.
+       (save_configuration): Save most of the non-server-account specific stuff, but I don't see 
+       how to do the rest.
+
+2007-04-06  Murray Cumming  <murrayc@murrayc.com>
+
+
+       * src/maemo/modest-account-settings-dialog.c:
+       * src/maemo/modest-account-settings-dialog.h:
+       Most (but not all) of the widgets are now present as specified by the UI spec.
+       No changes are yet saved, and some details are not properly shown yet.
+       
+       * po/en_GB.po: Added translations for logical IDs used by the Account Settings dialog.
+
+2007-04-06  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/Makefile.am:
+       * src/maemo/modest-account-settings-dialog.c:
+       * src/maemo/modest-account-settings-dialog.h: The beginnings of the dialog to edit 
+       an existing account. Doesn't do much yet.
+       
+       * src/maemo/modest-account-view-window.c: (on_edit_button_clicked): 
+       Open the accounts settings dialog when Edit is clicked.
+
+2007-04-05  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/Makefile.am:
+       * src/maemo/easysetup/modest-easysetup-country-combo-box.c:
+       (load_from_file): 
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (modest_easysetup_wizard_dialog_init):
+       Try to get the source directory from the build system,
+       instead of hard-coding "./" into the source code, when opening our local 
+       copies of the configuration files.
+       But this still fails when opening the wizard from the Accounts window, 
+       maybe because the working directory seems to change at runtime.
+       So the wizard will fail the second time unless you copy the files into their 
+       proper locations - see the stdout warnings for the paths.
+       
+       (create_page_account_details),(on_before_next): 
+       Check for existing non-server accounts instead of server accounts, 
+       so the default account name is useful again, and to prevent trying to 
+       recreate accounts.
+       
+       * src/maemo/modest-account-view-window.c: (on_new_button_clicked): 
+       Show the wizard when New is clicked.
+       
+       * src/widgets/modest-account-view.c: (on_account_default_toggled),
+       (init_view): Show the column headers, and show the columns that are 
+       specified in the UI specs. However, something in Maemo's GTK+ prevents 
+       us from handling the GtkCellRendererToggle::toggled signal, so we cannot 
+       change the default account. This is despite unsetting the special properties.
+
+2007-04-05  Murray Cumming  <murrayc@murrayc.com>
+
+       * docs/reference/modest-design.sgml: Corrected spelling mistake.
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account): 
+       Create the 2 server accounts and then create the account, using the names of the 2 server accounts.
+       This seems to be how this should be used. If it is, then I will update the AccountMgr documentation 
+       to make that clearer.
+       * src/modest-account-mgr.h: Corrected the modest_account_mgr_search_server_account() documentation.
+
 2007-04-05  Murray Cumming  <murrayc@murrayc.com>
 
        * src/maemo/modest-account-view-window.c: