From 16c6038a24a4634594de9c804e2cb6223b12fe2a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20K=C3=B6hler?= <andi5.py@gmx.net>
Date: Tue, 16 Oct 2007 22:18:27 +0000
Subject: [PATCH] #487317: Fix qif import druid when stepping back without a
 file selection.

Also gray out "unload selected file" and "back" buttons when there is no
file selected.

BP


git-svn-id: svn+ssh://svn.gnucash.org/repo/gnucash/trunk@16567 57a11ea4-9604-0410-9ed3-97b8803252fd
---
 src/import-export/qif-import/druid-qif-import.c | 13 +++++++++++--
 src/import-export/qif-import/qif.glade          |  4 ++--
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/src/import-export/qif-import/druid-qif-import.c b/src/import-export/qif-import/druid-qif-import.c
index a47b4ffb04..cb3383408e 100644
--- a/src/import-export/qif-import/druid-qif-import.c
+++ b/src/import-export/qif-import/druid-qif-import.c
@@ -701,7 +701,9 @@ gnc_ui_qif_import_select_loaded_file_cb (GtkTreeSelection *selection,
   GtkTreeModel *model;
   GtkTreeIter iter;
   gint row;
+  GtkWidget *button;
 
+  button = gnc_glade_lookup_widget(wind->window, "unload_file_button");
   if (gtk_tree_selection_get_selected(selection, &model, &iter)) {
     gtk_tree_model_get(model, &iter, FILENAME_COL_INDEX, &row, -1);
     if(SCM_LISTP(wind->imported_files) && 
@@ -710,11 +712,17 @@ gnc_ui_qif_import_select_loaded_file_cb (GtkTreeSelection *selection,
       wind->selected_file = scm_list_ref(wind->imported_files,
 					 scm_int2num(row));   
       scm_gc_protect_object(wind->selected_file);
+      g_object_set(button, "sensitive", TRUE, (gchar*)NULL);
+      gnome_druid_set_buttons_sensitive(GNOME_DRUID(wind->druid),
+                                        TRUE, TRUE, TRUE, TRUE);
     } 
   } else {
     scm_gc_unprotect_object(wind->selected_file);
     wind->selected_file = SCM_BOOL_F;
     scm_gc_protect_object(wind->selected_file);
+      g_object_set(button, "sensitive", FALSE, (gchar*)NULL);
+    gnome_druid_set_buttons_sensitive(GNOME_DRUID(wind->druid),
+                                      FALSE, TRUE, TRUE, TRUE);
   }
 }
 
@@ -731,9 +739,9 @@ gnc_ui_qif_import_loaded_files_prepare_cb(GnomeDruidPage * page,
 {
   QIFImportWindow * wind = user_data;
 
-  update_file_page(wind);
   gnome_druid_set_buttons_sensitive(GNOME_DRUID(wind->druid),
-                                    FALSE, TRUE, TRUE, TRUE); 
+                                    FALSE, TRUE, TRUE, TRUE);
+  update_file_page(wind);
 }
 
 
@@ -860,6 +868,7 @@ gnc_ui_qif_import_default_acct_next_cb(GnomeDruidPage * page,
   SCM    fix_default = scm_c_eval_string("qif-import:fix-from-acct");
   SCM    scm_name;
 
+  g_return_val_if_fail(wind->selected_file != SCM_BOOL_F, FALSE);
   if(!acct_name || acct_name[0] == 0) {
     gnc_warning_dialog(wind->window, _("You must enter an account name."));
     return TRUE;
diff --git a/src/import-export/qif-import/qif.glade b/src/import-export/qif-import/qif.glade
index 7fc862afd4..eb83e33143 100644
--- a/src/import-export/qif-import/qif.glade
+++ b/src/import-export/qif-import/qif.glade
@@ -514,7 +514,7 @@ Please enter a name for the account. If the file was exported from another accou
 	  <property name="visible">True</property>
 	  <property name="title" translatable="yes">QIF files you have loaded</property>
 	  <property name="title_foreground">#f5f5f5f5f5f5</property>
-	  <signal name="prepare" handler="gnc_ui_qif_import_loaded_files_prepare_cb"/>
+	  <signal name="prepare" handler="gnc_ui_qif_import_loaded_files_prepare_cb" after="yes"/>
 	  <signal name="next" handler="gnc_ui_qif_import_generic_next_cb"/>
 	  <signal name="back" handler="gnc_ui_qif_import_generic_back_cb"/>
 
@@ -589,7 +589,7 @@ Click &quot;Forward&quot; to finish loading files and move to the next step of t
 		  <property name="spacing">0</property>
 
 		  <child>
-		    <widget class="GtkButton" id="button70">
+		    <widget class="GtkButton" id="unload_file_button">
 		      <property name="visible">True</property>
 		      <property name="can_focus">True</property>
 		      <property name="label" translatable="yes">Unload selected file</property>