From: Jan Büren <jan@kivitendo.de>
Date: Thu, 14 Jan 2021 13:07:24 +0000 (+0100)
Subject: Kommentar zum Payment-Helper
X-Git-Tag: kivitendo-mebil_0.1-0~9^2~526
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=524bc23eb0c179bfa2acbf6c6f00dce3788fccc7;p=kivitendo-erp.git

Kommentar zum Payment-Helper

Außer in den Testfällen wird der Helper nicht mit einem
negativen Amount aufgerufen. Im Helper befindet sich etwas
zuviel automagie. Eigentlich braucht man den Param amount nicht,
wird aber überall in kivi übergeben. Eigentlich darf der negativ
sein, verfälscht aber das multiplizieren im Helper.
---

diff --git a/SL/DB/Helper/Payment.pm b/SL/DB/Helper/Payment.pm
index a778e927f..6dee0918f 100644
--- a/SL/DB/Helper/Payment.pm
+++ b/SL/DB/Helper/Payment.pm
@@ -778,7 +778,10 @@ to be lower than the payment or open invoice amount.
 
 Transdate can either be a date object or a date string.
 Chart_id is the id of the payment booking chart.
-Amount is either a postive or negative number, but never 0.
+Amount is either a positive or negative number, but never 0.
+
+CAVEAT! The helper tries to get the sign right and all calls from BankTransaction are
+positive (abs($value)) values.
 
 
 Example: