G. Richardson [Sun, 12 Feb 2017 12:16:25 +0000 (13:16 +0100)]
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).
G. Richardson [Fri, 10 Feb 2017 12:09:06 +0000 (13:09 +0100)]
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.
Moritz Bunkus [Fri, 24 Mar 2017 15:04:02 +0000 (16:04 +0100)]
»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.
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
Bernd Bleßmann [Sun, 19 Mar 2017 16:46:17 +0000 (17:46 +0100)]
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.
Jan Büren [Thu, 23 Mar 2017 13:37:07 +0000 (14:37 +0100)]
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.
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.
Moritz Bunkus [Fri, 17 Mar 2017 10:09:32 +0000 (11:09 +0100)]
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.
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.
Jan Büren [Sat, 4 Mar 2017 09:47:18 +0000 (10:47 +0100)]
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.
Bernd Bleßmann [Tue, 21 Feb 2017 10:53:37 +0000 (11:53 +0100)]
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.
Moritz Bunkus [Thu, 2 Mar 2017 15:16:39 +0000 (16:16 +0100)]
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.
Jan Büren [Thu, 2 Mar 2017 15:09:38 +0000 (16:09 +0100)]
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
Moritz Bunkus [Thu, 2 Mar 2017 15:02:44 +0000 (16:02 +0100)]
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.
Moritz Bunkus [Thu, 2 Mar 2017 14:13:47 +0000 (15:13 +0100)]
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.
Moritz Bunkus [Wed, 1 Mar 2017 15:54:59 +0000 (16:54 +0100)]
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.
Moritz Bunkus [Wed, 1 Mar 2017 14:12:02 +0000 (15:12 +0100)]
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.
Moritz Bunkus [Wed, 1 Mar 2017 13:16:31 +0000 (14:16 +0100)]
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:
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.
Moritz Bunkus [Wed, 1 Mar 2017 12:46:34 +0000 (13:46 +0100)]
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.
projects / mfinanz.git / log