Revision: 5551 http://geany.svn.sourceforge.net/geany/?rev=5551&view=rev Author: eht16 Date: 2011-03-02 20:00:17 +0000 (Wed, 02 Mar 2011)
Log Message: ----------- Fix wrong warning on empty backup directory setting if 'Backup Copy' is not enabled at all.
Modified Paths: -------------- trunk/ChangeLog trunk/plugins/saveactions.c
Modified: trunk/ChangeLog =================================================================== --- trunk/ChangeLog 2011-02-27 08:57:20 UTC (rev 5550) +++ trunk/ChangeLog 2011-03-02 20:00:17 UTC (rev 5551) @@ -1,3 +1,10 @@ +2011-03-02 Enrico Tröger <enrico(dot)troeger(at)uvena(dot)de> + + * plugins/saveactions.c: + Fix wrong warning on empty backup directory setting if + 'Backup Copy' is not enabled at all. + + 2011-02-22 Nick Treleaven <nick(dot)treleaven(at)btinternet(dot)com>
* src/templates.c:
Modified: trunk/plugins/saveactions.c =================================================================== --- trunk/plugins/saveactions.c 2011-02-27 08:57:20 UTC (rev 5550) +++ trunk/plugins/saveactions.c 2011-03-02 20:00:17 UTC (rev 5551) @@ -453,15 +453,18 @@ g_key_file_set_integer(config, "backupcopy", "dir_levels", backupcopy_dir_levels); g_key_file_set_string(config, "backupcopy", "time_fmt", text_time); setptr(backupcopy_time_fmt, g_strdup(text_time)); - if (*text_dir != '\0' && backupcopy_set_backup_dir(text_dir)) + if (enable_backupcopy) { - g_key_file_set_string(config, "backupcopy", "backup_dir", text_dir); + if (NZV(text_dir) && backupcopy_set_backup_dir(text_dir)) + { + g_key_file_set_string(config, "backupcopy", "backup_dir", text_dir); + } + else + { + dialogs_show_msgbox(GTK_MESSAGE_ERROR, + _("Backup directory does not exist or is not writable.")); + } } - else - { - dialogs_show_msgbox(GTK_MESSAGE_ERROR, - _("Backup directory does not exist or is not writable.")); - }
if (! g_file_test(config_dir, G_FILE_TEST_IS_DIR) && utils_mkdir(config_dir, TRUE) != 0)
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.