AUDIT: r22340 - gnucash/trunk/src/gnome-utils - Bug #610648 - Change Reconciled Split Dialog ignores pressing Cancel

[Geert Janssens]

Author: gjanssens
Date: 2012-08-22 09:22:02 -0400 (Wed, 22 Aug 2012)
New Revision: 22340
Trac: http://svn.gnucash.org/trac/changeset/22340

Modified:
   gnucash/trunk/src/gnome-utils/dialog-utils.c
Log:
Bug #610648 - Change Reconciled Split Dialog ignores pressing Cancel

Author: Robert Fewell <14ubobit at gmail.com>
BP

Modified: gnucash/trunk/src/gnome-utils/dialog-utils.c
===================================================================
--- gnucash/trunk/src/gnome-utils/dialog-utils.c	2012-08-22 10:18:36 UTC (rev 22339)
+++ gnucash/trunk/src/gnome-utils/dialog-utils.c	2012-08-22 13:22:02 UTC (rev 22340)
@@ -602,15 +602,17 @@
         return GTK_RESPONSE_NO;
     }
 
-    /* Save the answer? */
-    if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(perm)))
+    if (response != GTK_RESPONSE_CANCEL)
     {
-        gnc_gconf_set_int(GCONF_WARNINGS_PERM, gconf_key, response, NULL);
+        /* Save the answer? */
+        if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(perm)))
+        {
+            gnc_gconf_set_int(GCONF_WARNINGS_PERM, gconf_key, response, NULL);
+        }
+        else if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(temp)))
+        {
+            gnc_gconf_set_int(GCONF_WARNINGS_TEMP, gconf_key, response, NULL);
+        }
     }
-    else if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(temp)))
-    {
-        gnc_gconf_set_int(GCONF_WARNINGS_TEMP, gconf_key, response, NULL);
-    }
-
     return response;
 }