BankTransaction: "Kontoauszug verbuchen" überarbeitet für ap_transaction und credit_note

sowie Test erweitert

Anpassung Erfolgsrechnung

CsvImport: Warnungen: Uninitialisierte Variablen

DATEV: Warnungen: Uninitialisierte Variablen

Bankimport: Nicht alle Vorschläge beim ersten "Verbuchen" sichtbar

Beim Einbau der Berücksichtigung des SEPA Exports sind die gefundenen Regeln für jede Banktransaktion
vorher zurückzusetzen.

Lagerbuchungen: Fehler bei Sortieren nach Chargennummer

Can't use an undefined value as an ARRAY reference at SL/WH.pm line 445.

Es fehlte die Sortiervorschrift für "chargenumber"

Typo in Kommentar

SL::Dev::Record - Doku Beispiel für viele Aufträge

Doku zu DATEV.pm csv_export_for_tax_accountant

DATEV csv_export_for_tax_accountant nutzt generate_datev_data

DATEV.pm Einrückung

DATEV Daten Test

DATEV Export - nach Abteilung filtern

SL/GoBD.pm nutzt generate_datev_data statt _get_transactions

DATEV-Checks beim Buchen nutzen nun generate_datev_data

SL/DATEV.pm für KNE-Export überarbeitet / Zwischendaten eingeführt

_get_transactions war bisher eine interne Funktion von SL::DATEV, die vor dem
DATEV-Export aufgerufen wurde, und die Daten aus der Datenbank ausgelesen und
transformiert hat. In diesem Schritt wurde auch auf DATEV-Fehler geprüft, daher
war diese Funktion prinzipiell schon ausreichend für die DATEV-Checks beim
Buchen, und es muß nicht noch extra eine Datei-Export gestartet werden.

Im ersten Schritt wurde diese Funktion also umbenannt nach generate_datev_data.

Beim Erstellen des KNE-Formats aus den Daten wurde bisher direkt beim
Bearbeiten der Daten die KNE-Datei durch viele add_blocks aufgebaut. Jetzt
werden erst in einem Zwischenschritt alle Daten in einem "neutralen" Array von
Hashes gesammelt, so daß sie von dort in einem Rutsch nach KNE oder z.B. nach
CSV exportiert werden können.

Die so generierten Daten (generate_datev_lines) eignen sich auch gut für Tests.

In diversen Kundenerweiterungen werden auch gerne die zu exportierenden Daten
nochmal modifiziert, z.B. Ersetzen des Sammelkontos durch ein Personenkonto,
dies geschieht am Besten auch bei den neutralen Daten.

Weiterhin gab es beim KNE-Export noch Reste von Code, der die Buchungsdaten auf
mehrere Exportdateien "ED0001, ED0002, ..." verteilen konnte. Aktuell hat das
aber nicht funktioniert, und es wird immer alles nach ED0001 geschrieben, von
daher wurde hier auch Code entfernt. Das Postversendeformat (KNE) von DATEV
wird allerdings sowieso bald eingestellt werden (ab 2018).

DATEV Export - Vorbereitung für CSV Export

DATEV KNE Export Refactoring

Anstatt die Werte aus der DB direkt zu transformieren und per add_block
direkt die KNE-Datei zu bauen werden jetzt alle Daten in einem Array aus
Hashrefs gesammelt und unformatiert zwischengespeichert.
Aus diesem Zwischenstand wird dann erst in einem Rutsch die KNE-Datei
generiert. An dieser Stelle könnte man aber auch direkt die Daten als
CSV abgreifen.

CsvImport: Mehrdeutigkeit nur prüfen, wenn entsprechende Daten angegeben sind.

Fehler kam im commit
"CsvImport: bei nicht eindeutigen Kunden/Lieferanten Fehler melden."
rein.

CsvImport: customernumber/vendornumber statt number

Fehler kam im commit
"CsvImport: bei nicht eindeutigen Kunden/Lieferanten Fehler melden."
rein.

ActionBar: Verwendung bei »System« → »Vorlagen« → *

»System« → »Vorlagen« → »Stilvorlage« entfernt

Die Funktion funktioniert seit der Aufteilung der Stylesheets in
mehrere Unterdateien schlicht nicht mehr. Da sich bisher niemand
beschwert hat, wird die Funktion wohl auch nicht benötigt.

ActionBar: Verwendung bei »System« → »Historiensuchmaschine«

Kreditoren/Debitoren-Buchung zuletzt bebuchtes Konto laden

Wenn nur ein Konto ohne Wert vorhanden ist, gehen wir davon aus,
dass der Anwender einen anderen Kunden/Lieferanten buchen möchten und
hier das zuletzt bebuchte Konto als Default möchte

ActionBar: Verwendung bei »System« → »Benutzer«

ActionBar: Verwendung bei »System« → »Bücherkontrolle«

ActionBar: Verwendung bei »System« → »Korrekturen im Hauptbuch«

ActionBar: Verwendung bei »System« → »Lager«

CsvImport Mulitplex: Template: kein loop.count ausserhalb einer Schleife

CsvImport Aufträge: Rabatt richtig für Datenbank aufbereiten.

In der DB steht der Rabatt als Hundertstel.

CsvImport Aufträge: bei nicht eindeutigen Artikeln Fehler melden.

CsvImport Aufträge: Artikel auch nach EAN suchen

CsvImport: bei nicht eindeutigen Kunden/Lieferanten Fehler melden.

CsvImport: Kunden/Lieferanten auch nach GLN suchen können.

Für die Imports, die die Angabe eines Kunden oder Lieferanten brauchen und
check_vc verwenden (Aufträge, Ansprchpersonen, Lieferanschriften,
Debitorenbuchungen), kann neben Id, Nummer oder Name auch die GLN verwendet
werden.

Typo in Übersetzung behoben.

CsvImport Multiplex/Aufträge: CVars bei Fehlern nicht behandeln

Verbuchte Zahlungen auch in neuen Fällen von Bankbewegegungen berücksichtigen

Ergänzend zu af131a46:
Banktransaction: Kontoauszug verbuchen: Sonderfall für "credit_notes"

bank_transactions.t richtige Anzahl von Tests

Testfall test_ap_transaction auch ausführen

Seit Anbeginn des Testfalls vorhanden. Jetzt auch ausgeführt.

Testfall Verbuchen Kontoauszug zu Kreditorenbeleg ergänzt

Sehr viele Tests für ar, nur einen speziellen für ap.

Test: Lieferantengutschrift verbuchen, auch die Gegenseite Banktransaktion prüfen

Die Zahlung wurde korrekt gebucht, allerdings erwartet kivitendo jetzt auch
Änderungen in der bank_transactions invoice_amount, in anderen Testfällen
(test1) wird diese auch überprüft.

überflüssiges SQL-Upgrade-Script »parts_add_chart_foreign_keys.sql« entfernt

Im dem Script werden drei Fremdschlüssel für Spalten angelegt, die im
nachfolgenden Script »parts_remove_unneeded_fields.sql« gleich gedroppt
werden. Damit ist das Script überflüssig und sogar schädlich, falls
jemand noch Datenbankinhalt hat, bei dem die Spalten IDs enthalten, zu
denen es in den Zieltabellen keine Zeilen mehr gibt.

SL::X::DBError Serialisierung sollte auch die msg ausgeben

Ver-/Einkauf/Finanzbuchhaltung: automatisches Update nach Kunden-/Lieferantenwechsel

CsvImport Multiplex/Aufträge: Auch nicht angegebene CVars im Objekt speichern.

ActionBar: Verwendung bei »System« → »Benutzerdefinierte Variablen«

ActionBar: Verwendung bei »System« → »Lieferbedingungen«

ActionBar: Verwendung bei »System« → »Zahlungsbedingungen«

ActionBar: Verwendung bei »System« → »Sprachen und Übersetzungen« → »SEPA-Überweisungen«

ActionBar: Verwendung bei »System« → »Sprachen und Übersetzungen« → »Anreden«

ActionBar: Verwendung bei »System« → »Einheiten bearbeiten«

Rechnungssuchen Einkauf/Verkauf: Seitentitel an Menütitel anpassen

Damit wird klarer, dass nicht nur nach Debitorenbuchungen gesucht
wird, sondern auch Verkaufsrechnungen und Gutschriften.

ActionBar: Verwendung bei »Einkauf« → »Berichte« → »Einkaufsrechnungen…«

ActionBar: Unterstützung für »only_if«/»not_if« bei Links

ReportGenerator: Action-Bar-Actions über Hook-Funktion ausgeben lassen können

Damit ist es möglich, dass die Actions vom Report-Generator zwischen
den Actions der Hauptmaske eingefügt werden.

Kontoauszug verbuchen: Zeilen ohne Vorschläge trotzdem anzeigen

LXDebug: clone_for_dump als eigenständige Funktion zur Wiederverwendung

ActionBar: Und nochmal Korrektur zu html Menü scrolling

PriceSource: editable Dokumentation

Preisquellen: Support für editierbare Preisquellen

ActionBar: top-padding in html menü + actionbar

ActionBar: Scrolling von ausgeklapptem HTML Menü gefixt

DO::get_shipped_qty - typo

ActionBar: Standardaktion im Artikelstammdatencontroller auf »Speichern«

ActionBar: Standardaktion im Kundenstammdatencontroller auf »Speichern«

ActionBar: nicht »continue«, sondern »next« für Short-Circuit einer Schleife

Kreditorenbuchungen: Storno von bezahlten Rechnungen verhindern

Das Action-Bar-Setup nutzt den Wert $::form->{totalpaid} als Indikator
dafür, ob bereits Zahlungen verbucht wurden. Ist das der Fall, so darf
die Rechnung nicht storniert werden können.

Daher muss dieser Wert berechnet werden, bevor das Action-Bar-Setup
durchgeführt wird.

ActionBar: Standard-Aktions-Button optisch hervorheben

Der Standardbutton ist derjenige, der bei Druck auf Return/Enter
ausgelöst wird.

Aktuell ist die Hervorhebung über fette Schrift geregelt. Eine andere
Möglichkeit wäre, die Border von 1px auf 2px zu erhöhen, was das
Aussehen analoger zu klassischen GUIs machen würde.

SL::X: und ohne typo

Datenbankexceptions: SL::X::DBError korrekt ausgeben

ActionBar: ReportGenerator: get('actionbar') liefert Array von Bars

ActionBar: Hintergrundjob-Verwaltung: Action heißt destroy, nicht delete.

changelog - Features aus Unstable aktualisiert

Typos in changelog

ustva-report.html geierlein-js kosmetik

Zwei Leerzeichen-Einrückung ist scheinbar das gängigste
für JavaScript Funktionen im kivitendo-Projekt.

ustva html Template: geierlein-pfad Variable korrigiert

Die Prüfung weiter oben ist korrekt, der eigentliche Variablenname
aber dann für das Programm falsch. Scheinbar war dieser bei OD
hartkodiert und das Feature wurde nicht vor dem Commit im Standard
geprüft.

trigram Indizes mit may_fail versehen und UPGRADE entsprechend angepasst

CsvImportReport: Manager-Methode destroy löscht nicht aus aktiver Sitzung

Vorher wurden alle Reports bis auf den letzten aus der aktiven Sitzung gelöscht.
Da aber mit den Reports auch das Profile gelöscht wird und im Profil der
zufällige Dateiname der temporären Csv-Datei enthalten ist und dieser nach einem
Test-Import für weitere Test-Importe oder den eigentlichen Import benötigt wird,
darf dieser Report nicht gelöscht werden.

BankImport: Csv-Datei mit zufälligem Dateinamen speichern und im Profil übergeben

siehe auch commit "CsvImport: zufälligen Dateinamen für jeden Import. Ablegen in
temporärem Profil"

CsvImport: zufälligen Dateinamen für jeden Import. Ablegen in temporärem Profil

Damit können auch mehrere gleiche Imports von einer Session aus gleichzeitig
aufgerufen werden.

CsvImport: Controller erzeugt temporäres Profil, nicht der Hintergrund-Job.

CsvImport: temporäres Profil nicht beim Report speichern, …

… denn das passiert schon beim Anlegen des Hintergrund-Jobs.

BankImport: Refactoring: Profil über init-Methode laden.

BackgroundJob::CsvImport: unbenutztes Modul SessionFile::Random nicht laden

CsvImport: doppeltes delete-Statement entfernt

CsvImport: doppelt angegebene Methode "file" entfernt

POD: Typo in Autor-Abschnitt verbessert.

ActionBar/Dateimanagement: Verschicken von Dateien via E-Mail wieder implementiert

Kreditorenbuchungen: Zahlungen buchen gefixt

Seit Umstellung auf die Verwendung des Chart-Pickers heißt die
Form-Variable für das Verbindlichkeitskonto »AP_chart_id«, und nicht
mehr »APselected«. Außerdem enthält sie die Datenbank-ID des Kontos,
und nicht die Kontonummer.

WebDAV: Überflüssigen Aufruf webdav_path im Frontend entfernt

WebDAV::get_all_objects ruft in der Methode seit Anbeginn des Moduls
schon ein webdav_path auf, von daher schreddern bevor noch  mehr
copy & waste im Programm passiert

SL::Template::*: Form::template direkt nutzen, nicht mehr Form::init_template

Form::init_template wurde bei der Umstellung entfernt, bei der nur
noch die eine Template-Instanz im SL::Presenter genutzt wird, und Form
keine eigene mehr beinhaltet.

Mailer: Zeichensatzattribut bei Anhängen nur bei Text-Anhängen setzen

Ein als Binärdaten zu behandelnder Anhang darf nicht natürlich nicht
umcodiert werden. Normalerweise ignorieren E-Mail-Programme bei
Nicht-Text-Anhängen den Zeichensatz, wenn er gesetzt ist, aber wir
sollten diesbezüglich auch einfach saubere E-Mails senden.

Mailer: Attribut »path« bei »Content-Type« nicht setzen

Darin steht der Pfad auf dem lokalen System und ist damit ein
information leak, z.B. /var/www/kundenname/kivitendo/files/…

LXDebug::dump: Objekte aus Namensraum SL::* ohne »as_debug_info« wie Hash behandeln

LXDebug::dump: nur Form-Instanzen als Hash behandeln, nicht alle Hash-basierten Objekte

LXDebug::dump: Unterstützung für zirkuläre Strukturen

Durch die Umstellung auf vorheriges Reduzieren auf essenzielle
Informationen muss dump() sicherstellen, dass es bei zirkulären
Strukturen nicht in eine Endlosschleife gerät.

Weiterhin müssen alle Rückwärtsreferenzen aufgeweicht
werden (Scalar::Util::weaken), damit sie von der garbage collection
normal aufgeräumt werden.

kivi.File.js: Debugcode wieder entfernt

LXDebug::dump: Interna von Rose-DB- und DateTime-Objekten nicht mehr ausgeben

dump arbeitet nun nicht mehr direkt auf dem zu dumpenden Objekt,
sondern auf Kopien, die je nach Typ auf ihre essenziellen
Informationen zusammengeschrumpft werden. Dafür kann ein Objekt eine
Methode »as_debug_info« bereitstellen, die eine solche Essenz
zurückgibt.

Für SL::DB::Object ist eine Implementation beigelegt, die nur die
Spalten mit ihren stringifizierten Werten zurückgibt, nicht aber mehr
die ganzen Interna wie z.B. Meta-Informationen enthält.

Arrays und Hashes (und Objekte, die auf diesen simplen Typen basieren
und keine eigene »as_debug_info« zurückgeben) werden rekursiv geklont.

Alles andere definierte wird stringifiziert.

Dafür wurde die Funktion LXDebug::dump_object entfernt, die etwas
Ähnliches gemacht hat, aber nur für eine einzelne Rose-DB-Instanz.

Dateimanagement: Fehler beim Umbenennen richtig zurückgeben

Ist man z.B. bei den Artikelstammdaten und versucht, eine Datei
umzubenennen, und ändert den Namen nicht, so sollte eine Fehlermeldung
angezeigt werden. Wurde sie aber nicht, weil Folgendes passierte:

• Im ClientJS-Flash wird Fehlermeldung erzeugt und das ClientJS sofort
  gerendert, anschließend mit »return« der innerste Block verlassen
• Der innerste Block ist aber ein »eval { … }«, und nicht die Funktion
  selber. Daher geht der Kontrollfluss nach dem »eval« weiter. Dabei
  werden mehr Sachen auf das ClientJS raufgepusht und aschließend
  erneut gerendert.

Dieses zweite Rendern schreibt dann erneut HTTP-Response-Zeilen sowie
ein vollständiges Array von ClientJS-Aktionen.

Der JavaScript-Code auf der Browser-Seite bekommen somit:

• HTTP-Response-Zeilen #1
• ClientJS-Array #1
• HTTP-Response-Zeilen #2
• ClientJS-Array #2

Damit kann er natürlich nicht umgehen, ignoriert die komplette
Antwort, und die Fehlermeldung wird somit nicht angezeigt.

Der Fix ist, nur das Umbenennen innerhalb des »eval« zu machen, und
den Rückgabewert außerhalb des »eval« zu prüfen. Bei einem anderen
Wert als SL::File::RENAME_OK wird das ClientJS dann nur einmal
gerendert und anschließend die Funktion verlassen.

Dateimanagement: Stammdaten: DOM-Elemente bei multiples Tab-Aufrufen nicht duplizieren

Der Aufruf des Tabs »Dateianhänge« in den Artikelstammdaten liefert
einen Tab aus, in dem der HTML-Code für den Umbenennen-Dialog
vorhanden ist (initial versteckt). Ruft man den Dialog dann auf, so
verschieb jQuery den in dem Moment im DOM an eine ganz andere
Stelle (an den Ende vom <body>), sodass der Dialog-Code nun nicht mehr
innerhalb des <div> liegt, das für den Tab geladen wurde.

Wechselt man nur zu einem anderen Tab und ruft den Tab »Dateianhänge«
erneut auf, so wird die bisherige <div> für den Tab entfernt und neu
geladen. Beim Neuladen wird natürlich auch wieder eine Kopie des
HTML-Codes für den Dialog mitgeschickt.

Die erste Kopie befindet sich zu dem Zeitpunkt aber weiterhin im DOM,
weil sie ja kein Kind des ursprünglichen Tab-<div>s mehr ist. Somit
haben wir nun zwei Kopien des HTML-Codes im DOM.

Das führt dazu, dass auch die Inputs doppelt vorhanden sind. Es wird
dann die falsche Input (nicht die, die der Benutzer*in angezeigt wird)
an den Server geschickt.

Der Fix sorgt dafür, dass der Dialog beim Schließen wieder an seine
ursprüngliche Stelle im DOM verschoben wird. Dadurch wird der beim
Neuladen des Tabs zusammen mit dem alten Tab-<div> sauber entfernt.