----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129739/ -----------------------------------------------------------
(Updated Sty 7, 2017, 8:21 po południu) Review request for KMymoney. Repository: kmymoney Description (updated) ------- 1) "amount" (last column) has been replaced by "ending balance" because it's calculated specially for this report by QueryTable::constructPerformanceRow, 2) "return on investment" column is being calculated also when "ending balance" is zero, because zero ending balance doesn't invalidate ROI 3) "buys" and "sells" column take "fee amounts" into account during ROI calculation, because fees are inseparable from investments, 4) "buys" column has minus sign and "sells" column has plus sing, because the same notatnion is used by KMM importers. Diffs (updated) ----- kmymoney/reports/listtable.cpp 72b605f kmymoney/reports/querytable.cpp e44f74c Diff: https://git.reviewboard.kde.org/r/129739/diff/ Testing ------- Reports on stock securities with dividends, completely, partially and not sold at all. Thanks, Łukasz Wojniłowicz
