numbers back in tables

[pspp] / src / ui / gui / psppire-dialog-action-paired.c

diff --git a/src/ui/gui/psppire-dialog-action-paired.c b/src/ui/gui/psppire-dialog-action-paired.c
index b6bd72f67743223ea999edb4c0eb9378ac0d9c09..4d2fbb3aee06f2b9a3c13f02af2fcb297fa5a959 100644 (file)
--- a/src/ui/gui/psppire-dialog-action-paired.c
+++ b/src/ui/gui/psppire-dialog-action-paired.c
@@ -46,7 +46,7 @@ dialog_state_valid (gpointer data)
   struct variable *v = NULL;
   GtkTreeIter dest_iter;
 
-  if ( n_rows == 0 )
+  if (n_rows == 0)
     return FALSE;
 
   /* Get the last row */
@@ -58,8 +58,8 @@ dialog_state_valid (gpointer data)
 
   if (v == NULL)
     return FALSE;
-    
-  /* if ( NULL == pd->valid) */
+
+  /* if (NULL == pd->valid) */
   /*   return TRUE; */
 
   return TRUE;
@@ -93,7 +93,7 @@ select_as_pair_member (GtkTreeIter source_iter,
 
   n_rows = gtk_tree_model_iter_n_children  (GTK_TREE_MODEL (tt_d->list_store), NULL);
 
-  if ( n_rows > 0 )
+  if (n_rows > 0)
     {
 
       gtk_tree_model_iter_nth_child (GTK_TREE_MODEL (tt_d->list_store),
@@ -104,7 +104,7 @@ select_as_pair_member (GtkTreeIter source_iter,
   else
     v1 = NULL;
 
-  if ( n_rows == 0 || v1 != NULL)
+  if (n_rows == 0 || v1 != NULL)
     {
       gtk_list_store_append (tt_d->list_store, &dest_iter);
 
@@ -124,7 +124,7 @@ select_as_pair_member (GtkTreeIter source_iter,
 
 
 static gchar *
-generate_syntax (PsppireDialogAction *pda)
+generate_syntax (const PsppireDialogAction *pda)
 {
   PsppireDialogActionPaired *d = PSPPIRE_DIALOG_ACTION_PAIRED (pda);
   gchar *text = NULL;
@@ -149,56 +149,48 @@ generate_syntax (PsppireDialogAction *pda)
   return text;
 }
 
-static void
-psppire_dialog_action_paired_activate (GtkAction *a)
+static GtkBuilder *
+psppire_dialog_action_paired_activate (PsppireDialogAction *a, GVariant *param)
 {
   PsppireDialogAction *pda = PSPPIRE_DIALOG_ACTION (a);
   PsppireDialogActionPaired *act = PSPPIRE_DIALOG_ACTION_PAIRED (a);
 
-  GHashTable *thing = psppire_dialog_action_get_hash_table (pda);
-  GtkBuilder *xml = g_hash_table_lookup (thing, a);
-  if (!xml)
-    {
-      xml = builder_new ("paired-samples.ui");
-      g_hash_table_insert (thing, a, xml);
-      
-      GtkWidget *selector = get_widget_assert (xml, "psppire-selector3");
-      GtkWidget *bb = gtk_button_box_new (GTK_ORIENTATION_HORIZONTAL);
-      GtkWidget *button = gtk_button_new_with_mnemonic (_("O_ptions..."));
-      GtkWidget *box = get_widget_assert (xml, "dynamic-populate");
+  GtkBuilder *xml = builder_new ("paired-samples.ui");
 
-      pda->dialog = get_widget_assert   (xml, "t-test-paired-samples-dialog");
-      pda->source = get_widget_assert   (xml, "paired-samples-t-test-treeview1");
+  GtkWidget *selector = get_widget_assert (xml, "psppire-selector3");
+  GtkWidget *bb = gtk_button_box_new (GTK_ORIENTATION_HORIZONTAL);
+  GtkWidget *button = gtk_button_new_with_mnemonic (_("O_ptions..."));
+  GtkWidget *box = get_widget_assert (xml, "dynamic-populate");
 
-      gtk_window_set_title (GTK_WINDOW (pda->dialog), _("Paired Samples T Test"));
+  pda->dialog = get_widget_assert   (xml, "t-test-paired-samples-dialog");
+  pda->source = get_widget_assert   (xml, "paired-samples-t-test-treeview1");
 
-      act->pairs_treeview = get_widget_assert (xml, "paired-samples-t-test-treeview2");
-      act->list_store = GTK_LIST_STORE (gtk_tree_view_get_model (GTK_TREE_VIEW (act->pairs_treeview)));
+  gtk_window_set_title (GTK_WINDOW (pda->dialog), _("Paired Samples T Test"));
 
-      act->opt = tt_options_dialog_create (GTK_WINDOW (pda->toplevel));
+  act->pairs_treeview = get_widget_assert (xml, "paired-samples-t-test-treeview2");
+  act->list_store = GTK_LIST_STORE (gtk_tree_view_get_model (GTK_TREE_VIEW (act->pairs_treeview)));
 
-      g_signal_connect_swapped (button, "clicked", G_CALLBACK (tt_options_dialog_run), act->opt);
+  act->opt = tt_options_dialog_create (GTK_WINDOW (pda->toplevel));
 
+  g_signal_connect_swapped (button, "clicked", G_CALLBACK (tt_options_dialog_run), act->opt);
 
-      gtk_box_pack_start (GTK_BOX (bb), button, TRUE, TRUE, 5);
-      gtk_box_pack_start (GTK_BOX (box), bb, FALSE, FALSE, 5);
-      gtk_widget_show_all (box);
- 
 
-      psppire_dialog_action_set_valid_predicate (pda, dialog_state_valid);
-      psppire_dialog_action_set_refresh (pda, refresh);
+  gtk_box_pack_start (GTK_BOX (bb), button, TRUE, TRUE, 5);
+  gtk_box_pack_start (GTK_BOX (box), bb, FALSE, FALSE, 5);
+  gtk_widget_show_all (box);
 
-      g_object_set (pda->source,
-                   "predicate", var_is_numeric,
-                   NULL);
 
-      psppire_selector_set_select_func (PSPPIRE_SELECTOR (selector),
-                                       select_as_pair_member,
-                                       act);
-    }
-  
-  if (PSPPIRE_DIALOG_ACTION_CLASS (psppire_dialog_action_paired_parent_class)->activate)
-    PSPPIRE_DIALOG_ACTION_CLASS (psppire_dialog_action_paired_parent_class)->activate (pda);
+  psppire_dialog_action_set_valid_predicate (pda, dialog_state_valid);
+  psppire_dialog_action_set_refresh (pda, refresh);
+
+  g_object_set (pda->source,
+               "predicate", var_is_numeric,
+               NULL);
+
+  psppire_selector_set_select_func (PSPPIRE_SELECTOR (selector),
+                                   select_as_pair_member,
+                                   act);
+  return xml;
 }
 
 static void
@@ -212,7 +204,7 @@ static void
 psppire_dialog_action_paired_class_init (PsppireDialogActionPairedClass *class)
 {
   GObjectClass *object_class = G_OBJECT_CLASS (class);
-  psppire_dialog_action_set_activation (class, psppire_dialog_action_paired_activate);
+  PSPPIRE_DIALOG_ACTION_CLASS (class)->initial_activate = psppire_dialog_action_paired_activate;
   PSPPIRE_DIALOG_ACTION_CLASS (class)->generate_syntax = generate_syntax;
 
   object_class->finalize = psppire_dialog_action_paired_finalize;