https://bugs.kde.org/show_bug.cgi?id=328127
allan <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |CONFIRMED Ever confirmed|0 |1 --- Comment #1 from allan <[email protected]> --- 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. _______________________________________________ KMyMoney-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kmymoney-devel
