id ungleich Null setzen in tax_zones

In der Tabelle tax_zones gibt es sehr häufig noch Einträge mit id=0.
Dadurch kommt es bei der Anzeige, und auch beim Bearbeiten dieses
Eintrags zu fehlern. Dieser Commit vergibt verändert den Wert der
id, s.d. die id einen von 0 verschiedenen Wert hat.

Steuerzonen und Buchungsgruppen bearbeiten

Nachdem man Steuerzonen und Buchungsgruppen auf Datenbankebene
bearbeiten kann, ist jetzt auch eine Bearbeitung an der Ober-
fläche möglich.

Alter Code, der für die Anzeige/Bearbeitung/Erstellung von
Buchungsgruppen entwickelt wurde hat nicht mehr funktioniert
und wurde vollständig ersetzt.

Auf Datenbankebene Steuerzonen konfigurierbar gemacht

Damit können jetzt mehr als die 4 Standardsteuerzonen eingerichtet
werden. Die bisherigen Steuerzonen wurden aus der Tabelle
buchungsgruppen in die neue Tabelle taxzone_charts ausgelagert.

Konzeption siehe Ticket #2295

Es wurden noch keinerlei Veränderungen an der Kivitendo
Benutzeroberfläche durchgeführt, durch dieses Upgrade können die
Buchungsgruppen also nicht mehr konfiguriert werden, dies muß auf
Datenbankebene passieren.

Rose schon angepasst.

Conflicts:

SL/DB/MetaSetup/Buchungsgruppe.pm

Installationcheck: --silent switch um es in shell scripte einzubinden

Typo-Fix: Trennzeichen für CVar-Flags ist ':' nicht ';'.

round_amount: Fix für falsches Runden bestimmter Werte

Gewisse Werte wie z.B. 33,675 wurden bei 2 Stellen falsch gerundet,
nämlich auf 33,67 anstelle von 33,68. Bei anderen Werten hingegen
funktionierte es (beispielsweise 149,175 @ 2 → 149,18).

Grund war, dass durch das Addieren von 0.5 wieder Fließkommaberechnung
und damit die Ungenauigkeit der Präsentation der IEEE-Fließkommazahlen
ins Spiel kommt. Das anschließende int() schneidet dann die
Fließkommazahl falsch ab, ungefähr so:

- Initial: 33,675
- Linksshift um 2 Dezimalstellen, also * 100: 3367,5
- Dann + 0.5 und truncate, hier passierts: +0.5 =
  3367,499999999999999999999958 (auch wenn Perl das in der Ausgabe als
  3368 darstellen würde) oder so, davon int() ergibt nun mal 3367 vor
  anschließendem Rechtsshift um 2 Dezimalstellen

Lösung ist, bis auf das Links-/Rechtsshiften um die Dezimalstellen gar
keine Fließkommaberechnung zu verwenden. Eine Variante ist, eine Stelle
mehr zu shiften als man an Genauigkeit will, dann 5 zu addieren und
anschließend auf das nächst kleinere Vielfache von 10 zu
reduzieren (durch simples Abziehen vom Modulo 10).

Um die Logik leicht einfacher zu halten, wird das Vorzeichen anfangs
ermittelt und ab dann nur noch mit dem Absolutwert der Zahl
gerechnet. Das ursprüngliche Vorzeichen wird erst nach dem erneuten
Rechtsshift, also ganz am Schluss der Berechnung, wieder hergestellt.

GetModels: Funktion zum Setzen zusätzlicher URL-Parameter

L: Eindeutige ids für radio_tag wiederhergestellt.

Die waren in da2fecb453416b3731a896ab6d35e9992be3d9eb kaputtgegangen.

VERSION auf unstable gesetzt

Revert "POD-Dokumentation aktualisiert"
die methoden werden jetzt generisch generiert.
Doku ist korrekt und aktuell.  My bad.
This reverts commit b04128a335772d4cb9a30b0fde52413796981de0.

HTML-Tag richtig schliessen und Kosmetik

Standardmässiges Ein- und Auslagern von Dienstleistungen per Mandantenkonfiguration eingestellt

Mandantenkonfiguration erweitert:Dienstleistungen NICHT automatisch Auslagern
Standardmässig werden Dienstleistungen wie Waren und Erzeungnisse behandelt und
so auch beim Standard-Auslager-Verfahren. Entsprechend die Mandantenkonfiguration (defaults)
angepasst.

POD-Dokumentation aktualisiert
get_ Methoden werden nicht mehr einzeln für die Tabelle defaults gesetzt

HTML::Util: Tests, und Bugfixes für ein paar Randbedingungen

HTML::Util: Short circuit ohne $value. Behebt gleichzeitig eine Warnung.

Tests: neue Background Job Klasse auch erwarten

SL::Locale::String: @_ in neue Ref kopieren anstatt \@_

ansonsten verschwinden aus irgend einem Grund alle Elemente außer dem
ersten.

Hintergrundjob zum Löschen abgelaufener Sessions

CreatePDF-Helfer: Anpassung an API-Änderung SL::Template.*new

PreisTaxCalculator: Caching-Abfrage für »keine Positionen« gefixt

Kunden-/Lieferantenstammdaten: CVar-Input-Felder mit richtigem Namen versehen

Ging leider im Commit »CustomVariables: Verwendung mit RDBO als Writer
implementiert« kaputt, weil außerhalb dieses Templates »var_name«
bereits gesetzt war und somit alle CVar-Input-Felder denselben Namen
bekommen haben.

Handle zur LXDebug-Logdatei auch UTF-8 encoden

Wenn GREEK CAPITAL LETTER DELTA im Text, dann auch utf8 flaggen.

Wiederkehrende Rechnungen: Variablen für lange Monatsnamen gefixt

Ging im Commit »Wiederkehrende Rechnungen: Formatierung von
Datumsdruckvariablen über freie Formatstrings« kaputt. Danach waren die
Variablen <%current_month_long%> schlicht leer.

task_server: pro Job mit Sprache vom konfiguriert initialisieren

…und die Systemsprache nur als Fallback nutzen.

Hat in der Praxis vermutlich keinerlei Auswirkung, ist aber die
Implementation, wie sie sein sollte.

LXDebug::log_time: Differenz zum letzten Aufruf in ms mit ausgeben

Druckvorbereitung: Lieferschein-Model mit richtigem Primärattribut laden

Das ging im Commit »Einkauf/Verkauf: Lieferadressenfelder nie aus
Stammdaten vorbelegen« kaputt.

SL::DB::(Delivery)Order,(Purchase)Invoice: Aliase »add_items«

Die Relationships für die Positionen heißen in allen Klassen
unterschiedlich. Daher gibt es schon seit Längerem den Alias »items« in
allen Klassen.

Das Hinzufügen von Positionen hingegen erforderte bisher, dass man den
Namen der Relationship kennt, z.B. für Invoice:
$obj->add_invoiceitems. Um das zu Vereinfachen: neuer Alias »add_items«
in allen vier Klassen.

Aufträge bei Anzeige der Lieferungen in Kundenstammdaten richtig zuordnen.

Und richtigen Beleg (VK/EK) öffnen.

Behebt #2467.

Hinweis von Sven format_amount hat nichts in Template Verzweigung zu suchen.
Keine Ahnung warum mein Test am Freitag, dass nicht korrekt evaluiert hat.
Wahrscheinlich ist qty undef. Es reicht hier aber auch aus, einfach zu prüfen, ob
es sich um einen oder mehrere Einträge in STOCK_INFO handelt.

Hinweis von Sven format_amount hat nichts in Template Verzweigung zu suchen.
Keine Ahnung warum mein Test am Freitag, dass nicht korrekt evaluiert hat.
Wahrscheinlich ist qty undef. Es reicht hier auch aus, einfach zu prüfen, ob
es sich um einen oder mehrere Einträge in STOCK_INFO handelt

DB-Trigger, um sicher zu stellen, dass ein Lagerplatz auch zum Lager gehört.

Betrifft #2485.

Merge branch 'master' of github.com:kivitendo/kivitendo-erp

Einkaufslieferschein, Einlagern mit mehreren Lagern verbessert
Mehrfaches Aufrufen vom Fragezeichen Einlagern-Knopf, bzw. Erneuern der Posititonseinlagermaske
hat die Zuordnung zu Lager -> Lagerplatz fehlerhaft gesetzt, falls ein Standardlagerplatz in
den Stammdaten gesetzt war.
Details s.a. Ticket: 2485

UStVA: Position 47 in Summe 53 mit aufnehmen

2 Warnungen in den Tests beseitigen

Lieferadresse eingeben: Möglichkeit zum Kopieren aus Stammdaten

Im Dialog für die Eingabe einer individuellen Lieferanschrift gibt es
nun die Möglichkeit, die Felder aus den Stammdaten des
Kunden/Lieferanten vorzubelegen. Dabei werden sowohl die
Rechnungsadresse als auch alle Lieferadressen angeboten.

Ist beim Abschicken in mindestens einem Feld etwas eingetragen, so wird
eine eventuell vorher ausgewählte shipto_id abgewählt, ansonsten
beibehalten.

Was momentan noch nicht angeboten wird, ist die Firmenadresse aus den
Mandantendaten, weil hier das Adressfeld leider nicht nach Straße,
Postleitzahl, Ort aufgeteilt ist. Ansonsten wäre das für Einkaufsbelege
sinnvoll.

Verkauf: leeren Eintrag in Lieferadressendropdown klarer benennen

Angezeigt wird nun »Keine/individuelle Lieferadresse«, um es den
Benutzern klarer zu machen, dass die über den Button »Lieferadresse«
eingetragene Lieferadresse greift.

Einkauf/Verkauf: Lieferadressenfelder nie aus Stammdaten vorbelegen

Das neue Verhalten ist wie folgt:

- Weder die shipto_id (die Drop-Down-Box in den Belegmasken) noch die
  individuellen shipto*-Felder werden weder beim Neuanlegen eines
  Beleges noch bei Wechsel des Kunden aus den Datenbanken belegt.

- Beim Ausdruck werden die shipto*-Felder nicht mehr aus der
  Mandantenkonfiguration vorbelegt, wenn keine Lieferadresse gesetzt
  ist.

- Beim Ausdruck werden die shipto*-Felder mit den Daten aus den
  Kundenstammdaten belegt, wenn die shipto_id (die Drop-Down-Box in den
  Belegmasken) gesetzt ist.

Die ursprüngliche Intention war, möglichst viele Fälle abzudecken. Ganz
ursprünglich war es nämlich in den Druckvorlagen gar nicht möglich,
Kontrollstrukturen zu benutzen und damit die Ausgabe konditional zu
steuern. Es konnte also rein in den Druckvorlagen nicht unterschieden
werden zwischen »der Benutzer hat keine Lieferadresse eingegeben« und
»der Benutzer hat eine eingegeben oder ausgewählt«.

Daher wurde die ganze Logik immer im Perl-Code abgehandelt.

Das macht aber erhebliche Probleme für den Benutzer, für den es absolut
intransparent ist, wann welche Lieferadresse wie vorbelegt wird. Hinzu
kommt, dass in den Belegmasken nicht ersichtlich ist, dass eine
individuelle Lieferadresse eingetragen wurde.

Hinzu kommt, dass die Druckvorlagen inzwischen verschiedene Mechanismen
zur Verfügung haben, um Fallunterscheidungen zu treffen (z.B. die
kivitendo-Mechanismen $(if shipto…)$ oder die LaTeX-eigenen
\IfThenElse{\equal{$(shipto…)$}{…}}}…). Leider war es mit dem vorherigen
Code für die Druckvorlagen nicht mehr möglich festzustellen, ob der
Benutzer nun eine Lieferadresse eingegeben hat oder nicht.

Die neue Situation ist recht einfach:

Steht in »shiptoname« oder »shiptostreet« ein nicht leerer Wert, so ist
eine Lieferadresse vorhanden, ansonsten nicht.

Für die »nicht«-Fall kann dann jede Vorlage selber entscheiden, was zu
tun ist. Für Vorlagen im Verkaufsbereich sinnvollerweise gar keine
Lieferadresse ausgeben (oder einfach die Lieferadresse aus den
Kundenrechnungsdaten), für Vorlagen im Einkaufsbereich ebenfalls keine
Lieferadresse oder die Adresse aus der Mandantenkonfiguration.

Behebt #2345, #2400.

DateTime: (add|subtract)_businessdays Funktionen

Rundungsfehler bei periodischen Rechnungen mit Einzelpreisen ...

... mit einer Nachkommastelle und Rabatt behoben.

siehe auch commit a22b8118e0bd68acac8a2d7b02a2d4f9fd0eaff1

Zudem einen Test dazu angelegt. Allerdings weicht die Art, wie der
PriceTaxCalculator und die Beleg-Masken rechnen, von einander ab.
Da müsste nochmal geprüft werden, was die richtige Art ist
(erst Rabatt abziehen und dann runden,
oder gerundeten Rabatt abziehen und wieder runden).
Auch beim Errechnen des Amounts gibt es unterschiede
(netamount * (1+Steuersatz),
oder aufsummieren).

FollowUp(Access)-Rose-Models: sinnvollere Namen für Relationships

rose_auto_create_model.pl: Relationship-Namen anhand der Spaltennamen mappen

Bisher wurde das Umbenennen der generierten Relationships anhand des von
Rose vergebenen Namens der Relationship vorgenommen. Das ist
problematisch, weil diese wiederum von der Reihenfolge abhängen, in der
die Fremdschlüsseldefinitionen von der Datenbank zurückgeliefert werden.

Konkretes Beispiel: Tabelle »follow_up_access« mit den Spalten »who« und
»what«, die beide Fremdschlüssel auf employee sind.

Rose benennt die erzeugten Relationships nach dem Klassennamen
derjenigen Tabelle, auf die die Schlüssel verweisen. Tabelle »employee«
→ Klasse »SL::DB::Employee« → Default-Relationshipname »employee«.

Die erste von der Datenbank gemeldete Fremdschlüsseldefinition bekommt
nun den Namen »employee«. Die zweite sollte diesen ebenfalls bekommen,
aber da der schon vergeben ist, bekommt sie das Suffix »_obj«, ergo
»employee_obj«. Soweit, so gut. Ändert sich nun aber die Reihenfolge, so
werden die Namen genau umgekehrt herum vergeben.

Die Umbenennung anhand des Tripels <Domäne, Tabelle, Spaltennamen>
hingegen ist immer eindeutig.

Kunden-/Lieferantenstammdaten: Maps-Link mit Firmenadresse als Routing-Ausgang

Spaltentyp Text anstelle von varchar() in diversen Tabellen Teil 3

Kunden-/Lieferantenstammdaten: Beschränkungen für Feldlängen entfernt

…da die Datenbankspalten inzwischen alle vom Typ TEXT und nicht mehr
VARCHAR sind.

Spaltentyp Text anstelle von varchar() in diversen Tabellen Teil 2

Tabelle »shipto«

Lieferantenauftrag → Kundenauftrag: Verkaufspreis als Einkaufspreis übernehmen

Der Preis, den ich beim Lieferanten zahlen musste (alte Maske:
sellprice_N) ist dann im weiteren Verkaufsprozess der
Einkaufspreis (neue Maske: lastcost_N).

Generische Unterstützung für CTI: Click-to-dial

ReportGenerator: class für Links angeben können

SL::MoreCommon: benutztes Encode auch usen

USTVA: Analog zu b0f569c auch Position 46 umdrehen.

Neuer Bericht »Liquiditätsvorschau«

Verkaufsbelege: optional nur Projekte des Kunden anbieten

Auch dieses Feature kann über die Mandantenkonfiguration eingeschaltet
werden.

Instance-Conf am Anfang jedes Requests laden

Angebote: neue Felder Auftragswahrscheinlichkeit, Voraus. Abrechnungsdatum

Einkaufs-/Verkaufsprozesse: optionale Einschränkungen für gewisse Aktionen

Über die Mandantenkonfiguration kann verboten werden, dass gewisse
Aktionen in den Einkaufs- und Verkaufsprozesse durchgeführt
werden. Diese sind:

- Direkte umwandlung von Verkaufsangeboten und -aufträgen in
  Verkaufsrechnungen (nur über den Weg der Lieferscheine)
- Direktes Anlegen neuer Einkaufslieferscheine und -rechnungen (nur
  durch Umwandlung bestehender Belege)

Offene Transaktionen vor DB-Upgrades comitten

Hintergrund ist, dass Locks potenziell vorhanden sein können. Einfaches
Beispiel: $::instance_conf wird geladen (dadurch implizites
ACCESS-SHARE-Lock auf »defaults«), Upgrade will Schema von »defaults«
verändern, was dann hängt, weil dafür ACCESS-EXCLUSIVE benötigt wird –
das mit ACCESS-SHARE kollidiert.

CSV-Helfer-Test: In-Memory-Streams mit richtigem Encoding initlisieren

Einkauf/Verkauf: Vorgangsbezeichnung in Suche default an…

…sofern in der Mandantenkonfiguration das Erzwingen der Eingabe der
Vorgangsbezeichnung eingeschaltet ist.

Einkauf/Verkauf: Eingabe der Vorgangsbezeichnung optional erzwingen

Wird über eine Option in der Mandantenkonfiguration eingeschaltet.

Auftragsmaske: JavaScript-Checks für Speichern aktiver per. Rechnungen nach kivi.SalesPurchase verschoben

Lieferscheinmaske: JavaScript-Checks für Lagermenge nach kivi.SalesPurchase verschoben

DB-Schema-Anpassungen für »Spaltentyp Text anstelle von varchar() in diversen Tabellen«

Projektliste: kein Filterkriterium für Status == alle

…und nicht »nie benutzt«

Kundenauswahl bei Summen-/Saldenliste

Vor E-Mail-Versand prüfen, ob Empfänger+Betreff+Body vorhanden

Angebote/Aufträge drucken: auch das Order-Objekt ausliefern

Im Druck auch den Typ von Artikeln mit ausliefern

special_chars: U+00A0 NO-BREAK SPACE in latex erkennen und korrekt rendern.

Das Zeichen passiert oft, wenn Artikelbeschreibungen von Webseiten von
Lieferanten copy&pasted wird.  Webseiten padden ihre Daten gerne mit &nbsp; das
dann als U+00A0 gerendert wird, und landet so in der Datenbank.

SL::DB::Invoice: Verknüpfung auf die Buchungsobjekte

print_html_template auch mit scalarrefs als file aufrufbar gemacht

Ausserdem die Fehlermeldung bei nicht korrektem aufruf mit header angeben.

print_form optional ohne redirect

Form->error: confess anstelle von STDERR bei Fehlern von der Console aus benutzen

Locales: fehlende Übersetzungen aus CRM-Menü-Datei ergänzt

locales.pl: Task-Server-Fehlschlags-E-Mail-Template ignorieren

Spaltentyp Text anstelle von varchar() in diversen Tabellen

console: shortcuts für die häufigsten Belegtypen, nützlich beim debuggen

Term::ReadLine::Perl::Bind in Console nicht nutzen

Fixt zwar unter RXVT, macht aber unter anderen Emulatoren die Bindings
ziemlich kaputt.

Task-Server: vor jedem Job mehr Variablen re-initialisieren

Besonders wichtig: $::request, da sie zum Cachen genutzt wird und die
Garantie vom Cache ist, dass er nach jedem »Request« (beim Task-Server:
nach jedem Job) geleert wird.

SL::DB::Part,Helper::PriceTaxCalculator: agressiveres Caching

Damit wird für ziemlich jeden Objekttyp nur noch ein einziges Query
abgesetzt.

SL::DB::Object: generische Methoden zum Cachen von RDBO-Instanzen

SL::DB::Part: Cachen von Objekten gefixt Teil 2

get_active_taxkey hat vorher ganz einfach nicht gecachet.

SL::DB::Part: Cachen von Objekten gefixt

Über Steuerzonen erhaltene Informationen müssen pro Part-ID gecachet
werden, weil diese natürlich Artikel-spezifisch sind.

Locale: Warnungen

CVars: spelling: dependant -> dependent (wir haben uns irgendwann mal auf AE geeinigt)

PeriodicInvoices: korrekter Fallback, wenn kein email Template gesetzt ist.

PeriodicInvoices: Rechnugnen nicht einen Tag zu früh erstellen, das gibt massig Probleme

SL::DB::CustomVariable::{value,value_as_text}: Verwendung von $::request->cache()

Partpicker: Benutzerdefinierte Variablen mitschicken

SL::DB::Unit::base_factor: Verwendung von $::request->cache()

SL::DB::{Chart,Part}: Verwendung von $::request->cache()

SL::Request: Funktion zum Cachen von Objekten für Dauer des Requests

PriceTaxCalculator: chart->taxkey lookup vorberechnen

PartPicker: keine AJAJ-Requests nach Fat-Data ohne item.id schicken

LXDebug: Zeitstempel mit Millisekunden-Auflösung ausgeben

SL::Helper::CreatePDF: gewisse Variablen nicht mit Variablen überschreiben lassen

Tab Persistenz in allen masken ausser customer_vendor

War beim Umschreiben auf jquery-ui kaputtgegangen, weil der div.tabwidget eine
id braucht. CustomerVendor hatte das beim neuschreiben schon korrekt mit id
versehen.

SL::Controller::Base->send_file: neuer Parameter unlink zum Löschen nach Senden