{"issue":{"state":"open","created_at":"2011-11-29T20:41:24Z","updated_at":"2011-11-29T20:41:26Z","user":{"gravatar_id":"256406243346b9253f8610fb1d1e05f6","id":1215373,"url":"https://api.github.com/users/importer","avatar_url":"https://secure.gravatar.com/avatar/256406243346b9253f8610fb1d1e05f6?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"importer"},"comments":2,"body":"Die Funktion processFormData in der Form.php verarbeitet sämtliche Dateien, die unter $_SESSION['FILES'] zu finden sind. Befinden sich in der Session noch \"alte\" Einträge, so führt dies zu Seiteneffekten:\n1. falsche Uploads werden an eine zu versendene E-Mail angehängt\n2. Beim speichern der Formulardaten in der DB werden entweder \n   a) falsche Uplaods mit abgespeichert oder\n   b) das Script bricht mit einer Exception ab, da für die Uploads gar kein Datenbankfeld vorhanden ist.\n\nIch würde vorschlagen, dass in der Session bei jedem FILE auch die Id des Formulars mit abgespeichert wird. So kann man processFormData auch nur die Uploads berücksichtigen, die vom eigenen Formular kommen.\n\n--- _Originally created by Thraile on April 12th, 2011, at 09:26am (ID 3007)_","assignee":{"gravatar_id":"87d2e5aefadcb310f9155df28c2b3582","id":1192057,"url":"https://api.github.com/users/leofeyer","avatar_url":"https://secure.gravatar.com/avatar/87d2e5aefadcb310f9155df28c2b3582?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"leofeyer"},"id":2392576,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"labels":[{"name":"Defect","color":"bd2c00","url":"https://api.github.com/repos/contao/import/labels/Defect"},{"name":"frontend","color":"e6e6e6","url":"https://api.github.com/repos/contao/import/labels/frontend"}],"number":3007,"title":"processFormData verarbeitet Uploads nicht richtig","closed_at":null,"html_url":"https://github.com/contao/import/issues/3007","url":"https://api.github.com/repos/contao/import/issues/3007","milestone":{"state":"open","created_at":"2011-11-29T15:29:54Z","closed_issues":112,"due_on":null,"creator":{"gravatar_id":"87d2e5aefadcb310f9155df28c2b3582","id":1192057,"url":"https://api.github.com/users/leofeyer","avatar_url":"https://secure.gravatar.com/avatar/87d2e5aefadcb310f9155df28c2b3582?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"leofeyer"},"number":32,"description":"Contao 2.10.0 (minor release)","title":"2.10.0","open_issues":1,"url":"https://api.github.com/repos/contao/import/milestones/32"}},"action":"created","comment":{"created_at":"2011-11-29T20:41:26Z","updated_at":"2011-11-29T20:41:26Z","user":{"gravatar_id":"256406243346b9253f8610fb1d1e05f6","id":1215373,"url":"https://api.github.com/users/importer","avatar_url":"https://secure.gravatar.com/avatar/256406243346b9253f8610fb1d1e05f6?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"importer"},"body":"Ja, oder aber irgend ein anderes Feld verhindert die Validierung des Formulars und man verlässt die Seite. Dann bleiben die Daten in $_SESSION['FILES'] erhalten.\n\nBeispiel:\nBei einem Formular wurde das Pflichtfeld Kundennummer nicht ausgefüllt und das Formular validiert nicht. Der Beispieluser ist blöd und hat keine Ahnung, woher er diese Nummer nehmen soll. Da entdeckt er das Suchfeld und denk sich prima, hier finde ich bestimmt meine Antwort -> schon haben wir Daten in $_SESSION['FILES'] und wenn man Suchanfragen in der Datenbank speichert, fällt einem genau jetzt auf die Füße.\n\n--- _Originally created by Thraile on April 12th, 2011, at 11:02am_","id":2946990,"url":"https://api.github.com/repos/contao/import/issues/comments/2946990"}}