bank_transactions Testfälle erweitert

für #356 und #355.

part-template typo with_empty

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.

Das Flag einer Zahlung wird nun beibehalten, sofern:

• die Zahlung bereits vorher existiert hat (Präsenz der
  `acc_trans_id`)
• Wert und Konto gleich geblieben sind

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.

Das Flag einer Zahlung wird nun beibehalten, sofern:

• die Zahlung bereits vorher existiert hat (Präsenz der
  `acc_trans_id`)
• Wert und Konto gleich geblieben sind

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.

Das Flag einer Zahlung wird nun beibehalten, sofern:

• die Zahlung bereits vorher existiert hat (Präsenz der
  `acc_trans_id`)
• Wert und Konto gleich geblieben sind

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.

Das Flag einer Zahlung wird nun beibehalten, sofern:

• die Zahlung bereits vorher existiert hat (Präsenz der
  `acc_trans_id`)
• Wert und Konto gleich geblieben sind

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.
beim Warenimport das Anlegen neuer Artikel für jede leere Zeile.

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.

Das behebt z.B. einen Fehler mit falschen Preisquellen nach dem Workflow
Angebot -> Auftrag, wo immer die Preisquellen der ersten Postion im Dialog
verwendet wurde.

Lieferplan um Filter nach Abteilungen ergänzt

Fix: #354 Zahlungsbedingung falsch bei Workflow Lieferschein -> Rechnung

Bisher wurde nur nach dem ersten Treffen der richtigen Auftragsnummer gesucht.
Dabei konnten Zahlungsbedingungen vom Einkaufs-Auftrag nach Verkaufs-Rechnung übernommen werden.
Entsprechend zusätzlichen Filter nach vc_id eingebaut.

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

calculate_qty (Formel): auf Dialog umgestellt

Auftrags-Controller: Formelberechnung

Sollte evtl. auf Dialog umgestellt werden.

calculate_qty (Formel): Input- und Formel-Feld auch als Dom-Id übergeben können

calculate_qty (Formel): alu (?) und andere unbenutzte Parameter entfernt

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

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

SuSa - vergessene Hidden für department_id

t/ar/ar.t aufgeräumt und Refactoring

Typo in Beschreibung von chart_pos_er.sql

Refactoring payment.t

reset_state in init_state umbenannt und params entfernt. Die Daten werden nur
einmal aufbereitet.

Überall title an Anfang der Funktion verschoben, bzw. neu angelegt.

ALWAYS_RESET entfernt. Es werden nun während des Tests nie Rechnungen gelöscht.
Immer alle Daten der Einzeltests zu löschen und alles neu zu initialisieren
würde den Test auch recht langsam, und war eh nicht sauber implementiert.

t/db_helper/payment.t - Testcode verschoben

die eigentlichen Tests stehen jetzt vor den Testfunktionen und
Helperfunktionen, nicht ganz am Ende.

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
»Deaktiviert als Voreinstellung« ausgewählt hat.

Der Effekt ist, dass man die CVar in den Stammdaten nicht mehr
bearbeiten kann.

EK-Rechnung Stornos verbieten solange Zahlungen nicht ausgeglichen sind

S.a. #325 (https://redmine.kivitendo-premium.de/issues/325)

Stornos bei EK-Rechnungen verbieten, solange SEPA-Exports verknüpft sind

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.

doc: Liste der Debian-Pakete ergänzt

Somit gibt es wieder zwei einfach zu pastende Installationslisten.
Einmal Datenbank, Webserver und Perl und des weiteren die Druckvorlagenpakete.
Die Pakete poppler-utils, aqbanking-cli und postgresql-contrib sind somit hinzugefügt.

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.

SQL release Upgrade Skripte für 3.5.2

Merge pull request #22 from robert-scheck/wollen-moechten

s/Wollen/Möchten/g - less strong and likely a bit more polite

Merge pull request #20 from robert-scheck/fa-leonberg

Reflect changed data of tax office Leonberg (since 2018)

Font-Familie nur im <body>-Tag setzen, und auf Browser-Standard-Schrift belassen

sans-serif ist ein Meta-Name, der im Browser von den Anwendern selber
eingestellt werden kann.

Merge pull request #19 from robert-scheck/capital-sharp-s

Support German capital sharp s

Merge pull request #21 from robert-scheck/typo

Corrected typos, added missing tags and updated Fedora information

Merge pull request #18 from robert-scheck/vatno

Use consistently shortening "USt-IdNr." according to German BZSt

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

s/Wollen/Möchten/g - less strong and likely a bit more polite

Corrected typos, added missing tags and updated Fedora information

Reflect changed data of tax office Leonberg (since 2018)

See also:
 - http://www.fa-leonberg.de/pb/,Lde/Startseite
 - http://www.fa-leonberg.de/pb/,Lde/Startseite/Ihr+Finanzamt/Bankverbindungen

Support German capital sharp s

See also:
 - https://de.wikipedia.org/wiki/Gro%C3%9Fes_%C3%9F
 - https://en.wikipedia.org/wiki/Capital_%E1%BA%9E

Use consistently shortening "USt-IdNr." according to German BZSt

http://www.bzst.de/DE/Steuern_International/USt_Identifikationsnummer/FAQ/FAQ_Vergabe_USt_IdNr/Vergabe_FAQ_node.html

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

Changelog: Neue Verknüpfung E-Mail-Journal <-> Beleg

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
sich überwiegend an der Erweiterung der Verknüpfung von Letter.pm, bzw. dem ShopConnector.

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

Auftrags-Controller: Belegnummer in Titlezeile anzeigen

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

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

Inventur: Journal nach Artikel filtern können: Doppeltes "tr" korrigiert

Inventur: Journal nach Artikel filtern können

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.

Validator: time - striktere Prüfung und tests

Typo in Übersetzungen

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

Typos in POD

Validator: time

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
  geändert. Ziel sollte es aber sein, dass potentielle Kollisionen
  selten passieren.

* fragemnt ist die offizielle Bezeichnung für den #... Anteil einer URI.
  Blöderweise nennt Javascript das document.location.hash, und so ist
  das im Gedächtnis geblieben. Da aber hash eh schon in Perl und
  Programmieren extrem überladen ist mit Bedeutungen ist fragment hier
  der etwas weniger miese Begriff.

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.

js: tab-widget cookie ignorieren wenn url hash vorhanden

Unter bestimmten umständen funktioniert das direkte Springen auf einen
Tab sonst nicht.

Part/Customer/Vendor: Bei click auf ein autocomplete item sofort committen

Betrifft nur die beiden schon auf commit umgestellten Picker. Historisch
benutzen die Picker ein zweistufiges Modell: Nach Texteingabe wird bei
der ersten aktion (sei es tab oder enter) erstmal vervollständigt, aber
nicht committed. Erst wenn dadurch die Maske in einen definierten
zustand gerät wird beim zweiten tab committed.

Es gibt aber diverse Szenarien wo die Absicht des Benutzers eindeutig
ist. Dabei kann dann der zweite Schritt übersprungen werden. Bisher war
das vor allem Texteingabe die eindeutig gematcht hat und dann mit Enter
abgeschlossen wurde. Der Klick auf ein Element im Drop-Down ist aber
definitiv auch ein solcher Fall.

HTML Menü: Javascript ausgelagert in eigene Datei

Menü: CSS-Klasse an offene Menüeinträge setzen

Damit ist es möglich, in Stylesheets zwischen offenen und
geschlossenen Menüs zu unterscheiden.

ustva: verwaiste Funktion entfernt

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: Angebote/Anfragen: Übersetzungen

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
muss `IS::get_customer` den Wert immer aus den Kundestammndaten
übernehmen.

Auftrags-Controller: Telmplate: '$' vor T8-Filter

Hinweis in UPGRADE: Fallback-Module entfernt.

Behebt: #343 Kontoauszug verbuchen Skontoautomatik bei Verkaufsrechnungen defekt

p.link_tag anstatt p.link in bank_transactions Templates

DATEV: Testanpassungen

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
* List::MoreUtils
* List::UtilsBy
* PBKDF2::Tiny
* Regexp::IPv6
* Set::Infinite
* String::ShellQuote
* parent

Dabei waren DateTime::Set und Set::Infinite als Anhängigkeiten von
DateTime::Event::Cron reingekommen, das gibt es aber zusammen mit
Set::Crontab immernoch nicht in apt, also ist das weiter im fallback.

List::MoreUtils und List::UtilsBy sind meine ich für Debian 5
reingekommen, die Pakete sind aber seit Ewigkeiten in apt verfügbar.

parent ist seit 5.10.1 ein coremodul.

Die anderen wurden anscheinend einfach so reingeworfen, und hätten da
nie drin sein sollen.

scripts/dbupgrade2_tool.pl: Datenbankverbindungen mit SL::DB->client

AM::_get_warehouse: schnelleres Query

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

Typo in POD

Aufträge speichern: Warnung bzgl. existierender wiederkehrender Rechnungen gefixt

Behebt #341.