Finanzbuchhaltungsmasken: Flash anzeigen & <form> eine ID geben

ClientJS: Warnungen fixen

ClientJS: »pattern:« nur bei Abschnittskommentaren

Bei einem Kommentar hinter der Action ist der Kommentar selber bereits
das Pattern und darf das Stichwort »pattern:« nicht mehr
enthalten. Andernfalls würde es 1:1 in den JS-Code kopiert.

Verbleibenden all_vc-Aufruf entfernt

Reste von arap.pl entfernen

Merge branch 'f-use-proper-selects-and-pickers'

arap.pl entfernt

Sie enthielt nur noch drei Funktionen:

• select_project — wurde nirgends mehr verwendet
• project_selected — war nur eine Helferfunktion für select_project
• continue — diese wurde nach common.pl verschoben

vclimit entfernen

Form->all_vc: entfernt, da inzwischen unbenutzt

Berichte: $form->all_vc nicht mehr verwenden

Mahnungen: $form->all_vc nicht mehr verwenden

Funktion check_name() entfernt

Alle Anwender wurden auf Kunden-/Lieferantenpicker umgestellt.

Lieferschein: Kunden-/Lieferantenpicker nutzen

Presenter: button_tag, submit_tag, ajax_submit_tag von L nach SL::Presenter::Tag verschoben

Lieferscheinsuche: Kunden-/Lieferanten nicht mehr als Drop-Down

Lieferscheinsuche: kompaktere Maske

Zahlungsein-/-ausgänge: Kunden-/Lieferantenpicker verwenden

Mahnungssuche: Kundensuche immer nur via Eingabefeld

Multibox entfernen

Inzwischen wird das Template nicht mehr benutzt.

Verkaufswertbericht: keine Multibox mehr nutzen

Kontoauszug verbuchen: keine Multibox verwenden

Angebote/Aufträge: Währungsauswahl via L.select_tag

Angebots-/Auftragssuche: Maske kompakter und konsistenter gemacht

Angebote/Aufträge: Kunden-/Lieferantenwahl via Picker

Angebots/Auftragssuche: Style-Attribut zentral via Variable setzen

Angebots/Auftragssuche: keine Multibox verwenden

Einkaufsrechnungen: Lieferantenwahl über Picker und nicht Multibox

Einkaufsrechnungen: Abteilungs- & Projektauswahl nicht über Multibox

Verkaufsrechnungen: Abteilungen sortiert anzeigen

Einkaufsrechnungen: keine Multibox für Steuersatzauswahl

Verkaufsrechnungen: Kundenwahl über Picker

»ENDE LOESCHMICH in 2012«… ja dann… weg damit

Artikelsuche: überflüssige Fallunterscheidungen nach Artikeltyp entfernen

Seit der Einführung der Artikelklassifizierung gibt es nur noch eine
Suchmaske für Artikel, in der über Checkboxen gesteuert wird, nach
welchen Typen gesucht wird. Daher ergibt es keinen Sinn, im Template
danach zu unterscheiden, vor allem, da der aufrufende Code die Checks so
setzt, dass immer alle Controls angezeigt werden.

Artikelsuche: Inputs gleich lang gemacht

Artikelsuche: keine Multibox mehr verwenden

Warteschlangen-Druck: keine Multibox mehr verwenden

Einkaufsrechnungssuche: Inputs gleich lang gemacht

Naja gut… so gleich lang, wie Inputs nun mal werden, wenn man sie alle
auf eine einheitliche Breite setzt. Comboboxes werden trotzdem leicht
schmaler gerendert.

Verkaufsrechnungssuche: Inputs gleich lang gemacht

Naja gut… so gleich lang, wie Inputs nun mal werden, wenn man sie alle
auf eine einheitliche Breite setzt. Comboboxes werden trotzdem leicht
schmaler gerendert.

Einkaufsrechnungssuche: keine Multibox mehr nutzen

Kreditorenbuchungen: Kundenauswahl über Picker

Kreditorenbuchungen: Währungs-Drop-Down nicht als Hidden mitschleifen

Verkaufsrechnungssuche: keine Multibox mehr nutzen

Debitorenbuchungen: Kundenauswahl über Picker

Erweiterung Liquiditätsübersicht

wiederkehrende Rechnungen mit Periode einmalig wie einen normalen
offenen Auftrag in der Liquiditätsübersicht behandeln

locales und changelog für vorherigen commit

Wiederkehrende Rechnungen, um neue Periode 'einmalig' erweitert.

Wiederkehrenden Rechnungen die aus einem Auftrag automatisch generiert
werden können jetzt die Periode 'einmalig' (engl. 'one time') mit dem
Zeitwert 0 haben.
Falls diese Periode angewählt wird, wird die Konfiguration des Auftrags
nach Erzeugen der einmaligen Rechnung auf inaktiv gesetzt (s.a.
 perldoc SL/DB/PeriodicInvoicesConfigs.pm).
Der Empfänger der Rechnungslauf-Mail bekommt in diesem Fall zusätzlich
die Auftragsnummer, bei welchem die wiederkehrenden Rechnungen deaktiviert
wurden mitgeteilt.
Zusätzlich ist diese Benachrichtigung per Mail erweitert, sodass jetzt
auch eine Liste der versandten Rechnungen per E-Mail mitgeteilt wird.

Debitorenbuchungen: Währungs-/Verkäufer-Drop-Downs nicht als Hidden mitschleifen

Kreditorenbuchungen: nur zur Kontenkategorie passende Steuerschlüsselauswahl anbieten

Historien Suchmaschine: SQL-Injektion verhindern.

ProjectPicker: Taskendrücke auf Shift, Ctrl & Alt ignorieren

Andernfalls wird schon beim Drücken eines der Modifier eine Suche
ausgelöst. Das kann besonders nervig sein, wenn man den Focus vom
Browser wegnimmt:

• Alt+Tab drücken, um zum anderen Fenster zu wechseln
• Schon bei Alt wird eine Suche ausgelöst.
• Durch den Focus-Verlust wird ein Blur-Event ausgelöst.
• Und schon ist der interne Zustand ungültig, und das Element wird rot
  dargestellt.

Analoges passiert, wenn man z.B. mit Ctrl+F2 zu einem anderen virtuellen
Desktop wechselt.

Noch schlimmer ist, dass bei Shift+Tab ebenfalls zuerst eine Suche
ausgelöst und anschließend verhindert wird, dass der Focus das Element
verlässt.

ProjectPicker: unbenutzte Variable last_unverified_dummy entfernt

Sie wird nur gesetzt, nie gelesen, und vor allem auch nicht mit »var«
deklariert — wodurch es im Strict-Modus zu JavaScript-Fehlermeldungen
kommt.

ProjectPicker: Strict-Modus & Warnungen von jshint beseitigt

CustomerVendorPicker: Taskendrücke auf Shift, Ctrl & Alt ignorieren

Andernfalls wird schon beim Drücken eines der Modifier eine Suche
ausgelöst. Das kann besonders nervig sein, wenn man den Focus vom
Browser wegnimmt:

• Alt+Tab drücken, um zum anderen Fenster zu wechseln
• Schon bei Alt wird eine Suche ausgelöst.
• Durch den Focus-Verlust wird ein Blur-Event ausgelöst.
• Und schon ist der interne Zustand ungültig, und das Element wird rot
  dargestellt.

Analoges passiert, wenn man z.B. mit Ctrl+F2 zu einem anderen virtuellen
Desktop wechselt.

Noch schlimmer ist, dass bei Shift+Tab ebenfalls zuerst eine Suche
ausgelöst und anschließend verhindert wird, dass der Focus das Element
verlässt.

CustomerVendorPicker: unbenutzte Variable last_unverified_dummy entfernt

Sie wird nur gesetzt, nie gelesen, und vor allem auch nicht mit »var«
deklariert — wodurch es im Strict-Modus zu JavaScript-Fehlermeldungen
kommt.

CustomerVendorPicker: Strict-Modus & Warnungen von jshint beseitigt

Verkaufsbericht - Abteilung korrekt behandeln

und Umstellung auf select_tag

Historien Suchmaschine: Stacküberlauf vermeiden

Die Fehlermeldung
    'DBD::Pg::st execute failed: ERROR:  stack depth limit exceeded' wird durch
    Anpassung des queries vermieden

statt:
    WHERE (trans_id = 2088 OR trans_id=2090 OR trans_id=2092 ... (10000 fach)

nun:

    WHERE (trans_id = 2088 OR trans_id IN (2090,2092,2094, ...

am: fehlendes include für Historiensuche(2)

Undefined subroutine &main::conv_dateq called at bin/mozilla/am.pl line 1246

PDF Helper Erweiterungen: Formatkorrektur nach perltidy

t/backgorund_job/known_jobs.t

Dieser Test hat noch nie einen Fehler gefunden, aber produzierst ständig
false positives wenn jemand einen neuen Hintergrundjob einfügt und
vergisst den hier einzutragen. Weg damit.

Revert "focus: scroll to active element"

This reverts commit fdc625b4d1ba26782ff81ec2d79c1b533f26a9a7.

Das funktioniert so nicht. Grundsätzlich passiert das Scrollen passiert bei Fokus
automatisch. Bei force-reload kann es aber passieren, dass js/common
ausgeführt wird, bevor alle css Dateien geladen sind. Dann ändert sich
nachträglich die Größe der Elemente und das Element wird wieder aus dem
Bildschirm geschoben.

Ausserdem können Browser kein scroll offset, focus scrollt nur exakt
soweit dass das Element auf dem Screen ist, auch wenn das bedeutet, dass
es dann in den letzten 3% des Screen liegt.

Ich habe versucht das zu fixen, aber es wird dadurch nur komplizierter,
nicht besser. Also raus damit.

PDF Helper Erweiterungen: bothsided , out_path

merge_pdfs mit weiterem Parameter "out_path"

Statt den Inhalt als Ergebnis zu bekommen, kann nun auch gleich die exitierende Datei zurückgegeben werden

Es kann nun die Seitenzahl der Dokumente ermittelt werden
und ggf (falls param "bothsided" gesetzt) eine leere Seite dazwischengefügt werden

Paginierung schneller machen:

SQL-Injection raus

Neuer Test t/part/stock.t für Part Methode get_stock

SL::DB::Part - neue Methode get_stock

um den aktuellen Lagerbestand eines Artikels in einem bestimmten
Lagerplatz oder in einem bestimmten Lager zu einem bestimmten Stichtag
in der Standardeinheit auszulesen.

SL::Dev::Inventory - set_stock mit abs_qty, transfer_stock

Man kann Lagerbestände jetzt sowohl positiv als auch negativ ändern,
oder einen gewünschten Bestand vorgeben (abs_qty).

Weiterhin kann man per transfer_stock Lagermengen zwischen Lagerplätzen
umlagern.

SL::Dev::Record - Aufträge und Lieferscheine erstellen

Flash bei ReportGenerator-Verwendung nicht manuell anzeigen

Der ReportGenerator zeigt den Flash inzwischen selber an. Daher muss,
nein sollte, oder sogar darf der Aufrufer den Flash im Top-Info-Text
vorher nicht mehr anzeigen. Andernfalls erscheint er doppelt.

Paginierung schneller machen für Lager(Paginierung repariert)

Lagerbuchungen und Lagerbestand optimieren

1. Auf Rose umstellen grosser Aufwand.
2. Roseverhalten nachimplementieren:
2.1. Nur einen Query haben, danach kein herausnehmen von Zeilen
2.2. Dann funktioniert Paginierung mit LIMIT und OFFSET (wie Rose)
2.3. Rose macht ein zu Beginn  ein COUNT(*) um die Gesamtmenge zu ermitteln,wir merken uns das nach dem ersten Query

Der Mengenfilter kann schlecht in SQL integriert werden wegen Einheitenumrechnung (eventuell später noch eine PSQL Funktion einbauen)

jQuery-resetForm() anstelle von selbst geschriebenem Reset-Code nutzen #4

jQuery-resetForm() anstelle von selbst geschriebenem Reset-Code nutzen #3

jQuery-resetForm() anstelle von selbst geschriebenem Reset-Code nutzen #2

jQuery-resetForm() anstelle von selbst geschriebenem Reset-Code nutzen

Unsere bisherigen Reset-Implementationen haben die Inputs alle auf leer
gesetzt. Die Standard-Reset-Routinen der Browser setzen die Inputs
allerdings auf ihren Zustand zurück, den sie beim Laden hatten, was auch
mehr Sinn ergibt.

Alle Rechnungen: »als bezahlt markieren« auf neue Funktionen in SL::DB::(Purchase)Invoice umgestellt

SL::DB::(Purchase)Invoice: neue Funktion »mark_as_paid«

Sie macht nichts weiter, als »paid« auf »amount« zu setzen.

Form->redirect: übergebene Nachricht mittels »flash_later« anzeigen lassen

Der Funktion $::form->redirect kann eine Nachricht übergeben
werden. Diese wird bisher nur dann angezeigt, wenn es keinen
aufzurufenden Callback gibt. Gibt es einen Callback, verschwindet die
Nachricht bisher im Nirvana.

Mit dieser Änderung wird die Nachricht im Flash gespeichert. Damit
besteht zumindest eine Chance, dass die nachfolgend angezeigte Maske die
Nachricht auch anzeigt, sofern sie den Flash einbindet. Da das
langfristig eh alle Masken machen sollen, ergibt diese Übergabe auch
jetzt schon Sinn.

ReportGenerator: im HTML-Bericht den Flash anzeigen

Hintergrundjob zum Schließen bei geschlossenen Aufträgen verlinkten Projekten

Der Job ist per Default deaktiviert. Wenn eine solche Funktionalität für
eine Installation benötigt wird, so muss der Admin manuell einen
Hintergrundjob dafür anlegen.

Lager: Umstellung auf Verwendung vom Part-Picker

Dadurch wurden mehrere alte Artikel-Auswahl-Routinen und -Vorlagen
überflüssig, die gleich entfernt wurden.

Lager: Einlagernfunktionalität aus wh.pl entfernt

Diese wird inzwischen von SL::Controller::Inventory zur Verfügung
gestellt.

Form->show_generic_error: Parameter action, back_button entfernt

Der Funktion kann man eine spezifische Action mitgeben, damit ein Button
mit der Action anstelle des normalen Zurück-Buttons angezeigt wird. Es
gibt allerdings (schon lange?) keinen einzigen Aufruf mehr, der dieses
Feature »Action übergeben« nutzt.

Daher wird es hiermit entfernt.

Im Umkehrschluss gibt es keine Situation, wo der Zurück-Button nicht
angezeigt werden soll. Daher wird auch dieser Parameter entfernt.

Revert "kivi.call_jquery: Funktion zum Aufrufen beliebiger jQuery-Funktionen"

Dies nimmt Commit 8c93869920ec0bf012a0f59175b39449c90a4d1f zurück. War
kein schönes Design, und die Aufrufer wurden bereits auf andere
Interfaces umgestellt.

t/part/assembly.t: nicht mit undef vergleichen

io.pl: disconnect_standard_dbh bei _make_record wieder entfernt

Das war in 49707e3c eingefügt worden mit Hinweis, dass es ein temporärer
Fix ist bis single-dbh abgeschlossen ist. Kann also wieder raus.

SelfTests: check_overpayments verbessert

i)
  chart_link muss nicht mehr über JOIN chart c c.link mit der
  acc_trans vereint werden
ii)
  Der Test hat keine sinnvollen Daten an den Benutzer zurück-
  gegeben. Jetzt wird die Kunden- und die Rechnungsnummer gemeldet
iii)
  Die Datenmenge nach aktuellem Geschäftsjahr zu filtern ist ent-
  sprechend angepasst (aktuelle Zahlungen -> Analyse aller vorhergenden
  Daten (trans_id) unabhängig vom Zeitraum).

false positive in SelfTests gefiltert (Periode)

Hintergrund:
Um nur die Daten-Analyse auf das aktuelle Geschäftsjahr (Geschäftsjahr gleich
Kalender-Jahr) zu machen, wird für die meisten Tests mit transdate == current_year
gefiltert.
Problem: Periodenübergreifende Buchungen (Zahlungsein- und -ausgänge) werden
nicht berücksichtigt und schlagen entsprechend Fehlarlam.

Optimierung: Es werden nur die Buchungen des aktuellen Geschäftsjahr angeschaut
und weitere acc_trans Daten ohne Zeitfilter über die trans_id gruppiert.

Menu Icons können nun auch .svg Dateien sein

Vorrangig werden nun svg Dateien aus dem Verzeichnis ./image/icons/svg/*.svg verwendet.
Falls dort kein Icon auffindbar ist wird nach ./image/icons/16x16/*.png gesucht.

Es gibt seit langem ein svg Icon für den GoBD Export.
Das wird nun sichtbar. GGf anpassen.
Das Verzeichnis icon_svg wurde in icons/svg umbenannt.

Merge branch 'f-chart-picker-in-gl'

kivi.PriceRule.js: strict & Warnungen vermeiden

kivi.call_jquery: Funktion zum Aufrufen beliebiger jQuery-Funktionen

An vielen Stellen schicken wir vom Backend aus nur die Namen
aufzurufender JavaScript-Funktionen an den Browser. Der sucht dann die
auszuführende Funktion mittels »kivi.get_function_by_name« aus dem
Namensraum heraus und führt sie anschließend mit »func.apply(…)« aus.

Leider ist es damit nicht so einfach möglich, jQuery-Funktionen
auszuführen, da diese als »this«-Argument ein jQuery-fiziertes Objekt
erwarten. Außerdem geht ein »namespace("jQuery.fn.resetForm")« schlicht
nicht bzw. tut nicht das, was man erwartet.

Daher die Funktion »kivi.call_jquery«. Ihr übergibt man:

1. einen jQuery-Selektor, der das »this«-Argument der aufzurufenden
   Funktion repräsentiert,
2. den Namen der aufzurufenden Funktion und
3. optional weitere Argumente.

Würde man z.B. einen Wert in einem Input setzen wollen, so sähe der
Aufruf in der Console so aus:

  kivi.call_jquery('#some_input_field', 'val', '42')

Das ist äquivalent zo

  $('#some_input_field').val('42')

Interessant wird die Verwendung an Stellen, wo nur Funktionsnamen
übergeben werden dürfen, z.B. bei »call«-Actions bei der ActionBar. Hier
sähe z.B. eine Action zum Zurücksetzen einer Form wie folgt aus:

  action => [
    t8('Reset'),
    call => [ 'kivi.call_jquery', '#form', 'resetForm' ],
  ],

Beim Anklicken wird dann letztlich das Äquivalent hierzu ausgeführt:

  $('#form').resetForm()

kivi.history_back: wie history.back() aber über kivi.get_function_by_name auffindbar

kivi.submit_form_with_action: fügt Hidden-»action« mit Wert zu Form hinzu & submittet

Zuerst werden alle existierenden Inputs namens »action« entfernt, damit
sich die Inputs nicht ins Gehege kommen. Anschließend wird ein neues
Hidden namens »action« mit dem übergebenen Wert erstellt, der Form
hinzugefügt, und die Form wird über $(selector).submit() sumittet.

kivi.SalesPurchase.js: Warnung vermeiden

PrintOptions-Helfer: sub opthash Paket-lokal anstelle von sub-lokal

Wir nutzen Named-Sub-in-Named-Sub nie, daher auch hier
nicht. Ursprünglich war die Funktion in bin/mozilla/io.pl, da ergab es
noch Sinn, um den Funktionsnamensraum nicht zu überladen. Aber nach dem
Verschieben in ein eigenes Paket ist das nicht mehr nötig.

CSS: eine Klasse für versteckte Elemente

Mahnungen: Funktion »Diese Sprache verwenden« gefixt Teil 2

Wenn die Checkbox angehakt ist, so soll die Benutzer*in die Möglichkeit
haben, über die Druckoptionen einen Sprache für alle zu erzeugenden
Mahnungen auszuwählen. Ist die Checkbox aus, so wird die Sprache des
Kunden der jeweiligen Mahnung genommen, und dann muss die Sprachauswahl
deaktiviert werden.

Der erste Commit hat nur die Maske zum Erstellen neuer Mahnungen
abgedeckt. Dieser erledigt die Maske zum Anzeigen vorhandener Mahnungen.

Mahnungen: Funktion »Diese Sprache verwenden« gefixt

Wenn die Checkbox angehakt ist, so soll die Benutzer*in die Möglichkeit
haben, über die Druckoptionen einen Sprache für alle zu erzeugenden
Mahnungen auszuwählen. Ist die Checkbox aus, so wird die Sprache des
Kunden der jeweiligen Mahnung genommen, und dann muss die Sprachauswahl
deaktiviert werden.

Part Controller - als neu speichern repariert

alle Buttons (Speichern, Als neu speichern, Löschen) setzen die action
selber (siehe js/kivi.Part.js) , die form braucht daher kein verstecktes
action für dispatch.

Mahnungen: Typo in query

Fürhte zwar nicht zu einem Fehler, war aber sicher so nicht gewollt.

locale-Lauf