one line table

comment potentially empty lines

expl environment

withut xparse

\FakeTable

\ -> \tabularnewline

test faketable

remove unnecessary spaces

adjust Pricing tabular currency spacing

add a4paper, since outdated tl

fix ifkomavarempty

move title to infobox if enabled

infobox option

switch to toaddrhpos

adjust margins

location -> komavars

merge komavar changes for location/reffields

load expl3 first

table -> Pricing table

spacing adjustments in pagestyle

initial pagestyle config

letter environment

adjust head/foot structure

remove \ourfont

remove geometry -> DIN.lco

kivi scratch variables

class loading to .tex

Mandantenfähigkeit bis firma<n>

cleanup font selection

remove ifthenelse requirement

move inheader.tex -> kiviletter.sty

compiler independence

typographic corrections

initial commit

SL::Form: Kosmetik: auskommentierten Code entfernt

der ist schon 12 Jahre auskommentiert.

SL::Form: get_lists: Unterstützung für shipto entfernt

get_lists für shipto wird nirgends mehr verwendet, soweit ich das sehe.

Zudem hat _get_shipto module nicht berücksichtigt und da customer und ar
ids aus eigenen Sequenzen bekommen und somit zwei dieser ids für die
verschiedenen Tabellen gleich sein können, konnte es passieren, dass für einen
Kunden eine individuelle Lieferadresse aus einer Rechnung (für einen anderen
Kunden) zurückgelieferrt wird.

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