gnucash maint: Partially revert commit B555f495 and solve problem more simply by instead resequencing prior statements.

J. Alex Aycinena alex.aycinena at code.gnucash.org
Mon Jun 6 18:18:56 EDT 2016


Updated	 via  https://github.com/Gnucash/gnucash/commit/c0086e15 (commit)
	from  https://github.com/Gnucash/gnucash/commit/b555f495 (commit)



commit c0086e15e957df3589918abbea9c6a3579d24933
Author: Alex Aycinena <alex.aycinena at gmail.com>
Date:   Mon Jun 6 15:17:55 2016 -0700

    Partially revert commit B555f495 and solve problem more simply by instead resequencing prior statements.

diff --git a/src/gnome-utils/dialog-options.c b/src/gnome-utils/dialog-options.c
index 48d4267..3b7bab7 100644
--- a/src/gnome-utils/dialog-options.c
+++ b/src/gnome-utils/dialog-options.c
@@ -168,14 +168,6 @@ gnc_options_dialog_changed (GNCOptionWin *win)
 }
 
 void
-gnc_options_dialog_not_changed (GNCOptionWin *win)
-{
-    if (!win) return;
-
-    gnc_options_dialog_changed_internal (win->dialog, FALSE);
-}
-
-void
 gnc_option_changed_widget_cb(GtkWidget *widget, GNCOption *option)
 {
     gnc_option_set_changed (option, TRUE);
@@ -1321,14 +1313,16 @@ gnc_options_dialog_response_cb(GtkDialog *dialog, gint response, GNCOptionWin *w
 
     case GTK_RESPONSE_OK:
     case GTK_RESPONSE_APPLY:
-        gnc_options_dialog_changed_internal (window->dialog, FALSE);
         close_cb = window->close_cb;
         window->close_cb = NULL;
         if (window->apply_cb)
             window->apply_cb (window, window->apply_cb_data);
         window->close_cb = close_cb;
         if (response == GTK_RESPONSE_APPLY)
+        {
+            gnc_options_dialog_changed_internal (window->dialog, FALSE);
             break;
+        }
         /* fall through */
 
     default:
diff --git a/src/gnome-utils/dialog-options.h b/src/gnome-utils/dialog-options.h
index a981894..4cdcbb1 100644
--- a/src/gnome-utils/dialog-options.h
+++ b/src/gnome-utils/dialog-options.h
@@ -46,7 +46,6 @@ GtkWidget * gnc_options_page_list(GNCOptionWin * win);
 GtkWidget * gnc_options_dialog_notebook(GNCOptionWin * win);
 
 void gnc_options_dialog_changed (GNCOptionWin *win);
-void gnc_options_dialog_not_changed (GNCOptionWin *win);
 
 void gnc_option_changed_widget_cb(GtkWidget *widget, GNCOption *option);
 void gnc_option_changed_option_cb(GtkWidget *dummy, GNCOption *option);
diff --git a/src/gnome-utils/gnc-main-window.c b/src/gnome-utils/gnc-main-window.c
index 681f469..cff961f 100644
--- a/src/gnome-utils/gnc-main-window.c
+++ b/src/gnome-utils/gnc-main-window.c
@@ -3956,10 +3956,6 @@ gnc_book_options_dialog_apply_cb(GNCOptionWin * optionwin,
     {
         gnc_book_option_num_field_source_change_cb (use_split_action_for_num_after);
         gnc_gui_refresh_all ();
-        /* the previous stmt causes the 'apply' and 'OK' buttons to be set to
-           sensitive on the open Book Options dialog; the next stmt resets them
-           to insensitive, as they should be */
-        gnc_options_dialog_not_changed (optionwin);
     }
 }
 



Summary of changes:
 src/gnome-utils/dialog-options.c  | 12 +++---------
 src/gnome-utils/dialog-options.h  |  1 -
 src/gnome-utils/gnc-main-window.c |  4 ----
 3 files changed, 3 insertions(+), 14 deletions(-)



More information about the gnucash-changes mailing list