IntenseFinance/gnucash
3
0
Fork 0
mirror of https://github.com/Gnucash/gnucash.git synced 2025-02-25 18:55:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fix double-click for import-main-matcher.c

Browse Source 
The New multiselect feature used two boolean values that was set up as
pointers but there was a error with one if statement that stopped the
double-click to run the match dialogue. Changed all occurrences to just
use boolean values.
This commit is contained in:
Robert Fewell 2019-06-23 10:43:57 +01:00
parent b50ed47556
commit 262a1fac5d
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
gnucash/import-export/import-main-matcher.c
View File

@ -100,8 +100,8 @@ void on_matcher_help_close_clicked (GtkButton *button, gpointer user_data);
/* Local prototypes */ /* Local prototypes */
static void gnc_gen_trans_assign_transfer_account ( static void gnc_gen_trans_assign_transfer_account (
GtkTreeView *treeview, GtkTreeView *treeview,
gboolean *first, gboolean first,
gboolean *is_selection, gboolean is_selection,
GtkTreePath *path, GtkTreePath *path,
Account **new_acc, Account **new_acc,
GNCImportMainMatcher *info); GNCImportMainMatcher *info);
@ -377,8 +377,8 @@ gnc_gen_trans_update_toggled_cb (GtkCellRendererToggle *cell_renderer,
static void static void
gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview, gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
gboolean *first, gboolean first,
gboolean *is_selection, gboolean is_selection,
GtkTreePath *path, GtkTreePath *path,
Account **new_acc, Account **new_acc,
GNCImportMainMatcher *info) GNCImportMainMatcher *info)
@ -390,8 +390,8 @@ gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
gboolean ok_pressed; gboolean ok_pressed;
ENTER(""); ENTER("");
DEBUG("first = %s",*first?"true":"false"); DEBUG("first = %s", first ? "true" : "false");
DEBUG("is_selection = %s",*is_selection?"true":"false"); DEBUG("is_selection = %s", is_selection ? "true" : "false");
DEBUG("path = %s", gtk_tree_path_to_string (path)); DEBUG("path = %s", gtk_tree_path_to_string (path));
DEBUG("account passed in = %s", gnc_get_account_name_for_register (*new_acc)); DEBUG("account passed in = %s", gnc_get_account_name_for_register (*new_acc));
model = gtk_tree_view_get_model (treeview); model = gtk_tree_view_get_model (treeview);
@ -406,7 +406,7 @@ gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
{ {
ok_pressed = TRUE; ok_pressed = TRUE;
old_acc = gnc_import_TransInfo_get_destacc (trans_info); old_acc = gnc_import_TransInfo_get_destacc (trans_info);
if (*first) if (first)
{ {
ok_pressed = FALSE; ok_pressed = FALSE;
*new_acc = gnc_import_select_account (info->main_widget, *new_acc = gnc_import_select_account (info->main_widget,
@ -418,7 +418,7 @@ gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
ACCT_TYPE_NONE, ACCT_TYPE_NONE,
old_acc, old_acc,
&ok_pressed); &ok_pressed);
*first = FALSE; first = FALSE;
DEBUG("account selected = %s", DEBUG("account selected = %s",
gnc_account_get_full_name (*new_acc)); gnc_account_get_full_name (*new_acc));
} }
@ -480,7 +480,7 @@ gnc_gen_trans_assign_transfer_account_to_selection_cb (GtkMenuItem *menuitem,
DEBUG("passing account value = %s", DEBUG("passing account value = %s",
gnc_account_get_full_name (assigned_account)); gnc_account_get_full_name (assigned_account));
gnc_gen_trans_assign_transfer_account (treeview, gnc_gen_trans_assign_transfer_account (treeview,
&first, &is_selection, l->data, &assigned_account, info); first, is_selection, l->data, &assigned_account, info);
DEBUG("returned value of account = %s", DEBUG("returned value of account = %s",
gnc_account_get_full_name (assigned_account)); gnc_account_get_full_name (assigned_account));
DEBUG("returned value of first = %s",first?"true":"false"); DEBUG("returned value of first = %s",first?"true":"false");
@ -507,7 +507,7 @@ gnc_gen_trans_row_activated_cb (GtkTreeView *treeview,
first = TRUE; first = TRUE;
is_selection = FALSE; is_selection = FALSE;
gnc_gen_trans_assign_transfer_account (treeview, gnc_gen_trans_assign_transfer_account (treeview,
&first, &is_selection, path, first, is_selection, path,
&assigned_account, info); &assigned_account, info);
DEBUG("account returned = %s", gnc_account_get_full_name (assigned_account)); DEBUG("account returned = %s", gnc_account_get_full_name (assigned_account));
LEAVE(""); LEAVE("");