From d21f62b73b84086173901da197fe15eaa4e9dcce Mon Sep 17 00:00:00 2001 From: Gerald Combs Date: Fri, 8 Feb 2013 18:07:34 +0000 Subject: Add a directory name preference type (PREF_DIRNAME) and registration routine (prefs_register_directory_preference). Add PREF_FILENAME and PREF_DIRNAME support to the Qt module preferences. Change a couple of preferences to directory names. Clean up some names and default settings. svn path=/trunk/; revision=47573 --- ui/gtk/prefs_dlg.c | 38 ++++++++++--- ui/preference_utils.c | 4 ++ ui/qt/main_window_preferences_frame.ui | 2 +- ui/qt/module_preferences_scroll_area.cpp | 98 ++++++++++++++++++++++++++++---- ui/qt/module_preferences_scroll_area.h | 2 + ui/qt/preferences_dialog.cpp | 17 ++++-- 6 files changed, 135 insertions(+), 26 deletions(-) (limited to 'ui') diff --git a/ui/gtk/prefs_dlg.c b/ui/gtk/prefs_dlg.c index f3a2189593..fb6236c425 100644 --- a/ui/gtk/prefs_dlg.c +++ b/ui/gtk/prefs_dlg.c @@ -76,8 +76,8 @@ static gboolean prefs_main_delete_event_cb(GtkWidget *, GdkEvent *, gpointer); static void prefs_main_destroy_cb(GtkWidget *, gpointer); static void prefs_tree_select_cb(GtkTreeSelection *, gpointer); -static GtkWidget *create_preference_filename_entry(GtkWidget *, int, - const gchar *, const gchar *, char *); +static GtkWidget *create_preference_path_entry(GtkWidget *, int, + const gchar *, const gchar *, char *, gboolean); #define E_PREFSW_SCROLLW_KEY "prefsw_scrollw" #define E_PREFSW_TREE_KEY "prefsw_tree" @@ -213,10 +213,17 @@ pref_show(pref_t *pref, gpointer user_data) break; case PREF_FILENAME: - pref->control = create_preference_filename_entry(main_grid, pref->ordinal, + pref->control = create_preference_path_entry(main_grid, pref->ordinal, label_string, tooltip_txt, - pref->stashed_val.string); + pref->stashed_val.string, FALSE); + break; + + case PREF_DIRNAME: + pref->control = create_preference_path_entry(main_grid, pref->ordinal, + label_string, + tooltip_txt, + pref->stashed_val.string, TRUE); break; case PREF_RANGE: @@ -839,13 +846,21 @@ static void preference_filename_entry_cb(GtkWidget *button, GtkWidget *filename_te) { /* XXX - use a better browser dialog title */ - file_selection_browse(button, filename_te, "Wireshark: file preference", + file_selection_browse(button, filename_te, "Wireshark: File preference", FILE_SELECTION_READ_BROWSE); } +static void +preference_dirname_entry_cb(GtkWidget *button, GtkWidget *filename_te) +{ + /* XXX - use a better browser dialog title */ + file_selection_browse(button, filename_te, "Wireshark: Directory preference", + FILE_SELECTION_CREATE_FOLDER); +} + static GtkWidget * -create_preference_filename_entry(GtkWidget *main_grid, int grid_position, - const gchar *label_text, const gchar *tooltip_text, char *value) +create_preference_path_entry(GtkWidget *main_grid, int grid_position, + const gchar *label_text, const gchar *tooltip_text, char *value, gboolean dir_only) { GtkWidget *entry; GtkWidget *button, *file_bt_hb; @@ -867,8 +882,11 @@ create_preference_filename_entry(GtkWidget *main_grid, int grid_position, gtk_widget_set_tooltip_text(entry, tooltip_text); gtk_widget_show(entry); - g_signal_connect(button, "clicked", G_CALLBACK(preference_filename_entry_cb), entry); - + if (dir_only) { + g_signal_connect(button, "clicked", G_CALLBACK(preference_dirname_entry_cb), entry); + } else { + g_signal_connect(button, "clicked", G_CALLBACK(preference_filename_entry_cb), entry); + } return entry; } @@ -956,6 +974,7 @@ pref_check(pref_t *pref, gpointer user_data) case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: /* Value can't be bad. */ break; @@ -1054,6 +1073,7 @@ pref_fetch(pref_t *pref, gpointer user_data) case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: str_val = gtk_entry_get_text(GTK_ENTRY(pref->control)); if (strcmp(*pref->varp.string, str_val) != 0) { *pref_changed_p = TRUE; diff --git a/ui/preference_utils.c b/ui/preference_utils.c index 29e403f5f8..005817a10c 100644 --- a/ui/preference_utils.c +++ b/ui/preference_utils.c @@ -61,6 +61,7 @@ pref_stash(pref_t *pref, gpointer unused _U_) case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: g_free(pref->stashed_val.string); pref->stashed_val.string = g_strdup(*pref->varp.string); break; @@ -117,6 +118,7 @@ pref_unstash(pref_t *pref, gpointer changed_p) case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: if (strcmp(*pref->varp.string, pref->stashed_val.string) != 0) { *pref_changed_p = TRUE; g_free((void *)*pref->varp.string); @@ -166,6 +168,7 @@ reset_stashed_pref(pref_t *pref) { case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: g_free(pref->stashed_val.string); pref->stashed_val.string = g_strdup(pref->default_val.string); break; @@ -206,6 +209,7 @@ pref_clean_stash(pref_t *pref, gpointer unused _U_) case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: if (pref->stashed_val.string != NULL) { g_free(pref->stashed_val.string); pref->stashed_val.string = NULL; diff --git a/ui/qt/main_window_preferences_frame.ui b/ui/qt/main_window_preferences_frame.ui index 7a4e8af51c..42646127c3 100644 --- a/ui/qt/main_window_preferences_frame.ui +++ b/ui/qt/main_window_preferences_frame.ui @@ -70,7 +70,7 @@ - Choose... + Browse... diff --git a/ui/qt/module_preferences_scroll_area.cpp b/ui/qt/module_preferences_scroll_area.cpp index ada6e557d2..e30dbdbada 100644 --- a/ui/qt/module_preferences_scroll_area.cpp +++ b/ui/qt/module_preferences_scroll_area.cpp @@ -32,6 +32,7 @@ #include #include #include +#include #include #include #include @@ -152,6 +153,29 @@ pref_show(pref_t *pref, gpointer layout_ptr) vb->addLayout(hb); break; } + case PREF_FILENAME: + case PREF_DIRNAME: + { + vb->addWidget(new QLabel(pref->title)); + QHBoxLayout *hb = new QHBoxLayout(); + QLineEdit *path_le = new QLineEdit(); + QStyleOption style_opt; + path_le->setProperty(pref_prop_, qVariantFromValue(pref)); + path_le->setMinimumWidth(path_le->fontMetrics().height() * 20); + path_le->setStyleSheet(QString( + "QLineEdit {" + " margin-left: %1px;" + "}" + ) + .arg(path_le->style()->subElementRect(QStyle::SE_CheckBoxContents, &style_opt).left())); + hb->addWidget(path_le); + QPushButton *path_pb = new QPushButton("Browse..."); + path_pb->setProperty(pref_prop_, qVariantFromValue(pref)); + hb->addWidget(path_pb); + hb->addSpacerItem(new QSpacerItem(1, 1, QSizePolicy::Expanding, QSizePolicy::Minimum)); + vb->addLayout(hb); + break; + } case PREF_COLOR: { // XXX - Not needed yet. When it is needed we can add a label + QFrame which pops up a @@ -182,12 +206,20 @@ ModulePreferencesScrollArea::ModulePreferencesScrollArea(module_t *module, QWidg pref_t *pref = le->property(pref_prop_).value(); if (!pref) continue; - if (pref->type == PREF_UINT) { + switch (pref->type) { + case PREF_UINT: connect(le, SIGNAL(textEdited(QString)), this, SLOT(uintLineEditTextEdited(QString))); - } else if (pref->type == PREF_STRING) { + break; + case PREF_STRING: + case PREF_FILENAME: + case PREF_DIRNAME: connect(le, SIGNAL(textEdited(QString)), this, SLOT(stringLineEditTextEdited(QString))); - } else if (pref->type == PREF_RANGE) { + break; + case PREF_RANGE: connect(le, SIGNAL(textEdited(QString)), this, SLOT(rangeSyntaxLineEditTextEdited(QString))); + break; + default: + break; } } @@ -200,30 +232,34 @@ ModulePreferencesScrollArea::ModulePreferencesScrollArea(module_t *module, QWidg } } - foreach (QRadioButton *enum_rb, findChildren()) { - pref_t *pref = enum_rb->property(pref_prop_).value(); + foreach (QRadioButton *rb, findChildren()) { + pref_t *pref = rb->property(pref_prop_).value(); if (!pref) continue; if (pref->type == PREF_ENUM && pref->info.enum_info.radio_buttons) { - connect(enum_rb, SIGNAL(toggled(bool)), this, SLOT(enumRadioButtonToggled(bool))); + connect(rb, SIGNAL(toggled(bool)), this, SLOT(enumRadioButtonToggled(bool))); } } - foreach (QComboBox *enum_cb, findChildren()) { - pref_t *pref = enum_cb->property(pref_prop_).value(); + foreach (QComboBox *combo, findChildren()) { + pref_t *pref = combo->property(pref_prop_).value(); if (!pref) continue; if (pref->type == PREF_ENUM && !pref->info.enum_info.radio_buttons) { - connect(enum_cb, SIGNAL(currentIndexChanged(int)), this, SLOT(enumComboBoxCurrentIndexChanged(int))); + connect(combo, SIGNAL(currentIndexChanged(int)), this, SLOT(enumComboBoxCurrentIndexChanged(int))); } } - foreach (QPushButton *uat_pb, findChildren()) { - pref_t *pref = uat_pb->property(pref_prop_).value(); + foreach (QPushButton *pb, findChildren()) { + pref_t *pref = pb->property(pref_prop_).value(); if (!pref) continue; if (pref->type == PREF_UAT) { - connect(uat_pb, SIGNAL(pressed()), this, SLOT(uatPushButtonPressed())); + connect(pb, SIGNAL(pressed()), this, SLOT(uatPushButtonPressed())); + } else if (pref->type == PREF_FILENAME) { + connect(pb, SIGNAL(pressed()), this, SLOT(filenamePushButtonPressed())); + } else if (pref->type == PREF_DIRNAME) { + connect(pb, SIGNAL(pressed()), this, SLOT(dirnamePushButtonPressed())); } } @@ -400,6 +436,44 @@ void ModulePreferencesScrollArea::uatPushButtonPressed() qDebug() << "FIX Implement UATs:" << pref->title; } +void ModulePreferencesScrollArea::filenamePushButtonPressed() +{ + QPushButton *filename_pb = qobject_cast(sender()); + if (!filename_pb) return; + + pref_t *pref = filename_pb->property(pref_prop_).value(); + if (!pref) return; + + QString filename = QFileDialog::getSaveFileName(this, + QString("Wireshark: ") + pref->description, + pref->stashed_val.string); + + if (!filename.isEmpty()) { + g_free((void *)pref->stashed_val.string); + pref->stashed_val.string = qstring_strdup(filename); + updateWidgets(); + } +} + +void ModulePreferencesScrollArea::dirnamePushButtonPressed() +{ + QPushButton *dirname_pb = qobject_cast(sender()); + if (!dirname_pb) return; + + pref_t *pref = dirname_pb->property(pref_prop_).value(); + if (!pref) return; + + QString dirname = QFileDialog::getExistingDirectory(this, + QString("Wireshark: ") + pref->description, + pref->stashed_val.string); + + if (!dirname.isEmpty()) { + g_free((void *)pref->stashed_val.string); + pref->stashed_val.string = qstring_strdup(dirname); + updateWidgets(); + } +} + /* * Editor modelines * diff --git a/ui/qt/module_preferences_scroll_area.h b/ui/qt/module_preferences_scroll_area.h index 8a297ef86c..f5be4c7c1b 100644 --- a/ui/qt/module_preferences_scroll_area.h +++ b/ui/qt/module_preferences_scroll_area.h @@ -65,6 +65,8 @@ private slots: void stringLineEditTextEdited(const QString &new_str); void rangeSyntaxLineEditTextEdited(const QString &new_str); void uatPushButtonPressed(); + void filenamePushButtonPressed(); + void dirnamePushButtonPressed(); }; #endif // MODULE_PREFERENCES_SCROLL_AREA_H diff --git a/ui/qt/preferences_dialog.cpp b/ui/qt/preferences_dialog.cpp index 4c2da2c703..8332b37025 100644 --- a/ui/qt/preferences_dialog.cpp +++ b/ui/qt/preferences_dialog.cpp @@ -48,7 +48,6 @@ #include #include #include -#include #include #include @@ -427,6 +426,7 @@ bool PreferencesDialog::stashedPrefIsDefault(pref_t *pref) case PREF_STRING: case PREF_FILENAME: + case PREF_DIRNAME: if (!(g_strcmp0(pref->default_val.string, pref->stashed_val.string))) return true; break; @@ -597,10 +597,19 @@ void PreferencesDialog::on_advancedTree_itemActivated(QTreeWidgetItem *item, int break; } case PREF_FILENAME: + case PREF_DIRNAME: { - QString filename = QFileDialog::getSaveFileName(this, - QString("Wireshark: ") + pref->description, - pref->stashed_val.string); + QString filename; + + if (pref->type == PREF_FILENAME) { + filename = QFileDialog::getSaveFileName(this, + QString("Wireshark: ") + pref->description, + pref->stashed_val.string); + } else { + filename = QFileDialog::getExistingDirectory(this, + QString("Wireshark: ") + pref->description, + pref->stashed_val.string); + } if (!filename.isEmpty()) { g_free((void *)pref->stashed_val.string); pref->stashed_val.string = qstring_strdup(filename); -- cgit v1.2.1