From a345f992474c8c5c003c4636319bb83c354e9963 Mon Sep 17 00:00:00 2001 From: "G. Richardson" Date: Mon, 3 Dec 2012 12:05:17 +0100 Subject: [PATCH] Bug 1832 - GuV-Bericht Ist-Versteuerung wiederhergestellt MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit In dem Abschnitt ar sollen die Erlöse laut ac.amount relativ zu den Zahlungseingängen ausgewiesen werden. Dann kam eine Prüfung rein, ob der Rechnungsbetrag vielleicht 0 ist, um eine 0 im Nenner zu verhindern. Diese Prüfung greift aber sowohl bei Verkaufsrechnungen mit Betrag 0 als auch fälschlicherweise bei Eingangsrechnungen mit Betrag x. Diese Aufwände wurden dann mit 1 multipliziert und haben dann die Aufwände, die später im ap-Teil berechnet wurden, negiert, weshalb die Ausgaben alle 0 waren. Der Patch von Uwe Konrad macht also genau das Richtige, und ich habe noch ein paar überflüssige Prüfungen auf 0 rausgenommen. Damit sollte die Funktionalität wie von vor der 0-Prüfung wiederhergestellt sein. Ob dieses Verfahren dann noch richtig ist, wenn Bug 1793 angegangen wird, muß man dann noch sehen. --- SL/RP.pm | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/SL/RP.pm b/SL/RP.pm index a319cf59d..4bb82f2b2 100644 --- a/SL/RP.pm +++ b/SL/RP.pm @@ -516,12 +516,16 @@ sub get_accounts_g { if ($form->{method} eq 'cash') { $query = qq| - SELECT SUM( ac.amount * CASE WHEN COALESCE((SELECT amount FROM ar WHERE id = ac.trans_id and amount != 0 ), 0) != 0 THEN + SELECT SUM( ac.amount * CASE WHEN COALESCE((SELECT amount FROM ar WHERE id = ac.trans_id), 0) != 0 THEN + /* ar amount is not zero, so we can divide by amount */ (SELECT SUM(acc.amount) * -1 FROM acc_trans acc INNER JOIN chart c ON (acc.chart_id = c.id AND c.link LIKE '%AR_paid%') WHERE 1=1 $inwhere AND acc.trans_id = ac.trans_id) - / (SELECT amount FROM ar WHERE id = ac.trans_id and amount != 0 ) ELSE 1 END + / (SELECT amount FROM ar WHERE id = ac.trans_id) + ELSE 0 + /* ar amount is zero, or we are checking with a non-ar-transaction, so we return 0 in both cases as multiplicator of ac.amount */ + END ) AS amount, c.$category FROM acc_trans ac LEFT JOIN chart c ON (c.id = ac.chart_id) -- 2.20.1