Individuelle Lieferadresse hinzufügen: cp_gender nicht vergessen

shiptocp_gender darf bei der Prüfung, ob die Lieferadresse hinzugefügt werden
soll (also nicht leer ist), nicht berücksichtigt werden, da shiptocp_gender nie
leer ist.
Allerdings muss shiptocp_gender sehr wohl beim INSERT oder UPDATE
berücksichtig werden.

Der Fehler kam mit
"Individuelle Lieferadresse hinzufügen: cp_gender nicht auf leer prüfen …"
(commit b8fa93b5d2cfff453fd8a82206d1a288407a8be6) rein.

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

Inventur: beim Zählen/Korrigieren auch Artikel ohne MHD berücksichtigen, …

…, wenn MHD in der Mandantenkonfiguration eingeschaltet ist.

S:C:Inventory: Kosmetik/Kommentar: emacs happy

JSON-Payload: Content-Type-Header richtig parsen

Content-Type kann auch noch Attribute wie »; charset=UTF-8«
enthalten. Muss ignoriert werden.

Unterstützung für JSON-encodierte POST-Daten

Mit diesem Patch werden nun alle POST-Daten in $::request->post_data
gespeichert. Wie bisher auch werden Daten mit Content-Type
»multipart/form-data« zusätzlich in $::form gespeichert.

Will man aber JSON-encodierte Daten hochladen, so werden die
ausschließlich in $::request->post_data gespeichert, nicht in
$::form. Hintergrund ist, dass JSON-Dokumente auch aus Arrays bestehen
können, $::form aber halt immer ein Hash ist. Weiterhin ist es
sinnvoll, die Namensräume der übertragenen Parameter sauber
voneinander zu trennen.

LXDebug::clone_for_dump: Warnung wegen Hash-Zugriff mit Key undef vermeiden

Wiederkehrende Rechnungen: ge-e-mailte Rechnungen im WebDAV speichern

S:DATEV:CSV: Längenbegrenzung bei buchungstext wieder rein

Flog mit
"Fixt #348 DatevExport kommt mit bestimmten Zeichen im Buchungstext nicht klar"
(commit 324726acd30b8992854a2d59fec2a50265613eef) raus.
War sicher ein Versehen.

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

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

t/datev/invoices.t: Kosmetik: Alignment

S:DATEV:CSV: Kosmetik: Leerzeile in Array-Definition entfernt

S:D:Manager:ReconciliationLink: nicht initialisierter Wert: Warnungen vermeiden

Durch Tests getriggert, konkret: durch t/bank/bank_transactions.t

year_end.t - aktuelles Jahr nehmen

ging in 2020 kaputt...

Payment Helper Test nutzt create_ap_transaction

create_ap_transaction benutzen, statt per acc_trans-Einträge manuell
zusammenzubauen. Außerdem purchase_invoice in ap_transaction umbenannt.

Presenter Mapping für Dialogbuchungen repariert

War bei Umstellung vergessen worden.

add_{ap|ar}_amount_row berücksichtigen gldate

Ansonsten konnte gldate in ar/ap und acc_trans abweichen, wenn gldate
explizit gesetzt war und nicht dem aktuellen Tag entsprach.

t/datev/invoices.t nutzt Dev create_ap_transaction

Anstatt die acc_trans-Einträge manuell zusammenzubauen.

Dev Record - create_{ar|ap|gl}_transaction mit assign_attributes

Damit kann man auch Felder setzen, die nicht explizit abgefragt und
geprüft werden, z.B. itime manuell setzen. Wird für manche Tests
benötigt.

Dev Record : create_{ar|ap|gl}_transaction mit gldate

Damit man Belege für Tests mit abweichendem Erfassungsdatum erstellen
kann.

Typo in typos

Das war mir einen eigenen Commit wert ^^

Typos in DB.pm POD

Kosmetik: changelog.orig löschen

DATEV-Export Festschreibungskennzeichen für jede Buchung einzel setzen

Kundenstatistik: Defekten Link repariert.

id fehlte falls alter Auftrags-Controller in Mandanten-Konfiguration aktiviert.

Kundenstatistik: Defekten Link repariert.

id fehlte falls alter Angebots-Controller in Mandanten-Konfiguration aktiviert.

Version auf unstable gesetzt; Dokumentation für Forenlink geändert

Kundenbericht: richtig nach Kreditlimit, UStID und Amtsgericht sortieren

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
  Parameter« benötigt man keine sub; das geht einfach mit einem ternären
  Operator.
• »Keine Parameter« drückt man nicht durch »undef« aus, sondern durch
  die leere Liste »()«. Wird »undef« als einziger Parameter übergeben,
  so wird in der Funktion de Facto das hier gemacht:

  my %hash = (undef);

  und das ergibt eine Warnung, dass eine Liste mit ungerader Anzahl von
  Elementen in ein Hash umgewandelt werden soll.

(cherry picked from commit e5a76ed158219e3d2b19f142213f8a80c11627f1)

Cherry pick aus dem master selber, da dieser commit durch
54ce51443886623bb785e7c54492ecbc8dc9b3aa "Dateimanagement: Massendruck"
überschrieben wurde.

Behebt #411 (redmine).

Release 3.5.5

Datenbank-Upgrade-Skripte für Release 3.5.5

changelog: Bugfixes hinzugefügt

changelog: Weiterleitung zum Ziel, wenn abgemeldet

Versionsnummer im Changelog und Dokumentation hochgesetzt (3.5.5)

Changelog: Hinweis auf Dokumentation für neuen Hintergrund-Job

Login: blacklisting bei Callback für Redirect zum Ziel implementiert

Login: Callback nur bauen bei GET-Request und wenn action vorhanden

Login: Callback setzen und berücksichtigen, wenn abgemeldet

Wenn ein Ziel innerhalb des Programms aufgerufen wird (z.B. aus einem
Link, den man zugschickt bekommt) und man nicht eingeloggt ist, so
wird man zu dem Ziel weitergeleitet, nachdem man sich eingeloggt hat.

LoginScreen: Request beenden, wenn Benutzer nicht authentifiziert wird

Individuelle Lieferadresse hinzufügen: cp_gender nicht auf leer prüfen …

…, denn das ist nie leer.

Und sonst werden individuelle Lieferadressen angelegt, obwohl keine
Eingaben gemacht wurden.

Hierzu sollte evtl. noch ein DB-Upgrade-Skript erstellt werden, welches
die leeren shiptos entfernt. Das ist allerdings nicht ganz einfach (siehe
auch Ticket).

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

False Positive bei Überprüfung von Lieferant und Lieferaten-Rechnungsnummer verhindern

Falls eine Buchung noch am selben Tag verändert wird,
kommt es fälschlicherweise zu der Aussage, dass schon eine
Rechnung mit dieser Rechnungsnummer zum Lieferanten erfasst worden sei.
Entsprechend geändert

In calculate_qty.html number_format() durch kivi.format_amount() ersetzt

function number_format() gelöscht, da nicht mehr nötig

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