gnucash maint: Remove automatch_store_transactions() from import-main-matcher.
John Ralls
jralls at code.gnucash.org
Thu Jun 23 18:06:43 EDT 2016
Updated via https://github.com/Gnucash/gnucash/commit/53b45262 (commit)
from https://github.com/Gnucash/gnucash/commit/72f630e8 (commit)
commit 53b4526262111c2cae68625ce352b8c749163969
Author: John Ralls <jralls at ceridwen.us>
Date: Thu Jun 23 14:50:36 2016 -0700
Remove automatch_store_transactions() from import-main-matcher.
There hasn't been a clist to iterate through since 2002, calling
gnc_import_TransInfo_refresh_destacc immediately after
gnc_import_TransInfo_set_destacc(...TRUE) is a no-op, and refresh_model_row
gets called immediately after run_account_picker_dialog_returns.
As a side benefit this might fix bug 757078.
diff --git a/src/import-export/import-main-matcher.c b/src/import-export/import-main-matcher.c
index 3d60557..414da3e 100644
--- a/src/import-export/import-main-matcher.c
+++ b/src/import-export/import-main-matcher.c
@@ -96,11 +96,6 @@ void on_matcher_help_close_clicked (GtkButton *button, gpointer user_data);
/* Local prototypes */
static void
-automatch_store_transactions(GNCImportMainMatcher *info,
- GtkTreeModel *model,
- GtkTreeIter *iter,
- GNCImportTransInfo *trans_info);
-static void
refresh_model_row(GNCImportMainMatcher *gui, GtkTreeModel *model,
GtkTreeIter *iter, GNCImportTransInfo *info);
@@ -258,14 +253,7 @@ run_account_picker_dialog (GNCImportMainMatcher *info,
old_acc,
&ok_pressed);
if (ok_pressed)
- {
- gnc_import_TransInfo_set_destacc (trans_info,
- new_acc,
- TRUE);
-
- /* Iterate through the transactions in a given clist to auto match them */
- automatch_store_transactions(info, model, iter, trans_info);
- }
+ gnc_import_TransInfo_set_destacc (trans_info, new_acc, TRUE);
}
static void
@@ -914,20 +902,6 @@ void gnc_gen_trans_list_add_trans_with_ref_id(GNCImportMainMatcher *gui, Transac
return;
}/* end gnc_import_add_trans_with_ref_id() */
-/* Iterate through the rows of the clist and try to automatch each of them */
-static void
-automatch_store_transactions (GNCImportMainMatcher *info,
- GtkTreeModel *model,
- GtkTreeIter *iter,
- GNCImportTransInfo *trans_info)
-{
- /* returns TRUE if we changed this row, so update it */
- if (gnc_import_TransInfo_refresh_destacc(trans_info, NULL))
- {
- refresh_model_row(info, model, iter, trans_info);
- }
-}
-
GtkWidget *gnc_gen_trans_list_widget (GNCImportMainMatcher *info)
{
g_assert(info);
Summary of changes:
src/import-export/import-main-matcher.c | 28 +---------------------------
1 file changed, 1 insertion(+), 27 deletions(-)
More information about the gnucash-changes
mailing list