projekt kivitendo

CsvImport/Base: single-dbh

Achtung: der hier schreibt ein for {} continue um in deutlich
komplexeres Offset Handling. Muss extra getestet werden.

Form: single-dbh (partiell)

standard_dbh->rollback fehlt noch

CustomVariableConfig: single-dbh

CsvImport: mehr single-dbh

CsvImport/Base: single-dbh

CsvImport: single-dbh

CsvReport: single-dbh

TransNumber: single-dbh

meerge: IO: single-dbh return

RecordLinks: single-dbh

IO: single-dbh

Common: single-dbh

ARAP: single-dbh

CVar: single-dbh

GenericTranslations: single-dbh

AM: single-dbh und disconnects

AccTransCorrections: single-dbh

FU: single-dbh

SEPA: single-dbh

Notes: single-dbh

DATEV: single-dbh

TODO: single-dbh

DN: single-dbh und disconnects

SL:: single-dbh und disconnects

CP: single-dbh

GL: single-dbh

AP: single-dbh

AR: single-dbh

IR: single-dbh

IS: single-dbh

DO: single-dbh

OE: single-dbh

IC: single-dbh für delete und update_prices

IC: Umstellung IC::save auf single dbh

Nur noch ein DB-Hanlde pro Request zum Client aufmachen

Auftrags-Controller: lastcost nur vom Artikel nehmen, wenn die Position neu ist

Ansonsten werden die EKs von schon in der DB gespeicherten Aufträgen immer mit
denen des Artikels überschrieben.

siehe auch commit 370d43e2028e8a450055048f1354da48163d3954
"Order Controller - lastcost von Positionen mit speichern"...

BUG_FIX Userpreferences

Falls keine Id vorhanden muss ein insert gemacht werden

Fehlende impliziten Übersetzungen für Zahlungsart

Wird in bank_transfer_edit gebraucht

CSV-Import Artikel: Einige Erweiterungen

CSV-Import von Artikel hat nun für existierende Artikel folgende Optionen:

1. Eigenschaften von existierenden Einträgen aktualisieren
     2. Eigenschaften von existierenden Artikeln aktualisieren / Nicht vorhandene überspringen...

Geierlein: alter Elsterexport überarbeitet

Generierung der Params in ODGeierlein

Neuer Controller für AJAX Request
Verallgemeinern (Teile aus ustva.pl nach SL/USTVA.pm

elster_format angepasst

Die Formate der Steuernummern für Baden-Württemberg und Rheinland-Pfalz...

2. Version POD zu create_assembly

Probiert insgesamt mehr 'auf den Punkt' und trotzdem
user-verständlich zu beschreiben.

1. Version POD zu create_assembly

create_assembly wird transfer_assembly ersetzen.
Dies ist die erste Version alle aktuellen optionalen Verfahren
für 'Erzeugnis fertigen' zu ergänzen.
Aktuell offen ist die Liste der Parameter, sowie das
Verhalten bzgl. best_before

new_lastmtime mit $provided_dbh aufrufen

fix für #204

new_lastmtime um optionalen dbh erweitert

Kosmetik, bessere Beschreibung für neuen SelfTests (#24)

Copy und Paste-Fehler bei Beschreibung im Positiv-Testfall
übersehen, sowie Einrückungen verbessert.

Payment-Helfer: Rechnen mit undefinierten Werten vermeiden

Weiteren Transaction-Selftest

Falls ein datepaid im Nebenbuch gesetzt ist, so darf paid nicht
auf 0 sein, wenn es keine Zahlungen in der acc_trans hierzu gibt.

Merge branch 'f-use-with_transaction-instead-of-do_transaction'

»with_transaction« anstelle von »do_transaction« verwenden

Es sollte so selten wie möglich »do_transaction« verwndet werden, damit
man sich immer angewöhnt, »with_transaction« zu nutzen.

Hintergründe und Unterschiede zwischen den beiden Funktionen sind in der...

Auftragscontroller: nach Löschen Redirect zum Anlegen, nicht zum Bearbeiten

Pflichtenheft: Zugriff auf nicht vorhandenes »visible_item« verhindern

Sind im Baum gerade Textblöcke ausgewählt, so liefert die Funktion
»visible_item« undef zurück, weil aktuell kein Item (= Abschnitt oder
Funktionsblock) sichtbar ist.

Wird in so einem Moment ein Abschnitt oder Funktionsblock per Drag &...

Rechnungsmassenerstellen: keine DB-Transaktion um convert_to_invoice()

convert_to_invoice() macht selber intern bereits eine Transaktion auf;
daher ist die außen unnötig.

Weiterhin waren die Parameter merkwürdig übergeben:

• Für eine On-The-Fly-Unterscheidung für »diese Parameter« vs. »keine...

SL::DB::with_transaction: bessere Doku zu den zwei Hauptunterschieden

TaskServer-Steuerung: Temp-Datei automatisch entfernen lassen

Bankauszug: nur Warnung wegen Überzahlung bei tatsächlicher Überzahlung

CSV-Helfer: Error-Felder versionsunabhängig setzen

Unterschiedliche Versionen von Text::CSV_XS geben beim Aufruf von
»error_diag« unterschiedliche Felder zurück: neuere geben mehr
zurück.

Es gibt aber einen Testcase, der das Fehler-Array auf exakte
Übereinstimmung prüft. Da sorgt also jegliches neu hinzugekommenes Feld...

SL::DB::Part: Setter für type=assembly bzgl. inventory_accno_id fixen

inventory_accno_id muss für Typ »assembly« immer auf undef stehen. Das
testet auch der Testcase. »type« hat allerdings das Falsche gemacht,
wofür im Testcase versucht wurde, ein Workaround zu implementieren,...

Bankeinzug: bei Überzahlung eine Warnung ausgeben

Eine Überzahlung ist oftmals OK oder unvermeidbar, sollte aber von der
BenutzerIn begutachtet werden.

Bankauszug: Unterzahlung mehrerer Rechnungen verhindern

Wenn mehrere Rechnungen ausgewählt werden, so verteilt der Algorithmus
schlicht den Betrag der Überweisungen auf die Rechnungen in der
Reihenfolge, in der die Rechnungen ausgewählt wurden. Dabei wird so...

Bankauszug: Transaktionsrichtung mit Belegrichtung abgleichen

Erhält man eine Zahlung, so darf man diese nur mit Belegen verbuchen
können, die Zahlungen in Empfangsrichtung bedingen: Verkaufsrechnungen
und Gutschriften im Einkauf.

Analog gilt das auch für ausgehende Zahlungen. Hier passen nur...

Bankauszug verbuchen: etwas Dokumentation ergänzen

Bankauszug verbuchen: Warnungen/Fehler anzeigen; pro Zeile eine DB-Transaktion

Das Verbuchen von Bankauszügen wird nun in Datenbanktransaktionen
gekapselt. Damit die BenutzerIn bei einem Fehler nicht alles erneut
einstellen muss, wird eine Datenbanktransaktion pro...

Payment-Helfer: with_transaction() anstelle von do_transaction() nutzen

»do_transaction()« kommt von Rose::DB selber. Es schert sich nicht
darum, ob bereits eine Transaktion läuft, sondern macht einfach eine mit
»BEGIN« auf. Am Ende der an »do_transaction()« übergebenen Sub committet...

Payment-Helfer: Kosmetik (Einrückung, keine ; am Ende von if)

CsvImport: deutlich anzeigen, ob Vorschau oder echter Import

CsvImport: Kosmetik (Anordnung)

Letter: is_sales nicht in init_letter clobbern

Briefe: Sprache im Druck

Preisquellen: weitere TODO doku

Order Controller - lastcost von Positionen mit speichern

ansonsten ist die Margenberechnung im Verkaufsbericht falsch (lastcost
ist immer 0). Siehe Ticket #201

Doku: PriceSource Verhalten für Belegumwandlungen

Wie beschrieben in redmine#199:

Wenn ein Kunde einen Kundenrabatt hat, und man aus einem Auftrag einen
Lieferantenauftrag macht, wird die active_discount_source der Artikel
(z.B. "customer_discount/1162") mit auf die Einkaufsseite des Workflows...

CustomerVendor: agnostische number Methode für beide

Briefe: Lieferantenbriefe

BUG-FIX: LOCK TABLE parts in CVS-Import Artikel

Problematik, dass alles andere mit Rose gemacht wird,
die Teilenummern holen, jedoch nicht.

Siehe auch Redmine #190

BUG-FIX: Rose kann nur sort_by

order_by wird stillschweigend ignoriert

Weitere Fixes?

Tabelle parts: Fremdschlüssel für Konten - bessere Namen

Tabelle parts: Fremdschlüssel für Konten

Bug #199 - Kundenrabatt in Einkaufsrechnungspositionen

Behebt Fehler #199 (Redmine)

Erzeugnisfremdschlüssel auf Artikel 2

auch die Spalte id verlinken.

Die Tabelle assembly ist etwas Banane und nicht gerade Rose-freundlich.

• die Spalte id entspricht der id des eigentlichen Erzeugnisses, alle
  Artikel eines Erzeugnisses haben den gleichen Eintrag in id....

Rose Dateien für Tabelle user_preferences

Erzeugnisfremdschlüssel auf Artikel

Tabelle assemblies mit parts verknüpfen

nicht benutzte Spalte parts.alternate entfernen

Auftrag in Lieferschein wandeln: Rose-DB-Handle für Item-Verknüpfungen verwenden

Sonst wirkt die transaction nicht und es kann sein, dass record_links
angelegt werden, auch wenn die Transaktion abgebrochen wird.

Kunde: Datenbanktabelle "klass" nach "pricegroup_id" migriert

und einen Fremdschlüssel gesetzt.

S:C:BankTransaction: Kosmetik für bessere Lesbarkeit (Einrückung, Anordnung…)

POD für Methode closed in SL/DB/Invoice

Die derzeitige Umsetzung ist mMn falsch bei Überzahlungen, die sollten
auch offen sein.

Kontoauszug verbuchen - Beschriftung verbessert.

Übernommene Vorschläge zeigen die Rechnungsnummer und den offenen Betrag
der Rechnung an, die Beschriftung lautet nun auch "offener Betrag".

SL/DB/BankTransaction.pm - carp statt die

Controller BankTransaction - Einrückungen und Kommentare

Refactoring und Einrückungen in BankTransaction.pm

TopQuickSearch: Und Verlinkung auch wieder hergestellt

Sicherheit: ReDoS in trim() umgehen.

trim hat bisher whitespace mit dem regex /^\p{WSpace}+|\p{WSpace}+$/
getrimmt. Der ist aber anfällig gegen große Mengen Whitespace in der
Mitte, weil dann das Backtracking in O(n²) läuft:

$_ = "--".(" "x200000)."and more"; s/^\p{WSpace}+|\p{WSpace}+$//;...

Fertigen ohne Prüfung nach Bestand(2)

Das Prüfen ob Lager das "Standard-Lager für Auslagern ohne Prüfung auf Bestand"
ist genügt nicht, es können Bauteile dieses Lager auch als Standardlager haben

Fertigen ohne Prüfung nach Bestand

Wird bei einem Bestandteil das "Standard-Lager für Auslagern ohne Prüfung auf Bestand"
verwendet, wird ohne Prüfung ausgelagert.

Dies geschieht nur beim gewählten Feature "Zum Fertigen Standardlager des Bestandteils verwenden"

Funktion "Erzeugnis fertigen" sucht Bestandteile im falschen Lager.(4)

Falls das Bestandteil bei gesetztem "transfer_default_warehouse_for_assembly"
kein Standardlager besitzt und es kein "Standard-Lager für Auslagern ohne Prüfung auf Bestand"
in der Mandantenkonfig gesetzt ist,...

TopQuickSearch: typisierte Suche gefixt

TopQuickSearch: Alle Artikelsuchen haben nur Erzeugnisse gefunden

...hat keiner gemerkt...

Drafts: load kann jetzt zusätzliche Argumente aus form übernehmen

Kontoauszug verbuchen: Fehlermeldung und falscher Betrag

Werden mehrere Rechnungen mit einer Banküberweisung bezahlt,
bekommt letzte Rechnung immer den Rest des Betrages,
auch wenn sie überbucht wird.

fix Redmine #192

Eine Rückbuchung von ganz bezahlten Rechnungen kann über diesen Weg nicht gemacht werden

Payment-Helper - create_bank_transaction bekommt "purpose" param

und Beispiel für das Erstellen einer Zahlung für mehrere Rechnung mit
Skonto in POD.