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.

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.

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

Aufträge - Abteilung per ALL_DEPARTMENTS und L.select_tag

Verkaufsrechnung bearbeiten - ALL_DEPARTMENTS wie andere Belegemasken

Debitorenbuchung - Abteilung nutzt immer L.select_tag als Dropdown

Kreditorenbuchung - Abteilung nutzt immer L.select_tag als Dropdown

Dialogbuchen - Abteilung nutzt immer L.select_tag als Dropdown

Webdav-Datei-Aufruf direkt mit Backend-Funktion

Webdav Pfad und Dateiname muss nicht bei der Parameterübergabe
zusammengebaut werden. Besser direkt die Funktion Webdav->full_filedescriptor
hierfür nutzen

RB-Briefvorlage kivicompany vergessen zu setzen

Webdav: Funktion full_filedescriptor dokumentiert

Kreditorenbuchung um WebDAV-Funktion erweitert

changelog für WebDAV für Dialogbuchen geändert

Dialogbuchen um WebDAV-Funktion erweitert

gl html vergessenes schließendes table element

Alle neuen Features im Master-Branch in das Basis User-Menu

SL::DB::Helper::Manager: mehr Dokumentation

SL::DB::Helper::Manager: neue Funktion zum Pre-Cachen aller Einträge einer Tabelle

Tests: CTI-Test gefixt

Nach Übernahme von Commit cfb460aac »CTI: wenn kein dial_command
vorhanden callto: links rendern« aus der LINET-Installation war dies
aufgrund erweiterter Funktionalität nötig.

Part Controller - altes Template entfernt

wurde ersetzt durch templates/webpages/part/_multi_items_dialog.html

Alte Partsgroup Templates entfernt

Fehlende Übersetzungen ergänzt

_cvars.html: benutzte Plugins auch einbinden (hier: T8)

BUGFIX: Neuen Artikel anlegen verwendet alten Controller

Neuen Artikel anlegen springt in nicht mehr vorhandene Methode in ic.pl

Nun wird in neuen Part Controller gesprungen

Mehr als eine SelfTest-Modul prüfen

Der Aufruf der SelfTest funktionierte nur für genau eine
Datei - Mit einer leicht anderen Aufruf-Syntax werden auch weitere
Tests durchgeführt.

Revert "Mehr als eine SelfTest-Modul prüfen"

git add --patch falsch angewandt.

This reverts commit db4680753f5ec1de62bdc0557f850afd810444d1.

Mehr als eine SelfTest-Modul prüfen

Der Aufruf der SelfTest funktionierte nur für genau eine
Datei - Mit einer leicht anderen Aufruf-Syntax werden auch weitere
Tests durchgeführt.

BankTransaction: kein Push auf Skalarvariable

Neue Perl-Versionen werfen bei Pushes auf Skalare (auch wenn das
Arrayreferenzen sind) Fehler und nicht nur Warnungen:

push $array_ref, $value; # Experimental push on scalar is now forbidden

push @{ $array_ref }, $value; # OK

BankTransaction: Spaltennamen von CamelCase in snake_case umbenannt

kivitendo nutzt rein lower-case-Spaltennamen mit Unterstrichen als
Worttrenner, kein CamelCase. Betrifft Spalten transactioncode und
transactiontext in bank_transfers.

MT940-Import: Darstellungskorrekturen und fehlender Push eines Vorschlags

CTI: wenn kein dial_command vorhanden callto: links rendern

employee_company und salesman_company für flattentoform

Hintergrund: Bei automatisch generierten Rechnungen werden diese
beiden Druckvariablen nicht gefüllt. Mittlerweile sind diese aber
Mandantenspezifisch und nicht mehr an dem Benutzer gebunden.
Entsprechend den Wert aus instance_config geholt und für beide  Variablen

BUGFIX: Kreditorenbuchungen: Fehler beim Nutzen von Drafts(2)

Herausnahme der Anpassungen aus dem Draft Controller in die neuen Methoden show_draft der legacy scripts

BUGFIX: Benachrichtigung automatisch erstellter Rechnungen fehlerhaft

Dieses Problem wird im Redmine Ticket #218 angesprochen.

Durch Verwendung einer lokalen Variable behoben