Revert "Webdav-Komponente: Das Erstellen des Verzeichnisses ausgelagert. Ferner den...
authorJan Büren <jan@lx-office-hosting.de>
Fri, 6 May 2011 16:52:28 +0000 (18:52 +0200)
committerJan Büren <jan@lx-office-hosting.de>
Fri, 6 May 2011 16:52:28 +0000 (18:52 +0200)
commit6e56204df612176aa0300243483746f1b04c0a7c
tree583be6e90f3888d8907a58b4a99d272e43e6432a
parent66ddc388f22a78ff14f7705a0d3d601e81440e04
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.
SL/Common.pm
SL/Form.pm