Testfall Kosmetik (Alignment)

Typos in Doku (Dev)

DB Transaktionen - Array Context funktioniert nicht

Innerhalb des eval{} Blockes ist für wantarray ein neuer Context,
in diesem Falle ist wantarray hier nicht definiert.
Deshalb muss dies per Variable in den eval-Block übergeben werden

Geierlein: Abschalten der Steuernummer nur wenn Feature "ustva" eingeschaltet ist.

Alte Abfrage in der Mandantenkonfig falls keine Ustva ausgewählt,
z.B per Defaultmanager swiss"

CsvImport: Bearbeiter aus Front-End setzen, wenn nicht angegeben.

Wenn keine employee_id in der CSV-Datei angegeben ist, dann den Benutzer, der
den Import gestartet hat als, Bearbeiter setzen. Dieser wird in den Daten des
Hintergrundjobs gespeichert.
Falls hier nichts hinterlegt ist, dann den Benutzer aus dem Login setzen. Das
ist derjenige Benutzer, unter dem der Task-Server läuft.

kivi.Part.js: Optionen rekursiv mergen

Model-Presenter Bindung mit Proxyobjekten

Part: doku update

kivi/Part.js: commit_none hook korrekt verdrahten

Part: part picker commit hooks umbenannt

kivi.Part.js: on_enter_match_* umbenannt in commit_*

Part: exact_match robuster mit localized $::form

Behebt Bugs mit vergessenen Filtern wie unit_obj.convertible_unit

Part: Nach einfügen von Zeile den Partpicker clearen

kivi.Part.js: clear Methode für Partpicker

kivi.Part.js: im PICKED state auf enter/tab den commit hook auslösen

Part: multipicker verwenden

PartPicker: Multipicker als separate popupklasse

kivi.Part.js: real_id wird nicht mehr benötigt

kivi.Part.js: run_action als wrapper um functions und strings

kivi.Part.js: Datenübergabe Picker<->Presenter auf data umgestellt

Ist in Summe kompakter und einfacher erweiterbar.

kivi.Part.js: keydown ausgelagert in eigenen handler

kivi.Part.js: PartPicker von closure auf prototype style umgeschrieben

War notwendig, damit das Objekt sich selbst an andere Objekte weitrgeben
kann. Vorher musste es dafür eine anonyme Closure über die eigenen
Funktionen machen, die extrem schwer verständlich war.

Nachteil dafür jetzt, man kann keine Methoden mehr ohne closure als
callback verwenden und alles ist voll mit this.

kivi.Part.js: PickerPopup: bei Enter nicht timeout abwarten

kivi.Part.js: paginate aus Picker gezogen

kivi.Part.js: PickerPopup auf prototype style umgestellt

kivi.Part.js: Part Picker Popup in ein separates widget verschoben

kivi.Part.js: KEY konstanten aus Picker verschoben.

die brauche ich gleich in mehreren Objekten.

kivi.Part.js: init routine angelegt

kivi.Part.js: Formatierung

kivi.Part.js: PartPicker: key code für UP eingefügt

Partpicker: fat_set_item mit neuem Part-Controller gefixt

Der alte Part-Controller erwartete, dass die Part-ID als $::form->{id}
übergeben wird. Der neue hingegen nutzt $::form->{part}->{id}.

kivi.js + kivi.Part.js: reinit_widgets verschoben

PartPicker: Cursor down öffnet jetzt immer das autocomplete Menü

PartPicker: enter hooks überlagerbar gemacht

PartPicker von autocomplete_part.js nach kivi.Part.js verschoben

kivi.Part.js: jslint und use strict

record_links.t: nach Test aufräumen

sales_order_with_periodic_invoices_config.t: nach Test aufräumen

File-Controller: Warnungen wegen undefinierter Werte vermeiden

Warnungen traten bei t/file/filesystem.t auf.

WebDAV Pfad-Name im Dialog geändert

Referenz ist ein ungünstiges Freitext-Feld, um WebDAV Pfade
eindeutig anzulegen. Transaktions-, bzw. die Buchungsnummer
im Buchungsjournal ist hier eindeutig.

Kontoauszug verbuchen Link-Farbe grün einheitlich gesetzt

Kontoauszug verbuchen: Lieferantengutschrift buchen

Nun wird eine negativen Kreditorenbuchung (Lieferantengutschrift) richtig verbucht.

t/bank/bank_transactions.t um einen Test erweitert
und Stdout des Controllers (HTML) in String umgelenkt.

Kontoauszug verbuchen: Banktransaktionen mit Fehlern nicht zählen

Die OK Anzeige der Anzahl gebuchter Transaktionen ohne fehlerhafte Buchungen

Kontoauszug verbuchen: DB-Rollback nicht zusätzlich als Fehler melden

Bei einer negativen Kreditorenbuchung (Lieferantengutschrift)

wird nun nicht mehr
"do_transaction() failed - Died at SL/Controller/BankTransaction.pm line 733."

gemeldet, sondern der Fehler an der (nicht)verbuchten Rechnung angezeigt:
"Gesendete Zahlungen können nur mit Einkaufsrechnungen und Verkaufsgutschriften verbucht werden."

Kontoauszug verbuchen: Sonderzeichenbehandlung

Sonderzeichen beim Vergleich Verwendungszweck - Kunde/Lieferant escapen

t/file/filesystem.t robuster

Tests: t/structure/common_errors.t parallelisiert

Tests: t/structure/no_lexicals_in_postif.t parallelisiert

Tests: t/structure/no_indirect_object_notation.t parallelisiert

Tests: t/003safesys.t parallelisiert

Filesystem tests nun ohne anderen Output

Controller Downloaddaten werden umgelenkt und anschließend geprüft.

Fehlerbehandlung in SL/File.pm vereinfacht

Dokumentation: neue Perl-Modul-Abhängigkeiten für Entwickler dokumentiert

Tests: 001compile.t parallelisiert

CSV-Artikel-Import-Controller: Warnung wegen Vergleich mit undefiniert vermeiden

Trat bei Tests auf (genauer: t/controllers/csvimport/parts.t).

Tests: $::form nur über Helfermethode Support::TestSetup::create_new_form anlege

Diese Helfermethode erstellt und konfiguriert auch gleich das
Template-Objekt mit dem Pfad zum Test-eigenen
Template-Cache-Verzeichnis.

Bankauszug verbuchen: Gutschrifts-Fall über Aufruf der richtigen Funktion erkennen

»type« bezieht sich nur auf die Spalte »type«, in der nicht zwischen
Einkauf und Verkauf unterschieden wird.

Testwarnung: doppeltes »my $ap_chart« vermeiden

SL::Mailer: Header-Felder nicht zwei mal MIME-encoden

Das Modul Email::MIME kümmert sich bereits um das richtige MIME-Encoding
der Header. Daher darf es vorher nicht gemacht werden.

Tests: Template-Objekt in Form für Test-Cache-Verzeichnis anlegen

Dient dafür, Dateizugriffsprobleme wegen Berechtigungen zu vermeiden:
»users/templates-cache« wird normalerweise vom Webserveruser erzeugt und
beschrieben, die darin liegenden Dateien haben mode 0600. Tests werden
hingegen als normale User ausgeführt und haben damit nicht mal
Leserechte auf die Dateien in »users/templates-cache«.

Das Template-Objekt wird direkt in $::form abgelegt, wodurch dann auch
reguläre Routinen wie SL::Presenter::Base->render ins richtige
Verzeichnis schreiben.

Damit müssen auch keine Render-Tests mehr übersprungen werden, falls
keine Schreibrechte auf das Haupt-Cache-Verzeichnis
»users/templates-cache« besteht.

SL::Auth: Kosmetik (Alignment)

Auth-DB-Upgrades alle nach SL::DBUpgrade2::Auth verschieben

Das verhindert eine Warnung wegen Umdefinition der Funktion
»SL::DBUpgrade2::clients::run« während der Tests, weil es das Script
»clients.pl« sowohl für die Auth-DB als auch für die Haupt-DB gibt.

Auth-DB-Spalten-Infos nur lesen, wenn DB-Verbindung vorhanden

»reset« wird während des Request-Shutdowns ausgeführt. Falls aber noch
gar keine Auth-DB existiert (neue Installation z.B.), so wird versucht,
eine Verbindung dahin aufzubauen, was zu einer Fehlermeldung führt —
selbst, wenn man gerade versucht, besagte Auth-DB anzulegen.

Test with_transaction.t: deutsche und englische Fehlermeldungen erkennen

DB-Cluster sind bei uns in Deutschland normalerweise mit englischer oder
mit deutscher Locale angelegt. Daher sollte ein Test mit beiden
Sprachvarianten genügen.

Revert "Tests: Datenbank-Meldungen immer in English zurückgeben"

Dies revertet e38b0b21c1e924cb0b1a21f6e2fe5db7d3258f70, da der
Datenbankuser Superadmin sein muss, um lc_messages setzen zu
können.

MIME:*  aus dem Installcheck und Doku wieder rausgenommen

Für t/file/filesystem.t Anpassungen im Debug und Fehlermeldung

Nun läuft der Test wieder

E-Mail versenden wieder mit Email::MIME

Einschränkung: *.eml werden zwar mitverschickt,
aber tauchen beim Empfänger nicht als *eml auf sondern als einzelne Anhänge

Testfehler: keine indirekte Objektnotation nutzen

t/file/filesystem.t: keinen festen Namen für temporäres Verzeichnis nutzen

Weiterhin muss das Verzeichnis angelegt werden. Der Test darf sich nicht
darauf verlassen, dass er einfach durchläuft.

Test-Run-Script ohne Harness

Wenn man mal nur die rohe Ausgabe eines Tests sehen will.

SL::DB: Exception nur dann mit ->isa/->rethrow behandeln, wenn geblesstes Objekt

ref($error) gilt auch für z.B. geworfene Arrayreferenzen, welche aber
nicht geblesst sind, wodurch man auf ihnen keine Funktionen aufrufen kann.

Tests: Datenbank-Meldungen immer in English zurückgeben

Die Sprache, in der z.B. Fehlermeldungen von der Datenbank geliefert
werden, wird beim Erstellen des PostgreSQL-Clusters festgelegt. Dies ist
bei vielen Standardinstallationen Deutsch, da meinst nur
»--locale=de_DE.UTF-8« gesagt wird, und damit auch LC_MESSAGES gesetzt
wird.

Für Tests ist das ungünstig, weil sich diese darauf verlassen können
müssen, dass die Fehlermeldungen immer in einer bestimmten Sprache
sind. Unsere Tests vergleichen Fehlermeldungen mit den erwarteten
Mustern, und diese sind in Englisch.

Daher die Nachrichten-Locale Datenbankverbindungen von sowohl Auth- als
auch Hauptdatenbank am Anfang der Tests auf Englisch setzen.

Testfehler: typische Schreibfehler gefixt

Testfehler: Tabs entfernt

Template-Syntax-Test: eigenes Compile-Cache-Verzeichnis für Tests

CACHE_SIZE=0 sollte eigentlich dafür sorgen, dass das Template-Modul die
compilierten Templates nicht cachet. Das funktioniert aber wohl
nicht.

Daher ein eigenes Cache-Verzeichnis dafür definieren, um
Dateizugriffsprobleme wegen Berechtigungen zu vermeiden:
»users/templates-cache« wird normalerweise vom Webserveruser erzeugt und
beschrieben, die darin liegenden Dateien haben mode 0600. Tests werden
hingegen als normale User ausgeführt und haben damit nicht mal
Leserechte auf die Dateien in »users/templates-cache«.

t/Support/Files: auf strict umgestellt

fehlende Locales ergänzt

Bankauszug verbuchen: Umstellung auf Belegvorlagen & Filter-Fixes

Belegvorlagen: Form-Parameter beim Laden über Request vorbelegen können

Belegvorlagen: employee_id bei Umwandlung aus Drafts beibehalten

fehlende Locales ergänzt

Kontoauszug verbuchen: Beleg/Memo bei »Alle Buchungen« angeben können

Kontoauszug verbuchen: »Beleg«/»Memo« bei Vorschlägen angeben können

Kontoauszug verbuchen: bei Vorschlägen Rechnungsnummer in Verwendungszweck highlighten

Presenter: mehr Funktionen von L in Tag-Presenter verschieben

kivi.parse_amount: bei ungültigen mathematischen Ausdrücken 0 zurückliefern

Ansonsten würde eine Exception geworfen, was ein anderes Verhalten als
bei der Perl-Variante wäre.

kivi.parse_amount: bei ungültigen Zeichen 0 zurückgeben

Es werden nun nur noch mathematische Zeichen erlaubt.

js parse_/format_amount: Tests für Swiss-Zahlenformat & für Nicht-Oktal-Parsen

FiBu Schnellsuche - GL Habensumme entfernt, ist schon in oneline_summary

oneline_summary mit Rechnungsbeträgen

Bei Belegen wird der Bruttobetrag angezeigt.

Bei Dialogbuchungen wird die Summe aller Haben-Buchungen als Betrag
angezeigt.

FiBu Schnellsuche repariert

war in 5b981bb6a5c kaputt gegangen.

Typo in Übersetuzng

Part Csv Import - Preisgruppenpreisimport testen

name change: validator statt validater

Typos in Doku

console - login explizit in myconfig setzen, für Testdatenbank

Wenn man sich für die Testdatenbank einen Mandanten anlegt um sich an
der Oberfläche die generierten Testdaten anzuschauen, und die
Datenerstellung dort auch per console vorbereitet:

scripts/console --client testdb --login unittests

Ansonsten klappt SL::DB::Manager::Employee->current->id nicht.

Mailer: nur Adress-Anteil als Absender im SMTP-Dialog verwenden & nicht encodieren

Der SMTP-Dialog nutzt beim MAIL FROM nur den Adress-Anteil einer
E-Mail-Adresse:

MAIL FROM:<user@the.doma.in>

Daher sollte auch nur dieser Anteil übergeben werden, auch wenn in
$self->{from} potenziell eine komplette Adresse steht.

Da dieser Adress-Anteil ausschließlich aus ASCII-Zeichen besteht, muss
er dann auch nicht encodiert werden. Das macht die Strings im Log
leserlicher, und vermutlich sind MIME-Q-/-B-Encodings auch im
SMTP-Standard eigentlich nicht erlaubt (siehe RFC 5321).

SL::Mailer: Kosmetik für bessere Lesbarkeit (Alignment, Leerzeichen, Einrückung)

Banktransaction: Kontoauszug verbuchen: Sonderfall für "credit_notes"

"credit_notes" haben negativen Betrag,
diese werden nun gesondert  behandelt.

Ersetzt commit fbcd5580