Project

General

Profile

Actions

Fehler #699

closed

Einkaufsgutschrift mit Skonto per Kontenauszug verbuchen, erzeugt DATEV Fehler, bzw. eine Fehlbuchung

Added by Jan Büren 10 months ago. Updated 29 days ago.

Status:
Gelöst
Priority:
Normal
Assignee:
Target version:
Start date:
06/27/2024
Due date:
% Done:

0%

Estimated time:

Description

Der Skonto-Betrag wird mit dem falschen Vorzeichen verbucht.

Eigentlich sollte der Skonto-Betrag immer dasselbe Vorzeichen wie der Bank-Betrag haben.
Das passt auch zu (fast) allen bisherigen Testfällen außer diesem hier:

test_default_ap_transaction_two_charts_19_7_with_skonto (payment.t)

Im ersten Schritt wäre es gut, den Testfall einmal an der Oberfläche anzuschauen und dann die Prüfung genauer zu machen (wie sehen die acc_trans Einträge wirklich aus?)

Actions #1

Updated by Jan Büren 10 months ago

Ich hab jetzt im Kundenprojekt einen Hotfix der nur

next unless $skonto_booking->{'skonto_amount'} != 0;
- my $amount = -1 * $skonto_booking->{skonto_amount};
+ my $amount = $self->invoice_type eq 'purchase_credit_note' ? $skonto_booking->{skonto_amount} : -1 * $skonto_booking->{skonto_amount};

Alle aktuellen Tests laufen dagegen durch.

Warum das in diesem Fall logisch ein anderes Vorzeichen benötigt, ist mir allerdings (noch) schleierhaft.

jan@specht-kivitendo:/var/www/kivitendo-jan$ t/test.pl t/000setup_database.t t/db_helper/payment.t t/bank/bank_transactions.t
t/000setup_database.t ....... ok
t/db_helper/payment.t ....... ok
t/bank/bank_transactions.t .. ok
All tests successful.

Actions #2

Updated by Jan Büren 2 months ago

  • Target version set to Bugsprint Schönhagen
Actions #3

Updated by Max Wessendorf about 1 month ago

  • Status changed from Neu to In Bearbeitung
  • Assignee set to Jan Büren
Actions #4

Updated by Jan Büren 29 days ago

  • Status changed from In Bearbeitung to Gelöst
Actions

Also available in: Atom PDF