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

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