Übertragung in Datenbank

[romeodadon]

Was mach ich den nun falsch?
Es hat immer Funktioniert, seit gestern trägt es nix mehr in de DB ein.

Hier der script:
CODE
<?php
// Verbindung zum Server
$link = mysql_connect('localhost', 'XXX', 'XXX');
if (!$link) {
   die('Verbindung nicht möglich : ' . mysql_error());
}

// benutze Datenbank
$db_selected = mysql_select_db('user', $link);
if (!$db_selected) {
   die ('Kann Datenbank nicht benutzen : ' . mysql_error());
}

// Variablen
$name= $_POST['name'];
$vorname= $_POST['vorname'];
$straße= $_POST['straße'];
$plz= $_POST['plz'];
$ort= $_POST['ort'];
$email= $_POST['email'];
$IP=$_SERVER['REMOTE_ADDR'];
$anrede=$_POST['anrede'];
$fuehrerschein=$_POST['fuehrerschein'];
$telefon=$_POST['telefon'];
$land=$_POST['land'];
$datum=date("Y.n.j");
$zeit=date("H:i:s");
$checkbox=$_POST['checkbox'];

// Autos
$auto1= $_POST['auto1'];
$auto2= $_POST['auto2'];
$auto3= $_POST['auto3'];


// Pflichtfelder ausgefüllt?
if (($checkbox =='')){
echo "Um sich anzumelden müssen Sie die AGB bestätigen.";}

if (($name == '') OR ($vorname == '') OR ($telefon == '') OR ($straße == '') OR ($plz =='')) {
  echo "Bitte füllen Sie alle Pflichtfelder korrekt aus!<br><br>";
  }

else {

function PruefeMail($email) {
if (eregi("^[a-z0-9]+([-_\.]?[a-z0-9])+@[a-z0-9]+([-_\.]?[a-z0-9])+\.[a-z]{2,4}", $email))
{ return TRUE; }
else
{ return FALSE; }
}

//Wenn korrekt, wird der Eintrag angelegt und eine Meldung ausgegeben

if (PruefeMail($email)){
 echo "Der eintrag ist erfolgt, vielen Dank.";
 $sqladd=("INSERT INTO xxx(name,vorname,straße,plz,ort,email,ip,auto1,auto2,auto3,anrede,fuehrerschein,telefon,land,datum,zeit,) VALUES
 ('$name','$vorname','$straße','$plz','$ort','$email','$IP','$auto1','$auto2','$auto3','$anrede','$fuehrerschein','$telefon','$land','$datum','$zeit')");
 mysql_query($sqladd);}
 else{
 echo "Emailadresse ist fehlerhaft"; }}
 
// <!-------Mail to Empfänger------->

mail($email,
    "BETREFF",
    "
Sehr geehrte/er $anrede $name $vorname    
Vielen Dank für Ihre Bewerbung.
Ihre Daten wurden erfolgreich in unserer Datenbank eingetragen.

Über den Status Ihrer Bewerbungsdaten werden wir Sie informieren.


Ihr Team", "From:info@domain.de");

///////////////////////////////////////////////////

?>


Verstehe den Fehler nicht, er gibt aus das er es eineragen hätte, dabei steht nix in der DB!

 

[Michael Bieri]

QUOTE (Only4yu @ Fr 12.1.2007, 12:26) Verstehe den Fehler nicht, er gibt aus das er es eineragen hätte, dabei steht nix in der DB!

Die Bestätigung wird immer angezeigt, egal was im Query danach passiert..

CODE echo "Der eintrag ist erfolgt, vielen Dank.";


Bitte sag uns, dass dieser Code nicht auf einer öffentlichen Webseite in Betrieb ist. Nimms nicht persönlich, aber sowas ist das ideale, schlechte Beispiel wie man sowas _NICHT_ tun sollte. So wie der Code jetzt dasteht wäre SQL-Injection möglich, man könnte das Bestätigungsmail einfachso an tausende wildfremde Empfänger senden (~SPAM), das Fehlerhandling fehlt etc..

Daher macht es in meinen Augen keinen Sinn, diesen Code zu "reparieren". Hier hilft eher ein Neuanfang.

-> php.net

 

[romeodadon]

ok, fehelr beseitigt !

DANKE!

nein der Code ist intern zur Verwaltung, das mit der mailbestätigung war ein test.
danke