projekt kivitendo

Kosmetik (Einrückungen) zum Commit von gerade

CreatePeriodicInvoices: Transaktionssicher

Aufträge verarbeiten, Rechnungen erzeugen, Drucken und E-Mail-Versand
in einer Transaktion

SelfTests: Transactions um Prüfung auf laxe Buchungskonfiguration erweitert

SelfTest: Fehlermeldung beim E-Mail-Versand an den Task-Server hochreichen

SelfTest false positive vermeiden

Der #b4cc349b ist leider auch Schrott.
Im Array hab ich nur failed mit 0 oder 1 als zuverlässige Status-Meldung

SelfTest false positive vermeiden

$self->all_passed enthält nicht mehr den Zustand, ob alle
Tests erfolgreich waren. Als Workaround auf zwei negativ
Status-Meldungen prüfen, die bei Problemen gesetzt sind.

SelfTests: False Positive bei Buchungen auf ic_cogs als Sammelkonto vermeiden

Verknüpfte Belege um die Verknüpfung 'E-Mail Journal' erweitert.

Falls das E-Mail-Journal aktiviert ist wird zusätzlich zu der
archivierten E-Mail auch die Verknüpfung vom Beleg zu der E-Mail mitgespeichert
und ist im Beleg zusätzlich direkt anwählbar.
Etwas mehr Details im POD vom Mailer.pm, die Implementierung orientiert...

SelfTests/Transactions.pm erweitert

Da beim Löschen von Zahlungen mittlerweile auch kaskadieren die
reconciliation_links gelöscht werden, können verwaiste abgeglichen
Bankbewegungen entstehen.
Entsprechend Testfall ergänzt.

SelfTests: check_paid_ap ID in E-Mail-Bericht übergeben -> einfachere Analyse

fehlendes BCC in CreatePeriodicInvoices.pm(_email_invoice) hinzugefügt

POD-Fehler fixen

Wiederkehrende Rechnungen: Einmalige Rechnungen auch nur einmalig ausführen

Die kleinste Periode die calculate_dates verwendet ist monatlich.
Falls eine einmalige Rechnungen ein Startdatum älter als einen
Monat besitzt, werden mehrere Rechnungen erzeugt....

Wiederkehrende Aufträge mit einmaliger Periode verbessert

Die Deaktivierung erst nach der Generierung der Rechnung veranlassen.
Andernfalls werden diese automatisch bei anderen Rechnungsläufen deaktiviert.

WebshopApi: BackgroundJob ShopPartMassUpload.pm

WebshopApi: BackgroundJob ShopOrderMassTransfer

Task-Server & FailedBackgroundJobsReport: Encoding des E-Mail-Bodys gefixt

SL::Mailer erwartet die Einträge in Perls internem Encoding. Daher ist
jegliche Verwendung von Encode::decode(…) falsch.

Zusätzlich muss aber dem Template-Toolkit mitegeteilt werden, dass die...

PDFs werden nicht mehr bei wiederkehrenden Rechnung per E-Mail erzeugt/angehangen

fix redmine #238

CsvImport: Controller erzeugt temporäres Profil, nicht der Hintergrund-Job.

BackgroundJob::CsvImport: unbenutztes Modul SessionFile::Random nicht laden

Dateimanagement: Massendruck

Der Massendruck per BackgroundJob nutzt nun das FileManagement

Die erzeugten Dateien werden ggf. mit PDF Anhängen erweitert
und entsprechend abgespeichert.

Per Flag kann dem massendruck mitgeteilt werden,
dass eine leere Seite ggf hinzugefügt wird...

Wiederkehrende Rechnungen, um neue Periode 'einmalig' erweitert.

Wiederkehrenden Rechnungen die aus einem Auftrag automatisch generiert
werden können jetzt die Periode 'einmalig' (engl. 'one time') mit dem
Zeitwert 0 haben.
Falls diese Periode angewählt wird, wird die Konfiguration des Auftrags...

Hintergrundjob zum Schließen bei geschlossenen Aufträgen verlinkten Projekten

Der Job ist per Default deaktiviert. Wenn eine solche Funktionalität für
eine Installation benötigt wird, so muss der Admin manuell einen
Hintergrundjob dafür anlegen.

SelfTests: check_overpayments verbessert

i)
chart_link muss nicht mehr über JOIN chart c c.link mit der
acc_trans vereint werden
ii)
Der Test hat keine sinnvollen Daten an den Benutzer zurück-
gegeben. Jetzt wird die Kunden- und die Rechnungsnummer gemeldet...

false positive in SelfTests gefiltert (Periode)

Hintergrund:
Um nur die Daten-Analyse auf das aktuelle Geschäftsjahr (Geschäftsjahr gleich
Kalender-Jahr) zu machen, wird für die meisten Tests mit transdate == current_year
gefiltert.
Problem: Periodenübergreifende Buchungen (Zahlungsein- und -ausgänge) werden...

SelfTest Kosmetik

Ein überflüssiges JOIN entfernt und etwas sinnvoller den
Code formatiert.

SelfTests check_summe_stornobuchungen optimiert

Die Summenprüfung der Stornobuchungen muss
auch über den Jahreswechsel hinaus Buchungen berücksichtigen.

Mehr als eine SelfTest-Modul prüfen

Der Aufruf der SelfTest funktionierte nur für genau eine
Datei - Mit einer leicht anderen Aufruf-Syntax werden auch weitere
Tests durchgeführt.

Revert "Mehr als eine SelfTest-Modul prüfen"

git add --patch falsch angewandt.

This reverts commit db4680753f5ec1de62bdc0557f850afd810444d1.

Mehr als eine SelfTest-Modul prüfen

Der Aufruf der SelfTest funktionierte nur für genau eine
Datei - Mit einer leicht anderen Aufruf-Syntax werden auch weitere
Tests durchgeführt.

BUGFIX: Benachrichtigung automatisch erstellter Rechnungen fehlerhaft

Dieses Problem wird im Redmine Ticket #218 angesprochen.

Durch Verwendung einer lokalen Variable behoben

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

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

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.

»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...

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

CsvImport: deutlich anzeigen, ob Vorschau oder echter Import

CsvImport: Spaltenzuordnung für non multiplexed

Hintergrundjob periodische Rechnungen: bei Fehlern Fehlermeldungen ausgeben, …

… nicht die Anzahl der Fehler.

SelfTests: Weitere Randfälle zumindestens in der Analyse-Ausgabe erwähnen

Hintergrund: Steuerschlüssel können auch bei Zahlungskonten vorhanden sein,
damit wäre eine Verknüpfung mit einer Netto-Rechnung auch nicht sinnvoll.

SelfTests erweitert: arap.paid mit acc_trans.paid vergleichen

Der Bezahl-Wert in den Nebenbücher muss auch immer mit dem Bezahl-Wert
in der acc_trans(chart_link= '%ARAP_paid%' übereinstimmen.

SelfTests verbessert

Eingangsrechnungen können und dürfen diesselbe Rechnungsnummer haben,
entsprechend beim group by berücksichtigt.
Ferner amount auf Zahlungsausgangskonto und nicht Zahlungseingangskonten
berücksichtigt.

SelfTest: Überbuchte Bank-Transaktion finden

Es ist möglich, mehrere Rechnungen auf einen Schlag einer Bankbewegungen
zuzuordnen. Aktuell wird an der Oberfläche der Anwendung hier keine
Warnmeldung/Rückmeldung ausgegeben. Eine überbuchte Bankbewegung ist...

SelfTests Buchungen ohne Steuer auch mit Steuerschlüssel 11, etc filtern

Nicht nur Steuerschlüssel 0 kann steuerfrei sein, sondern auch
diverse andere (tax.rate beachten!)

SelfTests: Verwaiste invoice-Einträge auch auf Zeitraum beschränken

Wiederkehrende Rechnungen: Variablen Periodenstart-/-enddatum richtig setzen

Die Perioden beginnen nicht mehr immer nur am 1. des Monats, sondern an
dem Tag, der über das Vertragsstartdatum angegeben ist. Daher müssen
auch die Variablen <%period_start_date%> und <%period_end_date%> anhand...

Wiederkerende Rechnungen: Implementation automatischer Versand via E-Mail

SL::DB::Printer: »Dokument an Drucker schicken« zentralisiert

Die neue Funktion print_document übernimmt das Spawnen des externen
Prozesses und schickt das Dokument an den Drucker. Das Dokument kann
entweder direkt als Inhalt oder als zu sendender Dateiname übergeben...

Revert "Bugfix zu manuelle Lieferadresse im Auftrag wird nicht in wiederkehrende Rechung übernommen"

This reverts commit f46afb13bacfe1d838cb4a7a5b5b58e8145ff4b1.

Wird durch einen folgenden Commit besser implementiert, nämlich direkt
in der Umwandlungsroutine von Auftragsobjekten zu Rechnungsobjekten.

Selftest Transaction - Syntaxfehler entfernt

SelfTest - bessere Fehlerbehandlung für Einbinden der Test-Module

Fehlermeldung an den TaskServer zurückgeben.
Fehlermeldung(en) in der Status-E-Mail hinzufügen.

SelfTests falscher Zeitraum für Standard-Tests

Selftests konsequent auf fromdate -> todate umgestellt

Ferner Kosmetik von hässlichen SQL Formatierungen

Self-Tests Haupt- / Nebenbuchprüfung verbessert

Weitere Selftests

- Kreditoren Haupt und Nebenbuch Netto-Prüfung (ap.netamount == acc_trans.amount)
- Für alle Taxkeys <> 0 muss auch eine Steuerbuchung in acc_trans vorhanden sein

SelfTest -> Transaction. Test verbessert (ar.amount invoice.amount)

Summe laut Verkaufsbericht gleich Summe der Rechnungen (ar).
Der Test schlägt fehl, wenn es Debitorenbuchungen gibt, da hier keine
invoice Einträge gemacht werden. Entsprechend die Abfrage verbessert.

SelfTest: Datenbanknamen gefixt; Client-Namen auch mit ausgeben

SelfTest: Bei overpaid auch invnumber ausgeben, zur besseren Analyse

Copy & Paste Fehler, bei Fehlertext SelfTest #11

Csv-Import-Hintergrund-Job: Als Ergebnis einen Text zurückgeben, nicht das Job-Objekt.

Die implizite Rückgabe des Job-Objekts führte beim Speichern der Job-Historie zu
einem DB-Fehler, der nichts mit dem Import zu tun hat.

MassRecordCreationAndPrinting: SessionFiles für TaskServer Fix

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

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

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

MRCP.pm. Falls Webdav Archivierung auch PDF in Webdav kopieren

Beim Massenwandeln prinizipiell auch den WebDAV Ordner und die
Datei plus Zeitstempel erstellen. Entsprechende Fehlermeldungen
an die Oberfläche zurückgeben, wie allen anderen Meldungen in MRCP auch.

Wiederkehrende Rechnungen: Lastschrifteinzugsflag in Konfiguration setzen können

POD Fehler

Massenkonvertierung von Lieferscheinen nach Rechnung ink. Druck

Die Konvertierung als auch das Generieren des PDFs erfolgt als
Background-Job. Entsprechend muss der task_server für den.
Mandanten eingestellt sein.
Details und bekannte offene Punkte im POD der beiden Perl-Module....

CreatePeriodicInvoices: HTML-Formatierung beim automatischen Drucken richtig formatieren

Hintergrundjob für E-Mail-Bericht über fehlgeschlagene Jobs

Wiederkehrende Rechnungen: Testcase für Periodizitäten

Wiederkehrende Rechnungen: Berechnung für Auftragswertperiodizität angepasst

Wiederkehrende Rechnungen: Auftragswerts-Periodizität setzen können

Noch keine Anpassung der eigentlichen Berechnung. Wohl aber
Dokumentation von SL::DB::PeriodicInvoicesConfig.

CreatePeriodicInvoices: HTML-Formatierung in Langtexten berücksichtigen Teil 2

Nach Umstellung der Bemerkungsfelder in Einkaufs- und Verkaufsbelegen
muss auch das Feld »notes« als HTML-formatiert behandelt werden.

Verknüpfung von Einzelpositionen auch bei periodisch erzeugten Rechnungen (Auftrag -> Rechnung)
Für InvoiceItem in Invoice ein converted_from_*_id hinzugefügt wie bei der Verknüpfung von Auftrag zu
Rechnung, bzw. Lieferschein zu Rechnung. Letzterer Weg wird aktuell bei Objekten noch nicht...

CreatePeriodicInvoices: HTML-Formatierung in Langtexten berücksichtigen

Beim Ersetzen der Variablen muss das Format des Textes (HTML oder
normaler Text) berücksichtigt werden, damit Formatierungen richtig
angewandt und die Platzhalter überhaupt erst gefunden werden....

CreatePeriodicInvoices: refactoring der Parameterübergabe an _replace_vars

CreatePeriodicInvoices-Job: for anstelle von map für reine Nebeneffekt-Schleifen

PeriodicInvoices: employee aus Quellauftrag setzen

Wiederkehrende Rechnungen: Variable period_end_date gefixt

Das Periodenenddatum wird am Anfang der sub bereits richtig als »der
letzte Tag innerhalb des Abrechnungszeitraumes« berechnet und darf
hinterher daher nicht mehr auf den Monat abgeschnitten werden.

Hintergrundjob zum Löschen abgelaufener Sessions

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.

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

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

Test-BackgroundJob: Möglichkeiten, das Ergebnis zu steuern

SL::DB::PeriodicInvoicesConfig: Datumsberechnung aus Backgroundjob verschoben

Wiederkehrende Rechnungen: Formatierung von Datumsdruckvariablen über freie Formatstrings

Wiederkehrende Rechnung: "erste Rechnung erzeugen am" als Startdatum nutzen

Selftest - kaputten Test aus Commit ea8e8a62 repariert

Die Prüfung auf verwaiste invoice-Einträge war fehlerhaft. Weitherhin
wurde die Prüfung auf die gesamt Datenbank ausgeweitet.

Neuer SelfTest Testfall: verwaiste invoice Einträge

Suche nach verwaisten Einträgen in invoice, die weder mit ar noch mit ap
verknüpft sind.

Beim CSV-Import die Session-Id der "auftraggebenden" Sitzung verwenden.

Damit kann der CSV-Import via Task-Server auch von Benutzern verwendet
werden, die den Task-Server nicht in ihrer Sitzung gestartet haben.

Es bleibt das Problem, dass der Import nicht ausgeführt werden kann,...

Bugfix zu manuelle Lieferadresse im Auftrag wird nicht in wiederkehrende Rechung übernommen
closes #2296

Beim CSV-Import Einträge in die Historie machen.

Fixt #1998.

Zirkuläres Include von CsvImport BackgroundJob und Controller aufbrechen

Unterstützung für andere Datenbankencodings als Unicode/UTF-8 entfernt

'templates' nicht aus %::myconfig, sondern aus Defaults-Tabelle holen

Selftest verbesserung

Zähler noch hochgesetzt und von betroffenen Konten werden jetzt auch
Kontonummer/Kontobeschreibung angezeigt.

Test auf Steuerschlüssel

Der SelfTest wird in diesem Commit so erweitert, dass überprüft wird,
ob jedes Konto einen gültigen Steuerschlüssel hat.

Wiederkehrende Rechnung: Debugmeldungen immer, aber auf Level DEBUG1 ausgeben

Modus für Druckbefehl bei wiederkehrenden Rechnungen richtig einstellen.

Behebt #2246.

Periodische Rechnungen: Monatsnamen nicht im Script hardcoden

Abgesehen davon, dass es extrem unschöner Stil ist, werden ohne "use
utf8;" dann auch die Umlaute nicht richtig behandelt.