Formular versenden

Bandit

Du bist unfassbar!!

Du hast "Richtig debuggen" immer noch ignoriert! Keinerlei Ausgaben von Variablen und das Error-Reporting immer noch nicht hochgedreht! Und gerade das Error-Reporting würde dir wichtige Informationen ausgeben. Aber da du eindeutig zu faul bist, selbst einfachste Dinge umzusetzen, werde ich den Teufel tun und dir den Fehler nennen. Dass du nur auf eine fertige Lösung wartest ohne auch nur einen Hauch von Eigeninitiative zu zeigen, beweist du wieder mal eindrucksvoll!

Spider67

Du willst es einfach nicht kapieren: Ich habe keine Ahnung was Debuggen bedeutet. Bitte lass mich mit deinem unnötigen Geschwätz in Ruhe. Ich bin auf dein Geschwafel nicht angewiesen. Kapiere das endlich, oder lass Dir meine Postings von jemanden erklären, der die deutsche Sprache kapiert. Das ist anscheinend bei Dir nicht der Fall. Bitte lass mich in Ruhe, ich werde sicher jemanden finden, der mir hilft. Von dir will ich keine Hilfe, du bist aber auch außerstande mir welche zu geben mit deinem eingebildeten Supermanngehabe.

Bandit

Du bist also nicht in der Lage, diese zwei Zeilen in das Script zu schreiben:

Code

error_reporting(-1);
ini_set('display_errors', true);

Wenn das so ist, dann solltest du besser die ganze Sache vergessen!

Und ganz nebenbei, du hast schon längst die Lösung bekommen, aber die hast du ja elegant ignoriert!

Zitat

Ich würde eher explode nehmen

Ich habe es erfolgreich durch explode ersetzt.

Siehe https://www.php.net/manual/de/function.explode.php

Du hast 2x einen Hinweis auf explode bekommen!!!

Hast du es jetzt endlich begriffen??

Spider67

Ok, jetzt habe ich die beiden Zeilen hineingeschrieben. Wenn ich nun das Formular versende, erhalte ich folgende Fehlermeldungen:

Code

Notice: Undefined index: email in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 45

Notice: Undefined index: vorname in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 45

Notice: Undefined index: nachname in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 45

Notice: Undefined index: name in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 45

Warning: preg_split(): No ending delimiter ',' found in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 67

Warning: Invalid argument supplied for foreach() in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 69

Warning: Cannot modify header information - headers already sent by (output started at /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php:45) in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 99

Alles anzeigen

Ich werde es jetzt auch noch mit explode versuchen.

Bandit

explode wird dir erst ab der Meldung mit preg_split helfen, die Notice-Meldung davor werden bleiben, da du keine Formularfelder mit den Namen email, vorname, nachname und name hast. Ich behaupte mal, dass dieses Script so noch nie richtig funktioniert hat.

Spider67

Also vor der php Version 7.0 hat das Script immer einwandfrei funktioniert und ich habe per Mail alle Daten und jeweils 2 Dateien erhalten, die die Absender mitgeschickt haben. Auch derzeit erhalte ich beim Versand des Formulars alle Daten einwandfrei, nur der Versand der Dateien (Format pdf oder Format jpeg) funktioniert nicht.

Code

Sie haben folgende Anmeldung erhalten:
Aufnahme_: 01.01.2020
Alter: über 14
Name: Muster Max
Geburtsdatum: 01.01.1980
Geburtsort: Irgendwo
Adresse: Musterstraße 1
Ort: Musterstadt
Staat: Aut
Beruf: Angeswtellter
Telefonnummer: 01234/2345
Mail: muster@aon.at
KFZ_Kennzeichen: Ho-123X
Handynummer: 0680/1234567
Erfahrungen: ja
Vergehen: nein
Prüfung: ja
Verein: nein
Mitglied: ja
Name_Verein: Muster
Datenschutz: ja
Datum: 01.01.1900
Kommentar: 
button: Senden

 Datei IMG_5629.JPG (Feld Fischerkarte) wurde wegen nicht zugelassener Dateiendung nicht uebertragen. Erlaubt waren nur jpg, gif, png, pdf, docx, doc, txt

Alles anzeigen

Hier sieht man die korrekt übermittelten Daten, die ich im Formular erfasst habe und die ich per Mail erhalte und die Fehlermeldung, die bezüglich der Datei erscheint. Ich habe eine jpeg Datei mit 1,6 MB übertragen und erhalte diesen Fehler???

Alle anderen Fehlermeldungen (notice und warning preg_split sowie das header warning) sind verschwunden, nachdem ich die Funktion split durch explode ersetzt habe und in Zeile 45 die Bezeichungen der Formularfelder durch die ID meiner Formularfelder ersetzt habe bzw.die von mir nicht verwendeten Formularfelder entfernt habe.

Es ist also nur mehr das Problem, dass die Datei nicht übertragen wird, obwohl ein definiertes Format in der definierten Größe verschickt wird.

Bandit

Poste das aktuelle Formular und Script

Spider67

Bitte hier das html

formular.txt

und das Script

script.txt

wird jemand schlau daraus (als Code kann ich es nicht posten - das ist zu lange)

Bandit

IMG_5629.JPG

$upload_extensions = 'jpg, gif, png, pdf, docx, doc, txt';

Da fällt dir nichts auf?

Spider67

Muss ich dann bei den erlaubten Dateiformaten Groß- und Kleinschreibung berücksichtigen, funktioniert es nicht wenn die Dateiendung groß geschrieben ist und die Definition im Script klein. Bei der ebenfalls getesteten pdf Datei war nämlich alles klein geschrieben.

Oder muss man den Punkt vor der Dateiendung auch im Script in der Definition anführen? Soll ich dann im Script sowohl groß als auch klein geschriebene Formate definieren.

Bandit

Tipp: https://www.php.net/manual/de/function.strtolower

Gastl

Zitat von Spider67

Bitte hier das html

formular.txt

und das Script

script.txt

wird jemand schlau daraus (als Code kann ich es nicht posten - das ist zu lange)

Ich sehe keinen Code.

@Bandit Ich bewundere deine Sturheit.

Allein danach, hätte ich die Tastatur ruhen lassen.

Zitat von Spider67

Bitte lass mich mit deinem unnötigen Geschwätz in Ruhe. Ich bin auf dein Geschwafel nicht angewiesen.

Guten Rutsch und ein gesundes 2020 allen.

Bandit

Zitat von Gastl

@Bandit Ich bewundere deine Sturheit.

Ebenfalls einen Guten Rutsch und ein gesundes 2020.

Spider67

Und wieder war es nicht möglich, meine Fragen einfach zu beantworten. Ich dachte, ich hätte die Fragen in verständlichem Deutsch gestellt. Dem war aber anscheinend nicht so.

Oder mein Verdacht bestätigt sich, dass jemand hier der deutschen Sprache nicht mächtig ist.

Trotzdem allen Prosit 2020.

Bandit

Mitdenken ist definitiv keine Stärke von dir, ganz im Gegenteil!

Ich denke dann mal für dich:

Man hat ein Problem mit Groß/Kleinschreibung von Dateinamen
Dabei spielt nur die Extension des Dateinamens eine Rolle
Ausnahmsweise dem Link folgen und lesen: strtolower — Setzt einen String in Kleinbuchstaben um
Grübel, Grübel und AHHHHHHH, man wandele den Dateinamen einfach in Kleinbuchstaben um und schon wird aus JPG jpg, was dann zu meiner Liste der Extensions passt.
Man beachte die Beispiele in der Doku und baue strtolower ein und schon ist das Problem gelöst

Und nun möge sich dein Rechner in Rauch auflösen, mir wird das jetzt doch zu doof mit deiner Bequemlichkeit und deiner Unfähigkeit mal mitzudenken! Lass es einfach sein mit Programmierung und Internetseiten, mit deiner merkwürdigen Einstellung wird das nichts. Außerdem hoffe ich, dass jetzt nicht irgendjemand dir das Script zurecht biegt und deine Faulheit am Ende doch noch siegt.

Spider67

Und warum wird wieder nicht auf die Tatsache eingegangen, dass ich auch versucht habe eine .pdf Datei zu übertragen, wo die Dateiendung auch hinsichtlich Kleinschreibung mit meinen definierten Dateien übereinstimmt, und wo trotzdem dieselbe Fehlermeldung erscheint?

Bandit

Code

if (strpos($upload_extensions, strtolower(substr($data['name'], -3))) !== false)

Spider67

Ist dies nun auch ein Code der erforderlich ist oder nicht???

Ich habe die Funktion nun hier eingefügt

PHP

<?php
/**
 * myFormMail.php generated from http://www.dreamweaver-buch.de
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the BSD License.
 * (Es wird keine Haftung fuer Fehler uebernommen)
 *
 * To contact the author write to {@link mailto:hussein@morsye.de Hussein Morsy}
 * The Script is generated from
 * {@link http://www.dreamweaver-buch.de}
 *
 * @author Hussein Morsy <hussein@morsy.de>
 * @version 1.11  31.05.2014
 * @license http://www.gnu.org/copyleft/lesser.html GNU Lesser General Public License
 * @copyright 2014 Hussein Morsy
 */
error_reporting(-1);
ini_set('display_errors', true); 
/* Generierte Eintraege. Koeenen auch von "Hand" geaendert werden */
$recipients        = 'spider67@aon.at';
$sender            = 'spider67@aon.at';
$subject           = 'Anmeldung ';
$introtext         = 'Sie haben folgende Anmeldung erhalten:';
$redirect          = 'danke.html';
$max_filesize      = '5000';
$upload_extensions = 'jpg, jpeg, gif, png, pdf, docx, doc, txt';
$max_files         = '2';

/* Formularfelder einlesen */
$inputs = &$_POST;

/*Großbuchstaben in Kleinbuchstaben umwandeln */
$str = "JPEG, JPG, GIF, PNG, PDF, DOCX, DOC, TXT";
$str = strtolower($str);
echo $str;

Alles anzeigen

Oder muss ich diese Funktion hier bei den Uploads einfügen?

Code

//Attachments hinzufuegen
  $count_uploaded = 0;
  foreach($_FILES as $name=>$data) {
    if(!empty($data['tmp_name'])) {
      if($count_uploaded < $max_files) {
        $count_uploaded++;
        $file = $data['tmp_name'];
        $filesize = round($data['size'] / 1024, 2);
        if($filesize <= $max_filesize) {
          // alle Punkte falls vorhanden entfernen
          $upload_extensions = str_replace('.','',$upload_extensions);
          $extensions = explode(',',$upload_extensions);
          $ext_found = false;
          foreach($extensions as $ext) {
            $ext = trim($ext);
            if ( strrchr($data['name'],'.') == ".$ext") {
              $mail->AddAttachment($file,"$name.$ext");
              $ext_found=true;
              break;
            }
          }
          if(!$ext_found) {
            $body .= "\r\n Datei $data[name] (Feld $name) wurde wegen nicht zugelassener Dateiendung nicht uebertragen. Erlaubt waren nur $upload_extensions \r\n";
          }
        }
        else {
          $body .= "\r\n Datei $data[name] (Feld $name) wurde wegen ueberschreitung der Dateigroesse $filesize nicht uebertragen Erlaubt waren nur $max_filesize\r\n";
        }

      }
      else {
        $body.="\r\nAnzahl ($max_files) erlaubter Dateiuploads ueberschritten.\r\n";
      }
    }
    else {
      # Es liegt keine Datei vor.
      #outputError( "Datei existiert nicht: " , $data['tmp_name']);
      #die();
    }
  }

Alles anzeigen

Bei einem Test mit der JPEG Datei wurden jedenfalls wieder alle eingegebenen Daten übertragen, wobei wieder die Fehlermeldung erschienen ist, dass die Datei wegen einer falschen Endung nicht übertragen werden konnte.

Code

$mail->Body = $body;
  //senden der EMail
  if($mail->Send()) {
    header("Location: $redirect");
  }else {
    outputError("Aufgrund eines Fehlers beim Senden konnten die Formulareingaben nicht per E-Mail &uuml;bertragen werden.<br>Error-Message:" . $mail->ErrorInfo);
  }
}else {
  outputError("Das Formular wurde nicht abgeschickt, da das Formular nicht ausgef&uuml;llt wurde.");

Weiters werden die strings korrekt in Kleinbuchstaben ausgegeben und es erscheint diese Fehlermeldung zu dem o.a. Teil des Scripts:

jpeg, jpg, gif, png, pdf, docx, doc, txt

Warning: Cannot modify header information - headers already sent by (output started at /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php:36) in /homepages/44/d291036264/htdocs/html/Anwendungen/myFormMail.php on line 103

Bandit

Zitat von Spider67

Oder muss ich diese Funktion hier bei den Uploads einfügen?

Und wieder verweigerst du das Mitdenken. Dieser Thread dient echt als gutes Beispiel dazu, wie sehr manche Leute besser die Finger vom Programmieren lassen sollten, weil das mit dem logischen Denken nicht wirklich funktioniert.

Ich schubse dich noch einmal in die richtige Richtung: wo liegt das Problem, an den vorgegeben Extension oder an den unbekannten Dateinamen vom Upload?

Und jetzt seh' zu, von mir hast du keinerlei Hilfe mehr zu erwarten. Ich weiss, in deinen Augen habe ich dir ja nicht geholfen, aber das liegt definitiv an dir, da du nicht mitdenkst.

Spider67

Ich denke, der Fehler liegt an den vorgegeben Extensions, nur wird halt leider dafür keine Lösung geboten. Im Gegenteil: es macht Dir Spaß, irgendwelches Kauderwelsch von Dir zu geben und in Rätseln zu sprechen, Darüber, dass Du nie konkret auf meine Fragen antwortest, darf man sich wirklich nicht mehr wundern. Dass ist halt schwer möglich, wenn man nicht sinnerfassend lesen kann.

Ich habe es echt satt, zu Deinem Spaß hier herumzuraten. Ich hoffe ganz einfach, dass hier in diesem wirklich guten Forum auch noch Leute mit Fachwissen sind, die mir echt weiterhelfen können und das auch wollen. Von dir habe ich echt die Nase voll.