projekt kivitendo

POD Ergänzungen

BankTransaction::save_single_bank_transaction kann nur
1 noch niemals vorher verbuchte Bankbewegung mit n Belegen verbuchen.
Sollte etwas klarer im POD und später in der Methode deutlich gemacht werden.

Kontoauszug verbuchen -> Dialogbuchungsentwürfe verbessert

Nette Idee aus odyn (Start des Gedankens #f09c2b407faa7 Ende des Gedankens #765a3d421e7).
Zwei Sollbruchstellen in odyn, deshalb in kivi neu formuliert:

Sollbruchstellen:
a) Ein Aufruf von BankTransaction::action_list kann Zustände im Datenmodell verändern...

Bericht BankTransaction auch Verknüpfungen mit Dialogbuchungen anzeigen

Zahlungsverkehr -> Berichte -> Bankbewegung -> verknüpfte Rechnungen
zeigt die record_links zum Typ ar|ap, erweitert für den Typ gl.

Bankimport: Fehler beim Verbuchen von Teilzahlungen: Rollback bei Fehler

Falls ein Fehler auftritt wird kein Rollback von der bereits gemachten Zahlung und dem neuen Recordlink gemacht,
lediglich die Banktransaktion wird nicht verändert

Erweiterung durch ein Test test_bt_error...

Bankimport: Rundungsproblem beim Vergleich Rechnungsbetrag - Kontobetrag

Durch explizites Runden konnte die perl Floatingpoint Arithmetik nicht überzeugt werden,
deshalb werden nun die Formatierten Strings der Beträge noch zusätzlich verglichen.

Hiermit wird der "exact_match" beim Vergleich von z.B. 3456,28 und 3456,29 nicht mehr gefunden...

Bankimport: Prüfung des reinen Ziffernanteils der Rechnung

Falls Rechnungen in der Rechnungsnummer ein Prefix vor der Nummer haben
und dies nicht exakt im Verwendungszweck der Kontobewegung aufgeführt ist,
wurde dies nicht als Bewertungskriterium herangezogen....

SL::Mailer: content_type statt contenttype, so wie im POD beschrieben

RecordLinks: Support für Kreditore/Debitorenbuchungen

Conflicts:
SL/Controller/RecordLinks.pm

ergänzung cherry-pick: form->format_sellprice existiert nicht

BankTransaction/list: vorhandene sepa_export_items an get_agreement übergeben

Conflicts:
SL/Controller/BankTransaction.pm

Ergänzung zum cherry-pick von gerade format_sellprice ist nicht in Form.pm

Ferner doppelt gepickte Bibliothek entfernt.

BankTransaction/list: sepa_export_items mit partition_by splitten

Spart eine Doppelschleife

Conflicts:
SL/Controller/BankTransaction.pm

BankTransaction::get_agreement_with_invoice - sepa_export_items optional übergeben

CSV-Import Aufträge: Bei History-Eintrag zwischen Ver- und Einkauf unterscheiden

Auftrags-Controller: Lieferantendetails ("D"): keine Preisgruppe bei Lieferanten

CsvImport Aufträge: interne Bemerkungen aus Kunden-/Lieferantenbemerkunen setzen

falls keine intnotes in der CSV-Datei angegeben sind.

LaTeX-Escaping: gewisse Formen von »no line to end here« vermeiden

Passiert, wenn im HTML so ein Konstrukt existert:

…Text</p><p> <br>

Das wird zu einem Paragraphen, einem Leerzeichen und einem forcierten
Zeilenumbruch:

----schipp----
Text

\newline...

Finanzübersicht: »einmalige« Periodizität bei wiederkehrenden Rechnung gefixt

Die Finanzübersicht nutzt die Funktion
`SL::DB::PeriodicInvoicesConfig::calculate_invoice_dates`, um jedes
Ausführungsdatum innerhalb eines Zeitraumes zu berechnen. Dort wurde
»einmalig« fälschlicherweise wie »jeden Monat wiederholen«...

Kontoauszug verbuchen: Fehlermeldungen visuell (flash) besser hervorheben

Beim Analysieren von #355 aufgefallen. Falls Probleme beim
Verbuchen der Kontoauszüge auftreten, sollte auch die Flash-Anzeige
nicht grün sein.

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

Kreditorenbuchungen: Flag »cleared« beim Zahlungsbuchen beibehalten

Existierende Zahlungen werden aus acc_trans komplett gelöscht und neu
eingefügt. Dabei geht der Status des Flags »cleared« verloren, der
anzeigt, dass eine Zahlung mit dem Konto abgeglichen wurde....

Debitorenbuchungen: Flag »cleared« beim Zahlungsbuchen beibehalten

Existierende Zahlungen werden aus acc_trans komplett gelöscht und neu
eingefügt. Dabei geht der Status des Flags »cleared« verloren, der
anzeigt, dass eine Zahlung mit dem Konto abgeglichen wurde....

Einkaufsrechnungen: Flag »cleared« beim Zahlungsbuchen beibehalten

Existierende Zahlungen werden aus acc_trans komplett gelöscht und neu
eingefügt. Dabei geht der Status des Flags »cleared« verloren, der
anzeigt, dass eine Zahlung mit dem Konto abgeglichen wurde....

Verkaufsrechnungen: Flag »cleared« beim Zahlungsbuchen beibehalten

Existierende Zahlungen werden aus acc_trans komplett gelöscht und neu
eingefügt. Dabei geht der Status des Flags »cleared« verloren, der
anzeigt, dass eine Zahlung mit dem Konto abgeglichen wurde....

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.

Payment::pay_invoice with skonto -> Steuersatz ist eindeutig

tax_id in acc_trans definiert sicherer den Steuersatz als der
taxkey (Steuerschlüssel von DATEV), s.a. FK-Constraint:
"acc_trans_tax_id_fkey" FOREIGN KEY (tax_id) REFERENCES tax(id)

DATEV::CSV Typo in Datenstruktur

SelfTests: False Positive bei Buchungen auf ic_cogs als Sammelkonto vermeiden

CSV-Helfer: Leere Zeilen ignorieren.

Als leere Zeilen hier gelten auch Zeilen, die nur das Trennzeichen enthalten.

Mit leeren Zeilen gab es immer wieder Probleme, teils mit schwer zu
interpretierenden Fehlermeldungen, teils mit nicht gewünschtem Verhalten, z.B....

Auftrags-Controller: Workflow Auftrag VK <-> EK

SL::DB::Order->new_from: Prüfung auf Quell- und Ziel-Typ refactored

Typo in Fehlermeldung

Auftrags-Controller: fake id für Items nach Workflow setzen.

Für items, die hinzugefügt werden, also noch nicht in der DB gespeichert sind,
muss eine fake id gesetzt werden, damit diese bei den actions, die einzelne
items betreffen, auch richtig gefunden/zugeordnet werden können....

Lieferplan um Filter nach Abteilungen ergänzt

Auftrags-Controller: Kunde/Lieferant vorbelegen, wenn deren id übergeben wird

Für die Workflow-Links aus den Kunden-/Lieferantenstammdaten heraus werden
die customer_id oder vendor_id berücksichtigt und die Kunden-/Lieferanten-
abhängigen Attribute im Order-Objekt entprechend gesetzt.

Workflow aus Kundenstammdaten zum neuen Auftrags-Controller, …

… wenn experimentelle Features eingeschaltet sind

Auftrags-Controller: Formelberechnung

Sollte evtl. auf Dialog umgestellt werden.

Auftrags-Controller: Kunden-/Lieferantendetails ("D")

Hier sollte die alte Makse auch umgestellt werden, erstens auf Dialog und
zweitens damit via Rose zugegriffen wird.

Payment Helper - Parameter in POD korrigiert

Payment Helper - besser Fehlermeldung in pay_invoice

Dev create_sales_invoice - type ist invoice

CVars: in Artikelstammdaten sind alle CVars gültig

Der Code zum Rendern der CVars zeigt nur »Element deaktiviert«, wenn
die CVar als ungültig geflaggt ist. Das kann auch in den Stammdaten
passieren, nämlich dann, wenn man eine CVar in der Konfiguration als...

Order-Controller: Angebotsgültigkeitsintervall beachten

Wie beim alten Angeboten auch, soll die Gültigkeit des Angebots
aus der Mandatenkonfiguration gesetzt werden, falls hier ein Wert vorhanden ist.

Auch Verwendungszweck beim Import per CSV zusammenfügen

Beim MT940 Import wird purposeX und remote_name_X in einer
Zeichenkette zusammengefügt. Beim CSV-Import ist dies nicht
möglich, da an der Oberfläche das entsprechende Drop-Down-Element
diese Zuordnung (Spalte -> Import-Objekt) nicht zulässt.

Merge pull request #17 from robert-scheck/apache24

Update .htaccess for Apache 2.4 to avoid mod_access_compat dependency

Auftrags-Controller: bei als neu speichern auch ein neues Objekt erzeugen

Behebt #351 (redmine).

Order: new_from: auch gleiche Quell- und Ziel-Typen berücksichtigen

Update .htaccess for Apache 2.4 to avoid mod_access_compat dependency

See also: https://httpd.apache.org/docs/2.4/upgrading.html

vergessener Presenter::EmailJournal

Der Betreff der E-Mail soll anklickbar sein und direkt zum
Detaileintrag des Mail-Journals führen.
Überwiegend übernommen von Presenter::Letter

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

DATEV/CSV check_encoding, sollte wirklich das encoding checken

Entsprechend vier Testfälle für die Routine gesetzt und
Hintergründe in #348 dokumentiert.

Auftrags-Controller: Anzeige mit Infos über den Kunden-/Lieferantentyp

SL::DB::Order: keinen Fehler werfen, wenn Typ noch nicht zu ermitteln.

Das ist der Fall, wenn noch kein Lieferant oder Kunde gesetzt ist.

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.

Auftrags-Controller: Workflow Angebot->Auftrag nur, wenn gespeichert

Typos in POD

Auftrags-Controller: Mini-Refactoring: js-save-Funktionen zusammengefasst

Auftrags-Controller: Save as new

Controller::Base::url_for: nur noch fragment erlauben

Mit mosu darüber gesprochen, und folgende Argumente kamen:

• Es ist unschön dass quasi magische Elemente wie controller, action und
  jetzt hash und fragment in einer flachen Argumentliste übergeben
  werden. Das ist aber historisch gewachsen und wird jetzt nicht...

TopQuickSearch::Contact: fragment statt hash für for_url

InstallationCheck: String::Shellcode nicht zur compilezeit laden

TopQuickSearch::Contacts: direkt auf den Ansprechpartner tab springen

Controller::Base: url_for unterstützt jetzt URI fragments.

HTML Menü: Javascript ausgelagert in eigene Datei

Form: verwaiste Funktionen entfernt

Auftrags-Controller: Workflow von Angebot nach Auftrag (Ein- und Verkauf).

SL::DB::Order->new_from implementiert.

Im Moment nur von Angeboten zu Aufträgen (Ein- und Verkauf).

Auftrags-Controller: Speichern nicht accesskey "enter"

Auftrags-Controller: auch für Angebote/Anfragen

Zahlungsbedingungen bei Kundenwechsel aus Stammdaten holen

`IS::get_customer` kann die Entscheidung, ob die Zahlungsbedingungen
überschrieben werden sollen oder nicht, gar nicht treffen, weil es
nicht genug Informationen hat. Nur der Aufrufer hat die, und daher...

Behebt: #343 Kontoauszug verbuchen Skontoautomatik bei Verkaufsrechnungen defekt

modules/fallback: Module entfernt.

In modules/fallback sollten nur module stehen die:

a) für installationcheck gebraucht werden oder
b) nicht per apt installierbar sind

Das gilt für die folgenden Module schon lange nicht mehr:

• DateTime::Set
• Email::Address...

AM::_get_warehouse: schnelleres Query

Das alte war extrem langsam für Installationen mit vielen Lagerplätzen

Typo in POD

PriceSource: unknown Flag um "fast" korrekt durchzureichen

PriceSource: fast Flag für zukünftige Performanceoptimierungen

CsvImport: Makemodels bei bestehenden Waren stabil sortieren

Part: makemodels_sorted accessor

Makemodel Manager mit _sort_spec

Auftrags-Controller: Konfiguration wiederkehrender Rechnungen.

Auftrags-Controller: E-Mail-Dialog aus common verwenden.

Auftrags-Controller: Beim Drucken Sprache richtig berücksichtigen.

Auftrags-Controller: Fehler bei Anzeige von EK und Marge in 2. Zeile behoben

Wenn das Order-Objekt noch nicht gespeichert ist, kann darauf nicht über
$item->order zugegriffen werden. Darum muss an die 2. Zeile eine weitere
Variable durchgereicht werden, um zu bestimmen, ob wir im Verkauf sind und...

Auftrags-Controller: Workflow -> Rechnung (Speichern und Rechnung erfassen)

Auftrags-Controller: Verkäufer bei Kundenwechsel ändern

Auftrags-Controller: Sortimente mit Mehrfachauswahl hinzufügen können.

Behebt #220 (redmine).

Presenter::Tag::link - umbenannt in link_tag

um Namenskonflikte mit dem link builtin zu vermeiden

Controller::LoginScreen: AuthHandler::handle ohne Argumente aufrufen

Abgesehen davon, dass die Parameter nicht mehr benutzt werden, behebt
das einen ganz ekeligen Bug: "panic: attempt to copy freed scalar"

Das Problem ist, dass der AuthHandler selber das Einloggen übernimmt und...

PriceSource: Objekte cachen.

Im Moment werden die einzelnen Worker im normalen Workflow mehrfach
angelegt, einmal für die verfügbaren, und dann noch einmal für den
besten und zum wiederherstellen der existierenden.

Für größere Belege wird das merklich, und bei komplexen...

Presenter: Kosmetik: Anordnung

Presenter: Klasse für SL::DB::PurchaseInvoice ist SL::Presenter::Invoice

Sichtbarer Effekt: in der Liste der verknüpften Belege wurde die
Rechnungsnummer und damit der Link zur Einkaufsrechnung nicht
angezeigt.

LaTeX-Druck: Newlines nicht entfernen

LaTeX-Druck: Unicode-Control-Zeichen und nicht druckbare Zeichen entfernen

Filtert z.B. das »bidirectional left-to-right«-Kontroll-Zeichen raus,
an dem sich LaTeX verschluckt.

DATEV/CSV: Buchungstext exportieren

Durch einen Typo wurde da statt dem Slot "buchungstext" der Slot
"buhcungsbes" exportiert, der natürlich leer war.

PriceSource: Doku Update

Preisquelle Stammdaten: lastcost hat jetzt niedrige Priorität

Damit überstimmt der nicht mehr listprice sofern der vorhanden ist

fixt #337

Vergessene Presenter-Umstellung für Controller/Part.pm

SEPA-Sammelanweisung (is_batch_transaction): BUGFIX Loop richtig beenden

Wurde eine Sammelanweisung gefunden, ist die Suche für die betroffenen Banktransaktion beendet.
Fälschlicherweise wurde nur die innere Schleife beendet, es muss aber die äußere Schleife beendet werden.

Presenter/RequirementSpec: typo

Part: Speichern nicht accesskey enter