r17043 - gnucash/trunk/src/import-export - Avoid critical warnings in the import account matcher dialog.
Andreas Köhler
andi5 at cvs.gnucash.org
Mon Mar 24 13:02:39 EDT 2008
Author: andi5
Date: 2008-03-24 13:02:38 -0400 (Mon, 24 Mar 2008)
New Revision: 17043
Trac: http://svn.gnucash.org/trac/changeset/17043
Modified:
gnucash/trunk/src/import-export/import-account-matcher.c
Log:
Avoid critical warnings in the import account matcher dialog.
If no account is selected, simply do not call engine functions on it.
Modified: gnucash/trunk/src/import-export/import-account-matcher.c
===================================================================
--- gnucash/trunk/src/import-export/import-account-matcher.c 2008-03-23 22:09:22 UTC (rev 17042)
+++ gnucash/trunk/src/import-export/import-account-matcher.c 2008-03-24 17:02:38 UTC (rev 17043)
@@ -228,17 +228,19 @@
switch (response) {
case GTK_RESPONSE_OK:
retval = gnc_tree_view_account_get_selected_account(picker->account_tree);
- retval_name = xaccAccountGetName(retval);
- DEBUG("Selected account %p, %s", retval,
- retval_name ? retval_name : "(null)");
+ if (retval)
+ retval_name = xaccAccountGetName(retval);
+ if (!retval_name)
+ retval_name = "(null)";
+ DEBUG("Selected account %p, %s", retval, retval_name);
/* See if the selected account is a placeholder. */
- if (xaccAccountGetPlaceholder (retval)) {
+ if (retval && xaccAccountGetPlaceholder (retval)) {
gnc_error_dialog
(picker->dialog,
_("The account %s is a placeholder account and does not allow "
"transactions. Please choose a different account."),
- retval_name ? retval_name : "(null)");
+ retval_name);
response = GNC_RESPONSE_NEW;
break;
}
More information about the gnucash-changes
mailing list