Zerschossener PHP-Code

Farlif

Hallo,

und schon kommt die nächste Frage um die Ecke ...

Ich habe für ein Kontaktformular (bei dem mir hier im Forum auch schon beträchtlich geholfen worden ist - vielen Dank!!) php in eine Seite eingebunden (via Plugin Exec-php) - der Code ist dann, aufgrund eines Theme-Wechsels, von WP zerschossen worden.
Ich habe das hier einsortiert, weil ich argwöhne, dass das Plugin etwas mit dem Zerschießen zu tun hat. Vielleicht falsch, dann Entschuldigung.

Worin äußert sich das Zerschießen?
Das Formular springt nach Absenden auf die index-Seite; die Informationen sind verloren, und die eMail wird auch nicht verschickt.

Die ganze Seite hängt schief in den Angeln!

Firefox findet bei der html-Validierung plötzlich 200 Fehler (!) - es wird irgendwas ganz Blödes sein, eine spitze Klammer nicht geschlossen oder so ... Ich finde es nicht.
Dummerweise habe ich keine Kopie gemacht, so dass ich die Datei durch ein Original ersetzen könnte ...

Jetzt suche und suche und suche und suche und suche ich den Fehler (und suche und suche), viereinhalb Stunden jetzt, und nun stehe ich - ach ja: Newbie - kurz vor dem Wahnsinn.

Findet evtl. jemand von euch mit klarem Blick den Fehler? Fehlt irgendwo ein Leerzeichen oder ein weißer Punkt ...?

Vielen, vielen Dank! Hier ist der Code -->

PHP

<?php
 
$Empfaenger = "web@spondeus.de"; 
 
 
if($_REQUEST['Send']) 
{     
if(empty($_REQUEST['Name']) || empty($_REQUEST['eMail']) || empty($_REQUEST['Telefon']) || empty($_REQUEST['Strasse']) || empty($_REQUEST['Ort'])) 
{ 
 
echo"Bitte gehen Sie <a href=\"javascript:history.back();\">zur&uuml;ck</a> und f&uuml;llen Sie die Adress-Felder aus, damit wir Sie informieren k&ouml;nnen."; 
 
} 
else
{ 
 
$Mailbetreff = "Via Sonnenstromrechner.de: "; 
 
$Mailbetreff .= $_REQUEST['Betreff']; 
 
 
 
$Trenner = md5(uniqid(time())); 
 
 
 
$Header = "From: ".$_REQUEST['Email']; 
 
$Header .= "\n"; 
 
$Header .= "MIME-Version: 1.0"; 
 
$Header .= "\n"; 
 
$Header .= "Content-Type: multipart/mixed; boundary=$Trenner"; 
 
$Header .= "\n\n"; 
 
$Header .= "This is a multi-part message in MIME format"; 
 
$Header .= "\n"; 
 
$Header .= "--$Trenner"; 
 
$Header .= "\n";  
 
$Header .= "Content-Type: text/plain";
$Header .= "\n";
$Header .= "Content-Transfer-Encoding: 8bit"; 
 
$Header .= "\n\n"; 
 
$Header .= "Sie haben von Sonnenstromrechner.de folgende Nachricht erhalten: \n\n"; 
 
 
 
while(list($Formularfeld, $Wert) = each($_REQUEST)) 
 
{ 
 
if($Formularfeld != "Send") 
 
{ 
 
$Header .= $Formularfeld.": ".$Wert."\n"; 
 
} 
 
} 
 
 
 
$Header .= "\nDatum/Zeit: "; 
 
$Header .= date("d.m.Y H:i:s"); 
$Header .= "\n";
$Header .= "--$Trenner";        
 
$Header .= "\n"; 
 
$Header .= "Content-Type: text/mail; utf-8"; 
 
$Header .= "; name="; 
 
$Header .= "\n"; 
 
$Header .= "--$Trenner--";  
 
mail($Empfaenger, $Mailbetreff, "", $Header); 
 
echo"Vielen Dank f&uuml;r Ihre Anfrage! Sie wurde &uuml;bermittelt. Die von unserem Partner Quotatis benachrichtigten Fachleute werden sich so bald wie m&ouml;glich mit Ihnen in Verbindung setzen: <br />
Wir w&uuml;nschen Ihnen viel Erfolg bei der Umsetzung Ihres Projekts Solar-Anlage!"; 
 
} 
} 
else
{ 
?> 
<table border="0" cellpadding="1" cellspacing="2"> 
<form action="<?php echo$_SERVER['PHP_SELF']; ?>" enctype="multipart/form-data" method="post"> 
<tr>
Betreff:<br /></tr> 
 
<tr>
<select name="Betreff" size="1"><option>...</option><option>Beratung Solarstrom</option><option>Beratung Solarthermie</option><option>Heizungs-Beratung</option><option>Beratung Fenster/Hausdichtung</option></select>
<br /></tr>
<br />
<tr>
Bitte geben Sie Ihr Bundesland ein:<br /></tr> 
<tr><select name="Bundesland" size="1" style="width: 300px"><option>...</option><option>Baden-W&uuml;rttemberg</option><option>Bayern</option><option>Berlin</option><option>Brandenburg</option><option>Bremen</option><option>Hamburg</option><option>Hessen</option><option>Mecklenburg-Vorpommern</option><option>Niedersachsen</option><option>Nordrhein-Westfalen</option><option>Rheinland-Pfalz</option><option>Saarland</option><option>Sachsen</option><option>Sachsen-Anhalt</option><option>Schleswig-Holstein</option><option>Th&uuml;ringen</option></select>
<br />
</tr>
<br />
<tr>
Wohin ist das Dach ausgerichtet?<br /></tr> 
<tr><select name="Dachausrichtung" size="1"><option>...</option><option>Nordwesten</option><option>Westen</option><option>S&uuml;dwesten</option><option>S&uuml;den</option><option>S&uuml;dosten</option><option>Osten</option><option>Nordosten</option><option>Norden</option><option>Flachdach</option></select>    
<br /></tr> 
<br />
<br />
<tr>
Wieviele Quadratmeter Dach haben Sie f&uuml;r Ihre Sonnenstrom-Anlage zur Verf&uuml;gung? 
(Abz&uuml;glich Gauben, Lauffl&auml;chen, Fenster usw.) <br /></tr>
<tr>
<input name="Solaranlagen_Quadratmeter" size="50" type="text"> m<sup>2</sup><br /></tr><br />
<tr>
Wie groß ist die Dachfl&auml;che insgesamt? <br /></tr>
<tr>
<input name="Dachgesamt" size="50" type="text"> m<sup>2</sup><br /></tr>
<br /><tr>
Und um welche Dachform handelt es sich?<br /></tr>
<tr><select name="Dachform" size="1" ><option>...</option><option>Pultdach</option><option>Flachdach</option><option>Rundlaufendes Dach</option><option>Satteldach</option><option>Andere Dachform</option></select>
<br /></tr>
<br />
<tr>
Wie ist das Dach geneigt? <br />
(Ann&auml;herung gen&uuml;gt zun&auml;chst, falls Daten nicht zur Hand.)<br /></tr> 
<tr>
<tr><select name="Dachneigung" size="1" ><option>...</option><option>unter 25 Grad</option><option>25-30 Grad</option><option>31-35 Grad</option><option>36-45 Grad</option><option>46-55 Grad</option><option>56-65 Grad</option><option>66-75 Grad</option><option>&uuml;ber 75 Grad</option></select>
<br /></tr>
<br /> 
<tr>Mit welchem Material ist das Dach eingedeckt?
<br /><br /></tr>
<tr><select name="Dachdeckungsmaterial" size="1" ><option>...</option><option>Schiefer</option><option>Eternit</option><option>Tondachziegel/Pfanne</option><option>Faserzement</option><option>Profilblech</option><option>Betondachstein</option><option>Alu</option><option>Folie</option><option>Bitumenschindeln</option><option>Holzschindeln</option><option>Anderes Deckmaterial</option></select> 
<br /><tr>
<br /><br /> 
<tr>Vielen Dank! F&uuml;r eine Beratung zur <i>Solarthermie</i> brauchen wir <br />
bitte zusätzlich noch die beiden folgende Angaben -<br />
Baujahr des Hauses (nur zur Solarthermie-Beratung):<br /></tr>
<tr><input name="Hausbaujahr" size="55" type="text">
<br /></tr>
<br /><br />
<tr> 
Anzahl der Personen, die in dem Haushalt leben <br />
(Verbrauchsabsch&auml;tzung - nur Solarthermie-Beratung):<br />
</tr>
<tr><input name="Personenzahl" size="55" type="text">
<br /></tr>
<br /><br />
<tr> 
Vielen Dank. <br />
Wie k&ouml;nnen wir Sie per eMail erreichen?<br /><tr><br /> 
<tr>
</tr>
<tr>
<input name="eMail" size="55" type="text">
<br /></tr>
<br /><br />
<tr> 
Anrede:<br /></tr>
<tr>
<select name="Anrede" size="1" ><option>...</option><option>Frau</option><option>Herr</option><option>Dr.</option><option>Prof.</option></select>
<br /></tr>
<br /><br />
<tr>
Ihr Name (Vorname, Name bitte):<tr>
<input name="Name" size="55" type="text">
<br /></tr> 
<br /><br />
<tr>
Stra&szlig;e, Nr.:<br /></tr>
<tr>
<input name="Strasse" size="55" type="text">
<br /></tr>
<br /><br />
<tr>PLZ, Ort:<br /></tr>
<tr><input name="Ort" size="55" type="text"><br /></tr>
<br /><br />
<br />
<tr>
Telefonnr.:<br /></tr>
<br />
<tr>
<input name="Telefon" size="55" type="text">
<br /></tr>
<br /><br />
<tr>Haben Sie besondere Fragen, W&uuml;nsche, Anforderungen? <br />
Ihre Nachricht: <br /></tr>
<tr><textarea cols="40" name="Nachrichtentext" rows="5"></textarea>
<br /></tr>
<br /><br />
<tr>
"Bitte informieren Sie mich unverbindlich zu meiner geplanten Sonnenstrom-Anlage. Diese Anfrage ist f&uuml;r mich kostenlos."<br /></tr> <br />
<tr align="center" colspan="2"><input name="Send" type="submit" value="Absenden"><br /></tr>
<br /> <br />
<tr>Vielen Dank!</tr>
</form> 
</table> 
<?php 
} 
 
 
?>

Alles anzeigen

:confused:

Werbepost

Hallo!

Wenn du gerade an deiner Website arbeitest oder dein aktuelles Hosting überdenkst: Wir betreiben mit NetzLiving eine Hosting-Plattform, die speziell auf Performance, Sicherheit und einfache Verwaltung ausgelegt ist.

✔️ Schnelle Ladezeiten (optimiert für WordPress & Co.)
✔️ Deutsche Server & DSGVO-konform
✔️ Persönlicher Support (kein 0815-Ticket-System)

Mehr erfahren

Marcus[IS]

Hi Farlif,

ich habe den Code den du oben gepostet hast als Standalone php Seite einfach mal ausprobiert.

Also es scheint alles zu funktionieren, sonst würde ein parsing error, oder zumindest eine andere Fehlermeldung auftauchen.

Mir ist allerdings aufgefallen, das der Absender bei der verschickten Email leer bleibt.;)

Die Ursache muss also woanders liegen.

Ammaletu

Nur so als Tip: Du kannst mit cForms meines Wissens nach relativ bequem Formulare bauen und in Seiten einbetten. Wenn das Ergebnis eh nur per Mail verschickt werden soll, würde ich diesen Weg einer Eigenbau-Lösung mit phpExec in jedem Fall vorziehen.
http://www.deliciousdays.com/cforms-plugin/

Falls Du die jetzige Lösung behalten willst tue der Welt bitte einen Gefallen und maskiere/filtere die Nutzereingaben (insbesondere auf Zeilenumbrüche), ehe Du sie in die Mail-Header schreibst. Das hier z.B. ist eine Einladung, Spam zu verschicken:

PHP

$Header = "From: ".$_REQUEST['Email']; 
$Header .= "\n";

Was macht der Spammer da? Der schickt Dir einen Wert wie "mailadresse\nBCC: mailadresse1;mailadresse2;..." im From-Feld, et voila, Dein Server verschickt Spam. :-/

Genau aus solchen Gründen soll man das Rad auch nicht unnötig neu erfinden und lieber eine vorhandene Lösung nutzen. ;-)

Farlif

Hi Marcus, hi Ammaletu,

danke für eure Antworten, für Standalone-Test, Mühe und Link!

Zitat von Ammaletu

Genau aus solchen Gründen soll man das Rad auch nicht unnötig neu erfinden und lieber eine vorhandene Lösung nutzen. ;-)

Ja, vor allem, wenn die eigenen Räder irgendwie immer eckig werden ... :roll:
Werde mich also mal bei cForms umsehen.

Aber nur für den Fall, dass es dann evtl. doch leichter für mich Newbie ist, das alte Formular zu retten (und die Spammer zu stoppen ;) ) - am Anfang hat es ja geklappt .... jetzt nicht mehr ... also irgendeine andere php kaputt?
Ich habe nun Header, Footer, Single.php und page.php aus der aktuellen Installation mit den Erstdateien des Themes verglichen - da gab es keinen Unterschied. Die waren es also nicht.

Kann man so von außen sagen, was vielleicht noch kaputt sein könnte?
Oder soll ich es vielleicht doch lieber gleich ganz lassen? :???:
Danke!

Farlif

Tja, nochmal hallo,

und Entschuldigung für Doppelpost, aber lange Versuchsreihen haben ergeben: cForms war auch nicht die Lösung.
Nach der Aktivierung des PlugIns war der Blog erstmal nicht mehr erreichbar ...

Ich musste cForms extern über ftp umbenennen und damit deaktivieren, dann kam ich wieder an die Admin-Funktionen heran, habe die anderen plugins sämtlich deaktiviert und cForms wieder aktiviert - Fehlanzeige. Wieder abgestürzt.

Nun würde ich es also doch gerne über das oben zitierte halb-handgestricktes Formular machen. - Das aber, wie gesagt, einfach nicht läuft! Obwohl es ja, wie ihr richtig sagtet, eigentlich doch laufen sollte. Und die anderen phps im Editor scheinen ja auch in Ordnung zu sein.

Hat jemand irgendeine wilde Idee, wo ich noch nachgucken könnte, was das Formular so sehr ausbremst?

Danke euch!

Ammaletu

Ok, zum einen: Mach Dich damit vertraut, wo und wie auf Deinem Server PHP-Fehler geloggt werden. Spätestens wenn Du selbstgeschriebenen PHP-Code ausführst, solltest Du das wissen und das Log ab und an checken, zumindest am Anfang.

Wenn Du weißt, wo die Fehler geloggt werden, kannst Du genauso gut mal schauen, wieso cForms nicht geht. Da wird dann wohl eine Voraussetzung für das Plugin nicht stimmen, vielleicht kann man das ja ohne großen Aufwand beheben.

Zu Deinem eigenen Formular: Das ist jetzt nicht speziell verwunderlich, wenn Du Dir überlegst, wie WP intern funktioniert. Du rufst meinetwegen die statische Seite meinedomain.de/formularseite auf, aber i[COLOR=Black]ntern wird meinedomain.de/index.php?page_id=... aufgerufen. Wenn Du das Formular also auf [/COLOR][COLOR=#000000][COLOR=#dd0000][FONT=Verdana][COLOR=Black]PHP_SELF zeigen lässt, [/COLOR][/FONT][COLOR=Black]schickst Du es einfach an die index.php ab. Ohne weitere Parameter in der URL ist das einfach die Startseite. Klar, da wird dann nichts angezeigt und die Mail wird nicht verschickt.[/COLOR]

[COLOR=Black]Wenn Du das also beheben willst, ersetze das Formularziel durch den Namen der Seite. Du kannst das sicher auch mit WP-Mitteln dynamisch einsetzen (The_permalink() ?!), aber einfach[COLOR=Black]er ist es, das einfach fest reinzuschreiben.[/COLOR]

Und denk an den Schutz gegen Spamversand! Folgende Links helfen eventuell weiter dabei:
http://www.teris.de/workshop-nr1.php
http://www.heise.de/security/artik…lle-270758.html
[/COLOR]
[/COLOR][/COLOR]

Farlif

Danke nochmal! Das bringt mich wirklich weiter.
Farlif

Willkommen! Melden Sie sich an oder registrieren Sie sich.

Zerschossener PHP-Code

Jetzt mitmachen!