parse_amount ersetzt durch kivi.parse_amount, " statt ' bei Zahlenformat

1) Unnoetig doppelte Funktion gelöscht
2) [% MYCONFIG.numberformat %] darf nicht in einfache Hochkommas gesetzt
   werden, da dies beim Zahlenformat 1'000.00 zu Fehlern führt.

Massenerstellen von Rechnungen aus LS: Filtern nach Kundentyp

Massenerstellen von Rechnungen aus LS: filter-Template: vergessenes <tr>

Reconciliation: Veto, kein invoice_amount nachträglich setzen.

auch nicht für Vorschläge

partnumber gibt es warehouse template nicht mehr (part picker)

"Sortimente erfassen" nicht anzeigen, wenn dieses exp. Feature ausgeschaltet ist

… zu "Mandantenkonfiguration: experimentelle Features einzeln an-/abschaltbar
machen"

WebDav: Fehler beim Kopieren anzeigen / Verzeichnis zurück wechseln (2)

Der erste commit 108753a78b203dbe0ccbe6438cc16c8df33c04d3 hat das Drucken
ohne Fehler beim Ins-Webdav-Kopieren kaputt gemacht. Probleme waren:
- ein return vergessen
- chdir zurück auch ohne Fehler

Diese commit fixt das.

Bezieht sich auch auf #96 (redmine)
Refs #96

Auth: fehlerhafte JSON-Requests mit JSON und richtigem HTTP-Response-Code beantworten

WebDav: Fehler beim Kopieren anzeigen / Verzeichnis zurück wechseln

Wenn in SL::Form->parse_template bei Common::copy_file_to_webdav_folder etwas
schief ging, wurde dort ein "die" oder "Form->error" aufgerufen. Allderdings
wird in parse_template vorher das Arbeitsverzeichnis gewechselt, so dass die
web-templates zum Anzeigen des Fehlers nicht mehr gefunden werden.

Dies ist nur ein schlechter Fix. In #96 (redmine) sind einige bessere Lösungen
erwähnt, die aber etwas mehr Aufwand und vor allem Testen verlangen.

Bezieht sich auch auf #96 (redmine)
Refs #96

Massendruck: Attachment-/Archiv-Dateiname in Zielsprache

Massendruck: Kopieren ins Webdav in den Helper verschoben.

Damit erzeugen alle Druckvorgänge hierüber Kopien im Webdav

Massendruck Lieferscheine/Rechnungen: doc_storage_enabled importieren

Auftrags-Controller: Prüfung beim Holen der Kunden-/Lieferanten-Art.-Nr, …

… ob Option eingeschaltet ist und ob Kunde/Lieferant schon gewählt
wurde. Sonst gibt es einen Fehler beim Workflow von VK in EK und umgekehrt.

Part: typo

(cherry picked from commit 8ce71088e51f6bce25c8406ff65f72fad3c15d1a)

S:D:Invoice->new_from: Auftragsdatum aus Auftrag bei Erzeugen aus Lieferschein

Einkaufsrechnung -> Suche. Fälligkeitsdatum als Suchfilter hinzugefügt

SL::Mailer: Debug-Meldungen rausgeworfen. mail5 führte zu einem Fehler

Die Debug-Meldung mit "mail5" griff auf $attachment->{id} zu, aber
attatchment kann auch ein Scalar sein. Dann ergab sich hier ein Laufzeitfehler,
der zwar durch ein eval abgefangen wurde, aber das Verschicken der Mail
verhinderte. Als E-Mail-Status bekam man dann "no send attempt made".

Wer den Mailer debuggen möchte, möge dann nochmal eigene Statements
einfügen.

InstallationCheck: Exception::Class: Minimal-Version 1.44

Sonst gibt es nur ungenaue Rose-Fehlermeldungen (nur "generic exception")
und der Test t/db_helper/with_transaction.t läuft nicht durch.

Behebt #407 (redmine).

Changelog und Dokumentation zum Hintergrund-Job: Jahreszahl in Nummernkreisen hochzählen

SetNumberRange: Validitätsprüfung auf Benutzereingabe (data) im Job

SetNumberRange: Background-Job um Jahreszahlen im Nummernkreis zu erhöhen

Nimmt die vierstellige Zahl des nächsten Jahres und multipliziert
diese mit 1 oder einem anderen Faktor. Präfixe und Länge des
Nummernkreis bleibt erhalten. Bspw.:

Letzte Rechnungsnummer:
RE201900009 --> RE202000000
Folgende Werte:
multiplier: 100000
digits_year: 4

multiplier: beliebiger Wert (empfohlen: 1, 10, 100, 1000, default 1)
digits_year: 4 || 2 (Falls keine 2 gesetzt ist, wird 4 verwendet)
          2019 || 19

zu #406 nicht zuviele ids (email journal) als ar.ids verwenden

Bugfix #406 Fehlerhafte abzurechnende Werte, falls Gutschriften höher als Auftragswert

Bugfix (teil) #406 abzurechnender Betrag bei Aufträgen manchmal falsch falls Gutschriften

S.a. Kommentar redmine

CSV-Import Debitorenbuchungen: maximale Betragsabweichung (Einstellungen) parsen

S:C:Letter: Vergessene Presenter-Umstellung für ein select_tag

auth master rights: Skript-Abhängigkeiten und Positionen reparieren

Durch falsche Abhängikeiten können bei Updates bestimmte Rechte feste
Positionen größer 1000 zugewiesen bekommen, bevor das Upgrade-Skript zum
Platzschaffen (multiplizieren mit 100) durchlaufen wird.
Diese werden dann auch mit 100 multiplizert und landen ganz am Ende, was nicht
gewollt ist.
Dieser commit behebt sowohl das Problem bei neuen Updates
(v.A. ältere Versionen sind betroffen) und versucht die falschen Positionen
zu korrigieren.

Typo in de locale

Form->prepare_for_printing: Einheiten übersetzen

OE.pm: Form-Werte nicht direkt in SQL-Queries schreiben

Auftrags-Controller: Positions-Update: auch Artikelnummern updaten

Auftrags-Controller: Spracheinstellungen aus Druck-/E-Mail-Dialog speichern.

Rechte zum Bearbeiten von Preisen in Belgen getrennt nach VK/EK anwenden

Recht zum Bearbeiten von Preisen in Belgen nach Ver- und Einkauf trennen

Das Recht "edit_prices" galt bisher sowohl für Verkauf als auch für Einkauf.
Nun wird das getrennt in "sales_esit_prices" und "purchase_edit_prices".

Recht zum Einsehen von Einkaufsdokumenten aller Mitarbeiter anwenden

Trennung VK/EK in Berichten

Recht zum Einsehen von Einkaufsdokumenten aller Mitarbeiter (Trennung VK/EK)

Das Recht "sales_all_edit" galt bisher sowohl für Verkauf als auch für Einkauf.
Nun wird das getrennt.

Auftrags-Controller: PDF-Download per ajax-send_file

Controller::send_file: rendern bei ajax abschalten können

Damit kann man weitere client_js-Kommandos absetzen.

Controller::send_file schickt jetzt über client_js wenn ajax

(cherry picked from commit 0f16bc87b4804cf05cefb5ac1514c7f565cfbfbd)

kivi.js: kivi.save_file und client_js binding

(cherry picked from commit f68ea953a6a563172f12991d2ca3f9f17ad89dd2)

Controller::redirect_to: url_for nicht doppelt aufrufen, wenn ajax

Macht zwar nichts, muss aber auch nicht.

CustomerVendor POD korrigiert

t/datev/datev_format_2018.t nutzt Dev create_gl_transaction

Jahresabschluß - GLTransaction->post und Tests

Der YearEnd Controller nutzt nun GLTransaction->post, damit muß man die
acc_trans-Einträge nicht mehr von Hand zusammenbauen, und die Buchungen
passieren automatisch als Transaktion, die Buchungen werden validiert
und es wird ein Historieneintrag erstellt.

GLTransaction - Dialogbuchungen per Rose erstellen

neue Methoden in GLTransaction zum Erstellen von Dialogbuchungen
* post
* validate
* add_chart_booking

An einigen Stellen im Code werden Dialogbuchungen per Hand erstellt,
inkl. Steuern, das soll hiermit vereinheitlicht und vereinfacht
werden.

Acc_trans-Einträge können nun mit wenigen Parametern zu Dialogbuchungen
hinzugefügt werden, die Parameter orientieren sich dabei an den Werten,
wie sie auch an der Oberfläche eingegeben werden (Konto, Soll/Haben,
Steuer).  Dabei werden einige der Werte aus der GLTransaction
automatisch übernommen.

Beim Buchen wird eine neue Transaktion gestartet, die Buchung wird
validiert und es wird ein Historieneintrag erstellt.

SL Dev Record - neue Methoden für ar/gl/ap Transaktionen

Neue Methoden, um für Tests schnell Debitorenbuchungen,
Kreditorenbuchungen und Dialogbuchungen zu erstellen

ActionBar: Javascript: unbenutzte Variable entfernt

ActionBar: Javascript: jquery-Selektor zu Objekt auflösen

Sonst lassen sich die Funktionen removeTooltip, setTooltip, setDisabled,
setEnabled nicht mit einem Selektor via ClientJs->run

Danke Mosu.

ReportGenerator: set_options: auch bei csv_export nur Standardwerte beibehalten

ReportGenerator: set_options: "for" statt "map", da Rückgabewert nicht verwendet

S:C:YearEndTransactions: Warnung vermeiden: Variablen waren schon deklariert

S:C:YearEndTransactions: Kosmetik: Tab entfernt

Auftrags-Controller: Link bei Artikelnummer: Artikelstamm in neuem Tab öffnen

Auftrags-Controller: self an _row-Template übergeben …

… und nicht einzelne Variablen aus self, die dort abgefragt werden.

changelog zu "Auftrags-Controller: Positions-Update aus Artikel-Stamm"

Auftrags-Controller: Benutzereinstellung: Positions-Update aus Artikel-Stamm

Auftrags-Controller: Positions-Update aus Artikel-Stamm

locales-Lauf en

fehlende Übersetzung zum Jahresabschluß

Vorhandene Steuer speichern: richitge Anzahl von Bind-Variablen angeben

fix zu commit 543d78225ec609e9f67ecb1544e135ca88149234
"Spalte taxnumber aus Tabelle tax entfernt"

Alten YearEndTransactions Test entfernt

Jahresabschluß - YearEndTransactions neu implementiert

Der alte Jahresabschluß hatte eine Reihe von Schwächen, z.B. wurde nicht
zwischen Bestands- und Erfolgskonten unterschieden, und es wurde auch
kein Gewinn- oder Verlustvortrag gemacht. Der Anwender mußte selber
entscheiden, welche Konten abgeschlossen werden sollten.

* Saldenvortragskonto, sowie Gewinn- und Verlustvortragskonto müssen in
  der Mandantenkonfiguration unter "Standardkonten" konfiguriert werden

* Es wird nicht mehr für jedes Konto eine Dialogbuchung erstellt,
  sondern es werden alle Soll- und Habensalden zusammengefasst, und
  diese in Summe gegen das Saldenvortragskonto gebucht

* Der Jahresabschluß lässt sich für einen bestimmten Zeitraum mehrmals
  ausführen, falls später Buchungen für den Zeitraum hinzukommen.

AccountingPeriod - Methode an get_balance_starting_date übergeben

Für den Fall, daß man die Methode zur Ermittlung des Startdatums nicht
aus der Mandantenkonfiguration nehmen möchte, sondern diese selber
vorgeben, gibt es nun einen neuen Parameter, um die Methode selber zu
bestimmen. Der Default ist weiterhin der Wert aus der
Mandantenkonfiguration.

balance_startdate_method_options nach AccountingPeriod Helper

verlagert, wird auch für YearEndTransactions benötigt.

CSV-Import-Bericht: beim Löschen nur verwaiste Profile löschen

Unter gewissen Umständen kann es passieren, dass es mehrere
`csv_import_reports` mit derselben `profile_id` gibt. In so einem Fall
darf nur dann versucht werden, die Einträge aus `csv_import_profiles`
zu löschen, wenn der letzte `csv_import_reports`-Eintrag gelöscht
wird, der auf auf dieses Profil verlinkt.

Bericht VK-Rg/Debitorenbuchungen: nach Abteilungen sortieren können

Bericht EK-Rg/Kreditorenbuchungen: Abteilungen anzeigen können

Bericht Angebote/Aufträge: Abteilungen anzeigen können

template oe search: Kosmetik: Leerzeilen entfernt

Bericht EK-Rg/Kreditorenbuchungen: Filter nach Abteilung als Option anzeigen

Bericht EK-Rg/Kreditorenbuchungen: Filter nach Abteilung als hidden mitnehmen

Preisgruppenpreise als Preisliste bei Kunden anzeigen

Wenn dem Kunden eine Preisgruppe zugeordnet ist, werden die Preise in
den Kundenstammdaten als Preisliste (eigener Tab) angezeigt.

JS: CustomerVendor: Kosmetik: Tab entfernt

BankTransaction: keine Leerzeichen am Ende vom Verwendungszweck einfügen

Wenn man mehrere Felder einfach mit `join(' ', @felder)` zusammenfügt,
so entstehen nun mal ein Haufen Leerzeichen, auch wenn alle Felder
selber leer sind. Das führt konkret beim Import von Banktransaktionen
dazu, dass das Verwendungszweck-Feld, das aus den Feldern
`purpose` (ohne Suffix) und `purpose1` bis `purpose13` gebildet wird,
sehr viele Leerzeichen am Ende oder in der Mitte hat, wenn die
Felder (größtenteils) nicht benutzt werden.

Löscht auch bei bestehenden Positionen in der Datenbank Leerzeichen am
Ende. Leerzeichen am Anfang oder in der Mitte werden hingegen nicht
angefasst.

BankTransaction: Vorschlagslistencode aus action_list eigene Funktion verlagert

BankTransaction: Sortierungscode vereinfacht

Belege: E-Mail-Dialog: Sprache für Anhang-Übersetzung setzen

Rose: MetaSetups bzgl. Fließkommazahlen erneuert

bank_transaction_acc_trans: überflüssige Spalte »id« entfernt

Primärschlüssel sind zwei andere Spalten, und die Präsenz dieser
Nicht-Primärschlüssel-Spalte zusammen mit Typ »serial« macht Probleme,
weil Rose kein Default für solche Spalten kennt.

Kundenartikelnummern in Vorlagen verwenden können

Einsortierung in das template array wie Lieferantenartikelnummern als
customer_make und customer_model.

Standard-Druckvorlagen Angebot/Auftrag/Lieferschein/Rechnung für Verkauf
exemplarisch angepasst.

WH: Warnungen wegen nicht initialisierten Werten vermeiden

Durch Tests getriggert, konkret: durch t/wh/transfer.t

SL::DB::Unit: Warnungen wegen nicht initialisierten Werten vermeiden

Durch Tests getriggert, konkret: durch t/wh/transfer.t

AttrSorted-Helfer: Warnungen wegen nicht initialisierten Werten vermeiden

Durch Tests getriggert, konkret: durch t/controllers/csvimport/parts.t

SL::Auth: Warnungen wegen nicht initialisierten Werten vermeiden

Durch Tests getriggert, konkret: durch t/controllers/base/render.t

DB-Upgrades für Hintergrundjobs von Perl auf SQL umgestellt

Rose-Models dürfen in DB-Upgrade-Scripten nicht verwendet werden, weil
die Perl-Strukturdaten (MetaSetup) in dem Moment schon auf dem neuen
Stand, die Datenbankstrukturen aber auf dem alten Stand sind. Daher
schlagen bei Unterschieden (z.B. eine Spalte soll später noch angelegt
werden, sie existiert aber im neuen MetaSetup schon) halt sämliche
Operationen fehl.

Lagerstandsbericht: Listenpreis als Basis f. Bestandswert auswählbar

Lagerstandsbericht: Listenpreis anzeigbar machen

Kosmetik: doppeltes Leerzeichen weg

SelfTest: Geschwindigkeitssteigerung durch »NOT EXISTS« anstelle von »NOT IN«

Nicht ganz frische PostgreSQL-Versionen (mindestens bis 9.6 inklusive)
optimieren »NOT IN«-mit-Subquery nicht automatisch und müssen daher
für jede Zeile des äußeren Selects einen linearen Scan auf die
Subquery-Tabelle machen.

Deutlich effektiver ist das in diesem Fall äquivalente »NOT
EXISTS«-mit-Subquery.

Beispiel: Namen aller Kunden, für die noch keine Rechnung geschrieben
wurde. Falsch mit »NOT IN«:

SELECT name
FROM customer
WHERE id NOT IN (
  SELECT customer_id
  FROM ar
);

Besser und semantisch äquivalent:

SELECT name
FROM customer
WHERE NOT EXISTS (
  SELECT customer_id
  FROM ar
  WHERE customer_id = customer.id
);

Geschwindigkeitssteigerung ist auch ein Euphemismus. Vor der Änderung
war der Selftest bei der LINET-Produktivdatenbank nicht in der Lage,
seine Tests innerhalb von drei Tagen auszuführen. Nach der Änderung
dauert der Selftest weniger als eine Minute.

Neure PostgreSQL-Versionen (z.B. v11) erkennen dieses Pattern
automatisch.

Rose-Models anhand des tatsächlichen Schemas aktualisiert

Doku: Update nach Auth-Erweiterung auf multiple Module

Auth: Unterstützung für multiple Authentifizierungsbackends

Über den Parameter "module" kann man nun multiple Backends angeben,
die nacheinander versucht werden, bis ein Erfolg gemeldet wird oder
die Liste durchlaufen wurde.

Zusätzlich kann man LDAP-Module mehrfach angeben. Damit
unterschiedliche Konfigurationen für jede Modulinstanz benutzt werden
können, wurde die Syntax erweitert: für "LDAP:Config-Abschnitts-Name"
wird "[authentication/Config-Abschnitts-Name]" benutzt. Zwecks
Rückwärtskompatibilität sucht "LDAP" ohne Angabe eines Namens nach dem
bisher auch verwendeten Abschnitt "[authentication/ldap]".

Nützlich ist das Ganze z.B., um einen LDAP-Fallback-Server angeben zu
können, der benutzt wird, wenn der Hauptserver nicht erreichbar sein
sollte.

Auth: mini_error gefixt

$::auth->mini_error wird potenziell zu einem Zeitpunkt aufgerufen, an
dem es die Instanzen von $::form und $::request noch nicht gibt. Da
hier wirklich nur die Bare-Bones-Ausgabe der Fehlermeldung benötigt
wird, machen wir für den Fall manuell ein CGI-Objekt auf.

LDAP-Auth: Konfiguration über Konstruktur übergeben

Ist eine Vorarbeit dafür, das LDAP-Modul mit unterschiedlichen
Konfigurationen benutzen zu können.

LDAP-Auth: enter_sub/leave_sub entfernt

Task-Server auf unterschiedlichen Maschinen laufen lassen können

Jede Task-Server-Instanz und jeder Hintergrundjob haben nun ein neues
Attribute »node_id«. Darüber kann gesteuert werden, dass bestimmte
Jobs nur von einer bestimmten Instanz ausgeführt werden.

Die »node_id« eines neu angelegten Jobs ist standardmäßig leer. Das
bedeutet, dass ein Job von einer beliebigen Task-Server-Instanz
ausgeführt werden kann.

Die »node_id« eines Task-Servers ist standardmäßig der Hostname (siehe
ausgabe von »perl -MSys::Hostname -le 'print hostname()'«), kann aber
in der Konfigurationsdatei überschrieben werden (»[task_server]« →
»node_id«).

Zusätzlich gibt es den Konfigurationsparameter »[task_server]« →
»only_run_tasks_for_this_node«. Ist dieser Parameter gesetzt, so führt
der Task-Server nur diejenigen Jobs aus, deren »node_id«-Feld mit der
»node_id« der Task-Server-Instanz übereinstimmt. Andernfalls werden
auch diejenigen Jobs ausgeführt, deren »node_id«-Feld leer ist.

Achtung: es findet momentan keinerlei Locking statt. Das bedeutet,
dass es für jede Datenbank nur eine Task-Server-Instanz geben darf,
bei der »only_run_tasks_for_this_node« nicht gesetzt ist. Für
Load-Balancing eignet sich das also bisher noch nicht.

Typos in de locale

Workflow Lieferantenauftrag->Kreditorenbuchung: mini Performance-Optimierung

Später benötigte Relationen direkt mit laden.
Dank an Geoffrey.

Workflow Lieferantenauftrag->Kreditorenbuchung: Zahlungsbedingungen aus Auftrag

Um das Fälligkeitsdatum zu ermitteln, die Zahlungsbedingungen aus dem Auftrag
nehmen, nicht die aus den Lieferantenstammdaten.

Neuer Workflow Lieferantenauftrag->Kreditorenbuchung

Für jedes Aufwandskonto der Positionen im Lieferantenauftrag wird eine
Zeile in der Kreditorenbuchung erstellt. Gebucht wird standardmäßig
auf des entsprechende Aufwandskonto. In der Mandantenkonfiguration
kann unter Standardkonten ein Konto ausgewählt werden, auf das dann
alle Zeilen gebucht werden.
Die Steuern werden übernommen, sofern diese für das ausgewählte
Aufwandskonto gültig sind. Ansonsten wird die Default-Steuer für das
Aufwandskonto gesetzt.
Der Quellauftrag wird geschlossen, wenn der Betrag aller
Kreditorenbuchungen, die aus Workflows aus dem Quellauftrag entstanden
sind, gleich dem Betrag des Quellauftrags ist.