r22341 - gnucash/branches/2.4/src/gnome-utils - [22340] Bug #610648 - Change Reconciled Split Dialog ignores pressing Cancel

[Geert Janssens]

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

Modified:
   gnucash/branches/2.4/src/gnome-utils/dialog-utils.c
Log:
[22340] Bug #610648 - Change Reconciled Split Dialog ignores pressing Cancel

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

Modified: gnucash/branches/2.4/src/gnome-utils/dialog-utils.c
===================================================================
--- gnucash/branches/2.4/src/gnome-utils/dialog-utils.c	2012-08-22 13:22:02 UTC (rev 22340)
+++ gnucash/branches/2.4/src/gnome-utils/dialog-utils.c	2012-08-22 13:26:10 UTC (rev 22341)
@@ -983,15 +983,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;
 }