Projekt

Allgemein

Profil

Fehler #416

Tests datev

Von Bernd Bleßmann vor fast 5 Jahren hinzugefügt. Vor fast 5 Jahren aktualisiert.

Status:
Gelöst
Priorität:
Normal
Zugewiesen an:
-
Zielversion:
-
Beginn:
24.01.2020
Abgabedatum:
% erledigt:

100%

Geschätzter Aufwand:

Beschreibung

Durch die Erweiterung "DATEV-Export Festschreibungskennzeichen für jede Buchung einzel setzen" (e4edba0587fc8224bd69047fe57dbf435b776aa6) gingen Tests kaputt (t/datev/datev_format_2018.t und t/datev/invoices.t). Ich habe mal Patches angehängt, aber noch nicht eingespielt, weil ich noch Fragen/Anmerkungen habe:

  • wieviele Felder sollen es sein? Im Moment sind es 120, vorher wurde der Datensatz auf 116 "aufgebohrt". 116 würden ausreichen, um das Festschreibungskennzeichen unterzubringen - ist 120 also eine weitere Anpassung?
  • SL::DATEV/CSV::_kivitendo_to_datev : hier wird noch versucht auf 116 aufzufüllen, aber es gibt im Array schon 120 Einträge. Deshalb kommt eine Warnung: "Negative repeat count does nothing at /var/www/kivitendo-bernd/t/../SL/DATEV/CSV.pm line 525.". Die Routine sollte angepasst werden oder weg
  • evtl. sollten noch Tests mit und ohne Kennzeichen rein, damit man merkt, wenn jmd. das kaputt macht
  • Im POD bei SL::DATEV steht, dass für locked false der default-Wert ist. Im CSV-Array ist der Default-Wert für locked 1

Dateien

datev_tests.diff (9,42 KB) datev_tests.diff Bernd Bleßmann, 24.01.2020 18:24

Zugehörige Revisionen

Revision 2d9b08c7 (diff)
Von Bernd Bleßmann vor fast 5 Jahren hinzugefügt

DATEV-Tests nach Erweiterung f. Festschreibungskennzeichen gefixed

Die Tests gingen mit
"DATEV-Export Festschreibungskennzeichen für jede Buchung einzel setzen"
(commit e4edba0587fc8224bd69047fe57dbf435b776aa6) kaputt.
Das Array hat jetzt mehr Einträge und das Festschreibunungskennzeichen ist an.

Behebt #416 (redmine).

Revision 08175832 (diff)
Von Bernd Bleßmann vor fast 5 Jahren hinzugefügt

S:DATEV:CSV: Kein Auffüllen des Arrays mehr

Durch die Änderungen zum Festschreibungskennzeichen
("DATEV-Export Festschreibungskennzeichen für jede Buchung einzel setzen"
commit e4edba0587fc8224bd69047fe57dbf435b776aa6) wurde das Array ohnehin auf
120 Einträge erweitert, so dass die Funktion zum Auffüllen auf 116 Einträge
weg kann.

Zudem wird dadurch eine Warnung vermieden ("Negative repeat count does nothing").

Refs #416 (redmine).

Historie

#1

Von Bernd Bleßmann vor fast 5 Jahren aktualisiert

  • Status wurde von Neu zu Gelöst geändert
  • % erledigt wurde von 0 zu 100 geändert

Status geändert durch Changeset kivitendo|2d9b08c723af0fbab5153ba6c8603db3a44cf98d.

Auch abrufbar als: Atom PDF