search

TheInfiniteKind / moneydance_open

17 stars 12 forks source link

Bug: Tools/currency/security history screen allows duplicate price history records (snapshots) for the same date. This allows ‘orphans’... #54

Open yogi1967 opened 3 years ago

yogi1967 commented 3 years ago

Bug - Tools/currency/security history screen allows duplicate price history records (snapshots) for the same date. This allows ‘phantoms’... Refer to link for details:

https://infinitekind.tenderapp.com/discussions/problems/66755-bug-toolscurrencysecurity-history-screen-allows-duplicate-price-history-records-snapshots-for-the-same-date

When this occurs, you can delete the dated record and then the one hidden 'behind it' appears.....

yogi1967 commented 2 years ago

I suspect this is because the snapshots are stored in a TreeSet() which holds only one unique key at a time. Hence, you add a new snap for the same date, the treeset stores it and drops the reference to the already existing one, but it still exists in the dataset. The code needs to check for the presence of the date in the treeset before adding it.