[Kmymoney-devel] Fwd: extragear/office/kmymoney/kmymoney/converter

[Cristian Oneț]

Any ideas why automatic closing of the bug report does not work?


---------- Forwarded message ----------
From: Allan Anderson <agander93 at gmail.com>
Date: Fri, May 27, 2011 at 12:58 PM
Subject: extragear/office/kmymoney/kmymoney/converter
To: kde-commits at kde.org


SVN commit 1233773 by allananderson:

The account contained in the 'L' record gets passed correctly from
kmymoney/kmymoney/converter/mymoneyqifreader.cpp to mymoneystatementreader.cpp,
as 'brokerageaccount'.  There, however, the 'brokerageactid' does not get set
correctly.
BUG: 273950
REVIEW: 6688


 M  +3 -3      mymoneystatementreader.cpp


--- trunk/extragear/office/kmymoney/kmymoney/converter/mymoneystatementreader.cpp
#1233772:1233773
@@ -220,7 +220,7 @@
  // For testing, save the statement to an XML file
  // (uncomment this line)
  //
-  //MyMoneyStatement::writeXMLFile(s,"Imported.Xml");
+  MyMoneyStatement::writeXMLFile(s,"Imported.Xml");///

  //
  // Select the account
@@ -542,7 +542,7 @@
    // determine the brokerage account
    brokerageactid = m_account.value("kmm-brokerage-account").toUtf8();
    if (brokerageactid.isEmpty()) {
-      brokerageactid = file->accountByName(m_account.brokerageName()).id();
+      brokerageactid = file->nameToAccount(t_in.m_strBrokerageAccount);
    }

    // find the security transacted, UNLESS this transaction didn't
@@ -1031,7 +1031,7 @@
  if (! transfervalue.isZero()) {
    // in case the transaction has a reference to the brokerage
account, we use it
    if (!t_in.m_strBrokerageAccount.isEmpty()) {
-      brokerageactid = file->accountByName(t_in.m_strBrokerageAccount).id();
+      brokerageactid = file->nameToAccount(t_in.m_strBrokerageAccount);
    }

    if (!brokerageactid.isEmpty()) {