typo

Project: Picker in reinit_widgets registriert

css: common vor main/menu einbinden

So kann man common styles in den Stylesheets überlagern.

Email-Journal standardmäßig nach Datum absteigend sortieren

Doku: CREATE EXTENSION statt CREATE LANGUAGE

Berechnung des nächsten Arbeitstages aus oe.pl und OE.pm in DateTime-Helper …

… verschoben, damit der Code nicht doppelt in OE.pm und oe.pl ist und auch von
woanders verwendet werden kann.

Beim jedem Speichern von Kunden/Lieferanten prüfen, ob der Name angegeben ist.

Vorher wurde das nur beim reinen Speichern geprüft, nicht beim Speichern und
Schließen oder Fortfahren mit einem Beleg.

Behebt #84 (redmine).

Locales neu gebaut

Syntaxfehler in Template behoben

Test zur Prüfung der Syntax aller Template::Toolkit-Templates

EmailJournal: Typo in Klassennamen gefixt

Lx-Office-Stilvorlage: Klassen listrow_error* ergänzt

SEPA: Zeilen ohne Bankinfo, die vorausgewählt wären, gesondert markieren

Bei Überweisungen sind diejenigen Zeilen vorausgewählt, bei denen in der
Rechnung der Lastschrifteinzug nicht aktiv ist, die ich also manuell
überweisen muss.

Bei Einzügen sind diejenigen Zeilen vorausgewählt, bei denen in der
Rechnung der Lastschrifteinzug aktiv ist, die ich also einziehen muss.

Sprich es werden die Zeilen mit (2) markiert und mit
Fehler-Hintergrundfarbe dargestellt, bei denen ich aktiv werden muss
aber es momentan aufgrund fehlender Bankinformationen nicht kann.

Einkaufsrechnungen: Neu buchen SQL Fehler behoben

Fehler trat auf, wenn Aufwandsmethode eingestellt war, und eine
Einkaufsrechnung, für die bereits eine Verkaufsrechnung existierte
geändert wurde. Dann wurde fälschlicherweise beim Umkehren der Rechnung
versucht die Warenbestandsbuchungen zu ändern.

Ohne Verkaufsrechnung existieren die allocated < 0 Positionen nicht.

Form->format_amount: Warnung über negativen repeat count vermeiden

Warnung erscheint, wenn bei Konstrukten wie »'str' x $anzahl« die
$anzahl negativ ist.

DATEV: Falsche Variable für Nachkommastellen gefixt

Fun fact: das war seit Refactoring in 2008-12-15 kaputt (Commit
40d52f50).

DATEV: Dateinamen richtig in EV-Datei schreiben

Ein foreach über ein Array weist der Iteratorvariablen die Inhalte zu,
nicht die Indizes ins Array. Also die Variable nicht noch mal ans Index
nutzen.

Fun fact: das war mindestens seit dem perltidy-Commit in 2005 kaputt.

DATEV: Benutzung undefinierter Werte vermeiden

Wenn der DATEV-Export vom Payment-Helfer mit aktivierten DATEV-Checks
heraus aufgerufen wird, so sind diverse Felder wie die KNE-Kopfdaten
nicht gesetzt (werden auch nicht benötigt).

Hash-Zuweisung mit ungerader Anzahl Elemente vermeiden

In einem Hash wird eine leere Liste im ternären Ausdruck nun mal im
Listenkontext ausgewertet… und dadurch ist bei so einem Konstrukt die
rechte Seite potenziell leer (sie enthält kein Argument):

my %h = ( whatever => test() ? 1 : () );

Benutzung undefinierter Werte vermeiden

Alte DB-Indizes entfernt/aufgeräumt

werden seit Einführung von Primary Keys in den entsprechenden Tabellen
nicht mehr benötigt.

Fehlende Fremdschlüssel für buchungsgruppen.inventory_accno_id ergänzt

Typos in Doku

Doku Build

Entfernte Symlinks: Dokuupgrade

do.pl: bei Nutzung von croak auch Carp einbinden

Lieferschein: Übernahme einiger Felder aus Auftrag gefixt

Betroffen waren das Auftragsdatum, Angebotsnummer, BearbeiterIn,
Steuerzone. Ist in 464f44a kaputt gegangen.

Einkaufs-/Verkaufsmasken: Zahlungsbedingungen bei Kunden-/Lieferantenwechsel resetten

Das ist in 5bc87ade kaputt gegangen.

DBUpgrade-Vorschau, Verkaufsmasken: schönere Bezeichnung für Details-Buttons

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

SL::DB::Project: displayable_name-Methode

   Überflüssigen mode aus DeliveryValueReport entfernt

    Details: Diese Variable wird nicht mehr verwendet und braucht demnach
    auch nicht initialisiert zu werden.

SelfTest um Testfall ac.amount == ar.netamount ergänzt

In Ergänzung zu #7915128c einen ähnlich sinnvollen Testfall für
Selftest ergänzt.
Im Datenbestand muss zwingend ar.netamount == acc_trans.amount für Typ
AR_amount identisch sein.
Ansonsten kommt / kam es in der Vergangenheit immer zu cent Differenz in
der Rechnungsübersicht und in der Rechnungseinzelansicht.

Abhängigkeiten für Auth-Upgrades verbessert

behebt #100

Dokumentationslink bei Vorlagen bearbeiten angepasst

ReportGenerator Doku verbessert

Typos / style in Doku

round_amount: Anzahl Stellen nicht zu hoch werden lassen

format_amount - negative 0 vermeiden

Ist eine Zahl z.B. -0.002, und wird auf 2 Stellen gerundet, so wird erst
das Minuszeichen gemerkt, dann gerundet (0.00), und schießlich wird das
Minus wieder hinzugefügt, dadurch bekommt man -0.00. Mit diesem Patch
wird das Minus-Zeichen entfernt wenn die gerundete Zahl genau 0 ergibt.

Übersetzung nachreichen für "Verkäufer importieren" beim CsvImport.

Siehe auch commit d13c70502afaee0fec445b82a0e1db1e37d0dfd2
"CsvImport für Kunden und Lieferanten: Verkäufer importieren können."

Typo in Übersetzung.

PriceSources: Fehlerbehandlung in PriceRules.

PriceSources: discount_from_source analog zu price_from_source implemeniert.

Wenn keine zur Rabatt-Quelle passende Klasse gefunden werden kann, wird auch
hier ein spezieller leerer Rabatt zurückgeliefert.

PriceSources: Namen der Quellen für Kunden- und Lieferantenrabatt berichtigt.

Die Klassen SL::PriceSource::Customer und Vendor verwenden als name
'customer_discount' bzw. 'vendor_discount' während SL::PriceSource::ALL
'customer' und 'vendor' benutzte.
Dadurch ließen sich z.B. diese Rabattquellen nicht deaktivieren.

Revert "POD zu update_attributes ergänzt"

This reverts commit 2271551b46c84fa256c7a2e0a46c04cddf31e89e.

Standardlagerplatz in Stammdaten beim Umlagern ändern

Scheinbar hat dies bisweilen immer nur bei gleichbleibenden Lagern funktioniert
(s.a. Ergänzung POD in Object.pm->update_attributes)
Ferner, sinnvollerweise das bestehende Part-Objekt verwenden und kein neues
erzeugen.

POD zu update_attributes ergänzt

mit der Bitte an Sven einmal gegenzulesen.

Symlinks entfernt

Die bevorzugte Art das Programm an den Apache anzubinden ist seit Jahren
über ein AliasMatch der direkt dispatcher.<typ> aufruft.

Das letzte mal haben wir das 2010 probiert, da gab es aber noch viele
Installationen deren Update nicht trivial gewesen wäre. Das sollte heute
nicht mehr so sein.

Doku für basic cgi muss noch angepasst werden.

t8 für sinnvolle Lokalisierung (locales.pl) in mtime_ischanged

Ferner korrekten optionaler Fehlertext bei entsprechender Option ausgeben.

Preisquellen: Fehlerbehandlung in MakeModel

CsvImport für Kunden und Lieferanten: Verkäufer importieren können.

Rückgabewert von system-Aufrufen prüfen und ggf. Fehler ausgeben.

Kosmetik

FlattenToForm: nicht zu rundende Zahlen mit mind. zwei Nachkommastellen darstellen.

3. Überarbeitung Prüfen beim Speichern, ob Dokument geändert ist

Verbesserte Regex für API-Aufruf. Gorash Code-QS. Abfangen von
SQL-Injections auf API-Ebene (100%)

neue Aggregatfunktion "FIRST", für Auswertungen

typo in pod

Massenkonvertierung Lieferschein - Rechnungen. Detailverbesserungen

Aufgrund der Code-QS von Bernd folgende Details bei der Massenkonvertierung
von Lieferscheinen nach Rechnungen eingebaut:
a) Das Model filtert jetzt nur nach offenen Verkaufslieferscheinen und
 nicht noch zusätzlich nach offenen, aber gelieferten Verkaufslieferscheinen.
b) Felder die mit CKEditor editiert wurden, werden zwar völlig richtig
 in das Invoice-Objekt editiert, ABER beim Massenausdruck über den Background-Job
 wird nicht automatisch der HTML-Code in LaTeX Code gewandelt. Daher erfolgte
 der Ausdruck als HTML-Code (<strong>Dies ist fett</strong>) im PDF.
 Entsprechend für die manuelle Auswahl als auch für die Background-Job geändert.
c) Der Filter im Bericht filtert nach transdate (Belegdatum) und nicht nach Lieferdatum.
 Entsprechend hier die richtige Beschriftung gesetzt.

Testfall convert_invoice.t um html-formatierte texte erweitert

Beim Konvertieren von Lieferschein nach Invoice auch explizit die
Formate von CKEditor prüfen. Ferner leichte weitere Anonymisierung der
Warendaten.

Übersetzungsbüro Richardson Rechtschreib-QS von Commits in den Standard ...

Auslagern über Standardlagerplatz: Artikel mit Chargennummern auslagern

Standardmässig funktioniert das Auslagern über Standardlagerplatz nicht, sobald eine Chargenummer
oder eine Mindesthaltbarkeit definiert ist, da dass Programm nicht entscheiden kann, welche Charge oder MHD
gemeint ist.
Ein Sonderfall ist aber, wenn gar keine Chargen/MHDs mehr eingelagert sind, dann kann die Ware
wieder automatisch, da eindeutig über den Standardlagerplatz ausgelagert werden.

Changelog Update

2. Überarbeitung     Prüfen beim Speichern, ob Dokument geändert wurde.

API-Funktion minimale Sicherheitschecks auf Parameter.
Häßliche If-Abfrage mosufiziert
POD ergänzt

Erste Überarbeitung "Prüfen beim Speichern, ob Dokument geändert wurde."

Prüfen beim Speichern, ob Dokument geändert wurde.

Allgemeine Funktion in Forms zum Lesen und Vergleichen der mtime

überall Form variable lastmtime da mtime mehrfach auftaucht

Conflicts:
templates/webpages/ar/form_header.html
templates/webpages/do/form_header.html

Zahlungsbedingungen: falsche Beschreibung der Bank-Variablen gefixt

Die Variablen wie <%bank%> wurden bezeichnet als »Ihre Bank«, sprich die
Bank der kivitendo-Nutzerin. Das ist schlicht falsch, weil es die Bank
des Kunden/Lieferanten ist, der beim gerade ausgedruckten Beleg
verwendet wird.

Außerdem ist es technisch auch gar nicht möglich, weil beliebig viele
Bankkonten in kivitendo angelegt werden können. Beim Ausdruck ist
allerdings kein Bankkonto auszuwählen, in keinem der Belege, also steht
eine Information wie »die Bank der kivitendo-Nutzerin« überhaupt nicht
zur Verfügung.

Zahlungsbedingungen: Variablen für BIC, IBAN, Mandats-ID, Mandatsunterschriftsdatum

Kosmetik: Einrückung

Lagerbewegungen/-bestände auch mit Mindesthaltbarkeitsdatum importieren können.

Auch hier kann es Probleme geben, wenn eine Installation zuerst das
Mindeshaltbarkeitsdatum aktiviert hatte und es dann deaktiviert wird (siehe auch
"System->Mandantenkonfiguration", Reiter "Lager" unter "Mindesthaltbarkeit
anzeigen".

Rechnungen: vor Benutzen von SL::DB::Invoice Modul auch laden.

Sonst gibt es eine Fehlermeldung, zumindest ohne fcgi.

Request: Dateiupload in form arrays[] gefixt

_recode_recursively hatte die Weiche für Dateiupload nur bei
Hashzugriff, und hat im Array-Kontext die Daten überschrieben.

Rechtecheck für BankImport Controller

Typos

Typo in xml doku

Changelog für Massenkonvertierung angepasst

Vergessene locals

Optionale Sortierfunktion in Lieferscheinen

Den Commit aus dem Kundenprojekt entsprechend für den
Standard überarbeitet und als optionale Aufruf für ein HTML-Template
zu Verfügung gestellt.Entsprechend kurz im POD beschrieben.

Implementiert FB0017 Position nach Art.Nummer in Lieferschein sortieren

Ab dem Status Verkaufs-Lieferschein können die Positionen nach Artikelnummern sortiert werden,
 sodass diese Sortierreihenfolge in die Rechnung übernommen wird.
Implementierung mit Natural Sort, d.h. Buchstaben und Zahlen können gemischt sein.

Aktion kann entweder über einen Knopf ausgelöst werden oder wird standardmässig bei jedem Speichern ausgeführt.
Aktuell heißt der Knopf: Sortieren und Speichern und erscheint sobald, der Lieferschein persisitente
Positionen in der DB hat.

Massendruck weitere Optionen (zweiter Druckbefehl) implementiert

transdate, copy_printer_id (id des zweiten Druckbefehls) als optionale
Parameter beim Aufruf der Konvertierung inkl. Ausdruck in der API
bereitgestellt.

Details: perldoc SL/Controller/MassInvoiceCreatePrint.pm

convert_to_invoice params an Invoice::new_from(%params)

DeliveryOrder.pm übergab etwaige Parameter beim Aufruf von new_from
an Invoice->post(%params). Das macht für den Anwendungsfall chart_id
in 42ea98b4 Sinn (Order.pm), allerdings benötigt die convert_to_invoice in
DeliveryOrder.pm die new_from(attributes => {'custom_attr' => value })
(s.a. 20118160).

OpenDocument-Template: Umwandlung von HTML-Feldern gefixt

OpenDocument hat gewisse Probleme mit Verschachtelung von gewissen
Konstrukten, z.B. kein <text:p> innerhalb von <text:p><text:span>. Die
HTML-Felder aber sind immer in ein <p>…</p> eingeschlossen.

Bisheriger Ansatz war, davon auszugehen, dass der aktuell offene Tag im
XML ein <text:p> ist. Dafür wurde im HTML schlicht das erste <p> und das
letzte </p> entfernt. Das funktioniert, wenn das HTML-Feld der einzige
Inhalt in z.B. einer Tabellenzelle ist, z.B. eine Zelle, in der nur
<%longdescription%> steht.

Es geht aber in die Hose, wenn der innerste offene Tag eben nicht
<text:p> ist, was sehr schnell passiert, z.B. wenn man Konstrukte wie
<%description%><%if longdescription%><%longdescription%><%end%>
nutzt.

Lösung ist, die aktuell offenen Tags in einem Stack zu verfolgen. Das
HTML-Formatieren kann dann alle offenen Tags bis zum letzten <text:p>
schließen und am Ende wieder öffnen.

Potenzieller Fix für Redmine #83.

OrderItem POD hinzugefügt

Hinweis auf entfernte Funktionen und Verbesserungsmöglichkeiten.

OrderItem tote Funktionen entfernt

Modus-Aufruf (DeliveryPlan) im Menü entfernt

Lieferplan Modus Lieferwertbericht entfernt

Der Modus ist jetzt unnötig, da der Lieferwertbericht einen
eigenen Controller hat.

Recht prinzipiell setzen

Nicht nur auf den Menüaufruf verlassen, sondern den gesamten
Controller vor Ausführung auf das Recht delivery_value_report prüfen.

Druck: Bei template-toolkit auch die globals setzen

SEPA: Nachrichten-ID-Filter in Zusammenfassung anzeigen

Kreditorenbuchungen: In Kopf anzeigen, in welchen SEPA-Exporten der Beleg enthalten ist

Einkaufsrechnungen: In Kopf anzeigen, in welchen SEPA-Exporten der Beleg enthalten ist

Debitorenbuchungen: In Kopf anzeigen, in welchen SEPA-Exporten der Beleg enthalten ist

Rechnungen: In Kopf anzeigen, in welchen SEPA-Exporten der Beleg enthalten ist

Debitoren-/Kreditorenbuchungen: Kreditlimit-Anzeige angepasst/gefixt

Der Titel »Kreditlimit« wird nun wie andere Zeilentitel in der linken
Spalte angezeigt, was den Blickfluss stabilisiert.

Weiterhin wurde in der Debitorenbuchungsmaske auch ein negativer
verbleibender Kredit in grün anstelle von rot angezeigt. Grund war ein
Vergleich mit einer falschen Variablen.

SEPA-Suche: nach Nachrichten-ID suchen können

SEPA-Suche: Nachrichten-IDs als Spalte anzeigen

SEPA: speichern, in welchen Nachrichten-IDs (MsgId) Exporte verwendet wurden

Manche Banken zeigen in ihren Auszügen nur die MsgId an, und es gibt
keine Möglichkeit, die darin enthaltenen einzelnen Überweisungen
angezeigt zu bekommen.

Diese MsgId muss allerdings bei jeder eingereichten Nachricht eindeutig
sein. Daher wird sie bei jedem Download zufällig erzeugt. Weiterhin kann
jeder Download eine beliebige Kombination von Exporten beinhalten.

Um eine einfacherer Nachverfolgbarkeit für solche Fälle zu ermöglichen,
wird nun bei jedem Download die dort verwendete MsgId bei allen
beteiligten Exporten gespeichert.

Rechnungssuche: optionale Spalte Mahnstufe implementiert

Debitorenbuchungen: aktuellen Mahnstatus anzeigen

Der Mahnstatus wird nun analog zu Verkaufsrechnungen angezeigt.

Verkaufsrechnungen: aktuellen Mahnstatus anzeigen; Texte leichter verständlich

Zusätzlich zur höchsten Stufe aktuell noch offenen Rechnungen und dem
gesamten offenen Betrag angemahnter Rechnungen des Kunden wird nun auch
explizit angezeigt, wenn die aktuelle Rechnung selber angemahnt
wurde. Diese Warnung wird nicht nur fett, sondern sogar rot dargestellt.

Mahnungen: Links zu Debitorenrechnungen gefixt

In der ersten Maske, in der die anzumahnenen Rechnungen ausgewählt
werden können, wurden Links immer nur auf is.pl gesetzt, auch für
Debitorenbuchungen.