Verkaufsbericht: typo beim refactoren gefixt

Historien-Popup: nur definierte Parameter an URL übergeben

Nicht definierte Werte werden von `encodeURIComponent` stringifiziert,
sodass z.B. `what_done=undefined&…` in der URL steht, wonach
Perl-seitig dann gefiltert wird und keine Ergebnisse mehr liefert.

Kontauszug verbuchen -> Buchung erstellen: Filter-Option verbessert

Eine Filter-Option schließt sich für ar/gl aus und die
Annahme, dass es auch Vorlagen zum Kreditor gibt, wenn wir eine
IBAN-Übereinstimmung haben ist auch zu kurz gegriffen,
geprüft wird jetzt IBAN + Vorlage

Kreditorenbuchung: Fehlermeldung, wenn kein Gegenkonto ausgewählt wurde

Artikelstammdaten: kein Redirect auf »neuen Artikel anlegen«

Kommt man aus der Artikelsuche oder legt man einen neuen Artikel an,
so wird beim Speichern nun kein Redirect mehr auf die Suchliste
bzw. die Maske zum Neuanlegen eines Artikels gemacht, sondern auf die
Bearbeiten-Maske des Artikels, der gerade bearbeitet bzw. angelegt
wurde. Man bleibt also sozusagen im Artikel, den man gerade in der
Hand hatte.

Existierende Callbacks werden weiterhin honoriert, sprich kommt man
aus anderen Masken, so kommt man beim Speichern auch in die andere
Maske zurück.

Admin: anständige Fehlermeldung in Druckerverwaltung, wenn kein Mandant

Es gibt eine Prüfung, ob es Mandanten gibt, und falls nicht, so wird
eine entsprechende Fehlermeldung ausgegeben. Die kann aber nicht
erreicht werden, wenn vorher versucht wird, mit `$self->client->id`
auf ein nicht definiertes Objekt zuzugreifen.

Bankimport: CSV- und MT940-Menüpunkte in Untermenü in »Zahlungsverkehr«

Menüpunkte zu ähnlicher Funktionalität sollten auch räumlich nahe
beieinander sein. In diesem Fall gewinnt das Thema »Kontoauszüge«
gegenüber dem Thema »CSV-Import«, da der Arbeitsfluss, der gerade
erledigt werden soll, komplett im Menü »Zahlungsverkehr« zu finden
ist, und nicht im Menü »System«.

benutzerdefinierte Variablen Ansprechpersonen beim Ausdruck als cp_cvar_…

CKEditor: mehrfach Initialisiserung bei Langtexten behoben

CKEditor: Fixes für texteditor in popup_dialog

- data field kanonisch auf data('ckeditorInstance') geändert
- jQuery style $.ckeditor hooks auf editor.on('loaded') geändert, der
  Adapter unterstützt kein inline
- mehrfachinitialisierung unterbunden
- selectall und focus Funktionalität in zwei Funktionen getrennt
- focus auf die API Methode .focus() umgestellt

debugs raus

git add --patch, danach nur git commit und nicht
git commit filename

behebt #300 negative Einkaufsrechnungen per Kontenimport verbucht

bank_transaction Testfall erweitert

Sowohl negative Kreditorenbelege als auch negative Einkaufsrechnungen testen.

FileManagement: Konsistenzprüfung zwischen Backend und Datenbank, hier Backend Filesystem

Das script 'scripts/sync_files_from_backend.pl' prüft, ob die Dateien im Backend noch vorhanden sind.
Dabei wird nach der aktuellsten Version gesucht, ist diese vorhanden ist ok,
ist diese nicht vorhanden werden die Versionen davor gesucht und ggf. die Version in der DB heruntergesetzt.
Ist keine Dateimehr vorhanden wird der Datenbankrecord gelöscht.

fixt #275

Filemanagement: Zentralisierte Prüfung ob bei Drucken im DMS gespeichert wird

Prüfung aller Mandanteneinstellung vor Speicherung im DMS,

fixt #286

CKEditor inline_resize: initial drag handler verbergen

CKEditor: Styleanpassungen für kivi Integration

- <textarea> Rahmen, hover, active Effekte auch für den Inline Editor
- padding in der Toolbar reduziert, so dass das bei normalem Spacing
  so groß ist wie das Element dadrüber. Ähnlich dem inline button
  Styling.

CKEditor: Common styles für inline Modus.

Margins und active Rahmen entfernt.

CKEditor: config Hack bei inline initialisierung

Aus irgendeinem Grund ignoriert der Inline Modus sämtliche
Dimensionsangaben in der Konfiguration, und zwar dokumentiert. Das sorgt
aber im textarea Modus dafür, dass das erzeugte Editor <div> völlig
falsch dimensioniert ist, und nachträglich angepasst werden muss. Die
CKEDITOR.editor.prototype.resize funktioniert dafür auch nicht.
Nachträglich machen sorgt für optische Unruhe.

Dieser Hack fügt im minified Code in der Datei core/creators/inline.js
im Konstruktor die folgenden Anweisungen ein:

  if (config.width)
    element.setStyle('width', CKEDITOR.tools.cssLength(config.width));
  if (config.height)
    element.setStyle('height', CKEDITOR.tools.cssLength(config.height));

kurz bevor das Element ins DOM eingefügt wird.

CKEditor: inline_resize plugin

ckeditor update auf 4.7.2

AreaInputTag: kleines Icon neben Input zum Umschalten auf Textarea

In Masken, in denen es keinen »Erneuern«-Button gab (neuer
Artikelcontroller), hatten die Benutzer*innen somit auch keine
Möglichkeit, aus dem normalen Input eine Textarea zu machen. Dies hier
ist nun eine manuelle Möglichkeit dafür.

Artikelstammdaten: »Beschreibung« als Textarea setzen, wenn mehrzeilig

ActionBar: `accesskey` mit Aufruftyp `call` gefixt

Wenn eine Action mit Aufruftyp `call` eine JavaScript-Funktion aufruft
und mit `accesskey = 'enter'` auf Return reagieren soll, so hat der
bisherige Setup-Code den Keyboard-Handler an das komplette
Document-Objekt gebunden. Das funktioniert aber zumindest in Chrome
nicht: der Handler wird nie aufgerufen.

Also an Body binden, und nicht an Document, was zumindest in Chrome
und Firefox gut funktioniert.

Fixt damit Return im neuen Artikel-Controller.

E-Mail: Versandfehler oben auch anzeigen

CustomVariables: Caching bei nicht existierenden CVars für ein Modul gefixt

Buchungsjournal: Abteilungsfilter bei Export berücksichtigen

vergessene hidden

Quicksuche: bei Stammdatensuche keine ungültigen anbieten

Betrifft:

• Artikelstammdaten in allen Varianten (Waren, Dienstleistungen,
  Erzeugnisse, Sortimente)
• Kunden & Lieferanten
• Ansprechpersonen (hier werden nur diejenigen angeboten, die zu
  gültigen Kunden/Lieferanten gehören)

Rechnungen: Nach »als geschlossen markiert« suchen

Übersetzungen: Aktualisierung

Übersetzungen: kein HTML-Code in Übersetzungen

DB-Upgrade: Dauer jedes Upgrade-Scripts timen

DB-Upgrade: kein »DROP CONSTRAINT …« nutzen

Die Namen von Constraints wurden früher mit $1, $2
etc. durchnummeriert. Inzwischen werden sie hingegen aus Tabellen- und
Spaltennamen benannt.

Daher ist ein »DROP CONSTRAINT…« mit Namen nicht portabel. Statt
dessen müssen leider alle vorhandenen Constraints gedropt und neu
angelegt werden.

Dokumentation: Rechtschreibung/Grammatik Artikelklassifizierung u. miniDMS

$self statt $_ eingebaut

CustomVariables: Beim Cachen von überlagerten Invalids können mehrere module auftauchen

CustomVariables: Nicht überlagerte Variablen können invalid sein

Muss beim autovivifien geprüft werden

CustomVariables: all_config cache korrekt behandeln für überladene Module

Customvariables: typo in config caching

CustomVariables: Requestlevel Caching für cvars_by_config

- Configs werden jetzt gecacht.
- is_valid wird en bloc vorberechnet und im Objekt gecacht.

SL::Dev::File und t/file/filesystem.t überarbeitet

* Einrückungen / Syntax überarbeitet
* Englisch leicht verbessert

SL/Dev/File.pm:

* die %common_params aus SL/Dev/File sind nur für den speziellen Test
  filesystem.t, daher in den Test verlagert.

* ebenfalls die get_all, get_all_count entfernt, SL/Dev/ soll nicht nur die
  Anzahl der zu tippenden Zeichen in Tests verringern, sondern allgemeine
  Funktionen für verschiedene Tests bereitstellen oder sinnvolle Defaults
  vorbelegen

SL/Controller/File.pm - Codeausrichtung und Stilverbesserungen

LaTeX: Tilde mittiger setzen lassen

Merge pull request #11 from freiphone/patch-3

Einkaufsrechnungen: Bearbeiter*in & Verkäufer*in mit aktueller Benutz…

LaTeX-Vorlagen: Ersetzungstabelle für en an die für de anpassen

LaTeX-Vorlagen: ~ escapen

shipped_qty: Testfall uas realem Bug

datev tests: Warnungen

csv_import tests: Falsche Parameterübergabe

BankTransaction: Warnungen

SL::DB::BankTransaction: Warnungen

SL::DATEV: Warnungen

Tests: SL::Dev Änderungen übernommen

SL::Dev: EXPORT rewrite und create/new split

- SL::Dev module exportieren nicht mehr ungefragt
- SL::Dev::ALL exportiert jetzt den übergebenen tag aus allen Modulen
- Funktionen nicht nicht speichern (Part/CustomerVendor) heißen jetzt
  "new_X" anstatt "create_X"

IS::get_customer: ersten Kunden zurückliefern, wenn keine Id vorgegeben ist.

Durch Umstellung auf den Customer/Vendor-Picker der Belegmasken konnte der
Picker unter bestimmten Umständen initial leer sein. Dadurch wurden dann bei
einem Wechsel von leer auf einen Kunden nicht alle Form-Variablen richtig
gefüllt (z.B. taxzone_id) und z.B. ein Lieferschein konnte nicht gespeichert
werden.

Das jetzige Verhalten gleicht dann dem von IR::get_vendor.

Lieferschein erzeugen: name-Methode nicht auf undef. VC-Obkjekt ausführen

Sonst gibt es einen Fehler, wenn es keine Kunden gibt oder für den Mandanten
noch keine Verkaufslieferscheine vorhanden sind.

Behebt #296 (redmine)

t/helper/shipped_qty.t: Vergessene Items-Sortierung.

t/helper/shipped_qty.t: Items des richtigen Objekts vor dem Neuladen vergessen.

SL/ClientJS.pm - Typos in POD

E-Mail-Dialog: bei Einkaufsaufträgen Standardvorbelegung

Siehe #263.

Fixt: Auslagern über Standardlagerplatz ohne Auslagern von Dienstleistungen

Die Prüfung, ob es sich um eine Dienstleistung handelt, wird
über part_type gemacht.
Andernfalls wird die auszulagernde Menge in diesem Fall bei
allen Positionen auf 0 gesetzt.
Zusätzlich die Prüfung auf gesetzte Menge über das Fragezeichen-Feld
entfernt (ist mit ActionBar reingekommen).

Typo in Übersetzung für Lieferantenauftragsnummer

Workflow Lieferschein -> Rechnung. Liefertermin als Rechnungslieferdatum setzen

Falls im Lieferschein einen Liefertermin gesetzt hat, ist dieser
eindeutig der sinnvollere Liefertermin in der Rechnung anstatt
das Lieferscheindatum.

Tests CsvImport: key für Zeichensatz im Profile ist charset, nicht encoding.

Spielte hier zwar keine Rolle, da das Encoding nicht getestet wird, war aber
dennoch falsch (encoding ist der Parameter für den Csv-Heler).

Kam in commit cdc396f5543250ac2cd9ad5b9b45bd2806c8e6c5
"Tests CsvImport: run-Methode der Worker verwenden" rein

BankTransaction Code cleaning

längst überfällige und häßliche semikolons entfernt

Für #277 toten Code entfernen

Optimierung SEPA-Erkennung #277

23 : 11 England gegen Pfalz, überzeugt mich

code cleaning:  debug statements vergessen zu entfernen

changelog für Feature Dialogbuchungsvorlagen mit Bankimport

Kontoauszug verbuchen -> Buchung erstellen -> Dialogbuchung nur abs()

Ob die Zahlung positiv oder negativ ist, bestimmt die Soll und Haben
in der Maske Dialogbuchung und liegt somit in der Hand des Anwenders,
 in welche Vorlage er buchen will.

Kontoauszug verbuchen -> Buchung erstellen erweitert

a) Filter erweitert, um nach Vorlagenname zu suchen
b) Filter korrekt in <form> gepackt, um submit und reset form
   wie gewohnt zu unterstützen
c) Buchungsmöglichkeit um gl_transactions (Dialogbuchungsvorlagen)
   erweitert, dass heißt in der Auswahlliste der Vorlagen werden
   jetzt Dialogbuchungsvorlagen angezeigt, falls ein Buchungskonto
   mit dem aktuellem Bankkonto übereinstimmt:
    (bank_accounts.chart_id == record_template_items.chart_id)
d) Filter erweitert, um nach Referenz (nur in gl_transactions)
   zu suchen
e) gl.pl erweitert, sodass die Metadaten der Vorlage geladen werden
   und mit den sinnvollen Vorgaben aus bank_transactions gefüllt werden

BankTransaction vendor_of_transaction wurde nie in filter_drafts benutzt

copy & waste Symptom:
weiteren toten Code vom initialen Commit entfernt

Kontoauszug verbuchen. Vorfilter nach vendor.iban mit bt.iban setzen

der initiale commit hat bt.iban mit vendor.account_number verglichen

Controller/BankTransaction.pm vendor_id entfernen

vendor_id wird im template/ nicht mehr benötigt.
S.a.: #aae306cda6274d
Kontoauszug verbuchen: keine Multibox verwenden

Tests CsvImport: run-Methode der Worker verwenden

Vorher wurden der Code der run-Methode aus den Basis-Klassen der Worker
(copy/paste) hier einzeln aufgerufen.

CsvImport: worker kann Csv-Helper auch mit scalar-ref als Datei aufrufen.

Der Csv-Helper kann für die Datendatei auch eine Referenz auf Daten erhalten.
Die Basis-Klasse der Worker des CsvImports unterstützt das jetzt auch. Damit
kann dann z.B. in den Tests $worker->run auch verwendet werden, wenn der Worker
in file eben eine Referenz auf die Csv-Daten enthält.

console - POD: --client ergänzt. Damit taucht das dann auch in der Hilfe auf.

Whitespace

Whitespace

SEPA: collective_transaction nach batch_transaction umbenannt

DATEV-Export nach Erfassungsdatum filtern

Hat man einen DATEV-Export schon für einen bestimmten Zeitraum
exportiert, und muß nachträglich noch ein paar Buchungen in dem Zeitraum
tätigen, kann man diese nachträglichen Buchungen nun gesondert
exportieren, indem man ein Datum nach dem letzten Exportdatum
angibt.

common/flash.html: no trailing whitespaces

BankTransaction: Überarbeitung von "Kontoauszug verbuchen" , SEPA-Export wieder integriert

Die Punktebewertung findet wieder ausschließlich in "get_agreement_with_bank_transactions" statt,
auch die SEPA-Sammelüberweisung. Diese bekommt dor extra Punkte, da ggf. für bestimmte Rechnungen negative Punkte entstehen.
Auch gibt es dort keine Remote Banknummer etc.

Die Testdatei t/bank/bank_transactions.t wurde um zwei Tests erweitert,
1. ein Test der das Verbuchen ohne SEPA-Export macht,
2. ein Test mit SEPA-Export

fixt #277

create_periodic_invoices.t - Umstellung auf SL/Dev und Partname

t/wh/transfer.t - Refactoring mit SL::Dev::Part::create_part

SL/Dev/Record.pm - create_invoice_item Beispiel aktualisiert

t/db_helper/price_tax_calculator.t mit SL::Dev refactored

t/controllers/financial_overview/sales_orders.t auf SL::Dev umgestellt

 sales_order_with_periodic_invoices_config.t mit SL::Dev refactored

t/db_helper/convert_invoice.t mit SL::Dev refactored

FileManagement: Anpassung an neue Schnittstelle im Ordercontroller

fix redmine #280

GetModels Paginated: Historische Filterduplizierung deaktivieren

Die war noch aus der Zeit vor GetModels, und war nötig damit Paginating
die gleichen Filter hatte wie das Hauptquery. Mittlerweile ist das aber
nicht mehr nötig, weil die GetModels-Architektur die einzelnen Plugins
eh chained.

Beschleunigt einige Queries, wo der Queryplaner in Bedingungen in Joins
die Duplizierung nicht automatisch entfernen kann, und dann in nested
loops nachprüfen muss.

E-Mail-Dialog: keine Vorbelegung bei Lieferantenauftrag/-lieferschein

Siehe #263.

Einkaufslieferscheine: Fix: Anzeige Ansprechperson wenn geliefert

E-Mail-Dialog: Vorbelegung vom Kunden/Lieferanten, wenn Ansprechperson keine E-Mail hat

Siehe #263.

templates zu a33ef0ec2 Standardlager und Lagerplatz im Warenbericht

sind 2 Ebenen tiefer im stash staging bereich gelandet ...
jetzt auch im standard ;-)

In Ergänzung zu #283. Filter-Optionen auf experimentell setzen

fixt: #283 Typfilter bei Lieferplan|Lieferwertbericht

Korrekte Filterübergabe an das Backend.
Zusätzlich weiteren Filterparameter 'Sortiment' eingebaut.
Ticket-Ziel: 100% done

S:C:H:ReportGenerator: Ergebnis von layout->get als Liste verwenden

Sonst ist der Export (PDF/CSV) von Controllern, die diesen Helper verwenden,
kaputt. Fehlermeldung war:
Can't locate object method "add" via package "1" (perhaps you forgot to load "1"?)
at SL/Controller/Helper/ReportGenerator.pm line 25.

ShippedQty: Abgefangene API Fehler auf Carp::croak umgestellt.

So werden die aus der Sicht des callers geschmissen.