slanack: The problem was that <split:quantity> received the `transaction

commodity' instead of the correct `account commodity'.  There should really be
a check if the account commodity has been defined.

1 files changed, 3 insertions, 7 deletions

diff --git a/gnucash.cc b/gnucash.cc
index 0eb10219..7d31526d 100644
--- a/gnucash.cc
+++ b/gnucash.cc
@@ -162,13 +162,9 @@ static void endElement(void *userData, const char *name)
     // the transaction is worth.
     amount_t value;
     commodity_t * default_commodity = NULL;
-    if (entry_comm) {
-      default_commodity = entry_comm;
-    } else {
-      account_comm_map::iterator ac = account_comms.find(xact->account);
-      if (ac != account_comms.end())
-	default_commodity = (*ac).second;
-    }
+    account_comm_map::iterator ac = account_comms.find(xact->account);
+    if (ac != account_comms.end())
+      default_commodity = (*ac).second;
 
     if (default_commodity) {
       curr_quant.set_commodity(*default_commodity);