projekt kivitendo

Kosmetik, bessere Beschreibung für neuen SelfTests (#24)

Copy und Paste-Fehler bei Beschreibung im Positiv-Testfall
übersehen, sowie Einrückungen verbessert.

Weiteren Transaction-Selftest

Falls ein datepaid im Nebenbuch gesetzt ist, so darf paid nicht
auf 0 sein, wenn es keine Zahlungen in der acc_trans hierzu gibt.

»with_transaction« anstelle von »do_transaction« verwenden

Es sollte so selten wie möglich »do_transaction« verwndet werden, damit
man sich immer angewöhnt, »with_transaction« zu nutzen.

Hintergründe und Unterschiede zwischen den beiden Funktionen sind in der...

Rechnungsmassenerstellen: keine DB-Transaktion um convert_to_invoice()

convert_to_invoice() macht selber intern bereits eine Transaktion auf;
daher ist die außen unnötig.

Weiterhin waren die Parameter merkwürdig übergeben:

• Für eine On-The-Fly-Unterscheidung für »diese Parameter« vs. »keine...

CsvImport: deutlich anzeigen, ob Vorschau oder echter Import

CsvImport: Spaltenzuordnung für non multiplexed

Hintergrundjob periodische Rechnungen: bei Fehlern Fehlermeldungen ausgeben, …

… nicht die Anzahl der Fehler.

SelfTests: Weitere Randfälle zumindestens in der Analyse-Ausgabe erwähnen

Hintergrund: Steuerschlüssel können auch bei Zahlungskonten vorhanden sein,
damit wäre eine Verknüpfung mit einer Netto-Rechnung auch nicht sinnvoll.

SelfTests erweitert: arap.paid mit acc_trans.paid vergleichen

Der Bezahl-Wert in den Nebenbücher muss auch immer mit dem Bezahl-Wert
in der acc_trans(chart_link= '%ARAP_paid%' übereinstimmen.

SelfTests verbessert

Eingangsrechnungen können und dürfen diesselbe Rechnungsnummer haben,
entsprechend beim group by berücksichtigt.
Ferner amount auf Zahlungsausgangskonto und nicht Zahlungseingangskonten
berücksichtigt.