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

Artikelsuche: Überschrift in Bericht wieder ergänzt

Es gibt inzwischen nur noch einen Menüpunkt für Artikelsuche, nicht mehr
einen pro Typ. Der Typ wird dann anhand von Checkboxen unterschieden.

Da dabei durchaus nach mehreren Typen gleichzeitig gesucht werden kann,
ergibt eine Unterscheidung der Überschrift nach zu suchendem Typ wenig
Sinn. Ergo wird als Überschrift immer »Artikel« genutzt.

Kreditorenbuchungen auf Verwendung des Chart-Pickers umgestellt

Debitorenbuchungen auf Verwendung des Chart-Pickers umgestellt

Form::create_links: überflüssige Kommentare entfernt

Dialogbuchen auf Verwendung des Chart-Pickers umgestellt

Mahnungen: Metadaten der Mitarbeiter (Bearbeiter/Verkäufer) setzen

Vorher: salesman_name hat den Namen von employee_name bekommen. Weitere
Daten wurden nicht (mehr) gesetzt.

Aktuell: Jetzt werden die Metadaten von employee_ und salesman_ sauber
gesetzt und die Dokumentation ist entsprechend angepasst.
CAVEAT: Bei mehreren Rechnungen die zusammengefasst werden, werden nur beim
ersten Treffer (teilweise LIMIT 1 in SQL-Anweisungen) die entsprechenden
Daten gesetzt (Hinweis in Doku geschrieben).

RB-Druckvorlagen: mehr Mitarbeiter-Metadaten für Mahnungen

Revert "Verkauf/Rechnungen/Massenerstellung: Zahlungsbedigungen werden nicht übernommen"

This reverts commit 90e59318411f63c95a7e803b4383d75ec72427a8.

gl.pl: Kommentar mit falschem Encoding ersetzt

ChartPicker: Lupe wie beim PartPicker stylen

ChartPicker: 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.

ChartPicker: 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.

ChartPicker: Strict-Modus & Warnungen von jshint beseitigt

ChartPicker: Test-Seite um vorausgefüllte Picker ergänzt

PartPicker: 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.

PartPicker: 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.

PartPicker: Strict-Modus & Warnungen von jshint beseitigt

Verkauf/Rechnungen/Massenerstellung: Zahlungsbedigungen werden nicht übernommen

Falls im Lieferschein keine Zahlungsbedingungen gesetzt sind,
wird die des Kunden für die Rechnung eingesetzt

kivi.js: alle jshint-Warnungen beseitigt & auf strict umgestellt

Part Controller - Artikelnummer von unbenutzten Artikeln ändern

LXDebug: bei SHOW_CALLER ersten Caller außerhalb von LXDebug.pm nehmen

Andernfalls wird beim Aufruf von z.B. »SL::Controller::CustomerVendor →
LXDebug::dump() → LXDebug::message()« fälschlicherweise
»LXDebug::dump()« als Caller ausgegeben. Das interessiert die Nutzer*in
aber im Zweifelsfall nicht, sondern »SL::Controller::CustomerVendor«.

Zum Erfassen neuer Artikel neuen Part-Controller verwenden

Die Knöpfe zum Erstellen neuer Artikel im Artikelreport springen nun in den neuen Part Controller

Prüfen der Bestandteile eines Erzeugnisses beim Hinzufügen

Erst Prüfung innerhalb des Erzeugnisses,
dann recursive Prüfung der das Erzeugnis enthaltenen Erzeugnisse,
Abbruch nach 100 Rekursionen.

Die Abfrage ist so, dass nur vom Erzeugnis abwärts der Baum in die Tiefe geprüft wird.
Dabei darf auf einem Graph kein Erzeugnis doppelt vorkommen.

Erzeugnisse sind nun editierbar, wenn sie von einem anderen Erzeugnis verwendet werden
solange sie in keinem ERP-Dokument verwendet werden.

Implementiert in einem Helper für SL::Controller::Part.
Er wird auch im Test t/part/assembly.t verwendet

Dokumentation neu generiert

Artikel-Klassifizierung: Dokumentation

partpicker mit classification_id als filter erweitert

Vereinfachte Abkürzungsmethode "typeclass_abbreviation" im Part Presenter

Zusammengefasstes Listen von Ware,Erzeugnis,Dienstleistung und Sortimenten

Die Suchmaske für Ware,Erzeugnis,Dienstleistung und Sortiment gilt nun einheitlich für alle Artikeltypen,
deshalb wird nur noch eine Menueauswahl dazu angeboten

Falls ein Artikel nicht gefunden wird, kann gleich in die Erfassung gegangen werden

Dies ist in der Manantenkonfigurierung einstellbar.

Artikel-Klassifizierung: "Preis separat ausweisen"

Dieses neue Attribut an Artikelklassifizierung erlaubt in Aufträgen und Rechnungen
bestimmte Artikel extra auszuweisen.
Dazu werden diese als extra Variable der Dokumentengeneierung zur Verfügung gestellt.
Siehe neue Dokumentation Kapitel 3.7

Hintergrund:
       Preise von Artikeln wie "Verpackung" oder "Transport" müssen
       oftmals separat ausgewiesen werden, genauso wie der reine Warenwert

Artikel-Klassifizierung

    Die Klassifizierung von Artikeln dient einer weiteren Gliederung um zum Beispiel den Einkauf vom Verkauf zu trennen, etc.

    Gekennzeichnet durch eine Beschreibung (z.B. "Einkauf") und ein Kürzel (z.B. "E")
    Flexibel änderbar und erweiterbar.

    - Neue Datenbanktablle und Rose-Objekte, sowie Controller zum Bearbeiten der Tabelle

    - Zwei-Zeichen Abkürzung:

    Der Typ des Artikel und die Klassifizierung werden durch zwei Buchstaben dargestellt.
    Der erste Buchstabe  ist eine Lokalisierung des Typs des Artikel ('P','A','S') ,
    deutch 'W', 'E', und 'D' für Ware Erzeugnis oder Dienstleistung, ggf. weitere Typen.
    Der zweite Buchstabe ist eine Lokalisierung der Klassifizierungsabkürzung (abbreviation).

    Die Abkürzungen sind aus dem Part Presenter abholbar:
    -  SL::Presenter::Part->type_abbreviation($part_type)
    -  SL::Presenter::Part->classification_abbreviation($classification_id)

    Anpassung des CSV Import,
    nun wird alternativ zur 'part_type'-Spalte die 'pclass'-Spalte mit zwei Buchstaben geparsed und entsprechend
    classification_id, part_type gesetzt.

Banktransaction: Kontoauszug verbuchen überarbeitet(2)

Korektere Initialisierung von Arrays

CsvImport Aufträge: Variable besser benannt und Kommentar berichtigt.

CsvImport Aufträge: Positionen nicht fehlerhaft markieren, wenn Auftrag fehlerhaft.

Das zeigt dann im Bericht nicht alle Zeilen als Fehler an, die selber keinen
Fehler enthalten und bei denen nur die Auftragszeile fehlerhaft ist.
Die Markierung ist auch nicht mehr nötig, da nur Aufträge und nicht
Postitionszeilen gespeichert werden.

CsvImport Aufträge: Keine Aufträge speichern, wenn alle fehlerhaft sind.

Dazu die zu speichernden Einträge mit einem leerem Array initialisieren, denn
sonst nimmt die save_objects-Routine aus der Basis-Klasse alle Einträge.
Das war bisher zwar kein Problem, weil auch alle Positionszeilen vorher bei
fehlerhaften Auftragszeilen als fehlerhaft markiert und somit nicht gespeichert
wurden. Diese Markierung ist jetzt überflüssig (extra commit).

Banktransaction: Kontoauszug verbuchen überarbeitet

kein _as_number für amount verwenden

Fehlermeldung bei Transaktionen bearbeitet

Bei Sammelanweisung Zuweisung korrigiert

Die per SEPA-Export gefundenen Übereinstimmungen werden nun auch bewertet,
damit gleichartige Rechnungen genauer zugeordnet werden

kivi.detect_duplicate_ids_in_dom: Funktion zum Auffinden doppelter IDs im DOM

do/form_header.html: Element »department_id« nicht doppelt ausgeben

kivi.popup_dialog: neuer Callback »load« nach erfolgreichem AJAX-Laden des Inhalts

CsvImport Aufträge: Auftragsnummer nicht mit SL::TransNumber setzen, …

denn dies wird automatisch über SL::DB:Order beim Speichern gemacht.

CsvImport: Unbenutzte Variable/Statement entfernt.

Typo im Kommentar korrigiert.

SelfTest Kosmetik

Ein überflüssiges JOIN entfernt und etwas sinnvoller den
Code formatiert.

SelfTests check_summe_stornobuchungen optimiert

Die Summenprüfung der Stornobuchungen muss
auch über den Jahreswechsel hinaus Buchungen berücksichtigen.

Mehr Fokusoptionen in Belegen

focus: scroll to active element

js/common.js: hash focus funktionalität wiederhergestellt

focus: Zeilenfokus in die normale Behandlung verschoben

LinkedRecord: Batch mode

Performance: inject_results in linked_records

LinkedRecords: with_objects erlauben für shipped_qty

Verkaufs-/Einkaufsbelege: beim Laden der Lieferadresse $form->{shipto_id} nicht überschreiben

Die Semantik in der Datenbank sieht so aus, dass z.B. oe.shipto_id nur
dann gesetzt ist, wenn eine der Lieferadressen aus den Stammdaten
ausgewählt ist. Genauer:

• Stammdatenlieferadresse:
  oe.shipto_id == shipto.shipto_id,
  shipto.module = 'CT',
  shipto.trans_id = customer.id (oder vendor.id)

• Individuelle Lieferadresse:
  oe.shipto_id == NULL,
  shipto.module = 'OE',
  shipto.trans_id == oe.id

Der Inhalt von $form muss das genau so reflektieren, denn anhand dieses
Feldes $form->{shipto_id} wird dann in den Masken entschieden, was wie
angezeigt wird.

LXDebug: Flag SHOW_CALLER für Ausgabe Dateiname:Zeilennummer bei message() & dump()

kivi.popup_dialog(): übergebenen close-Handler auch ausführen

Die Funktion kivi.popup_dialog() installiert ihren eigenen
close-Handler, um den in dem Moment noch vorhandenen Dialog zu
entfernen (sofern der Dialog aus HTML-Code erzeugt oder von einer URL
nachgeladen wurde) bzw. zu schließen (sofern ein vorher bereits im DOM
existierendes Element angezeigt wurde). Dadurch werden vom Aufrufer
vergebene close-Handler überschrieben.

Da es mehrere Stellen im Programm gibt, an denen ein close-Handler
übergeben wird, wurde kivi.popup_dialog() nun so erweitert, dass es den
übergebenen Handler speichert und zuerst ausführt und erst anschließend
das Entfernen respektive Schließen durchführt.

Abfangen von leerer warehouse_id bevor Objekt gesucht wird

Will man kein Default Lager einstellen kommt es zu einem Rose Fehler.
Nun wird verfahren wie wenn Lager nicht gefunden wird.

installation_check.pl: LxOfficeConf safe_require gefixt

doc binaries

documentation.xml eigenen typo gefixt

create doc binaries aktualisiert

Installationsanweisungen für debian/ubuntu geändert

Seit ca. 2015 ist CGI.pm nicht mehr im Perl Core und das
Paket libcgi-perl seit Intrepid nicht mehr vorhanden:
S.a.   2015-04-30 09:26:14 UTC   Deleted   Intrepid

https://launchpad.net/ubuntu/+source/libcgi-perl/+publishinghistory

Zahlungsbedingungen obsolet setzen

Damit können alte und ungenutzte Zahlungsbedingungen für neue
Belege/CV/Artikel ausgeblendet werden.

Außerdem Umstellung von generic/multibox auf L.select_tag in den Masken

Lieferscheine - Abteilung per ALL_DEPARTMENTS und L.select_tag