✔ Problem bei Übergabe der Textarea
- Themenstarter Malchor
- Beginndatum
DeeJTwoK
Erfahrenes Mitglied
OK, here we go:
Also ich denke das sind eine ganze Menge Kommentare, Tips, Gedankenanstöße und Fragen mit denen sich sein Code in vielen Aspekten verbessern lässt.
Wenn du eine neue Version und immer noch Probleme hast, kannst sie sie hier posten.
Gruß
PHP:
<?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))
// Wenn das adden neuer Piloten nicht klappt, dann würde ich versuchen mit echo-Befehlen herauszufinden, wo denn was ausgeführt wird und was in den Variablen drin steht. Hier könntest du also folgenden Befehl einbauen:
echo $_POST['submit_button'];
// Dann siehst du auch, was hier drin steht und warum er evtl. nicht in den IF-Teil, sondern in den ELSE-Teil springt
if ($_POST['submit_button'] == "Add pilot") {
// Wo ist eigentlich der Unterschied zwischen $id und $pilot_id?
// Das Passwort sollte man möglichst nicht im Klartext speichern, sondern vorher mit einer Hashfunktion "unlesbar" machen:
// http://php.net/manual/de/function.sha1.php
// http://www.webmasterpro.de/coding/article/php-sicherheit-passwoerter-sicher-speichern.html
$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";
// Warum hinten und vorne einen leeren String dranhängen?
$empfaenger = "".$dataset['mail']."";
$betreff = "Your Application on AmericanBlue-VA";
// Das Englisch ist sicher auch verbesserungswürdig ;)
$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")
{
// Eine generelle Frage/Tip: Die $id ist ja wahrscheinlich eine Zahl (Integer), dafür gibt es in MySQL einen eigenen Datentyp: INTEGER (im Gegensatz zu VARCHAR). Den solltest du dann auch nutzen, dann würden bei der Query auch die ' um die $id wegfallen.
$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 />Also ich denke das sind eine ganze Menge Kommentare, Tips, Gedankenanstöße und Fragen mit denen sich sein Code in vielen Aspekten verbessern lässt.
Wenn du eine neue Version und immer noch Probleme hast, kannst sie sie hier posten.
Gruß
So ich habe mir das ganze mal angeschaut, auch schon was geändert, und einfügen tut er es immer noch nicht, aber ich glaube ich habe auch vergessen was umzuschreiben, bin aber gerade nicht sicher.
Ich habe den Quellcode umm meine Kommentare ergänzt und damit man meine Kommentare schneller von deinen trennen kann habe ich vor meine immer //** gesetzt
Ich habe den Quellcode umm meine Kommentare ergänzt und damit man meine Kommentare schneller von deinen trennen kann habe ich vor meine immer //** gesetzt
PHP:
<?php
// **Diese Seite wird nur dann ausgeführt wenn überhaupt ein Eintragauswählbar ist
$id = isset($_GET['id']) ? intval($_GET['id']) : NULL;
// **wichtige Variablen für die folgenden Funktionen Bestätigung und Löschen
$passwort=md5($_POST["passwort"]); //** PW wird verwaschen
$passwort2=($_POST["passwort"]); //** PW als Klaranzeige für die Bestätigungsmail
$mailinput = ($_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.
// **Limit 1 gesetzt weil er beim löschen das Echo 4 mal wiedergab!
//**Die Vermutung ist genau das ist der Vorgang, den ich dieses Formular ausführen sollte.
//**Die Spalten piloten wollte ich gleich mit ansprechen, um sie später nicht mehr ansprechen zu müssen
$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.
//** Verständnis Problem, wieso soll ich diese Klammern schreiben, müsste doch auch ohne gehen
if ($ergebnis)
$dataset = mysql_fetch_array($ergebnis);
echo $_POST['submit_button'];
// **Er schmeißt als Result nur Submit Button aus mit der Else Echo Ausgabe
if ($_POST['submit_button'] == "Add pilot") {
// Wo ist eigentlich der Unterschied zwischen $id und $pilot_id?
//** ID ist eine Autocountspalte aus Bewerbung
// **PW verwasch Funktion war oben bereits in der Variablen Deklaration geplant
$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";
// Warum hinten und vorne einen leeren String dranhängen?
//***Verstehe nicht was du meinst
$empfaenger = "".$dataset['mail']."";
$betreff = "Your Application on AmericanBlue-VA";
// Das Englisch ist sicher auch verbesserungswürdig ;)
$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")
{
// **ID ist bereits ein Integer daher habe ich die '' entfernt!
$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";
}
?>DeeJTwoK
Erfahrenes Mitglied
Und meine Kommentare mit //++:
Poste nachdem du den Code verbessert hast auch mal die Seite 1 mit dem Formular.
UND: Ich rate dir ganz stark, die ganzen hidden-Felder zu löschen. Du hast ja alle Daten in der Datenbank piloten_bewerbung. Da kannst du mit der $id ja alles rausholen!
PHP:
<?php
// **Diese Seite wird nur dann ausgeführt wenn überhaupt ein Eintragauswählbar ist
// ++Normalerweise schon, aber was, wenn jemand die Seite manuell aufruft (in den Browser tippt)? Man sollte auf alles gefasst sein. Also entweder du gehst davon aus, dass $_GET['id'] IMMER gesetzt ist, dann reicht auch
$id = intval($_GET['id']);
// ++oder du gehst davon aus, dass $_GET['id'] evtl. nicht gesetzt ist, dann würde ich abbrechen:
if (isset($_GET['id'])) {
$id = intval($_GET['id']);
} else {
exit("Keine ID vorhanden"); // oder schöner formatiert
}
// ++dein Code: $id = isset($_GET['id']) ? intval($_GET['id']) : NULL;
// **wichtige Variablen für die folgenden Funktionen Bestätigung und Löschen
$passwort=md5($_POST["passwort"]); //** PW wird verwaschen
$passwort2=($_POST["passwort"]); //** PW als Klaranzeige für die Bestätigungsmail
$mailinput = ($_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.
// **Limit 1 gesetzt weil er beim löschen das Echo 4 mal wiedergab!
// ++Dann würde ich mir aber Gedanken darüber machen, WARUM 4 echos kamen, weil du sonst einen Bug hast! Dein SQL Query macht nämlich nicht das, was du willst!
//**Die Vermutung ist genau das ist der Vorgang, den ich dieses Formular ausführen sollte.
//**Die Spalten piloten wollte ich gleich mit ansprechen, um sie später nicht mehr ansprechen zu müssen
// ++Das kapier ich nicht. Das SQL statement macht einen JOIN über beide Tabelle. Lass dir das Ergebnis dieses Queries mal in phpMyAdmin ausgeben. Die Query macht bestimmt nicht was du eigentlich willst.
$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.
//** Verständnis Problem, wieso soll ich diese Klammern schreiben, müsste doch auch ohne gehen
// ++Es mag funktionieren, es ist aber ganz schlechter Programmierstil. Ohne Klammern und Einrückungen macht man auch ganz schnell Fehler, die schwer zu finden sind. Ich würde fast sagen, dass ich mich weigere bei Quellcode weiterzuhelfen, der nicht ordentlich geklammert und eingerückt ist******
if ($ergebnis)
$dataset = mysql_fetch_array($ergebnis);
echo $_POST['submit_button'];
// **Er schmeißt als Result nur Submit Button aus mit der Else Echo Ausgabe
// ++OK, dann scheint das nicht so zu funktionieren mit den zwei Buttons. Versuch mal den Buttons unterschiedliche Namen zu geben "submit_button_add" und "submit_button_delete". Dann kannst du hier mit
echo "ADD: --".$_POST['submit_button_add']."--<br>";
echo "DELETE: --".$_POST['submit_button_delete']."--";
// ++mal schauen, was da rauskommt, wenn du den einen oder den anderen Button drückst. Dann kannst du wahrscheinlich deine IF-Abfrage mit isset($_POST['submit_button_add']) machen
if ($_POST['submit_button'] == "Add pilot") {
// Wo ist eigentlich der Unterschied zwischen $id und $pilot_id?
//** ID ist eine Autocountspalte aus Bewerbung
// ++Ist der Pilot denn schon in der Pilot Tabelle, wenn er sich nur beworben hat, aber noch nicht besätigt wurde? Ich vermute nein, deswegen ist auch deine SQL Query von oben (SELECT...) falsch. Die $pilot_id ist an dieser Stelle wahrscheinlich überhaupt nicht verfügbar oder?
// **PW verwasch Funktion war oben bereits in der Variablen Deklaration geplant
$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";
// Warum hinten und vorne einen leeren String dranhängen?
//***Verstehe nicht was du meinst
// ++ Du hängst hier einen Leerstring "" vor und hinter die E-Mail-Adresse. Warum? Das sollte reichen: $empfaenger = $dataset['mail'];
$empfaenger = "".$dataset['mail']."";
$betreff = "Your Application on AmericanBlue-VA";
// Das Englisch ist sicher auch verbesserungswürdig ;)
$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")
{
// **ID ist bereits ein Integer daher habe ich die '' entfernt!
$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";
}
?>Poste nachdem du den Code verbessert hast auch mal die Seite 1 mit dem Formular.
UND: Ich rate dir ganz stark, die ganzen hidden-Felder zu löschen. Du hast ja alle Daten in der Datenbank piloten_bewerbung. Da kannst du mit der $id ja alles rausholen!
Zuletzt bearbeitet:
So ich habe das mal angepasst, aber irgendwo ist mal wieder ein Denkfehler!
Fehlermeldung auf Webseite:
Die Skripte:
Admin_pilot_anfragen2.php
Admin_pilot_anfragen_bearbeiten.php
Für das letzte Skript:
Zeile 144: $passwort=md5($_POST["passwort"]); //** PW wird verwaschen
Zeile145: $passwort2=($_POST["passwort"]); //** PW als Klaranzeige für die Bestätigungsmail
Zeile180: echo "ADD: --".$_POST['submit_button_add']."--<br>";
Zeile181: echo "DELETE: --".$_POST['submit_button_delete']."--";
Fehlermeldung auf Webseite:
Die Skripte:
Admin_pilot_anfragen2.php
PHP:
<?php
$id = isset($_GET['id']) ? $_GET['id'] : null;
$abfrage = "SELECT * FROM piloten_bewerbung Where `id` = '".$id."'";
$ergebnis = mysql_query($abfrage);
$dataset = mysql_fetch_array($ergebnis);
$num = mysql_Num_Rows ($ergebnis);
$nachname = ($dataset['nachname']);
$vorname = ($dataset['vorname']);
$mail = ($dataset['mail']);
$passwort = ($dataset['passwort']);
$plz = ($dataset['plz']);
$ort = ($dataset['ort']);
$heimatflughafen = ($dataset['heimatflughafen']);
$basis = ($dataset['basis']);
$ivaoid = ($dataset['ivaoid']);
$vatsimid = ($dataset['vatsimid']);
$other = ($dataset['other']);
$reg_datum = ($dataset['reg_datum']);
$geburtstag = ($dataset['geburtstag']);
if ($num != 0):
?>
<FORM ACTION="Admin_pilot_anfragen_bearbeiten.php?id=<?php print $id; ?>" METHOD="post">
<table cellspacing="2" cellpadding="1" width="88%" class="tableLine" align="center">
<TR bgcolor="#B6B6B6">
<TD width="30%"><div align="left"><strong> Pilot ID:</strong></div></TD>
<TD width="19%"><div align="left">
<INPUT TYPE="text" NAME="pilot_id" VALUE="<?php echo mysql_result($ergebnis, 0, "pilot_id");?>"
SIZE="4" MAXLENGTH="8">
<TD width="29%"><div align="left"><strong>Date of Birth:</strong></div></TD>
<TD width="22%"> <?php echo "$geburtstag" ?></TD>
</TR>
<TR bgcolor="#666666">
<TD width="30%"><div align="left"><strong> First Name:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$vorname" ?></div></TD>
<TD width="29%"><div align="left"><strong> Last Name:</strong></div></TD>
<TD width="22%"><div align="left"><?php echo "$nachname" ?></div>
</TD>
</TR>
<TR bgcolor="#B6B6B6">
<TD width="30%"><div align="left"><strong> E-Mail Addy:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$mail" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td><strong> Password:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$passwort" ?></div></TD>
</TR>
<TR bgcolor="#666666">
<TD width="30%"><div align="left"><strong> IVAO ID:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$ivaoid" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td><strong> Vatsim ID:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$vatsimid" ?></div></TD>
</TR>
<TR bgcolor="#B6B6B6">
<TD width="30%"><div align="left"><strong> Location:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$ort" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td><strong> Zip-Code:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$plz" ?></div></TD>
</TR>
<TR bgcolor="#666666">
<TD width="30%"><div align="left"><strong> Home Airport:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$heimatflughafen" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td width="196"><div align="left"><strong> First Location:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$basis" ?></div></TD>
</TR>
<TR bgcolor="#B6B6B6">
<TD><div align="left"><strong> Remarks:</strong></div></TD>
<TD><div align="left">
<?php echo "$other" ?></div></TD>
<TD><div align="left"><strong> Registrierungsdatum:</strong></div></TD>
<TD><div align="left"><?php echo "$reg_datum" ?></div></TD>
</TR>
<TR bgcolor="#666666">
<TD><div align="left"></div></TD>
<TD><div align="left"></div></TD>
<TD><div align="left"><strong>Comment for Rejecting:<br /><i>(Pls. edit for each Application)</i></strong></div></TD>
<TD><div align="left">
<textarea name="reject_comment" cols="40" rows="">Dear <?php echo "$vorname" ?> <?php echo "$nachname" ?>,
we are sorry, but we can not accept your Application.
For more Informations fell free to conact us on info@americanblue-va.com
Best Regards
Your AmericanBlue Support Team
</textarea>
</div></TD>
</TR>
<TR bgcolor="#FFF">
<TD></TD>
<TD><div align="center"><b>
<input type="submit" name="submit_button_add" value="Add Pilot">
</b></div></TD>
<TD><div align="center"><b>
<input type="submit" name="submit_button_delete" value="Delete Pilot Application">
</b></div> </TD>
<TD></TD>
</TR> </TABLE>
</FORM>
<?php else: echo "Eintrag nicht vorhanden";
endif;
mysql_freeresult($ergebnis);
mysql_close();
?>Admin_pilot_anfragen_bearbeiten.php
PHP:
<?php
// **Diese Seite wird nur dann ausgeführt wenn überhaupt ein Eintragauswählbar ist
if (isset($_GET['id'])) {
$id = intval($_GET['id']);
} else {
exit("Keine ID vorhanden");
}
// **wichtige Variablen für die folgenden Funktionen Bestätigung und Löschen
$pilot_id = ($_POST['pilot_id']);
$passwort=md5($_POST["passwort"]); //** PW wird verwaschen
$passwort2=($_POST["passwort"]); //** PW als Klaranzeige für die Bestätigungsmail
$mailinput = ($_POST['reject_comment']);
//**Nur noch ein JOIN ein piloten_bewerbung!
$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`
FROM
piloten_bewerbung
Where `piloten_bewerbung`.`id` = $id";
$ergebnis = mysql_query($abfrage);
// ++Es mag funktionieren, es ist aber ganz schlechter Programmierstil. Ohne Klammern und Einrückungen macht man auch ganz schnell Fehler, die schwer zu finden sind. Ich würde fast sagen, dass ich mich weigere bei Quellcode weiterzuhelfen, der nicht ordentlich geklammert und eingerückt ist******
// Das mit dem Einrücken sollte kein Problem darstellen aber das Klammern schon
if ($ergebnis)
$dataset = mysql_fetch_array($ergebnis);
// **Er schmeißt als Result nur Submit Button aus mit der Else Echo Ausgabe
// ++OK, dann scheint das nicht so zu funktionieren mit den zwei Buttons. Versuch mal den Buttons unterschiedliche Namen zu geben "submit_button_add" und "submit_button_delete". Dann kannst du hier mit
echo "ADD: --".$_POST['submit_button_add']."--<br>";
echo "DELETE: --".$_POST['submit_button_delete']."--";
// ++mal schauen, was da rauskommt, wenn du den einen oder den anderen Button drückst. Dann kannst du wahrscheinlich deine IF-Abfrage mit isset($_POST['submit_button_add']) machen
if ($_POST['submit_button'] == "Add pilot") {
// Wo ist eigentlich der Unterschied zwischen $id und $pilot_id?
//** ID ist eine Autocountspalte aus Bewerbung
// ++Ist der Pilot denn schon in der Pilot Tabelle, wenn er sich nur beworben hat, aber noch nicht besätigt wurde? Ich vermute nein, deswegen ist auch deine SQL Query von oben (SELECT...) falsch. Die $pilot_id ist an dieser Stelle wahrscheinlich überhaupt nicht verfügbar oder?
// **PW verwasch Funktion war oben bereits in der Variablen Deklaration geplant
$abfrage2="INSERT INTO piloten
SET pilot_id='$pilot_id',
vorname='{$dataset['vorname']}',
nachname='{$dataset['nachname']}',
mail='{$dataset['mail']}',
passwort='$passwort',
plz='{$dataset['plz']}',
ort='{$dataset['ort']}',
heimatflughafen='{$dataset['heimatflughafen']}',
letzter_standort='{$dataset['letzter_standort']}',
ivaoid='{$dataset['ivaoid']}',
vatsimid='{$dataset['vatsimid']}',
geburtstag='{$dataset['geburtstag']}'";
$sender = "info@americanblue-va.com";
$empfaenger = $dataset['mail'];
$betreff = "Your Application on AmericanBlue-VA";
// Das Englisch ist sicher auch verbesserungswürdig ;)
$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")
{
// **ID ist bereits ein Integer daher habe ich die '' entfernt!
$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";
}
?>Für das letzte Skript:
Zeile 144: $passwort=md5($_POST["passwort"]); //** PW wird verwaschen
Zeile145: $passwort2=($_POST["passwort"]); //** PW als Klaranzeige für die Bestätigungsmail
Zeile180: echo "ADD: --".$_POST['submit_button_add']."--<br>";
Zeile181: echo "DELETE: --".$_POST['submit_button_delete']."--";
DeeJTwoK
Erfahrenes Mitglied
Hallo,
genau, du hast ja alle input-Felder aus der ersten Seite rausgenommen, daher sind die Daten natürlich nicht mehr per $_POST[] abgreifbar. Daher auch "Undefined index: passwort". Also: undefinierter Index 'passwort' in dem Array $_POST[], weil $_POST[] nur die Daten enthält, die über das form abgeschickt werden.
Lösung:
Diese Daten solltest du nicht mehr über $_POST[] holen, sondern aus der Datenbank, weil sie da ja eh drin stehen. Dann musst du natürlich die entsprechenden Statements HINTER die Datenbankabfrage setzen.
Bezüglich deines Problems mit den unterschiedlichen Aktionen (ADD, DELETE) und der Entscheidung welcher Button denn nun gedrückt worden ist: Ich habe mal einen kleinen Test bei mir gemacht:
post1.php:
post2.php:
Jetzt der ganz einfache Test: Was kommt raus, wenn ich welchen Button drücke:
ADD (submit_button):
DELETE (submit_button):
ADD (submit_button_add):
DELETE (submit_button_delete):
Also anhand dieser Ergebnisse sieht man eigentlich, dass es zwei Lösungen für dein Problem geben kann:
Eine andere Möglichkeit wäre auch (wenn es bei dir überhaupt nicht klappen will), zwei Formulare zu nehmen und auf zwei unterschiedliche Seiten weiterzuleiten.
genau, du hast ja alle input-Felder aus der ersten Seite rausgenommen, daher sind die Daten natürlich nicht mehr per $_POST[] abgreifbar. Daher auch "Undefined index: passwort". Also: undefinierter Index 'passwort' in dem Array $_POST[], weil $_POST[] nur die Daten enthält, die über das form abgeschickt werden.
Lösung:
Diese Daten solltest du nicht mehr über $_POST[] holen, sondern aus der Datenbank, weil sie da ja eh drin stehen. Dann musst du natürlich die entsprechenden Statements HINTER die Datenbankabfrage setzen.
Bezüglich deines Problems mit den unterschiedlichen Aktionen (ADD, DELETE) und der Entscheidung welcher Button denn nun gedrückt worden ist: Ich habe mal einen kleinen Test bei mir gemacht:
post1.php:
HTML:
<html>
<body>
<form action="post2.php" method="POST">
<input type="submit" name="submit_button" value="ADD (submit_button)">
<input type="submit" name="submit_button" value="DELETE (submit_button)">
<input type="submit" name="submit_button_add" value="ADD (submit_button_add)">
<input type="submit" name="submit_button_delete" value="DELETE (submit_button_delete)">
</form>
</body>
</html>post2.php:
PHP:
<?php
echo "<pre>";
echo "submit_button: -".$_POST['submit_button']."-\n";
echo "submit_button_add: -".$_POST['submit_button_add']."-\n";
echo "submit_button_delete: -".$_POST['submit_button_delete']."-\n\n\n";
echo var_dump($_POST);
echo "</pre>";
?>Jetzt der ganz einfache Test: Was kommt raus, wenn ich welchen Button drücke:
ADD (submit_button):
Code:
submit_button: -ADD (submit_button)-
submit_button_add: --
submit_button_delete: --
array(1) {
["submit_button"]=>
string(19) "ADD (submit_button)"
}DELETE (submit_button):
Code:
submit_button: -DELETE (submit_button)-
submit_button_add: --
submit_button_delete: --
array(1) {
["submit_button"]=>
string(22) "DELETE (submit_button)"
}ADD (submit_button_add):
Code:
submit_button: --
submit_button_add: -ADD (submit_button_add)-
submit_button_delete: --
array(1) {
["submit_button_add"]=>
string(23) "ADD (submit_button_add)"
}DELETE (submit_button_delete):
Code:
submit_button: --
submit_button_add: --
submit_button_delete: -DELETE (submit_button_delete)-
array(1) {
["submit_button_delete"]=>
string(29) "DELETE (submit_button_delete)"
}Also anhand dieser Ergebnisse sieht man eigentlich, dass es zwei Lösungen für dein Problem geben kann:
- Du nimmst zwei Buttons mit gleichem Namen. Auf der nächsten Seite ist dann unter diesem Name im POST-Array der entsprechende Wert des Buttons der gedrück wurde.
- Oder du nimmst zwei Buttons mit unterschiedlichen Namen. Auf der nächsten Seite musst du dann prüfen welcher von den beiden im POST-Array gesetzt ist (Inhalt hat).
Eine andere Möglichkeit wäre auch (wenn es bei dir überhaupt nicht klappen will), zwei Formulare zu nehmen und auf zwei unterschiedliche Seiten weiterzuleiten.
Ich habe das mal in beiden Skripten bearbeitet. Leider verstehe ich das jetzt gar icht mehr wirklich. Ich stoße an meine Grenzen im Verständnis!
Hier mal beide Datein sowie die Fehlermeldung.
piloten_anfrage2.php
piloten_anfrage_bearbeiten.php
Fehlermeldung:
Hier mal beide Datein sowie die Fehlermeldung.
piloten_anfrage2.php
PHP:
<?php
$id = isset($_GET['id']) ? $_GET['id'] : null;
$abfrage = "SELECT * FROM piloten_bewerbung Where `id` = '".$id."'";
$ergebnis = mysql_query($abfrage);
$dataset = mysql_fetch_array($ergebnis);
$num = mysql_Num_Rows ($ergebnis);
$nachname = ($dataset['nachname']);
$vorname = ($dataset['vorname']);
$mail = ($dataset['mail']);
$passwort = ($dataset['passwort']);
$plz = ($dataset['plz']);
$ort = ($dataset['ort']);
$heimatflughafen = ($dataset['heimatflughafen']);
$basis = ($dataset['basis']);
$ivaoid = ($dataset['ivaoid']);
$vatsimid = ($dataset['vatsimid']);
$other = ($dataset['other']);
$reg_datum = ($dataset['reg_datum']);
$geburtstag = ($dataset['geburtstag']);
if ($num != 0):
?>
<FORM ACTION="Admin_pilot_anfragen_bearbeiten.php?id=<?php print $id; ?>" METHOD="post">
<table cellspacing="2" cellpadding="1" width="88%" class="tableLine" align="center">
<TR bgcolor="#B6B6B6">
<TD width="30%"><div align="left"><strong> Pilot ID:</strong></div></TD>
<TD width="19%"><div align="left">
<INPUT TYPE="text" NAME="pilot_id" VALUE="<?php echo mysql_result($ergebnis, 0, "pilot_id");?>"
SIZE="4" MAXLENGTH="8">
<TD width="29%"><div align="left"><strong>Date of Birth:</strong></div></TD>
<TD width="22%"> <?php echo "$geburtstag" ?></TD>
</TR>
<TR bgcolor="#666666">
<TD width="30%"><div align="left"><strong> First Name:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$vorname" ?></div></TD>
<TD width="29%"><div align="left"><strong> Last Name:</strong></div></TD>
<TD width="22%"><div align="left"><?php echo "$nachname" ?></div>
</TD>
</TR>
<TR bgcolor="#B6B6B6">
<TD width="30%"><div align="left"><strong> E-Mail Addy:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$mail" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td><strong> Password:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$passwort" ?></div></TD>
</TR>
<TR bgcolor="#666666">
<TD width="30%"><div align="left"><strong> IVAO ID:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$ivaoid" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td><strong> Vatsim ID:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$vatsimid" ?></div></TD>
</TR>
<TR bgcolor="#B6B6B6">
<TD width="30%"><div align="left"><strong> Location:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$ort" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td><strong> Zip-Code:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$plz" ?></div></TD>
</TR>
<TR bgcolor="#666666">
<TD width="30%"><div align="left"><strong> Home Airport:</strong></div></TD>
<TD width="19%"><div align="left">
<?php echo "$heimatflughafen" ?></div></TD>
<TD width="29%"><table cellspacing="0" cellpadding="0">
<tr>
<td></td>
<td width="196"><div align="left"><strong> First Location:</strong></td>
</tr>
</table></TD>
<TD width="22%"><div align="left"><?php echo "$basis" ?></div></TD>
</TR>
<TR bgcolor="#B6B6B6">
<TD><div align="left"><strong> Remarks:</strong></div></TD>
<TD><div align="left">
<?php echo "$other" ?></div></TD>
<TD><div align="left"><strong> Registrierungsdatum:</strong></div></TD>
<TD><div align="left"><?php echo "$reg_datum" ?></div></TD>
</TR>
<TR bgcolor="#666666">
<TD><div align="left"></div></TD>
<TD><div align="left"></div></TD>
<TD><div align="left"><strong>Comment for Rejecting:<br /><i>(Pls. edit for each Application)</i></strong></div></TD>
<TD><div align="left">
<textarea name="reject_comment" cols="40" rows="">Dear <?php echo "$vorname" ?> <?php echo "$nachname" ?>,
we are sorry, but we can not accept your Application.
For more Informations fell free to conact us on info@americanblue-va.com
Best Regards
Your AmericanBlue Support Team
</textarea>
</div></TD>
</TR>
<TR bgcolor="#FFF">
<TD>
</TD>
<TD><div align="center"><b>
<input type="submit" name="submit_button" value="ADD (submit_button)">
<input type="submit" name="submit_button_add" value="ADD (submit_button_add)">
</b></div>
</TD>
<TD><div align="center"><b>
<input type="submit" name="submit_button" value="DELETE (submit_button)">
<input type="submit" name="submit_button_delete" value="DELETE (submit_button_delete)">
</b></div>
</TD>
<TD>
</TD>
</TR>
</TABLE>
</FORM>
<?php else: echo "Eintrag nicht vorhanden";
endif;
mysql_freeresult($ergebnis);
mysql_close();
?>piloten_anfrage_bearbeiten.php
PHP:
<?php
// **Diese Seite wird nur dann ausgeführt wenn überhaupt ein Eintragauswählbar ist
if (isset($_GET['id'])) {
$id = intval($_GET['id']);
} else {
exit("Keine ID vorhanden");
}
// **wichtige Variablen für die folgenden Funktionen Bestätigung und Löschen
$pilot_id = ($_POST['pilot_id']);
$mailinput = ($_POST['reject_comment']);
//**Nur noch ein JOIN ein piloten_bewerbung!
$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`
FROM
piloten_bewerbung
Where `piloten_bewerbung`.`id` = $id";
$ergebnis = mysql_query($abfrage);
if ($ergebnis)
$dataset = mysql_fetch_array($ergebnis);
$passwort=md5($dataset['passwort']); //** PW wird verwaschen
$passwort2=($dataset['passwort']); //** PW als Klaranzeige für die Bestätigungsmail
echo "<pre>";
echo "submit_button: -".$_POST['submit_button']."-\n";
echo "submit_button_add: -".$_POST['submit_button_add']."-\n";
echo "submit_button_delete: -".$_POST['submit_button_delete']."-\n\n\n";
echo var_dump($_POST);
echo "</pre>";
// **Er schmeißt als Result nur Submit Button aus mit der Else Echo Ausgabe
// ++OK, dann scheint das nicht so zu funktionieren mit den zwei Buttons. Versuch mal den Buttons unterschiedliche Namen zu geben "submit_button_add" und "submit_button_delete". Dann kannst du hier mit
echo "ADD: --".$_POST['submit_button_add']."--<br>";
echo "DELETE: --".$_POST['submit_button_delete']."--";
// ++mal schauen, was da rauskommt, wenn du den einen oder den anderen Button drückst. Dann kannst du wahrscheinlich deine IF-Abfrage mit isset($_POST['submit_button_add']) machen
if ($_POST['submit_button'] == "Add pilot") {
// Wo ist eigentlich der Unterschied zwischen $id und $pilot_id?
//** ID ist eine Autocountspalte aus Bewerbung
// ++Ist der Pilot denn schon in der Pilot Tabelle, wenn er sich nur beworben hat, aber noch nicht besätigt wurde? Ich vermute nein, deswegen ist auch deine SQL Query von oben (SELECT...) falsch. Die $pilot_id ist an dieser Stelle wahrscheinlich überhaupt nicht verfügbar oder?
// **PW verwasch Funktion war oben bereits in der Variablen Deklaration geplant
$abfrage2="INSERT INTO piloten
SET pilot_id='$pilot_id',
vorname='{$dataset['vorname']}',
nachname='{$dataset['nachname']}',
mail='{$dataset['mail']}',
passwort='$passwort',
plz='{$dataset['plz']}',
ort='{$dataset['ort']}',
heimatflughafen='{$dataset['heimatflughafen']}',
letzter_standort='{$dataset['letzter_standort']}',
ivaoid='{$dataset['ivaoid']}',
vatsimid='{$dataset['vatsimid']}',
geburtstag='{$dataset['geburtstag']}'";
$sender = "info@americanblue-va.com";
$empfaenger = $dataset['mail'];
$betreff = "Your Application on AmericanBlue-VA";
// Das Englisch ist sicher auch verbesserungswürdig ;)
$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")
{
// **ID ist bereits ein Integer daher habe ich die '' entfernt!
$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";
}
?>Fehlermeldung:
Neue Beiträge
-
DataGrid Virtualisierung - Komischer Fehler beim scrollen- Letzte: AnnaBauer21
-
-
