Summary: | QIF investment import containing L[parent:sub-account] flagged | ||
---|---|---|---|
Product: | [Applications] kmymoney | Reporter: | timothy <egroegboyle> |
Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | agander93 |
Priority: | NOR | ||
Version: | 4.5 | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: | patch file |
Description
timothy
2011-05-23 11:35:52 UTC
Created attachment 60290 [details]
patch file
Proposed fix for this bug.
Can you submit it through reviewboard, if possible? (In reply to comment #0) > Version: 4.5 (using KDE 1.2) > OS: Linux > > This query/response from Allan (CSV importer plugin author) & Thomas Baumgart > explains the problem better than I can. <snip> > Actual Results: > The file is imported without problems. However in the ledger view of the > investment account/securityA, the transaction is flagged (yellow > triangle/exclamation). The account field is blank and no money is transferred > to the chq account as above <snip> 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. The attached patch appears to fix the problem. With the patch applied, an 'L' record of 'L[Bank-Accounts:chq]' will generate automatically those two accounts and correctly transfer money into/from 'Bank-Accounts:chq' . However the asset account automatically created is of type 'checking'. So, to achieve what the OP requires, an account of type 'asset' first needs to be created. Then, the required result is obtained. If both the 'Bank-Accounts' and 'chq' accounts are created manually, the same result is achieved. Tested with buy, sell and dividend transactions. Both QIF and CSV imports now work correctly. (In reply to comment #2) > Can you submit it through reviewboard, if possible? OK On Wed, 25 May 2011 01:51:02 +0200 (CEST) allan <agander93@gmail.com> wrote: > https://bugs.kde.org/show_bug.cgi?id=273950 > > > > > > --- Comment #1 from allan <agander93 gmail com> 2011-05-25 01:51:00 > --- Created an attachment (id=60290) > --> (http://bugs.kde.org/attachment.cgi?id=60290) > patch file > > Proposed fix for this bug. > The patch included with the reviewboard entry should be used, rather than this one. Logically, they are identical, but the header kdesvn produced was incomplete. Allan |