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.

customernotes auch als notes Form Variable(fixmaster)

Durch den query ct.* werden die notes und per ct.notes as customernotes eingelesen

fixup durch löschen von notes vor kopieren

Einkaufsstornos auch verknüpfen

Analog zur Verkaufs-Ansicht auch die Einkaufsrechnungen bei
Stornos verknüpfn

Verkauf Storno-Rechnungen auch verknüpfen

Ähnlich wie bei Gutschriften sollten Stornorechnungen auch mit
der Ursprungsrechnung verknüpft sein. closes #132

Presenter/Part: Type in Doku.

Order: Doku: weniger Bugs. Typo.

Kosmetik/Lesbarkeit

Ein klares delete statt ein kompliziertes map mit delete.

Order: Mehr Bugs

Order: und nochmal header nach oben

Order: <h1> statt <div class=listtop> für einheitliches spacing

Order: mehr fehlende Sachen in Doku ergänzt

Order: javascript Kleinigkeiten

Order: POD aufgeräumt, reformatiert, Bugs notiert.

Order: Überflüssige Includes entfernt

Weitere Änderungen für den revert von 7735019
Damit ist das Ticket #144 erledigt.

Revert "Ticket 29 Verknüpfte Belege -> keine Verknüpfung von Rechnung nach Auftrag"

This reverts commit 77350196300e930abc7de0df7dbf42e8cd6193b8.

Conflicts:
SL/OE.pm

Auftrags-Controller: Doku und Kommentare

Vergessene hidden Abteilung für FiBu->Berichte->Offene Forderungen/Verbindlichkeiten

neue Vorlagen mit Schweizer Bank-Einzahlungsscheinen mit OpenDocument-Makro

CsvImport: Aufträge: Fallback Lieferbedingungen auf die vom Kunden/Lieferanten.

Setzt die Lieferbedingungen auf die des Kunden oder Lieferanten des Auftrags,
falls in der CSV-Datei keine Lieferbedingungen angegeben sind.

Partpicker styling

- Lupe jetzt inline
- Lupe in svg, kann also mitskalieren
- Inputfeld ist jetzt Model padding-box, size Angaben propagieren besser
  auf die umliegenden Elemente
- getestet in lx-office-erp und kivitendo css

generic_translations: DB-Upgrade in Perl geschrieben wg. Constraint-Namen

In alten PostgreSQL-Versionen hießen Foreign-Key-Constraints oft noch so
was wie »$1«. Da sich das Upgrade-Script also bzgl. des Namens nicht
sicher sein kann, gibt's momentan nur die Möglichkeit, einmal alle
Foreign-Keys zu einer Tabelle wegzuwerfen und diese neu anzulegen.

Dafür wiederum haben wir Support-Funktionen in SL::DBUpgrade2::Base, die
wie nutzen können. Also Umstellung des Scripts auf Perl.

PriceSource: doku update

Mehrere Anforderungen die sich über die Zeit gesammelt haben

Artikelstammdaten: Verkaufspreisänderungen nicht mehr an Erzeugnisse propagieren

changelog-Update zu Zahlungsbedingungen

FlattenToForm: Zahlungsbedingungen des Kunden/Lieferanten nicht kopieren

Es haben die ZB des Beleges zu gelten, nicht die des
Kunden/Lieferanten. Die Variable »payment_terms« wird zwar später wieder
anhand von »payment_id« überschrieben (in »OE::order_details« und dann
»Form::set_payment_options«), aber nur dann, wenn im Beleg auch wirklich
Zahlungsbedingungen ausgewählt sind.

Sind keine ausgewählt, so würde das »payment_terms« von den Kunden-/
Lieferantenstammdaten gesetzt bleiben; das wäre schlicht inkorrekt.

DO.pm: wo croak() genutzt wird auch Carp usen

generic_translations: Einträge automatisch löschen, wenn Sprache gelöscht wird

Zahlungsbedingungen: Unterscheidung zwischen Angeboten/Aufträgen und Rechnungen

Dies führt ein neues Attribut »payment_terms.description_long_invoice«
und dazugehörige Übersetzungen in »generic_translation« ein.

Die Druckvariable »payment_terms« wird nun in Abhängigkeit vom
auszudruckenden Beleg gesetzt:

1. Für Verkaufsrechnungen wird zuerst eine Übersetzung von
   »description_long_invoice« für die ausgewählte Sprache gesucht. Falls
   die leer ist oder keine Sprache ausgewählt, so wird die nicht
   übersetzte »description_long_invoice« genommen. Ist auch die leer, so
   erfolgt ein Fallback auf 2.

2. Für alle anderen Belege oder falls bei 1. nichts heraus gekommen ist,
   wird wie vor dieser Änderung eine Übersetzung von »description_long«
   für die ausgewählte Sprache gesucht. Falls die leer ist oder keine
   Sprache ausgewählt, so wird die nicht übersetzte »description_long«
   genommen.

Preisquellen: Darstellung Lieferrantenrabatt gefixt

analog zu: 5f6e919f09cdbeeac184a138d62586d055e9ee0f
"Preisquellen: Darstellung Kundenpreise gefixt"

locales: Update

Verkaufspreisinformationen: Preisentwicklung der Stammdaten anzeigen

Verkaufspreisinformationen: Typfilter in der Überschrift anzeigen

Dadurch wird auf einen Blick etwas klarer, was die Tabelle anzeigt.

Verkaufspreisinformationen: »Seite:« nicht anzeigen, wenn es nur eine gibt

In dem Fall ist $self->{pages}{cur} 0 oder undef und nicht 1.

Typos in Doku

Typos

Typo in Doku

Recnungen: action nicht in die hiddens schreiben aber id setzen

Jetzt wird ein Schuh draus, Javascript braucht bei OD die #action

Preisquellen: Darstellung Kundenpreise gefixt

Rechnungen: resubmit dispatching gefixt

Rechnungen: action nicht in die hiddens schreiben

Fix für das doppel-action Problem

Revert "Rechnungsmasken auf dispatcher Mechanismus (fixmaster)"

This reverts commit b48c9c84b3b070011b50ae41216c3903a369f661.

Das sollte genau andersrum gefixt werden.

Einkaufsrechnungen: Suche nach Artikelnummer/-beschreibung in Positionen(2)

BUGfix für substr($where, 0, 4, " WHERE "):
$where muss am Anfang innerhalb 4 Zeichen das AND stehen haben,
sonst kommt es zu einem SQL Syntaxerror

BUG-Fix: Falsche Links zu Lieferscheinen.

In Spalte Lieferscheine falsche Links korrigiert.
In der Spalte Rechnungen 'Tote Links' korrigiert.

Einkaufs-/Verkaufsmasken: Zahlungsbedingungen bei Kunden-/Lieferantenwechsel resetten …

… auch nach Mehrfachauswahl. Das ist in 5bc87ade kaputt gegangen.
Siehe auch d5a3eba276c5e18e51511878fc74ee717969410b

BUG-Fix: Beim Stornieren einer Rechnungen wird der bezahlte Betrag verdoppelt.

Storniert man eine als bezahlt markierte Rechnung,
so wird der bezahlte Betrag ("paid") verdoppelt.
Im SQL wird, fälschlicherweise, amount zu paid dazu addiert. Soll aber nur zugewiesen werden.

Menu: Fehlerchecks beim yaml einlesen(2)

Abfangen der doppelten ID erst mal rausgenommen,
da die ID sehr wohl mehrfach in einer Datei referenziert werden kann.

Bücherkontrolle in Zahlungseingängen und Zahlungsausgängen fehlerhaft

- Erweiterung der Header und Footer Funktionen um das Buchungsdatum anhand der Bücherkontrolle zu prüfen
- Fällt Buchungsdatum ausserhalb des zulässigen Zeitraum der Bücherkontrolle so werden die betreffenden Einträge bei Zahlungsein-/ausgang deaktiviert
- Einbinden der Prüfung des maximal zukünftigen Buchungsdatums von Zahlungen ausgehend vom heutigen Datum
- Erweiterung der Prüfung bei Neueingabe von Zahlungsein-/ausgängen:
  - Es werden nur noch die editierbaren Einträge geprüft
  - Einträge ohne Zahlungen werden nicht geprüft und auch nicht gebucht

fix redmine #164

SQL-Fehler falls -- in quotiertem String vorkommt.

 z.B. In dem script steht folgende Zeile
INSERT INTO partstypes values(0,'-------','-','f','f','f');

da aber in SL/DBUpgrade2.pm generell alle Zeilenteile die -- haben,
weggeworfen werden:

s/--.*$;

wird der Rest der Datei nach Quote Ende durchsucht und nix mehr passiert.

Also schön nach zwei hintereinanderkommenden -- suchen aber nicht innerhalb von Quotestrings

BUG-Fix: Verkaufslieferscheine: Preis Faktor Formatierung angepasst.

Beim Lesen des Preisfaktors wurde die Formatierung in eine Zahl vergessen.

doppeltes gleiches Statement entfernt

Typo in comments

Artikelpreishistorie: Fix für NULL-Behandlung im Trigger

ChartPicker: Auch auf Paste-Events reagieren.

Siehe auch commit 2cf7a2cd78d812d917fb4c92f8ee57f6a32f3685
(PartPicker: Auch auf Paste-Events reagieren.)

CustomVendorPicker: Auch auf Paste-Events reagieren.

Siehe auch commit 2cf7a2cd78d812d917fb4c92f8ee57f6a32f3685
(PartPicker: Auch auf Paste-Events reagieren.)

ProjectPicker: Auch auf Paste-Events reagieren.

Siehe auch commit 2cf7a2cd78d812d917fb4c92f8ee57f6a32f3685
(PartPicker: Auch auf Paste-Events reagieren.)

PartPicker: Auch auf Paste-Events reagieren.

Damit wird bei eindeutigem eingefügtem Text der entsprechende Artikel
ausgewählt. Ansonsten wird der Text rot (undefined) dargestellt.

Vorher war es möglich, z.B. eine eindeutige Artikelnummer einzufügen, aber
intern war dennoch kein Artikel ausgewählt. In der Anzeige sah es aber so aus,
als sei alles in Ordnung.

ProjectPicker: Die (Un)-Gültigkeits-Spalte/Methode heißt valid, nicht obsolete.

Der Fehler führte dazu, dass bei Eindeutigen Eingaben im Picker dennoch kein
Ergebnis ausgewählt wurde, sondern der ajax-Call einen Fehler meldete, der dann
unterging.

ProjectPicker: SL::DB::Manager::Project hat (noch) keinen type_filter.

Es gibt zwar einen project_type, aber der ist als Filter noch nicht
implementiert.

Dieser Fehler führte dazu, dass bei eindeutigen Eingaben im Picker dennoch
kein Projekt ausgewählt wurde. Der ajax-Call lieferte einen Fehler zurück,
der dann unterging.

Rechnungsmasken auf dispatcher Mechanismus (fixmaster)

Fixup: Doppelte Variable in 0a50b37c

Forms Input Variable "action" existiert doppelt, die letztere ist auf dispatcher gesetzt,
die erste hat aber die id "action" und wird von Javascript gefunden.

(habe ich nun 4 h rumgesucht !!)

Artikelpreishistorie automatisch über Trigger speichern

changelog: no tabs

config: maildelivery.method kann jetzt auch Mailversand unterdrücken

Drafts: gl auch eingebunden

Drafts: Ausgelagert in Controller(2)

fixup: vergessenes Löschen von remove_draft()

Projektliste: Kundenname bei PDF-/CSV-Export richtig ausgeben

Der Controller-Helfer für den ReportGenerator muss aus Objekten Werte
machen können. Wie das geschieht, wird über die Spaltendefinition
festgelegt. Für Nicht-HTML-Anzeige wird entweder eine zur Verfügung
gestellte Unterfunktion benutzt, oder aber auf dem Objekt wird der
Spaltenname als Funktion aufgerufen.

Für die Spalte »customer« wird bei einem Projekt also das
SL::DB::Customer-Objekt genutzt, wenn keine manuelle Sub angegeben
wurde.

Fixt #166.

FlattenToForm: nur für Positionsartikel gültige CVars übernehmen

Benutzerdefinierte Variablen, die in den Artikelstammdaten als ungültig
markiert sind, dürfen dem Drucksystem gar nicht erst zur Verfügung
stehen.

CVars: beim Einlesen für Sub-Modules Gültigkeit richtig bestimmen

Werden für CVars für Belege eingelesen (z.B. Aufträge), wo also
»sub_module« gesetzt ist (hier: »orderitems«), so steht in der
CVar-Spalte »trans_id« die Datenbank-ID des referenzierten
Sub-Items (hier: »orderitems.id«) drin und nicht die ID des Items, auf
das sich die Konfiguration selber bezieht.

Die Gültigkeit einer CVar wird hingegen nicht am Beleg selber sondern
eine Ebene darüber, am Warenstammdatum, festgemacht. Das bedeutet, dass
in der Spalte »custom_variables_validity.trans_id« die Artikel-ID
enthalten ist.

Übergeben bekommt die Funktion zum Einlesen der CVars aber die ID des
Orderitems.

Also muss das Datenbankquery unterschiedliche Tabellen und Spalten
abfragen, je nachdem, ob »sub_module« gesetzt ist oder nicht.

Übersetzung für Buchungsgruppe -> Booking group

Nicht-übersetzte Deutsche Begriffe entfernt und übersetzt

Shopartikel, Steuersatz, Abteilung, Preisgruppe (gab es auch als
Preisklasse).

Typos in POD

fixt#165 shippingdates in inventory nachträglich setzen

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.

Belege runden, und Rundungsdifferenzen auf Extrakonten buchen.

Erfolgsrechnung

Revert "Erfolgsrechnung"

This reverts commit e3f8237101c331c52098cf6f7db86e1db2a2b36a.

Revert "Belege runden, und Rundungsdifferenzen auf Extrakonten buchen."

This reverts commit 422f2f17365ea7879c14b902c5db1e4cd7023e48.

Drafts: Ausgelagert in Controller

AuthHandler: auth Fehler auch bei ajax requests korrekt behandeln

ClientJS: controller default auf leeren Controller

Damit SL::ClientJS->new->render funktioniert

Rechnungen löschen mit confirm nachfragen, und nicht mit Extrarequest

Rechnungsmasken auf dispatcher Mechanismus umgestellt

Verkauf->Berichte->Rechnungen: Bestellnummer des Kunden nicht per default anhaken

Hintergrund: Die Bestellnummer des Kunden nimmt eine Menge Platz in der
Breite weg und ist i.d.R. nur für einige Fälle ein sinnvoller voreingestellter
Wert.
i.A. thw

Neuer Minimaltestfall für Rabattrundung im PTC

Beim PTC wird vor der Multiplizierung mit der Menge der gerundete Rabatt vom
Verkaufspreis abgezogen, statt erst die Zeilensumme zu berechnen und
dann den Rabatt zu ziehen.

6 Artikel zu 0.6€ mit 3% Rabatt
0.6 * 6 * ( 1 - 0.03 ) !=  ( 0.6 - round(0.6*0.03) ) * 6

In diesem Test wird anhand der PTC-Regel getestet. Es wird also nicht
angemahnt, daß das Ergebnis von der Berechnung in den Belegmasken abweicht.

Siehe auch Ticket 82.

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
aber auf jeden Fall nicht buchungskonform und muss entsprechend in der
Datenbank korrigiert werden.
Entsprechend einen Test hierfür geschrieben.

create_backorder entfernt

Kontoauszug verbuchen, Startdatum der Verbuchung greater/equal

Wie bei fromdate und auch sonst überall im Programm, entsprechend
das Startdatum der Kontoverbuchung noch als gültigen Wert mitnehmen.

TopQuickSearch: changelog Eintrag

Changelog für Customer-Picker bei Projekt und Kosmetik

Projekt erfassen: CustomerPicker verwenden

Changelog Update für Abteilung.

Abteilung in Rechnungsübersicht angeben

Dispatcher: end_of_request() sub aus main namespace verbannt

Dispatcher: FCGI-Neustart bei modifizierten Dateien via exec anstelle von exit

Gleicher Grund wie in 655f9081d19d201a1084a609beca1c06bef91e5c:

Wenn sich das Script beendet, so kann es sein, dass der Webserver
bereits den nächsten Request zum Script geschickt hat. Ist das der Fall,
kommt es zu einem internal server error für den User.

Statt dessen kann sich das Script selber ausführen. Dadurch werden die
Kommunikationskanäle zwischen Webserver und Script (STDIN, STDOUT,
STDERR) aufrechterhalten.

Dispatcher: Restart bei hohem Memory-Verbrauch via exec anstelle von exit

Wenn sich das Script im Fall von zu hohem Speicherverbrauch beendet, so
kann es sein, dass der Webserver bereits den nächsten Request zum Script
geschickt hat. Ist das der Fall, kommt es zu einem internal server
error für den User.

Statt dessen kann sich das Script selber ausführen. Dadurch werden die
Kommunikationskanäle zwischen Webserver und Script (STDIN, STDOUT,
STDERR) aufrechterhalten.

Dispatcher: Requstloop vom dispatcher.fpl nach Dispatcher.pm verschoben

Projekt »keep your main namespace clean«.

select styling in lx-office-erp.css

Irgendwer bei Firefox 46 scheint was geraucht zu haben. Das Stylesheet
hatte alle anderen windowmanager decorations überschrieben, aber FF
hat seit 32 select appearance ignoriert. Angeblich aus
Sicherheitsgründen.  Das haben sie glücklicherweise gerade rechzeitig
gefixt, weil sie nun das unity7 Theme nochmal kaputter gemacht haben.
Ansonsten wäre das unbenutzbar.

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