[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Pspp-cvs] pspp/src/ui/gui ChangeLog compute-dialog.c
From: |
John Darrington |
Subject: |
[Pspp-cvs] pspp/src/ui/gui ChangeLog compute-dialog.c |
Date: |
Wed, 11 Jul 2007 03:59:57 +0000 |
CVSROOT: /sources/pspp
Module name: pspp
Changes by: John Darrington <jmd> 07/07/11 03:59:57
Modified files:
src/ui/gui : ChangeLog compute-dialog.c
Log message:
Made type-and-label subdialog behave rationally when the target
variable
already exists.
CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/pspp/src/ui/gui/ChangeLog?cvsroot=pspp&r1=1.66&r2=1.67
http://cvs.savannah.gnu.org/viewcvs/pspp/src/ui/gui/compute-dialog.c?cvsroot=pspp&r1=1.6&r2=1.7
Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/pspp/pspp/src/ui/gui/ChangeLog,v
retrieving revision 1.66
retrieving revision 1.67
diff -u -b -r1.66 -r1.67
--- ChangeLog 9 Jul 2007 05:21:12 -0000 1.66
+++ ChangeLog 11 Jul 2007 03:59:57 -0000 1.67
@@ -1,3 +1,7 @@
+2007-07-11 John Darrington <address@hidden>
+
+ * compute-dialog.c : Set type-and-label dialog to sensible state,
and made syntax generation depend upon existence of target variable.
+
2007-07-08 John Darrington <address@hidden>
* data-editor.c data-editor.glade data-editor.h: Implemented
Index: compute-dialog.c
===================================================================
RCS file: /sources/pspp/pspp/src/ui/gui/compute-dialog.c,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -b -r1.6 -r1.7
--- compute-dialog.c 7 Jul 2007 09:23:52 -0000 1.6
+++ compute-dialog.c 11 Jul 2007 03:59:57 -0000 1.7
@@ -47,6 +47,7 @@
struct compute_dialog
{
GladeXML *xml; /* The xml that generated the widgets */
+ PsppireDict *dict;
};
@@ -177,13 +178,17 @@
string = g_string_sized_new (64);
+ if ( NULL == psppire_dict_lookup_var (cd->dict, target_name ))
+ {
if ( gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (string_toggle)))
{
const char *w = gtk_entry_get_text (GTK_ENTRY(width_entry));
- g_string_append_printf (string, "STRING %s (a%s).\n", target_name, w);
+ g_string_append_printf (string,
+ "STRING %s (a%s).\n", target_name, w);
}
else
g_string_append_printf (string, "NUMERIC %s.\n", target_name);
+ }
if ( gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (user_label_toggle)))
label = gtk_entry_get_text (GTK_ENTRY (label_entry));
@@ -208,6 +213,74 @@
}
static void
+reset_type_label_dialog (struct compute_dialog *cd)
+{
+ const gchar *target_name;
+ struct variable *target_var;
+
+ GtkWidget *width_entry =
+ get_widget_assert (cd->xml, "type-and-label-width");
+
+ GtkWidget *label_entry =
+ get_widget_assert (cd->xml, "type-and-label-label-entry");
+
+ GtkWidget *numeric_target =
+ get_widget_assert (cd->xml, "radio-button-numeric");
+
+ GtkWidget *string_target =
+ get_widget_assert (cd->xml, "radio-button-string");
+
+
+ target_name = gtk_entry_get_text
+ (GTK_ENTRY (get_widget_assert (cd->xml, "compute-entry1")));
+
+
+ if ( (target_var = psppire_dict_lookup_var (cd->dict, target_name)) )
+ {
+ /* Existing Variable */
+ const gchar *label ;
+ GtkWidget *user_label =
+ get_widget_assert (cd->xml, "radio-button-user-label");
+
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (user_label), TRUE);
+
+ label = var_get_label (target_var);
+
+ if ( label )
+ {
+ gtk_entry_set_text (GTK_ENTRY (label_entry), label);
+ }
+
+ gtk_widget_set_sensitive (width_entry, FALSE);
+
+ if ( var_is_numeric (target_var))
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (numeric_target),
+ TRUE);
+ else
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (string_target),
+ TRUE);
+
+ gtk_widget_set_sensitive (numeric_target, FALSE);
+ gtk_widget_set_sensitive (string_target, FALSE);
+ }
+ else
+ {
+ GtkWidget *expression =
+ get_widget_assert (cd->xml, "radio-button-expression-label");
+
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (expression), TRUE);
+
+ gtk_widget_set_sensitive (width_entry, TRUE);
+ gtk_widget_set_sensitive (numeric_target, TRUE);
+ gtk_widget_set_sensitive (string_target, TRUE);
+
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (numeric_target),
+ TRUE);
+ }
+
+}
+
+static void
run_type_label_dialog (GtkButton *b, gpointer data)
{
struct compute_dialog *cd = data;
@@ -218,10 +291,41 @@
gtk_window_set_transient_for (GTK_WINDOW (subdialog), GTK_WINDOW (dialog));
+ reset_type_label_dialog (cd);
response = psppire_dialog_run (PSPPIRE_DIALOG (subdialog));
}
+static void
+on_expression_toggle (GtkToggleButton *button, gpointer data)
+{
+ struct compute_dialog *cd = data;
+
+ GtkWidget *entry =
+ get_widget_assert (cd->xml, "type-and-label-label-entry");
+
+ if ( gtk_toggle_button_get_active (button))
+ {
+ gtk_entry_set_text (GTK_ENTRY (entry), "");
+ gtk_widget_set_sensitive (entry, FALSE);
+ }
+ else
+ {
+ const char *label;
+ struct variable *target_var;
+ const gchar *target_name = gtk_entry_get_text
+ (GTK_ENTRY (get_widget_assert (cd->xml, "compute-entry1")));
+
+ target_var = psppire_dict_lookup_var (cd->dict, target_name);
+ label = var_get_label (target_var);
+
+ if ( label )
+ gtk_entry_set_text (GTK_ENTRY (entry), label);
+
+ gtk_widget_set_sensitive (entry, TRUE);
+ }
+}
+
/* Pops up the Compute dialog box */
void
compute_dialog (GObject *o, gpointer data)
@@ -245,11 +349,19 @@
GtkWidget *func_selector = get_widget_assert (xml, "compute-selector2");
GtkWidget *type_and_label = get_widget_assert (xml, "compute-button1");
+ GtkWidget *expression =
+ get_widget_assert (xml, "radio-button-expression-label");
+
+
GtkSheet *var_sheet =
GTK_SHEET (get_widget_assert (de->xml, "variable_sheet"));
vs = PSPPIRE_VAR_STORE (gtk_sheet_get_model (var_sheet));
+ scd.dict = vs->dict;
+
+ g_signal_connect (expression, "toggled",
+ G_CALLBACK(on_expression_toggle), &scd);
gtk_window_set_transient_for (GTK_WINDOW (dialog), de->parent.window);
@@ -291,6 +403,7 @@
+
response = psppire_dialog_run (PSPPIRE_DIALOG (dialog));
- [Pspp-cvs] pspp/src/ui/gui ChangeLog compute-dialog.c,
John Darrington <=