Bademeister: GET funktioniert nicht

Beitrag lesen

Hi Stefan,

Du solltest nicht vergessen, Deine uebergebenen Formulardaten zu pruefen, bevor Du sie der Funktion copy uebergibst.

Erstens ist es eleganter, den Fall abgefangen, in dem Nonsens eingegeben wird und das Kopieren nicht klappt. Dein Skript wuerde in dem Fall das Kopieren versuchen, scheitern und dem User dann sagen, die Datei sei erfolgreich erstellt worden worden. Gib diese Meldung nur dann aus, wenn Du geprueft hast, dass es auch stimmt.

Zweitens

der inhalt (sowie die datei heißen soll) soll via einem Formularfeld übergeben werden.

solltest Du daran denken, dass der User bei Deinem Skript in der Lage ist, nicht nur einen Dateinamen, sondern einen ganzen Pfad anzugeben, in dem die Datei gespeichert wird. Du musst also unbedingt pruefen, ob der angegebene Name nur ein Dateiname oder ein Pfad mit Verzeichnissen ist.
Gegebenenfalls auch noch (der Zweck des ganzen Skripts ist mir ueberhaupt nicht klar), ob die angegebene Datei schon existiert, weil jemand anders sie bereits eingegeben hat.

Falls Du das Skript nur lokal selbst benutzt (wofuer auch immer), wuerde ich Dir immer noch zu obigem raten. Es kann auch nicht schaden, sich vor sich selbst zu schuetzen.

Viele Gruesse,
der Bademeister