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.
Moritz Bunkus [Wed, 1 Mar 2017 12:45:05 +0000 (13:45 +0100)]
kivi.popup_dialog: Dialog vor »custom close function« schließen
Wenn die »custom close function« den Dialog im DOM verschieben möchte,
so macht sie das mit $dlg.remove().appendTo('#new_parent_id'). Dabei
geht aber die Dialog-Initialisierung flöten.
Wird also erst anschließend $dlg.dialog('close') ausgeführt, so hagelt
das eine Fehlermeldung.
Moritz Bunkus [Wed, 1 Mar 2017 11:52:16 +0000 (12:52 +0100)]
generic/exception.html wiederhergestellt
Die Vorlage wurde im Commit 9d8f72a0f92d01e1e25b14788b193cd662cad0d3
entfernt, weil fälschlicherweise gedacht wurde, dass sie nicht mehr
benutzt wird, da locales.pl eine Warnung diesbezüglich ausgab.
Tatsächlich wird sie aber noch benutzt, und zwar als generische
Fehler-Seite für das Template-Modul. Dieser Fall wurde von locales.pl
mangels Markup aber nicht erkannt.
Moritz Bunkus [Tue, 28 Feb 2017 16:00:23 +0000 (17:00 +0100)]
Dateimanagement: Anhänge nicht als Referenz an SL::Mailer übergeben
SL::Mailer erwartet, dass der Inhalt der Anhänge, die in
$mailer->{attachments} übergeben werden, direkt im Attribut »content«
gespeichert ist.
Das Interface von SL::File hingegen gibt nur eine Skalarreferenz auf
den Dateiinhalt zurück. Daher kann diese nicht 1:1 an den SL::Mailer
übergeben werden, da es ansonsten zu Fehlermeldungen von Rose beim
Speichern im E-Mail-Journal kommt (»cannot bind reference«).
Moritz Bunkus [Tue, 28 Feb 2017 12:00:23 +0000 (13:00 +0100)]
Versehentlich entfernte Fremdschlüssel auf sepa_export_items wieder hinzugefügt
Das DB-Upgrade-Script
»auto_delete_sepa_export_items_on_ap_ar_deletion.pl« hat via
»SL::DBUpgrade2::Base::drop_constraints« alle Constraints auf
»sepa_export_items« entfernt, dann aber nur zwei davon (mit anderen
Bedingungen) neu angelegt — nicht aber die für die Spalten »chart_id«
und »sepa_export_id«.
Resultat ist, dass die Relationships aus den MetaSetups rausfliegen,
wenn man die von einer sauberen DB erzeugen lässt (z.B. mit
Scriptoption »--test-client«).
projects / kivitendo-erp.git / log