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«).
Moritz Bunkus [Fri, 13 Jan 2017 09:25:48 +0000 (10:25 +0100)]
ActionBar: ComboBox mit nur einem Eintrag wie Eintrag rendern
Das erleichtert, wenn man in einer ComboBox mehrere Einträge evtl. gar
nicht anzeigt. Der Aufrufer muss dann nicht prüfen, ob er der ComboBox
einen oder mehrere Einträge übergibt.
Moritz Bunkus [Fri, 13 Jan 2017 09:22:12 +0000 (10:22 +0100)]
ActionBar: Auslassen von Actions über Parameter »only_if«/»not_if« steuern können
Gedacht für Buttons, die z.B. aufgrund der Mandantenkonfiguration nie
angezeigt werden können. Nicht gedacht für Buttons, die nur aufgrund des
Belegzustands nicht benutzt werden können (z.B. »Löschen« bei einem noch
nicht gespeicherten Beleg).
projects / mfinanz.git / log