Bug 358129 - Changing dividend transaction to reinvested dividend transaction after import of QIF/QFX file on online banking update.
Summary: Changing dividend transaction to reinvested dividend transaction after import...
Status: RESOLVED UNMAINTAINED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: 4.7.1
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-17 15:06 UTC by Joe Byers
Modified: 2018-01-08 21:23 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joe Byers 2016-01-17 15:06:24 UTC
I have used the import and online banking updates for my investment
accounts.  There is a problem with the cash sweep/money market and other
transactions that are imported as a Dividend activity.

A dividend requires a brokerage  account, a reinvested dividend does
not.  I try and change the dividend to a reinvestment which causes the
transaction to be flagged error needing an account.  It seems that logic
does not delete the account field for the dividend transaction during
the update, even if I delete from the gui.

The only way I can get this in the system is to manually enter the
transaction adn delete the imported one.  I am glad to work with the
team on this.

As suggested by AGA, I entered a manual test dividend transaction.  I then edited to a reinvested dividend transaction, updating the required fields.  This worked as expected with no errors like the one  mentioned above.  To close the test, I edited the transaction back to a dividend transactions.  This also completed with no issues.



Reproducible: Always

Steps to Reproduce:
1. Import a QIF or OFX either as a local file or using the online banking OFX update with a divided transaction in it.
2. Try and update the dividend transaction to a reinvested dividend tranaction type, updating the required fields
3.  The edited transaction will flag an error.

Actual Results:  
There is no way to clear the account field that the dividend transaction type requires, but is not required by the reinvested dividend transaction type.

Expected Results:  
The account field that the dividend transaction type requires will be cleared or deleted..
Comment 1 allan 2016-01-18 00:08:36 UTC
> As suggested by AGA, I entered a manual test dividend transaction. I then edited to a reinvested dividend > transaction, updating the required fields. This worked as expected with no errors like the one mentioned 
> above. To close the test, I edited the transaction back to a dividend transactions. This also completed with > no issues.

Oh, that is not what I was expecting.  I had just tested with 4.7.2, and, as I recollect, manual entry resulted in the same problem.  I'll have to go back, later, and check.  Meanwhile, testing with the development branch, and my patch for this problem, all seems OK, so far.

After some more testing, I'll publish this patch and, hopefully, go via Reviewboard to commit.
Comment 2 Joe Byers 2016-12-24 17:52:20 UTC
Can we revisit this for year end?  It also has a problem with online banking.

I have transactions generated by an online banking update that are dividend transactions.  I need to change them to reinvest dividend Transactions.

When I do this, I get an missing assignment issue flagged.  My only fix is to entered a new reinvest dividend transaction and delete the merged transactions.  This defeats the purpose of the automation:).

I appreciate the help and Happy Holidays.
Comment 3 Jack 2016-12-24 18:03:12 UTC
First, what version are you using, and on what platform?  I have plenty of OFX transactions that import as dividend, that I change to reinvest-dividend, and it works fine for me.  Can you provide the full text of the error?
Comment 4 Joe Byers 2016-12-26 13:44:13 UTC
Jack,

I am in kmymoney version 4.7.2 on a fedora 24 linux, kde version 4.14.26.
Comment 5 Joe Byers 2016-12-29 15:03:42 UTC
I have upgraded to version 4.8 on fedora 24 linux workstations and will test with online updates over the weekend.
Comment 6 NSLW 2017-08-27 05:18:14 UTC
KMM 4.7.1 is old and unmaintained. Please update to 4.8. Besides:

(In reply to Jack from comment #3)
> [...]  I have plenty of
> OFX transactions that import as dividend, that I change to
> reinvest-dividend, and it works fine for me.  [...]
Comment 7 Joe Byers 2018-01-08 21:23:45 UTC
See similar bug 388700.