gnucash maint: [new-owner-report] modify report row->value accessor

[Christopher Lam]

Updated	 via  https://github.com/Gnucash/gnucash/commit/f0701da8 (commit)
	from  https://github.com/Gnucash/gnucash/commit/a4773d2b (commit)



commit f0701da8685ec0b8bbd593099a561fb9cc60503f
Author: Christopher Lam <christopher.lck at gmail.com>
Date:   Sat Nov 9 06:14:17 2019 +0800

    [new-owner-report] modify report row->value accessor
    
    Previously each owner-table row's value would access split's
    amount. But this fails for payments spanning multiple invoices,
    because these (with a single split to bank) will have a split for each
    invoice. Hence split->amount gets only 1 invoice payment amount.
    
    This change modifies to obtain the sum total of split->amounts in the
    APAR account thus handles multi-invoice payments correctly.

diff --git a/gnucash/report/business-reports/new-owner-report.scm b/gnucash/report/business-reports/new-owner-report.scm
index 549601f78..c99dee194 100644
--- a/gnucash/report/business-reports/new-owner-report.scm
+++ b/gnucash/report/business-reports/new-owner-report.scm
@@ -447,7 +447,7 @@
       (let* ((split (car splits))
              (txn (xaccSplitGetParent split))
              (date (xaccTransGetDate txn))
-             (value (xaccSplitGetAmount split))
+             (value (xaccTransGetAccountValue txn acc))
              (value (if payable? (- value) value))
              (invoice (gncInvoiceGetInvoiceFromTxn txn))
              (invoice-splits



Summary of changes:
 gnucash/report/business-reports/new-owner-report.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)