gnucash maint: Fix double-click for import-main-matcher.c
Robert Fewell
bobit at code.gnucash.org
Sun Jun 23 05:47:27 EDT 2019
Updated via https://github.com/Gnucash/gnucash/commit/262a1fac (commit)
from https://github.com/Gnucash/gnucash/commit/b50ed475 (commit)
commit 262a1fac5d910dd60e0eb0f3048a1dba9c9af8cd
Author: Robert Fewell <14uBobIT at gmail.com>
Date: Sun Jun 23 10:43:57 2019 +0100
Fix double-click for import-main-matcher.c
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.
diff --git a/gnucash/import-export/import-main-matcher.c b/gnucash/import-export/import-main-matcher.c
index c5aca00df..981bf8dee 100644
--- a/gnucash/import-export/import-main-matcher.c
+++ b/gnucash/import-export/import-main-matcher.c
@@ -100,8 +100,8 @@ void on_matcher_help_close_clicked (GtkButton *button, gpointer user_data);
/* Local prototypes */
static void gnc_gen_trans_assign_transfer_account (
GtkTreeView *treeview,
- gboolean *first,
- gboolean *is_selection,
+ gboolean first,
+ gboolean is_selection,
GtkTreePath *path,
Account **new_acc,
GNCImportMainMatcher *info);
@@ -377,8 +377,8 @@ gnc_gen_trans_update_toggled_cb (GtkCellRendererToggle *cell_renderer,
static void
gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
- gboolean *first,
- gboolean *is_selection,
+ gboolean first,
+ gboolean is_selection,
GtkTreePath *path,
Account **new_acc,
GNCImportMainMatcher *info)
@@ -390,8 +390,8 @@ gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
gboolean ok_pressed;
ENTER("");
- DEBUG("first = %s",*first?"true":"false");
- DEBUG("is_selection = %s",*is_selection?"true":"false");
+ DEBUG("first = %s", first ? "true" : "false");
+ DEBUG("is_selection = %s", is_selection ? "true" : "false");
DEBUG("path = %s", gtk_tree_path_to_string (path));
DEBUG("account passed in = %s", gnc_get_account_name_for_register (*new_acc));
model = gtk_tree_view_get_model (treeview);
@@ -406,7 +406,7 @@ gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
{
ok_pressed = TRUE;
old_acc = gnc_import_TransInfo_get_destacc (trans_info);
- if (*first)
+ if (first)
{
ok_pressed = FALSE;
*new_acc = gnc_import_select_account (info->main_widget,
@@ -418,7 +418,7 @@ gnc_gen_trans_assign_transfer_account (GtkTreeView *treeview,
ACCT_TYPE_NONE,
old_acc,
&ok_pressed);
- *first = FALSE;
+ first = FALSE;
DEBUG("account selected = %s",
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",
gnc_account_get_full_name (assigned_account));
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",
gnc_account_get_full_name (assigned_account));
DEBUG("returned value of first = %s",first?"true":"false");
@@ -507,7 +507,7 @@ gnc_gen_trans_row_activated_cb (GtkTreeView *treeview,
first = TRUE;
is_selection = FALSE;
gnc_gen_trans_assign_transfer_account (treeview,
- &first, &is_selection, path,
+ first, is_selection, path,
&assigned_account, info);
DEBUG("account returned = %s", gnc_account_get_full_name (assigned_account));
LEAVE("");
Summary of changes:
gnucash/import-export/import-main-matcher.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
More information about the gnucash-changes
mailing list