<?php
// Diese Zuweisung ist zwar nett, aber was ist wenn $_GET['id'] nicht gesetzt ist? Dann wird $id einfach auf NULL gesetzt und weitergemacht. Aber hier sollte doch dann alles weitere abgebrochen werden und eine Fehlermeldung angezeigt werden. Es macht ja keinen Sinn mit einer $id weiterzuarbeiten die NULL ist.
$id = isset($_GET['id']) ? intval($_GET['id']) : NULL;
// Hier generell die Frage: Wieso hast du auf Seite 1 alle Daten in hidden-Feldern und holst sie hier wieder aus dem POST-Array. Du hast die Daten ja auch in der Datenbank in der Tabelle piloten_bewerbung. Ich würde die Daten auf Seite 1 nur anzeigen (ohne hidden-Feld). Das Comment Feld als Textarea natürlich drinnen lassen. Und die ID dann entweder über ein hidden-Feld per POST oder über die URL per GET (Ich weiß nicht wie deine Seite 1 momentan aussieht).
$pilot_id=$_POST["pilot_id"];
$vorname=$_POST["vorname"];
$nachname=$_POST["nachname"];
$mail=$_POST["mail"];
$passwort=md5($_POST["passwort"]);
$passwort2=($_POST["passwort"]);
$postleitzahl=$_POST["plz"];
$ort=$_POST["ort"];
$heimatflughafen=$_POST["heimatflughafen"];
$letzter_standort=$_POST["basis"];
$ivaoid=$_POST["ivaoid"];
$vatsimid=$_POST["vatsimid"];
$other=$_POST["other"];
$reg_datum=$_POST["reg_datum"];
$geburtstag=$_POST["geburtstag"];
//Für die Mail musst du den Comment eigentlich nicht escapen. Nur wenn du ihn in der Datenbank speichern musst. Sonst sind nachher in der Mail hässliche Escape-Zeichen.
$mailinput = mysql_real_escape_string($_POST['reject_comment']);
// Ist denn der Pilot überhaupt schon in der Tabelle piloten? Ich würde hier nur die Tabelle piloten_bewerbung nehmen. Ich verstehe es so, dass sich ein Pilot bewirbt und damit in die Tabelle piloten_bewerbung kommt. Nach der Aktion des Admins kommt dann der Pilot aus der Tabelle piloten_bewerbung raus und wird in die Tabelle piloten geschoben. Das LIMIT 1 kannste dir auch schenken, da du hoffentlich keine doppelten IDs vergibst.
$abfrage = "SELECT
`piloten_bewerbung`.`id`,
`piloten_bewerbung`.`vorname`,
`piloten_bewerbung`.`nachname`,
`piloten_bewerbung`.`mail`,
`piloten_bewerbung`.`passwort`,
`piloten_bewerbung`.`plz`,
`piloten_bewerbung`.`ort`,
`piloten_bewerbung`.`heimatflughafen`,
`piloten_bewerbung`.`basis`,
`piloten_bewerbung`.`ivaoid`,
`piloten_bewerbung`.`vatsimid`,
`piloten_bewerbung`.`other`,
`piloten_bewerbung`.`reg_datum`,
`piloten_bewerbung`.`geburtstag`,
`piloten`.`pilot_id`,
`piloten`.`passwort`,
`piloten`.`vorname`,
`piloten`.`nachname`,
`piloten`.`geburtstag`,
`piloten`.`strasse`,
`piloten`.`postleitzahl`,
`piloten`.`ort`,
`piloten`.`mail`,
`piloten`.`ivaoid`,
`piloten`.`vatsimid`,
`piloten`.`heimatflughafen`,
`piloten`.`benutzergruppe`,
`piloten`.`letzter_standort`
FROM
piloten_bewerbung,piloten
Where `piloten_bewerbung`.`id` = $id
Limit 1";
$ergebnis = mysql_query($abfrage);
// Du solltest unbedingt mit { und } und mit Einrückungen arbeiten. Sonst erkennt man nachher gar nicht mehr welches Statement in welchem Fall ausgeführt wird.
if ($ergebnis)
// Eine Schleife über ein einzelnes Dataset macht auch wenig Sinn. Das Statement würde so reichen: $dataset = mysql_fetch_array($ergebnis);
while ($dataset = mysql_fetch_array($ergebnis))
if ($_POST['submit_button'] == "Add pilot") {
$abfrage2="INSERT INTO piloten
SET pilot_id='$pilot_id',
vorname='$vorname',
nachname='$nachname',
mail='$mail',
passwort='$passwort',
plz='$plz',
ort='$ort',
heimatflughafen='$heimatflughafen',
letzter_standort='$letzter_standort',
ivaoid='$ivaoid',
vatsimid='$vatsimid',
geburtstag='$geburtstag'";
$sender = "info@americanblue-va.com";
$empfaenger = "".$dataset['mail']."";
$betreff = "Your Application on AmericanBlue-VA";
$mailtext = "Dear $vorname $nachname,\n thank you for applicate on AmericanBlue-va.com.\n We have accept your application.\n\nYour Data:\n<b>Pilot ID:</b> $pilot_id \n<b>Password:</b> $passwort2\n\n With immediate effect you can login on http:\\www.americanblue-va.com\n\n The hole staff wish you a plasent time with us and the Community\n\n <b>Your AmericanBlue-Va Team</b>";
mail($empfaenger, $betreff, $mailtext, "From: $sender ");
print"<p>You add <blockquote>Pilot <b>$pilot_id</b>$vorname $nachname!<blockquote></p>";
}
else if ($_POST['submit_button'] == "Delete Pilot Application")
{
$loesch = mysql_query("DELETE FROM piloten_bewerbung WHERE id = '$id'");
$sender = "info@americanblue-va.com";
$empfaenger = "".$dataset['mail']."";
$betreff = "Your Application on AmericanBlue-VA";
$mailtext = "$mailinput";
mail($empfaenger, $betreff, $mailtext, "From: $sender ");
print "You has rejected this Application";
}
else {
print "There is an failure in the script";
}
?><br />