On 25/11/13 01:49, Chris Tucker wrote:
Vanguard Index FDS Mid-Cap Value Index (VOE)
@ Thomas please, below.
Hi Chris
I've had a quick look at your patch, and, while I've not sat and thought
about it in detail, it does look OK.
However, my first thought was 'How do I test it, as I don't have OFX?'
Then, your patch is for mymoneystatementreader.cpp, which is common to
all input methods, so I knocked up a simple CSV file. First, I entered
into KMM, your original security, with a single Buy transaction. Then,
I imported my CSV file containing a Dividend Reinvest for your imported
security. The CSV i9mporter, prior to the actual import, displays the
data from the file, to give the user the chance to edit security names
to correspond to existing securities. This recognises the existing
symbol and security, and doesn't show the new data. I proceed, and
continue the import. The import statement stats show one transaction
processed and one added, with no duplicates. Looking now at the ledger
for the import account, both transactions appear with their own distinct
security names, and the investment view contains both securities with
their own separate names. Back in the Ledger, starting a new
transaction gives a choice of either security, so everything seems OK,
unless you actually wanted the two securities to match, rather than be
independent. You say "...the transaction import fails to find the
security by name.", and it can't because it's dealing with a different
name, but it actually finds it by symbol.
I have not used your proposed patch. Forgive my asking, but I'm
assuming you tested your patch? Did it give the result you wanted, and
to be clear, you do want the two to match?
@ Thomas please, below.
The next point is of principal. I've always taken it that a symbol
needs to be unique within KMM, and I can see possible problems having
two different securities with the same symbol. Of course, this would
resolve if the two securities can actually match.
Allan
_______________________________________________
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel
