gnucash maint: Multiple changes pushed

[Geert Janssens]

Updated	 via  https://github.com/Gnucash/gnucash/commit/5bbfb8ef (commit)
	 via  https://github.com/Gnucash/gnucash/commit/34ed91ea (commit)
	 via  https://github.com/Gnucash/gnucash/commit/894f8241 (commit)
	 via  https://github.com/Gnucash/gnucash/commit/2d3d0506 (commit)
	 via  https://github.com/Gnucash/gnucash/commit/8e6fb15d (commit)
	 via  https://github.com/Gnucash/gnucash/commit/dd50c7af (commit)
	from  https://github.com/Gnucash/gnucash/commit/34be3c8b (commit)



commit 5bbfb8efd49f171b0674e04046cd8689155452f5
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Jan 29 23:11:49 2023 +0100

    Add proper indent to scheme changes from previous commit
    
    Committed seperately to make it easier to look at the actual changes.
    This commit only adds a 2 space indent to one function body.

commit 34ed91eac969a3fb5cff8802a38293abeedc6ba6
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Jan 29 22:50:44 2023 +0100

    Bug 798734 - Aging Reports don't handle mixed currency payments and invoices without Trading Accounts
    
    Revisit the fix. It still made assumptions about the way
    payment transactions were created. It can now handle
    payment transaction either in the payment account
    currency or in the post account currency.

commit 894f8241e127ce180eb74ee280a49c4a95ba88cf
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Jan 29 22:48:50 2023 +0100

    Revisit invoice payment in multi-currency context
    
    - Show proper amount in dialog when applying or editing an existing transaction as payment
    - Be more careful not to waste the existing payment split
    - If the user changed the payment amount while starting from an existing transaction
      unreconcile the changed payment split
    - Avoid needlessly changing transaction currency (only do so if the user chose
      a new transfer account and the old currency is neither the new transfer account's
      currency nor the post account's currency)

commit 2d3d05068d5a0d8f1349324616e37dc6fbc7f551
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Jan 29 11:37:54 2023 +0100

    Edit/assign payment - be more prudent not to lose data
    
    We can't handle the case where a transaction has splits in more than one
    APAR account. Instead of only warning that some of them will be lost
    refuse to continue and leave it to the user to fix the transaction first.

commit 8e6fb15d44a497b11962772fb6ce3d064ca8a8b7
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Jan 29 11:32:58 2023 +0100

    Improve a few messages related to business payments

commit dd50c7af13ed3954492e997884fa962b76df6328
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Sun Jan 29 11:31:24 2023 +0100

    Multi-currency - show split amount rather than transaction value
    
    This issue was found while evaluating bug 798734, but not
    the immediate topic of that bug.



Summary of changes:
 gnucash/gnome/dialog-payment.c      | 108 ++++++++++++++++------------
 gnucash/report/report-utilities.scm |  88 ++++++++++++-----------
 libgnucash/engine/gncOwner.c        | 140 ++++++++++++++++--------------------
 3 files changed, 172 insertions(+), 164 deletions(-)