OE: single-dbh disconnects

IR: single-dbh disconnects

GL: single-dbh disconnects

Form: single-dbh disconnects

Common: single-dbh disconnects

CT: single-dbh disconnects

CP: single-dbh disconnects

CA: single-dbh disconnects

AR: single-dbh disconnects

AP: single-dbh disconnects

CsvImport: Aufträge: Auf ungültige Artikel prüfen.

t/controllers/csvimport/parts.t: weg mit den überflüssigen init Aufrufen

mailmap update

UserPrefs: Transaktionen für alle

CsvImport/Part: uninitialized hash value warning behoben

WH: spellcheck

USTVA: Leerzeichen am Zeilenende entfernt

CSV-Import: single-dbh

CSV-Import Artikel: Einige Erweiterungen: Testfehler behoben

WH::transfer_assembly: single-dbh

DBUpgrade2: single-dbh

Betrifft das Surrogat-dbh-handling in db-upgrades.

Der alte Code versucht $::form->get_standard_dbh mit dem Upgrade Handle
zu überschreiben, damit Code der im Upgrade läuft das benutzen kann. Das
geht mit dem SL::DB->client handling nicht.

Angenehmer Nebeneffekt ist, dass man jetzt sofort bei Testläufen auf der
Konsole sieht wenn das passiert, weil das Handle nicht funktioniert und
damit abstürzt.

User: single-dbh: vor upgrade nicht mehr disconnecten

task_server: single-dbh: Form::disconnect entfernt

single-dbh: Fehler in Transaktionen immer weiterwerfen

Dispatcher: Form::disconnect nicht aufrufen solange kein client da ist

task_Server: single-dbh

DBUpgrade: temporäre dbh Behandlung

get_standard_dbh entfernt und auf SL::DB->client->dbh umgebogen

dbconnect_noauto entfernt

User: Kommentare entfernt

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.pl: 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"
und refs #201 (redmine)

Kreditoren- und Debitorenbuchung: Erneuern per JS bei Wechsel des Beleg-Projekts

behebt #207 (redmine)

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
     3. Preise von vorhandenen Artikeln aktualisieren
     4. Preise von vorhandenen Artikel aktualisieren / Nicht vorhandene überspringen
     5. Mit neuer Artikelnummer einfügen
     6. Eintrag überspringen

    Zusätzlich können nun Spalten "Lager","Lagerort" als Name oder ID eingelesen werden,
    sowie Übersetzungen z.B. als 'description_EN' oder 'description_IT'.
    Auch cvars können als 'cvars_<name>' importiert werden.
    Ebenfalls sind weitere Bemerkungen an den einzelnen Importzeilen eingebaut.

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
sind in Geierlein anders als früher in taxbird

Geierlein auf gleichem Webserver

Da LocalStorage nur innerhalb einer Domaine funktioniert,
Wurde nun innerhalb Kivitendo der crome/content Baum von Geierlein installiert.

Also
cd geierlein/chrome/content
mkdir {kivitendoDocumentRoot}/geierlein
tar cf - * | tar xf - -C {kivitendoDocumentRoot}/geierlein

Natürlich gibt es auch Möglichkeiten, dies per alias-Config im apache2 zu machen

Konfigurerierbar in kivitendo.conf

Falls geierlein_path gestetzt ist das Feature enabled
Pfadnahme zeigt zu geierleinverzeichnis innerhalb des kivitendo Verzeichnisses

UStVa Einstellungen in Datenbank

Kein finanzamt.ini mehr sondern fa_bufa_nr in Defaults,
über diese BundesFinanzamtnummer greift man auf Finanzamt Rose-Objekt zu.
Das ist nun auch modifizierbar.
Nutzung von Rose,
Sonderfälle in stnr pattern (BaWü)

Falls keine Daten in Defaults wird ein ggf. vorhandenes finanzamt.ini gelesen
und die Daten von dort übernommen

Die Steuernummer wird nun nicht mehr in der Mandantenkonfig
sondern in den UStVa Einstellungen editiert

Bisher Keine Umstellung auf IBAN, BIC (ist dies notwendig?)

Übergabe ohne kz83
Geierlein berechnet das Ergebnis selbst,
es werden nun nur Eingangsbeträge übergeben

UTF-8 Umwandlung finanzamt.ini Texte

Links zu Mandantenkonfig eingearbeitet

Implementiert RedMine Feature #168

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

special_chars: Weitere Symbole für LaTeX

special_chars: Omega für LaTex als mathematisches Symbol.

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
Dokumentation von SL/DB.pm beschrieben.

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 &
Drop verschoben, so darf daher kein Zugriff auf Funktionen von
»visible_item« stattfinden. Die Prüfung, ob aktuell überhaupt Abschnitte
zu sehen sind, muss daher vorher erfolgen.

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
  Parameter« benötigt man keine sub; das geht einfach mit einem ternären
  Operator.
• »Keine Parameter« drückt man nicht durch »undef« aus, sondern durch
  die leere Liste »()«. Wird »undef« als einziger Parameter übergeben,
  so wird in der Funktion de Facto das hier gemacht:

  my %hash = (undef);

  und das ergibt eine Warnung, dass eine Liste mit ungerader Anzahl von
  Elementen in ein Hash umgewandelt werden soll.

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

Admin: Task-Server-Benutzer in Mandantenliste anzeigen

TaskServer-Steuerung: Temp-Datei automatisch entfernen lassen

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

Bankauszug: Dialog »Rechnung zuweisen«: Fokus auf Rechnungsnummernfeld

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
dafür, dass der Test fehlschlägt.

Also nur die Felder explizit an SL::Helper::Csv::Error übergeben, die
uns wirklich interessieren, und nicht das von »error_diag«
zurückgebebene Array 1:1 durchreichen.

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,
indem »inventory_accno_id« explizit überschrieben werden.

Leider schlug der Test manchmal fehl, trotz des Workarounds: je nachdem,
ob beim Anlegen (»new_assembly«) zuerst der Parameter für »type« oder
der für »inventory_accno_id« ausgeführt wurde.  Die Reihenfolge der
Ausführung ist deswegen nicht deterministisch, weil in Perl das
Iterieren über Hashes gewollt nichtdeterministisch ist, um
Sicherheitsproblemen vorzubeugen.

Wir wandeln Funktionsparameter beim Anlegen von Rose-Objekten explizit
von Listen in Hashes (SL::DB::Object::assign_attributes); anschließend
wird über das Hash iteriert. Im konkreten Fall enthält das Hash sowohl
»type« als auch »inventory_accno_id«, und je Perls internem Seed
bzgl. der Hashiteration wurde mal die eine, mal die andere
Setter-Funktion zuerst ausgeführt.

Merge branch 'bank-transaction-improvements'

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
lange der volle offene Betrag bezahlt wie möglich, der Rest kommt auf
die folgende Rechnung.

Allerdings ist für das Programm nicht ersichtlich, welche Anteile
welcher Rechnungen des Kunden/Lieferanten tatsächlich damit beglichen
wurden.

Also muss verhindert werden, dass das passiert; eine Warnung genügt
nicht.

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
Einkaufsrechnungen sowie Gutschriften von Verkaufsrechnungen.

Alle anderen Zuordnungen werden mit einem Fehler zurückgewiesen.

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
Banktransaktion benutzt.

Treten dabei Warnungen oder Fehler auf, so werden diese nun in einer
Tabelle übersichtlich dargestellt. Die Tabelle enthält sowohl die Daten
der Banktransaktion, bei der das Problem auftrat (entfernte
Kontonummer/BIC, entferne KontobesitzerIn, Betrag, Transaktionsdatum),
als auch Links zu den mit dieser Transaktion verknüpften
Rechnungen. Damit wird die BearbeiterIn in die Lage versetzt, die Fehler
genauer zu analysieren.

drafts/load.html entfernen, nach Umstellung auf Controller nicht mehr benötigt