projekt kivitendo

tests: Warnungen

Assortment und Assembly Tests nutzen SL::Dev::Part

Part CsvImport - Umbenennung parts_type nach part_type

t/part/assortment.t

assortment Test mit sellprice und lastcost sum

Sortiment - Test für Sortimentsartikel

Test angepasst nach income_accno_id-Entfernung in parts

Ware/Erzeugnis/Dienstleistung per parts.part_type unterscheiden 2

kivitendo Code angepasst.

Startup: Include-Pfade mittels FindBin ermitteln

Neue Perl-Versionen werden das aktuelle Verzeichnis '.' aus dem
Standard-Include-Pfad @INC entfernen. Das bedeutet für uns, dass wir
nicht mehr einfach »use SL::Dispatcher;« und ähnliche Konstrukte machen
können....

Auth::ColumnInformation: fetch nur in reset, nicht in init

die Version aus e7a7492e macht Probleme im taskserver.

ausserdem das reset nur dann machen, wenn es auch sinnvoll ist. Bisher
wurde das als silver Bullet aufgerufen, wannimmer sich etwas an der DB...

convert_invoice.t: purchase invoices vor parts aufräumen

Erstellen von Jahresabschluss-Buchungen(4)

- Einrückungen von G.Richardson,
- dates in DateObjekte konvertiert
- Eine Buchung und Gegenbuchung als eine Transaktion

Erstellen von Jahresabschluss-Buchungen(3)

Für Auswahl moderne presenter Methoden statt alte cgi Methoden eingebaut.
Fehlende Umbenennung in html Templates für Events korrigiert
Die Auswahl der 9xxx Konten nicht mehr auf 4 Zeichen begrenzt

Erstellen von Jahresabschluss-Buchungen(2)

Auf Wunsch exaktere Bezeichnung des Controllers etc,
also statt yearly ==> yearend

Erstellen von Jahresabschluss-Buchungen

Das Erstellen von Schluss- und Eröffnungsbuchungen (mit Saldovortrag auf 9000-Konten)
wird erleichtern bzw. automatisieren.

- Neuer Menüpunkt "Finanzbuchhaltung > SB/EB buchen"
Buchungsdatum:
SB: dem 31.12. des Vorjahres...

t/controllers/csvimport/parts.t: weg mit den überflüssigen init Aufrufen

CSV-Import Artikel: Einige Erweiterungen

CSV-Import von Artikel hat nun für existierende Artikel folgende Optionen:

1. Eigenschaften von existierenden Einträgen aktualisieren
     2. Eigenschaften von existierenden Artikeln aktualisieren / Nicht vorhandene überspringen...

»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...

SL::DB::Part: Setter für type=assembly bzgl. inventory_accno_id fixen

inventory_accno_id muss für Typ »assembly« immer auf undef stehen. Das
testet auch der Testcase. »type« hat allerdings das Falsche gemacht,
wofür im Testcase versucht wurde, ein Workaround zu implementieren,...

Tests: ic.pl nicht mehr vom html-tag counter ausschließen

Nachdem top100 rausgeflogen ist, ist das mit 13 tags sogar noch gut im
Mittelfeld der Sünder.

Tests: closed bei Überzahlungen auf TODO gesetzt

Tests: warnings in t/part/assembly.t

Spellcheck - separate

Test für Erzeugnisse per Rose erstellen erweitert

Test für Erzeugnisse

Test bank_transactions - am Ende aufräumen

Test für Rechnungen bezahlen per 'Kontoauszug verbuchen'

UserPreferences: erste version

Payment Helper kann nun auch Kursverluste/Gewinne

+ neue Helper-Methode exchange_rate für Rechnungen

Paymenthelper kann Fremdwährung mit Steuer inkl. und exkl.

PTC Test um Rundungsgenauigkeit ergänzt

Der PriceTaxCalculator hat als Datenwert neuerdings die
Rundungsgenauigkeit für den Modus Schweiz. Dies muss entsprechend
im Tests des Moduls auch berücksichtigt werden.

Csv: mapping support

Neuer Minimaltestfall für Rabattrundung im PTC

Beim PTC wird vor der Multiplizierung mit der Menge der gerundete Rabatt vom
Verkaufspreis abgezogen, statt erst die Zeilensumme zu berechnen und
dann den Rabatt zu ziehen.

6 Artikel zu 0.6€ mit 3% Rabatt
0.6 * 6 * ( 1 - 0.03 ) != ( 0.6 - round(0.6*0.03) ) * 6...

ParseFilter: neuer Filter "trim"

Ausserdem autotrim bei substr, head und tail Filtern

trim-Funktion zum Entfernen führender und anhängender Whitespaces

Typos und spellcheck.t

Test zu SL::Request::unflatten, der prüft, ob nicht Hash-Keys erzeugt werden.

Nicht, dass in einem Array aus Hashes key in einem Hash auftauchen, die nur in
einem anderen drin sind.

Payment Helper setzt $self->transactions zurück

Als es noch eine Methode transactions in SL::DB::Helper::Payment gab
wurden die acc_trans-Einträge bei Zugriff per $self->transactions jedes
Mal aus der Datenbank ausgelesen:
(SL::DB::Manager::AccTransaction->get_all(query => [ trans_id => $self->id ]);...

Debitorenbuchungen als CSV importieren

Ähnlich wie der Auftragsimport wird hier gemultiplexed, d.h. es gibt separate
Zeilen für die Debitorenbuchung (ar) und die Buchungszeilen (acc_trans).

Es handelt sich allerdings nicht exakt um acc_trans-Zeilen, die direkt...

Neue Methoden um Debitorenbuchungen zu erstellen

Vorbereitung für Debitorenbuchungsimport, neue Methoden für SL::DB::Invoice
Objekte:

add_ar_amount_row - Erlösbuchungen hinzufügen, mit Steuerschlüssel
create_ar_row - acc-trans für Forderung hinzufügen
validate_acc_trans - Prüfen ob alle acc_trans-Einträge aufgehen...

tests: Spellcheck auf allen Dateien machen

tests: html Vorlagen auch in Testdateien aufnehmen

Test zur Prüfung der Syntax aller Template::Toolkit-Templates

round_amount: Anzahl Stellen nicht zu hoch werden lassen

format_amount - negative 0 vermeiden

Ist eine Zahl z.B. -0.002, und wird auf 2 Stellen gerundet, so wird erst
das Minuszeichen gemerkt, dann gerundet (0.00), und schießlich wird das
Minus wieder hinzugefügt, dadurch bekommt man -0.00. Mit diesem Patch
wird das Minus-Zeichen entfernt wenn die gerundete Zahl genau 0 ergibt.

Testfall convert_invoice.t um html-formatierte texte erweitert

Beim Konvertieren von Lieferschein nach Invoice auch explizit die
Formate von CKEditor prüfen. Ferner leichte weitere Anonymisierung der
Warendaten.

Tests erweitert

Neuer BackgroundJob und den Standard Employee nicht in meinem
Testfall löschen, da dieser bei späteren Tests noch verwendet wird.

convert_invoice auch LinkedRecords und Items testen

DeliveryOrder um convert_invoice erweitert

Diesselbe Idee wie bei SalesOrder->convert_invoice. Der ursprüngliche
Lieferschein wird geschlossen und das neue Objekt mittels record_links
verknüpft.
Entsprechend Testfall mitgeliefert.

Typos - kein "wether" mehr!

AttrDuration: Test muss Sprachinvariant sein

AttrDuration-Test: Anzahl Testcases gefixt

ParseFilter: Komplexe Methoden ermöglichen, die auch den Key ändern

AttrDuration: Implementation für Spalten, die Dauer in Minuten speichern

Zahlungsbedingungen: Flag »Fälligkeitsdatum automatisch berechnen«

Tests: Indirekte Objektnotation für Instanzierung verbieten.

t/002goodperl: TODO für strict DBUpgrades entfernt.

Die müssen seit langem in packages liegen und strict sein.

Tests: 2 Tests in db_helper/payment.t auf TODO gesetzt

Tests: Hilfreiche aber false positive warning unterdrücken

Tests: Den einen fehlschlagenden 002goodperl Test auf TODO gesetzt.

Test all the things!

Form::parse_amount: Parsen als Oktalzahlen verhindern

Merge branch 'bankerweiterung_und_skonto'

Conflicts:
doc/changelog
locale/de/all

Sammelcommit Bankerweiterung und Skonto

Überarbeitung der Bankerweiterung vom Stand Niclas, und Einführung von
Bezahlung mit Skonto (alter payment Branch). Mehr Details siehe
changelog.

• changelog

• Ungültige Bankkonten ausblenden
• Punktesystem in Hash %points ausgelagert...

Aufräumen der DB am Ende der individuellen Tests

die Tests sollten am Ende ihre eigenen angelegten Daten löschen, damit
andere Tests beim Löschen ihrer Daten keine Abhängigkeitsprobleme
bekommen:

create_periodic_invoices.t
price_tax_calculator.t
record_links.t...

Typo gefixed

ParseFilter: typo + test

login nicht aus $::form nehmen. Teil 2

Dabei ergibt sich ein kleines Problem. User spezifische bin/mozilla/
Controller werden geladen bevor das User handling abgeschlossen ist.
Nach dieser Änderung sind die also de facto kaputt.

Tests: fehlender Backgroundjob

ParseFilter: Laundern schon vor dem eigentlichen parsen.

Erlaubt es indirekte Filter im Template roundtrip safe zu machen.

Tests: price_tax_calculator.t muss vor Kunden löschen richtig aufräumen

Finanzcontrolling: Testcase für Aufträge mit wiederkehrenden Rechnungen

Finanzübersicht: Testcase für Spalten Auftragsvorlauf/-eingang

Wiederkehrende Rechnungen: Testcase für Periodizitäten

Ich schreibe immer postition statt position.

PriceTaxCalculator: für Items berechnete flüchtige Werte zurückgeben Teil 3

Die items können nicht als Hash-Referenz gespeichert werden, weil das
kaputt geht, wenn die Items noch nicht gespeichert wurden und damit noch
keine ID besitzen. Daher Umstellung auf Array-Speicherung....

SL::CTI::sanitize_number: nur Ziffern zurückliefern

Rundung bei Debitorenbuchung, Kreditorenbuchung und Dialogbuchung

Zwei neue Hilfsfunktionen für Form eingeführt die von ap/ar/gl genutzt
werden:

• calculate_tax wird zur Berechnung der Steuer bei
  • update in ar, ap und gl
  • post_transaction in AR.pm und AP.pm innerhalb von calculate_arap...

Form::round_amount: Perls Wissen über Stringifizierung nutzen

Perl weiß am besten, wann eine nicht ganz exakte Fließkommazahl
eigentlich eine für Menschen sinnvoll lesbare Fließkommazahl ist (also
dass mit 143.19999999999998863132 eigentlich 143.2 gemeint ist, wenn ich...

round_amount: Perls Repräsentationsalgorithmus fürs exakte Runden nutzen

Als Erläuterung paste ich schlicht den relevanten Teil des Kommentars,
der nun auch in der Funktion steht:

Trying to round with more precision first only shifts the problem to rarer...

ParseFilter: with_objects merging bei Klassen mit Filtered Plugin

ParseFilter kennt 3 Quellen für with_objects Klauseln:

1. explizit übergebene
2. aus dem Filter inferierte
3. aus custom filtern gesetzte

Wenn nun ein Model das Custom Filter Plugin hat, muss bei jedem Filter...

Steuerzone - neue Customer/Vendor-Objekte brauchen Steuerzone

Durch den not-NULL Constraint bei Kunden und Lieferanten muß
beim Anlegen eines neuen Objekts nun zwingend die Steuerzone mit
übergeben werden, ähnlich wie bei der Währung.
Dies wurde bei einigen automatischen Tests nachgeholt.

test.pl: --fast switch, skippt 001compile und 003safesys

Nachtrag Steuerzone - Tests angepasst

Steuerzone in price_tax_calculator.t angepasst

round_amount: Fix für falsches Runden bestimmter Werte

Gewisse Werte wie z.B. 33,675 wurden bei 2 Stellen falsch gerundet,
nämlich auf 33,67 anstelle von 33,68. Bei anderen Werten hingegen
funktionierte es (beispielsweise 149,175 @ 2 → 149,18).

Grund war, dass durch das Addieren von 0.5 wieder Fließkommaberechnung...

HTML::Util: Tests, und Bugfixes für ein paar Randbedingungen

Tests: neue Background Job Klasse auch erwarten

DateTime: (add|subtract)_businessdays Funktionen

Rundungsfehler bei periodischen Rechnungen mit Einzelpreisen ...

... mit einer Nachkommastelle und Rabatt behoben.

siehe auch commit a22b8118e0bd68acac8a2d7b02a2d4f9fd0eaff1

Zudem einen Test dazu angelegt. Allerdings weicht die Art, wie der
PriceTaxCalculator und die Beleg-Masken rechnen, von einander ab....

Generische Unterstützung für CTI: Click-to-dial

CSV-Helfer-Test: In-Memory-Streams mit richtigem Encoding initlisieren

Rose-Attr-Helfer: _as_timestamp

…mit verschiedenen Präzisionsstufen und Tests!

Locale::parse_date_to_object: Unterstützung für volle Timestamps & explizite Formate

…und dafür Tests.

Locale::format_date_object: explizites Setzen der Formate; Millisekunden als neues Format

…und dazu Tests.

SL::DB::Helper::LinkedRecords: rekursive Suche in linked_records

LinkedRecords: bidi Verhalten den docs angepasst und direction Parameter optional gemacht

t/structure/globals: Lexikalische Variablen sind schwer exportierbar.

t/test.pl: Neuere Tests kommen sich bei multicore in die Quere. Vorerst ausschalten

t/test.pl: File::Find macht nur dann lstat wenn follow oder follow_fast als option gesetzt ist

LinkedRecords: Test und Doku Update