Bug 415409

Summary: False detection of a matching investment cash dividend transaction
Product: [Applications] kmymoney Reporter: Thomas Baumgart <tbaumgart>
Component: onlinebankingAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 5.0.7   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://commits.kde.org/kmymoney/6ed6175828ac1b027d1120eb7fabd3e1d239e5c9 Version Fixed In: 5.0.8
Sentry Crash Report:

Description Thomas Baumgart 2019-12-20 17:53:49 UTC 
SUMMARY
Importing two cash dividend transactions that reference different investment accounts but have the same amount are treated as the same transaction and matched.

STEPS TO REPRODUCE
1. Import an OFX cash dividend transaction into the corresponding account
2. Import an OFX cash dividend transaction with the same amount into a different account

OBSERVED RESULT
Second transaction is matched with the first

EXPECTED RESULT
Two transactions in different accounts are shown
Comment 1 Thomas Baumgart 2019-12-20 17:59:54 UTC 
Git commit c17568b8c82b59ad72b07d2db4ab7181757b4bfe by Thomas Baumgart.
Committed on 20/12/2019 at 17:59.
Pushed by tbaumgart into branch '5.0'.

Assign bankid to correct split

The bankid used to detect duplicate transactions was assigned to the
wrong split (the one that identifies the category). This change places
the bankid in the split that references the investment account.
FIXED-IN: 5.0.8

M  +22   -20   kmymoney/converter/mymoneystatementreader.cpp

https://commits.kde.org/kmymoney/c17568b8c82b59ad72b07d2db4ab7181757b4bfe
Comment 2 Thomas Baumgart 2019-12-20 18:01:19 UTC 
Git commit 6ed6175828ac1b027d1120eb7fabd3e1d239e5c9 by Thomas Baumgart.
Committed on 20/12/2019 at 18:00.
Pushed by tbaumgart into branch 'master'.

Assign bankid to correct split

The bankid used to detect duplicate transactions was assigned to the
wrong split (the one that identifies the category). This change places
the bankid in the split that references the investment account.
FIXED-IN: 5.0.8
(cherry picked from commit c17568b8c82b59ad72b07d2db4ab7181757b4bfe)

M  +22   -20   kmymoney/converter/mymoneystatementreader.cpp

https://commits.kde.org/kmymoney/6ed6175828ac1b027d1120eb7fabd3e1d239e5c9