After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 512497 - Investment transactions ignore payee/memo mappings
Investment transactions ignore payee/memo mappings
Status: VERIFIED FIXED
Product: GnuCash
Classification: Other
Component: Import - QIF
2.2.x
Other All
: Normal normal
: ---
Assigned To: Derek Atkins
Derek Atkins
Depends on:
Blocks: backport
 
 
Reported: 2008-01-28 06:04 UTC by Charles Day
Modified: 2018-06-29 22:00 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
QIF file demonstrating the bug (293 bytes, text/plain)
2008-01-28 06:06 UTC, Charles Day 		  Details
Proposed patch (293 bytes, patch)
2008-01-28 11:13 UTC, Charles Day 		none Details | Review
Proposed patch (ignore previous) (2.07 KB, patch)
2008-01-28 11:17 UTC, Charles Day 		committed Details | Review

Description Charles Day 2008-01-28 06:04:25 UTC 
Please describe the problem:
The druid asks the user to create mappings of payees and memos to accounts. This mapping is supposed to be used when an account or category is missing. These mappings are being ignored for investment transactions.

Steps to reproduce:
1. Begin importing the attached QIF file.
2. At the druid's "Match payees/memos to GnuCash accounts" step, change each entry to something other than "Unspecified".
3. Finish importing the QIF file.


Actual results:
The imported transactions all use the Unspecified account.

Expected results:
The imported transactions use the accounts specified in the "Match payees/memos to GnuCash accounts" step.

Does this happen every time?
Yes.

Other information:
Comment 1 Charles Day 2008-01-28 06:06:38 UTC 
Created attachment 103859 [details]
QIF file demonstrating the bug
Comment 2 Charles Day 2008-01-28 11:13:02 UTC 
Created attachment 103870 [details] [review]
Proposed patch

I've tested this patch on several different types of investment transactions. It works on those that require an account or category, such as BuyX, SellX, and XIn. Transactions that don't use accounts or categories, such as Buy and Sell, are unaffected (as expected).
Comment 3 Charles Day 2008-01-28 11:17:08 UTC 
Created attachment 103871 [details] [review]
Proposed patch (ignore previous)

Whoops... I attached the wrong file previously. This is the right one.
Comment 4 Josh Sled 2008-02-01 00:54:56 UTC 
trunk@16906, awaiting backport for 2.2.4.  Thanks!
Comment 5 Andreas Köhler 2008-02-03 02:19:47 UTC 
Applied to branches/2.2 as r16917 for GnuCash 2.2.4.
Thanks a lot!
Comment 6 John Ralls 2018-06-29 22:00:15 UTC 
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=512497. Please update any external references or bookmarks.