Bug 358129

Summary:	Changing dividend transaction to reinvested dividend transaction after import of QIF/QFX file on online banking update.
Product:	[Applications] kmymoney	Reporter:	Joe Byers <ecjbosu>
Component:	general	Assignee:	KMyMoney Devel Mailing List <kmymoney-devel>
Status:	RESOLVED UNMAINTAINED
Severity:	normal	CC:	agander93, lukasz.wojnilowicz, ostroffjh
Priority:	NOR
Version:	4.7.1
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

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 wojnilowicz 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.