Revert "Webdav-Komponente: Das Erstellen des Verzeichnisses ausgelagert. Ferner den Beleg auch in das Webdav-Verzeichnis kopieren."

This reverts commit 4cb3ecfd6f144901ed89dae7acafdf1c124c53f4.

Svens Anmerkungen:

> Hehe, ich check die ein, da die Erweiterung sowieso nur die webdav-
> komponente betrifft. ansonsten reverte ich direkt, falls du anders
> denkst.

Ich denke anders, wie immer.

- schonwieder deutsche Kommentare *grrr*

- return $main::lxdebug->leave_sub() unless ($path); <- solche kontrukte liefern 1
zurück (ich hab das irgendwann mal auf definiert 1 gesetzt nachdem vorher alles
mögliche bei rauskommen konnte). Wenn dann benutz lieber sowas:

$::lxdebug->leave_sub && return unless $path;

- copy_file_to_webdav_folder setzt $form->{type} auf '' und stirbt anschliessend
auf die Bedingung !$form->{type}. Das soll wahrscheinlich nicht so sein oder?

- Nicht jede print Operation hat tmpfiles. Im schlimmsten Fall hast Du damit n
directory traversal eingebaut, das musst Du unbedingt dreifach prüfen und nicht
einfach aus form holen. Sonst schreibt einer einfach tmpdir=/etc&&tmpfile=passwd
in die URL.

Und zu Deinem ursprünglichen Problem: Ich vermute das liegt an der Positionierung
in parse_template. Die Funktion ist ziemlich destruktiv geschrieben, die macht
halt form kaputt um was druckbares zu produzieren. Lass Dir mal form dunpen, ich
vermute dass da schon irgendwas kaputt ist. Die Fehlermeldung funktioniert in der
Konsole einwandfrei.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Webdav-Komponente: Das Erstellen des Verzeichnisses ausgelagert. Ferner den Beleg auch in das Webdav-Verzeichnis kopieren.
Offen: Fehlerbehandlung in Common.pm - ggf. gleich revert

Rückstände aus Umbenennung von SL/DB/Helpers nach SL/DB/Helper gefixt Teil 2

Umstellung Form::throw_on_error auf Verwendung von Exception::Lite anstelle von Hashrefs

Exception::Lite als neues Standard-Exception-Modul in Fallback-Module aufgenommen

Request abbrechen, wenn alte Konfigurationsdateien vorhanden sind

dbupgrade2_tool.pl: Option "--unapplied", zeigt nicht angewandte Upgradescripte an

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Bugfix: Subcent Buchungen führten zu unbalanced ledger in DATEV Export.

Wenn eine 1 cent Buchung auf ein normal versteuertes Konto gebucht wurde
tauchten die Buchungen in der folgenden Reihenfolge in der Datenbank auf:

-0.00840
-0.00160
+0.01000

Der Algorithmus, der die Buchungen aufsummiert, ist aus historischen Gründen
recht nachgiebig und bricht das aufsummieren ab, sobald die Summe der
bisherigen Buchungen kleiner ist als ein Cent. In diesem Fall ist das schon
nach der ersten Buchung der Fall, und bei der dritten Buchung wird dann ein
unbalanced ledger Fehler geworfen.

Der Fix prüft auf solche kleinen Werte und wechselt dafür in einen subcent
Modus, in dem auf zehntel Cent geprüft wird.

Beim Speichern eines Kontos chart.taxkey_id aus taxkeys aktualisieren/setzen

Fix für Bug 1627.

Nach $form->redirect() den Request beenden

Text bei Iconv-Konvertierung nach UTF-8 in Perls internes Encoding umwandeln

Und das auch bei Objekt-Aufrufsyntax:

  $text = SL::Iconv::convert('ISO-8859-15', 'UTF-8', $some_other_text);

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Bugfix für 1614

Created attachment 293 [details]
RP.pm.patch

Das Problem war, dass unterschiedliche Kostenstellen innerhalb desselben
Lieferanten nicht gefiltert wurden.

Sprich:

1.) Finde offene Verbindlichkeiten mit Abteilung x
2.) Lieferant A Treffer
3.) Hole alle offenen Verbindlichkeiten zu Lieferant A

Anbei der Hotfix für dieses Verhalten.

3.patch) Hole alle offenen Verbindlichkeiten zu Lieferant A die Abteilung x
gebucht haben ...

SL::DB::Customer: Fehler in der shipto Relation.

$::auth nicht löschen sondern nur resetten. (Experimentell)

Unechte Multiboxen auf L.select_tag umgeschrieben.

L ist genauso mächtig und deutlich schneller. Mulibox sollte nur noch für Customer/Vendorboxen verwendet werden.

CVar Filter wurden in Kundensuche nicht weitergereicht an den Export.

noch ein index der ein bestimmtes Query deutlich beschleunigt.

Cvar partial template auf PROCESS umgebaut. Spart bei sehr grossen masken ~5 Sekunden.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Zwei Indices in cvar Tabellen um komplexe Suchen zu beschleunigen.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

>>>
Hi Jan,
Ich versuche gerade die Einkaufsrechnungen zur Abteilung als pdf zu
exportieren. Der Bericht stimmt soweit (Screenshot). Doch leider exportiert er mir
in der pdf ALLE Rechnungen.
<<<
department als hidden variable für den csv-export unter Einkauf -> Berichte -> Einkaufsrechnungen hinzugefügt

Schreibfehler beseitigt

Always test your code, Moritz...

False positive in Test "my $foo = @bar" vermeiden

Konstruktue "my ($foo) = shift" durch eindeutigere Konstrukte ersetzt

Testscript zum Auffinden häufiger Fehler (z.B. "my @foo = shift" oder "%bar->something()")

alte Config Variablen in bin/mozilla/ustva.pl gefixt.

Zwei weitere globals die ok sind.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

typo: $::from

$::self typo

Update auf List::MoreUtils v0.30

Testscript um auf globale Variablen zu testen, die bei der Umstellung auf lx_office.conf vergessen wurden.

bin/mozilla/ustva.pl strict.

Falscher Variablenname

Aus versehen in bc433711 entfernte Funktion "show_history" wieder eingebaut

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Verkauf -> Berichte -> Filter nach Abteilungen wieder aktiviert
<<
 ähnlich wie commit 0bbfb33b6aa8e38bb6c81d1684ab7d08e5b5c5af abteilung
 wird so nicht mehr als zeichenkette zusammengebaut
 hätte zu ee9f9f9aa4c3b9d5d20ab10a45c12bcaa6aa78d0 auffallen können ;-) jan

Verkauf -> Berichte -> Filter nach Abteilungen wieder aktiviert
<<
 ähnlich wie commit 0bbfb33b6aa8e38bb6c81d1684ab7d08e5b5c5af abteilung
 wird so nicht mehr als zeichenkette zusammengebaut
 hätte zu ee9f9f9aa4c3b9d5d20ab10a45c12bcaa6aa78d0 auffallen können ;-) jan

Kontenübersicht. Bei Auswahl Bilanz und keinem closedto (Bücher-Abschluss) wurden keine Konten ausgewählt, falls das Installationsjahr gleich dem laufenden Jahr ist

Zwei Warnungen vermeiden

Default-Namen des Debug-Logs auf den üblichen Wert zurückgesetzt

Alle Datenbankverbindungen über DBIx::Log4perl laufen lassen, sofern dies in der Konfiguration aktiviert ist

Standard-Debug-Datei ist /tmp/lxdebug.log

Ein strict Problem beheben.

Iconv get_converter in interne Funktion umbenannt, und den einzigen externen Aufruf durch ein SL::Iconv->new ersetzt.

Erklärung: cbd1249e5c245ee605c32c6cdbdc1e26f92643d0 hat die Handle im
Objektzugriff von SL::Iconv gecacht, hat dabei aber das Verhalten im
Klassenzugriff auf get_converter leicht geändert. Dadurch bekommen
Datenbankupgradescripte kein SL::Iconv Objekt mehr sondern ein Text::Iconv, was
sich minimal anders verhält. Um potentielle Bugs dabei zu vermeiden wird das
hiermit abgeschafft.

SL::Iconv::get_converter bitte nicht mehr verwenden, und dafür SL::Iconv->new
benutzen.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Überflüssiges "form->unescape()" sorgte für falsche Zeichencodierung

Leaks in SL::Template::Plugin::T8

Filter werden in Template::Toolkit in einer anonymen Closure einfach im Kontext
verlinkt. Dadurch geht _jedes_ Filter Objekt verloren und leakt. Diese Änderung
cacht das T8 Filterobjekt, da es sowieso statisch ist. Das beschränkt die Leaks
auf einmal ~100 Byte.

Minimales Beispiel zum Templatesystem

Auf Romans Anregen - Ein simples POD basiertes Hilfesystem

So funktionierts:
- 1. Hilfe in eine pod Datei schreiben
- 2. Pod Datei nach doc/online/<lang>/<tag>.html legen,
      wobei <lang> der countrycode der locale ist,
      und <tag> ein /[a-zA-Z0-9_]+/ Identifier ist.
- 3. HTML erzeugen mit:

  scripts/make_docs.pl

- 4. Im Template einen Hilfelink einbinden mit:

  [% L.online_help_tag('tag') %]

Iconv Handle cachen.

Text::Iconv ist sloppy programmiert und leakt bei FCGI Anwendungen etwa 1MB pro
grossem Request. Das entsprechende Ticket [1] ist 3 Jahre alt, und es ist nicht
damit zu rechnen, dass das in nächster Zeit behoben wird.

Die Handle werden jetzt sortiert gecachet und im Klassenraum vorgehalten, so
dass der Overhead für die Iconv Objekte abschätzbar bei 4,4kb pro Encoding
Kombination ist.

[1] https://rt.cpan.org/Public/Bug/Display.html?id=31101

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Title erst nach create_invoice setzen, dadrin wird der typ sanitized.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

"SET default_with_oids = true;" muß nicht mehr in der postgresql.conf geändert werden.

$::auth erst nach dem session expire zerstören.

templates/webpages/is/_payments.html: Ein ; zuviel

Wechselkurse werden im footer nicht angezeigt.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Bei vielen Lagerplätzen ist es hilfreich die Liste der bin(s) alphabetisch vorzusortieren

Zirkuläre Referenzen auflösen

$::auth erst nach $::form und $::locale anlegen, weil $::auth beide benötigt (SL/Auth/LDAP.pm)

Conflicts:

SL/Dispatcher.pm

Zirkuläre Referenzen in $::auth auflösen, damit DB-Verbindung am Ende des Requests geschlossen wird

Conflicts:

SL/Auth/DB.pm

Merge branch 'master' of lx-office.linet-services.de:lx-office-erp

Funktion "send_file" im Controller, um Dateien herunterzuladen

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Passwort für DB richtig einpflegen

Hinweis von Moritz bzgl. Rechnungsnummer für Gutschrift:
> so, wie du das Feature implementiert hast, funktioniert es aber nur
> genau in dem Moment, in dem du die Gutschrift direkt nach der Umwandlung
> aus der Rechnung heraus ausdruckst. Wenn du sie später via Verkauf ->
> Berichte aufrufst, so wird das Feld invnumber_for_credit_note leer
> sein, weil du es nicht in der Datenbank speicherst.
Indeed. Das hatte ich in meinem ersten Enthusiasmus übersehen. Gleich mehr
...

Update changelog

Bei Verkaufsgutschriften zusätzlich noch die Rechnungsnummer mitnehmen, damit der Bezug klarer ist.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Wahrscheinlich wird $form->department seit der T8 Umstellung nicht mehr korrekt zusammengesetzt.
Offen: Interne DB-ID wird immer noch bei Abteilung angezeigt (?!)
Ferner: Erfolgs- und Kostenstelle ist noch defekt. War meiner Meinung auch noch nie ganz sauber.
Ansonsten, wichtigster Schritt zum Fixen von 1613

Konfigurationsort für viele Flags ist Abschnitt 'features', nicht 'system'

Hoffentlich Fix für Bug 1624.

Sortierfunktion für Pricegroup-Model

Model SL::DB::Prices in SL::DB::Price umbenannt

Conflicts:

SL/DB/Helper/Mappings.pm
SL/DB/Part.pm

Label für falsche Checkbox

Vor jedem Request neue Instanz von $::auth anlegen

$::auth cachet Informationen wie Gruppenzugehörigkeit und -rechte. Das
führt dazu, dass Änderungen im Adminbereich für laufende FCGI-Prozesse
nicht sichtbar waren, wenn die entsprechende Gruppe schon einmal
abgefragt wurde.

Merge branch 'master' of lx-office.linet-services.de:lx-office-erp

Alle Dateien in SL rekursiv nach Übersetzungen durchsuchen

Hinweis auf Remote XUL Manager.

Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp

Fünf-Sekunden-Verzögerung bei falschem Login auch bei nicht existierendem Benutzernamen erzwingen

Beim Neuanlegen von Artikeln aus Beleg heraus $::form in Session speichern

Methoden zum Speichern von eindeutigen Keys in der Session

Außerdem: Form dumpen und wiederherstellen; Werte nur für einen
bestimmten Zeitraum speichern.

Session immer am Ende des Requests speichern

Momentan wird eh am Anfang der Session einmal gespeichert, wenn
Benutzername und Passwort überprüft wurden. Somit erspart man sich
späteres Speichern.

Bearbeiten von Nutzern mit Umlauten im Namen gefixt

HTML.url() nimmt Perls interne Stringrepräsentation und ersetzt darin
die Umlaute. In UTF-8-Installationen muss das Ersetzen aber auf einer
UTF-8-Repräsentation erfolgen (so wie es Form::escape() auch macht).

Zweiter Teil des Fixes für Bug 1606.

Hashen der Passwörter mittels Digest::SHA1 (Fallback auf Digest::MD5)

Teil eines Fixes für Bug 1606. Hintergrund ist, dass bei Nutzernamen
mit Umlaut als zweitem Char crypt() ungültiges UTF-8 erstellt: Crypt
nimmt die ersten zwei Bytes (nicht Zeichen) einer Zeichenfolge und
stellt diese als Salt dem Ergebnis vorweg. Das führt zu einem
ungültigen UTF-8-Char, falls das erste Zeichen ASCII 7 bit und das
zweite Zeichen UTF-8 ist. Beispiel: "möp"

Legacy-Interface: Auth::authenticate unterstützt "is_crypted" nicht mehr

Sowohl Standard-DB-Handle als auch Auth-DB-Handle am Ende des Requests schließen

Bei Auth-DB-Verbindung nur Transaktionen starten, wenn tatsächlich Dinge verändert werden; dann explizit locken

Fix für Bug 1565.

Hochgeladene Dateien nicht mittels _recode_recursively umcodieren

Sortierfunktionen für Buchungsgruppen-Model

options_for_select: Default "0" ermöglichen

Hmpf! Wieso hat den Schwachsinn noch keiner bemerkt?

Namen lx-erp.conf + authentication.pl

Merge branch 'master' of lx-office.linet-services.de:lx-office-erp