[Kmymoney-devel] [kmymoney4] [Bug 328127] mymoneystatementreader.cpp matching problem

[allan]

https://bugs.kde.org/show_bug.cgi?id=328127

allan <agander93 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |CONFIRMED
     Ever confirmed|0                           |1

--- Comment #1 from allan <agander93 at gmail.com> ---
Not having an OFX source, I tested using the CSV importer, expecting that this
was a reasonable approach because the provided patch was to code common to all
import sources.  However, whereas the user received a message box, indicating
that the transaction would be dropped because no security was found, this did
not happen with CSV.  Instead, a new security was created.
This was traced to MyMoneyStatementReader::processSecurityEntry line 497 -
"} else if ((*it).tradingSymbol() == sec_in.m_strSymbol)" which was not finding
a match because the two symbols were of different case, and '.toLower()' needed
to be added to both sides.  The CSV importer now produced the same message box
as in OFX import.
The immediate cause of this was that, at the point were the patch had been
installed, the symbol there was empty.
Finally, this was traced to void
InvestProcessing::investCsvImport(MyMoneyStatement& st), where tr.m_strSymbol =
(*it_s).m_strSymbol needed to be added.

I'll be adding a patch for the full fix.

-- 
You are receiving this mail because:
You are the assignee for the bug.