Menus: ICON ohne .png

migrate script für menu

Menu: ICON property

Migrationsscript das Iconmappings enthält

Icons strukturunabhängig benannt.

weblogo entfernt

Bei Kontenabgleich Differenz der ausgewählten Zeilen richtig berechnen

Bei round_amount muß man immer die nötige Anzahl der Stellen als
Parameter mitgeben, ansonsten wird auf 0 Stellen gerundet.

behebt #66

Sammelaufträge - fehlerhaftes Verhalten

Standardmäßig wird der Sammelauftrag, zusammen mit den Positionen, per
RecordLinks verlinkt. Eine Ausnahme besteht wenn der Sammelauftrag aus
genau einem Auftrag entstanden ist, in dem Fall wird angenommen, daß
"als neu speichern" gemeint ist, und es gibt keine Verlinkung.

Dies entspricht dem eigentlich gewünschten Verhalten aus Commit d40a8e2 .

Belegpositionen nicht mehr mit ordnumber, transdate, cusordnumber speichern

stattdessen für das Drucktemplate der Rechnung ordnumber_oe, transdate_oe und
cusordnumber_oe aus Recordlinks auslesen, und auch entsprechende
Druckvariablen für Angebot und Lieferschein bereitstellen.

Diese Informationen sollen in Zukunft nur noch aus record_links bestimmt
werden, aus Kompatibilitätsgründen werden die alten Werte aber vorerst
noch in der DB belassen, aber eben nicht mehr bei neuen Aufträgen oder
Lieferscheinen gespeichert. Dadurch werden sie auch nicht mehr im Rahmen
des Workflows weitergereicht.

Ursprünglich wurden diese Datenbankfelder wahrscheinlich für
Sammelaufträge konzipiert, d.h. sie sollten nur befüllt werden, wenn man
einen neuen Auftrag aus mehreren bestehenden Aufträgen erstellt hat.
Das passt insofern, als daß diese Felder beim initialen Speichern eines
Auftrags nicht gefüllt wurden. Allerdings wurden die Felder schon
gefüllt, wenn man einen Auftrag zum zweiten Mal gespeichert hat, es war
also nicht allein auf das Zusammenfügen von Aufträgen beschränkt.
Außerdem wurden diese Felder im Rahmen des Workflows von Auftrag zu
Lieferschein oder Auftrag zu Rechnung dann in delivery_order_items oder
invoice gefüllt.

Bei "als neu speichern" eine Auftrags wurde auch noch die alte
Auftragsnummer in die neue Position übernommen.

Weiterhin wurde nicht berücksichtigt, daß man mittlerweile auch aus
mehreren Lieferscheinen eine Rechnung erstellen kann, die auch
unterschiedliche Aufträge haben können.

Für das Rückverfolgen der ursprünglichen Belege ist generell nun
record_links eine gute Möglichkeit, die Rückverfolgung von Positionen zu
ermöglichen. Das Verhalten, daß die Variablen nur dann gefüllt sind,
wenn sie aus Sammelaufträgen stammen, ist nun nicht mehr vorgesehen (und
hat vorher auch nicht richtig funktioniert).

In der Druckvorlage gibt es für Rechnungspositionen nun auch neue
Druckvariablen, nämlich die Angebotsnummer, Angebotsdatum,
Lieferscheinnummer und Lieferscheindatum für die Belege, aus denen die
Positionen im Rahmen des Workflows ursprünglich stammten. Siehe Doku.

Einheitliche displayable_name Methode für ar/ap/oe/do Objekte

Bestehend aus Dokumentenname und Dokumentennummer, z.B.
Rechnung 12
Gutschrift 20
Verkaufslieferschein 15b

Einheitliche Methode record_number für ar/ap/oe/do Objekte

entspricht jeweils invnumber/ordnumber/donumber

Methode displayable_type auch für Lieferscheine

record Aliase für Items

damit man von DeliveryOrderItem, OrderItem und InvoiceItem direkt auf
das entsprechende ar/ap/do/oe Objekte verweisen kann.

Beispiel in console:

die erste Position aus der ersten Rechnung:
 my $item = SL::DB::Manager::Invoice->get_first()->items->[0];
alle dorthin verknüpften Items (wenn aus Angebot, Auftrag und Lieferschein generiert)
 my $linkeditems =  $item->linked_records( direction => 'from', recursive => 1 );
in was für records befinden sich diese verknüpften Positionen:
 foreach ( @$linkeditems ) { print $_->record->type, "\n" };
 sales_quotation
 sales_order
 sales_delivery_order

Items in Belegen können durch LinkedRecords angesprochen werden

kivi.js: Support für jQueryUI-ToolTips wegen ToolTipster entfernt

Siehe Commit c0713b6. Damit nur ein ToolTip-System genutzt wird, und da
jQueryUI-ToolTip bisher nirgends in Templates verwendet wird, fliegt nun
der Support aus kivi.js. Grund ist auch, dass jQueryUI-ToolTip kein HTML
in ToolTips kann.

wzToolTip durch jQuery ToolTipster ersetzt

wzToolTip hat ein uraltes, ist in der Benutzung ausgesprochen
unkomfortabel und unflexibel und muss zwingend in jedem benutzenden
Template manuell _nach_ dem <body>-Tag eingebunden werden. Dadurch kann
es z.B. nicht im Layout mit ausgegeben werden.

SL::DB::CustomVariable->value für Typ Nummer auch wirklich Nummer zurückgeben

Da die Spalte number_value in der DB vom Typ numeric() ist, wird das von
Rose als String eingelesen. Das bedeutet, dass ->number_value z.B. der
Wert '0.00000' liefert, was im Booleschen Kontext trueish ist – nicht
das, was der Programmierer erwarten würde.

Statt dessen erwartet der Programmierer, dass !$zahl für den Wert 0 auch
wirklich zutrifft.

Daher sollte ->value für CVars vom Typ Nummer auch wirklich eine Zahl
zurückgeben, was durch ein * 1 erzwungen wird. Ausnahme: undef, was
weiterhin undef bleibt.

Projektliste in Detailsanzeige bei Angeboten, Aufträgen und VK-Rechnungen füllen (v2)

oe und is speichern die Projekt-Dropdowns, die im jeweiligen form_header
zusammengebaut werden, in $TMPL_VAR{ALL_PROJECTS}.

Das Projekt-Dropdown in io.pl für die Detailsanzeige (zweite
Positionszeile) greift hingegen für alle Belege auf
$form->{ALL_PROJECTS} zu, daher werden die Elemente nochmal gesondert in
$form gespeichert.

Revert "Projektliste in Detailsanzeige bei Angeboten und Aufträgen füllen"

This reverts commit fd71c09ec391d9ae8c9276597d0f0e574cc0b4c1.

Lieferscheinsuche: initialer Fokus auf Kunden-/Lieferantenname

Dadurch ist die Maske konsistent mit allen anderen Suchmasken im Verkauf
und Einkauf.

Focus: »erstes fokussierbares Element« nicht »Element mit initial_focus« überschreiben

Projektliste in Detailsanzeige bei Angeboten und Aufträgen füllen

Das Projekt-Dropdown in der Detailsanzeige bei den Positionen war leer.

Kontenabgleich: Gesamtsummen unten in richtigen Spalten anzeigen

Summen für "Betrag Bank" and "Betrag Buchungen" waren um eine Spalte
verschoben.

RB Druckvorlagen - Anpassungen für Brieffunktion

Straße hinzugefügt
Ihr Zeichen und Referenz optional anzeigen, nur wenn nicht leer.
Übersetzt.

GetModels: Optional die action setzen.

GetModels Filtered: gelaunderten Filter nicht mehr in Controller schreiben

Statt dessen kann der gelaunderte Filter über das Filtered-Plugin
abgefragt werden, z.B. im Controller:

$self->models->filtered->laundered

oder im Template:

SELF.models.filtered.laundered

GetModels Filtered: manuelles Setzen von launder_to entfernt

Dies ist ein Zwischenschritt. Das Ziel ist, den gelaunderten Filter nur
noch im Filtered-Plugin selber zu speichern und ihn von dort abzuholen.

Da launder_to bisher noch nirgends benutzt wird, sondern immer nur der
Default, kann es problemlos entfernt werden.

GetModels Filtered: gelaunderten Filter im Plugin zur Verfügung stellen

AttrDuration-Test: Anzahl Testcases gefixt

ParseFilter: Komplexe Methoden ermöglichen, die auch den Key ändern

ParseFilter: re-factor das Parsen von Filter- und Methodennamen

ParseFilter: re-factoring des Anwendens von Filtern und Methoden

GetModels: Disablen von Plugins auch für undef

Die Dokumentation sagt Folgendes zum Deaktivieren von Plugins:

> Configuration for plugins. If the option for any plugin is omitted,
> it defaults to enabled and is configured by default. Giving a
> falsish value as first argument will disable the plugin.

Das bisherige Verhalten war aber, dass bei Übergabe von
undef (z.B. »paginated => undef«) die Standardeinstellungen gegriffen
haben.

AttrDuration: Implementation für Spalten, die Dauer in Minuten speichern

DBConnect-Cache: undefined-Warnungen vermeiden

Treten z.B. beim Aufsetzen der Testdatenbank auf.

Bankerweiterung - bei Rechnung zuweisen Verkaufsrechnungen anzeigen

Das Feature war kaputt, offene Verkaufsrechnungen wurden nicht angezeigt.

Skonto - korrekte Variablenprüfung payment_term

für den Fall, daß kein payment_term bei den ausgewählten Rechnungen gesetzt ist.

Revert "Banktransactions - beim Speichern der Rechnung Zahlungsart prüfen"

This reverts commit 8809a4d1a4b4d3e5b3de31af94c0679dabfacc28.

Das funktioniert so nicht.

Typos in Doku

kivitendo/main.css: Encoding auf UTF-8

VK-Rechnungsmaske: "zurück"-Link hinzugefügt, um z.B. zur Berichtsliste zurückzukehren

Dieser Link ruft das gespeicherte callback auf, falls vorhanden.

R&B-Briefvorlage: Seitenumbruch ermöglichen und Kopfzeile ab zweiter Seite.

Body war in minipage verpackt. Damit fand kein Seitenumbruch innerhalb des
Body-Textes statt.

R&B-Briefvorlage: mit Logo und Fußzeile, wie bei den anderen Vorlagen.

Revert "RB-Druckvorlagensatz: abweichende Lieferadresse für Kunden- Lieferantenauftrag"

This reverts commit f167d00fa21c107f8f5c3b636d7031a739ea3ca5.

Aufträge sollten nicht an die Lieferadresse, sondern an die Stammdaten-Adresse
gehen. Zudem sind seit commit b6213d3539ccd179cd1f21b9afc54b8de8970774
"Einkauf/Verkauf: Lieferadressenfelder nie aus Stammdaten vorbelegen" die sipto-
Felder leer, wenn keine Lieferadresse angegeben ist.

ClientJS: Ziel-Argument bestimmter Funktionen vorne trimmen

jQuery kann aus HTML-Strings DOM-Objekte bauen:
$("<p>stuff</p>"). Beginnt der HTML-String mit Leerzeichen, so croakt
jQuery daran. Daher bei den betroffenen Funktionen, die immer auf einem
so gebauten DOM-Objekt hantieren, das Ziel-Argument um führende
Leerzeichen bereinigen.

ClientJS: cosmetics (alignment/indentation)

ClientJS: Kommentar korrigiert

Typos in Kommentaren

history - Vereinheitlichung von what_done für is/ir/ap/gl

Es wird nun immer "invoice" dort gespeichert.

Kreditorenbuchung: Stornos korrekt in history speichern

invnumber statt ordnumber

history für Zahlungen ar/ap/is/ir und Zahlungsein- und ausgang

Die Spalte "addition" in history_erp hat jetzt einheitlich den Eintrag
"PAYMENT POSTED", übersetzt als "Zahlung gebucht".

Bei Verkaufsrechnungen, Kreditor- und Debitorenbuchung wird jetzt zum
ersten Mal die Zahlung in history protokolliert, zumindest via "Zahlung
buchen".

history - bei parts what_done mit abspeichern

Verhalten von what_done an andere history-Einträge angepasst.

Bank - Reiter 'Set cleared entfernen' im Kontenabgleich

nicht mehr benötigter Entwicklungsstand.

Typos in Doku

Preisgruppenüberbleibsel entfernt

Aus der Preisgruppenumstellung

SL::DB::{Invoice,DeliveryOrder}->new_from: manuelles Filtern von Items via coderef

DB-Handle Caches: DATESTYLE korrekt setzen

Payment - falsche geschweifte Klammer bei Skonto Zahlungsvorschlag

Zahlungsbedingungen: Flag »Fälligkeitsdatum automatisch berechnen«

Bankbewegungsimport - detailliertere Fehlermeldungen

bei Fehlern oder nicht zuordbaren lokalen Bankdaten

Banktransactions - beim Speichern der Rechnung Zahlungsart prüfen

ob es einem gültigen payment type entspricht.

Außerdem korrekte Variablenprüfung, für den Fall, daß kein payment_term
gesetzt ist.

Zahlungsbedingungen bei Lieferscheinen; veraltete Spalte »terms« entfernt

Kosmetik: Ausrichtung

Typos in Doku

$&, $' und $` grösstenteils entfernt.

Dispatcher: Client DB Handles cachen.

Spart etwa ein Drittel an Requestoverhead, von 27ms auf 17ms.

Banktransactions Import - bessere Fehlermeldung wenn BLZ nicht stimmt

Beim Import wird sowohl Kontonummer oder IBAN als auch die BLZ geprüft.
Für den Fall, daß ein Konto anhand der importierten Kontonummer gefunden
wurde, die importierte BLZ aber nicht mit der dazu gespeicherten BLZ
übereinstimmt, gibt es nun eine aussagekräftigere Fehlermeldung.

Wahrscheinlich ist in diesem Fall die BLZ in den Bankkonteneinstellungen
falsch und muß dort aktualisiert werden.

Zumindest für den Fall, daß die IBAN übergeben wird, könnte man auf die
BLZ Prüfung wahrscheinlich auch verzichten.

Indirekte Objektnotation fixen

Tests: Indirekte Objektnotation für Instanzierung verbieten.

Rechte für Bankbewegungen in Bankerweiterung setzen

Als sinnvollen Default erhalten beim Upgrade Gruppen, die schon das
Recht für "Zahlungseingang, Zahlungsausgang, Kontenabgleich" (cash)
besitzen, auch alle Rechte für die Bankerweiterung, also die Arbeit mit
den importierten Bankbewegungen.

SL::DB::Helper::TransNumberGenerator: alle beteiligten Spalten aufführen

SL::DB::Helper::TransNumberGenerator: Belegnummern einmal direkt auslesen

Es werden alle vorhandenen Belegnummern benötigt. Diese wurden bisher so
ausgelesen, dass die Belege vom Rose-Manager via ->get_all komplett
geladen wurden und dann jeweils die Belegnummernspalte davon genommen
wurde. Das ist sehr langsam, vor allem da es potenziell sogar gleich ein
zweites Mal gemacht wurde.

Die Umstellung hier nutzt dafür ein direktes SQL-Query und umgeht Rose
dafür. Außerdem werden die nur einmal ausgelesen.

SL::DB::Helper::TransNumberGenerator: Tabellen und Zeilen locken

Die Tabelle, aus der die Liste der bereits benutzten Belegnummern
ausgelesen wird, muss exklusiv gelockt werden, um zu verhindern, dass
danach zwischen dem Auslesen und der Vergabe der neuen Belegnummer eine
andere DB-Verbindung dasselbe macht und dieselbe Nummer verwendet.

Dieses Locking muss daher vor dem Auslesen der Daten geschehen.

Weiterhin müssen die Zeilen in den Nummernkreistabellen (defaults
bzw. business) gelockt werden. Hier reicht aber das Locking der
entsprechenden Zeile.

Beide Locks müssen analog zu SL::TransNumber gehandhabt werden, um einen
potenziellen Deadlock zu vermeiden, sprich zuerst die Belegtabelle,
danach die Zeile in der Nummernkreistabelle.

SL::TransNumber: Belegtabelle vor Auslesen locken

Die Tabelle, aus der die Liste der bereits benutzten Belegnummern
ausgelesen wird, muss exklusiv gelockt werden, um zu verhindern, dass
danach zwischen dem Auslesen und der Vergabe der neuen Belegnummer eine
andere DB-Verbindung dasselbe macht und dieselbe Nummer verwendet.

Dieses Locking muss daher vor dem Auslesen der Daten geschehen.

Task-Server: vor Schlafen aufräumen

Dabei werden unter Anderem potenziell noch laufende Transaktionen
beendet und zumindest das Standard-DBH (nicht das von Rose)
geschlossen. Dadurch sollten alle Locks, die durch die Jobs
evtl. entstanden sind, wieder aufgelöst worden sein.

Task-Server: Initialisierung vereinheitlicht

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

io.pl::display_row: Felddefinitionen vereinheitlicht.

Ob eine Zelle eine volle Spalte bekommt oder in der zweiten Zeile landet
wird jetzt nur noch durch @header_sort und @row2_sort bestimmt.

HTML::Util: "nbsp" als HTML-Entity durch " " (space) ersetzen.

"nbsp" wird hier zu space, obwohl U+00A0 (non-breaking space) richtig wäre.
non-breaking space kann allerdings zu schwer zu findenden Fehlern zum Beispiel
beim CSV-Export führen, wenn ein Benutzer dieses nicht sichtbare Zeichen dann
per cut-and-paste irgendwo einfügt.

t/002goodperl: TODO für strict DBUpgrades entfernt.

Die müssen seit langem in packages liegen und strict sein.

ClientJS: Values nicht trimmen

Wenn numerische Werte (also IVs und NVs) per Regex getrimmt werden,
wird dabei das POK Flag gesetzt, das anzeigt, dass der Scalar auch ein
valider String ist.

JSON kann dann nicht mehr unterscheiden welcher Typ der Scalar ist, und
nimmt String. Das führt aber dazu, dass _alle_ Zahlen als Strings
encodiert werden.

Auch das wäre prinzipiell kein Problem, ausser dass Javascript keine
separaten Operatoren für Strings und Zahlen hat.

  json.val1 + json.val2

wird also immer als concat aufgefasst und nicht als Addition, und

  json.val ? true : false

ist immer true, weil "0" und "1" beide true in Javascript sind.

Tests: 2 Tests in db_helper/payment.t auf TODO gesetzt

ConventionManager: foreign_key Benennung

Der Standard Conventionmanager von Rose versucht irgendwelche
singular/plural Magic um aus dem key einen guten Namen zu kriegen.

Das klappt antürlich mal wieder nicht.

Wir haben aber einen besseren Mechanismus dafür,
SL::DB::Helper::Mappings.

Also werden jetzt die dafür benutzt.

Mappings: Direktzugriff auf package names Methode

ReconciliationLink: acc_trans relationship fixen

Locale: Bei parase_date_string undef sofort undef zurückgeben.

parse_amount: Bei amount undef sofort 0 zurückgeben.

RDBO Update: Tax

RDBO Update: Letter

Rechnungen: amount, netamount, paid NOT NULL DEFAULT 0

Tests: Hilfreiche aber false positive warning unterdrücken

BankTransactions: typo in deref

Drafts: keys auf hashref ist experimental

DBUpgrade2: Warnings in Tests unterdrücken

DBUpgrade: Warnings in Tests unterdrücken

DBUpgrade: Bei 3 alten Upgrades print Noise entfernt

Im Fall einer leeren Datenbank schreiben die jetzt keine
Statusnachrichten mehr nach STDOUT, das verwirrt sonst die Tests.

rose_auto_create_model: Optional auf frischer Datenbank arbeiten

Beim mergen hat man öfter das Problem, dass die Datenbank schon weit
mehr Upgrades enthält aber trotzdem aktuelle MetaSetups fixen muss.

Mit
  scripts/rose_auto_create_model.pl --test-client --all

werden jetzt alle Operationen auf einer frisch angelegten Testdatenbank
gemacht. Nachteil ist natürlich, dass das Anlegen gute 20s extra dauert.

Typos

Doku zu RB-Vorlagen erweitert

Typos in Dokumentation angepasst