SF.net SVN: geany:[5551] trunk

[eht16 at users.sourceforge.net]

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.