anpera.net

Hallo zusammen,
ich habe mir grad was gebastelt, aber es will nicht zu 100% funktionieren.
Daher poste ich hier einfach nur die Stelle aus dem Script, die nicht klappt. Falls mehr aus dem Script gebraucht wird, bitte sagen, dann gibt's den Rest auch. Der Fehler besteht darin, dass die Datenbank nicht geupdatet wird... Anzeige ist einwandfrei, aber was zu ändern klappt nicht... Ein wenig Hilfe, wäre nicht schlecht. Danke schonmal im Vorraus!
[php]if ($_GET['op']=="edit"){
output("Hier kann der Bestand geändert werden.
`n`c`b`\$IN ARBEIT!!!`b`c`0");
addnav("Bestand ansehen","su-test.php");
addnav("Zurück zur Grotte","superuser.php");
//Beginn der Edit-Tabelle
if ($_GET['subop']=="save") {
$_POST['menge'] = addslashes(closetags(stripslashes($_POST['menge']),'`i`b`c`H'));
$_POST['ankauf'] = addslashes(closetags(stripslashes($_POST['ankauf']),'`i`b`c`H'));
$_POST['verkauf'] = addslashes(closetags(stripslashes($_POST['verkauf']),'`i`b`c`H'));
$sql = "UPDATE rohstofflager SET menge='".$_POST['menge']."',ankauf='".$_POST['ankauf']."',verkauf='".$_POST['verkauf']."' WHERE anzeigename='".$_GET['anzeigename']."'";
$result = db_query($sql) or die(db_error(LINK));
redirect("su-test.php?op=edit");
}else{
output("<form action='su-test.php?op=edit&subop=save&anzeigename=".$row['anzeigename']."' method='POST'>",true);
output("<table cellpadding=2 cellspacing=1 bgcolor='#000000' align='center'><tr class='trhead'><td align=\"center\" valign=\"middle\">`b`@Rohstoff`b</td><td align=\"center\" valign=\"middle\">`b`&Vorrat`b</td><td align=\"center\" valign=\"middle\">`b`RKaufpreis`b</td><td align=\"center\" valign=\"middle\">`b`ZVerkaufspreis`b</td></tr>",true);
$sql = "SELECT anzeigename,menge,ankauf,verkauf FROM rohstofflager ORDER BY anzeigename ASC";
$result = db_query($sql) or die(db_error(LINK));
if (db_num_rows($result)==0) {
output("<tr class='trdark'><td colspan=5 align='center'>`&`iEs gibt keine Rohstoffe`i`0</td></tr>",true);
}else{
$bgclass = '';
while ($row = db_fetch_assoc($result)) {
$bgclass = ($bgclass=='trdark'?'trlight':'trdark');
output("<tr class='$bgclass'>
<td align=\"center\" valign=\"middle\"><input name='anzeigename' value='".htmlentities($row['anzeigename'])."' type='hidden'>`@".$row['anzeigename']."</td>
<td align=\"center\" valign=\"middle\">`&<input name=menge' value='".htmlentities($row['menge'])."' maxlength='5' size='5'></td>
<td align=\"center\" valign=\"middle\">`R<input name='ankauf' value='".htmlentities($row['ankauf'])."' maxlength='5' size='5'> Gold</td>
<td align=\"center\" valign=\"middle\">`Z<input name='verkauf' value='".htmlentities($row['verkauf'])."' maxlength='5' size='5'> Gold</td></tr>",true);
}
}
output("</table>",true);
output("`n`n");
output("<input type='submit' class='button' value='Ändern'></form>",true);
addnav("","su-test.php?op=edit&subop=save&anzeigename=".$row['anzeigename']);
//Ende der Edit-Tabelle
}
}[/php]

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

1. Ersetze das $_HTTP_GET_VARS mit $_GET xD

2. Könnte es sein, dass der Anzeigename nicht übermittelt wird?

Hab ich, klappt nicht...Wird doch aber selected... Wird ja auch in der Tabelle ausgegeben...

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

Ja, ausgegeben wirds, aber eben nicht im FOrmualar weitergegeben...

ersetze:
$this->bbcode_second_pass_code('', '<td align="center" valign="middle">`@".$row['anzeigename']."</td>')
durch

$this->bbcode_second_pass_code('', '<td align="center" valign="middle"><input name='anzeigename' value='".htmlentities($row[anzeigename'])."' type='hidden'>`@".$row['anzeigename']."</td>')

Funktioniert leider auch nicht...

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

Du willst einen Eintrag in deiner DB Zeile vornehmen. Schau dir mal das an:
[php]
$sql = "UPDATE rohstofflager SET menge='".$_POST['menge']."',ankauf='".$_POST['ankauf']."',verkauf='".$_POST['verkauf']."' WHERE anzeigename='".$_GET['anzeigename']."'";
[/php]

Du übergibst doch garnicht den Anzeigenamen. Siehe dein "form", wo du scheinbar auswählst:
[php]
output("<form action=\"su-test.php?op=edit&subop=save\" method='POST'>",true);
[/php]

Also ich würde ersteinmal deine SQL-Abfrage machen, damit du deine Daten komplett zur verfühung hast. Erst dann würde ich mit dem Form anfangen. Gut wenn ich ehrlich bin, würde ich es ganz anders machen. Aber wie oben beschrieben erst die SQL-Abfrage und dann erst das "form" anfangen.

[php]
output("<form action='su-test.php?op=edit&subop=save&anzeigename=".$row['anzeigename']."' method='POST'>",true);
[/php]

und den addnav für den Cheatschutz:
[php]
addnav("","su-test.php?op=edit&subop=save&anzeigename=".$row['anzeigename']);
[/php]

Dann weiss er auch im oberen SQL-Befehl, was er denn Updaten soll.....

_________________
Das schwarze Schaf der LotGD-Community, 2. platzierter, beim Giga.de Homepage-Award 2007 und 1. platzierter beim German Web Award 2008.
PC-ACTION meint: "Sehr gut" für MotWD!
Mind of the White Dragon => http://my-logd.com/motwd

.................................. .... egal...

Also danke erstmal, für die Änderung, aber auch die hat leider nicht geklappt...

Also zur Zeit sind da insgesamt 10 Rohstoffe drin. Ich erklärs jetzt mal genauer:
Ich möchte nicht immer per DB die Preise, oder den Bestand ändern und wollte mir daher diesen "Editor" zusammenflicken.
Ich wollte aber auch nicht jeden Rohstoff EINZELN bearbeiten müssen, also für jeden Rohstoff einen eigenen save-button, sondern einen Save-Button, für die komplette Liste, die da angezeigt wird...

Vielleicht geht das ja auch einfach nicht so... Ich würde mich aber trotzdem über weitere Ideen/Anregungen sehr freuen!

Danke an alle, die daran mitwirken!

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

Poste einfach mal das ganze Script. Ich werde dann mal morgen im laufe des Tages drüberschauen.

_________________
Das schwarze Schaf der LotGD-Community, 2. platzierter, beim Giga.de Homepage-Award 2007 und 1. platzierter beim German Web Award 2008.
PC-ACTION meint: "Sehr gut" für MotWD!
Mind of the White Dragon => http://my-logd.com/motwd

Ich habe jetzt das ganze Script als Anhang gepostet reingepackt.

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

Beim schnellen Überfliegen, denke ich mal, dass Du was in der SQL vergessen hast, den Rest hab ich mir nicht wirklich angeguckt, ich gehe davon aus, das ist so wie du es eigentlich willst?

`listorder` int(10) unsigned NOT NULL default '1' fehlt noch in der Tabelle, meiner Meinung nach Falls nicht, ICQ

? Nee, die Tabelle im PMA ist schon ok, ich hab sie gestern abend nurschnell rauskopiert, damit die, die mir helfen wollen/können, auch nachvollziehen können, was da für ne tabelle abgefragt wird... Kann sein, dass ich da was der vollständigkeit halber nicht angehakt haben ich hab nur das create und die inserts kopiert... oder was meinst du grad?
(Ich hab doch garkein listorder?!)

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

Doch du benutzt listorder in diesem 'Editor', musste nur schauen...

Also, du wolltest ja alle aufeinmal Ändern. Dies funktioniert aber so nicht, da du deine $_POST immer wieder überschrieben würdest, anhand der gleichen Namen, die du verwemdest.

Ich habe das man schnell ein wenig umgebaut. Versuche mal, ob es geht. Ich habe es nicht getestet, aber es sollte gehen....

_________________
Das schwarze Schaf der LotGD-Community, 2. platzierter, beim Giga.de Homepage-Award 2007 und 1. platzierter beim German Web Award 2008.
PC-ACTION meint: "Sehr gut" für MotWD!
Mind of the White Dragon => http://my-logd.com/motwd

Tatsache... Entschuldigung... Ich hab die Grunidee aus dem su-jobs genommen und hab wohl irgendwie was falsch "kopiert". Mal anders testen...
Edit: Tatsache, völlig falsch... Wie wäre denn deine Berichtigungsvorschlag, für das "save"?
Nein, klappt leider nicht weil unexpected ; in Zeile 67 (hab es dann aus der 70 rausgenommen, weil wohl das gemeint war) und das nav "Bestand ändern" war unsichtbar gemacht und auf sichtbar ging's dann nicht mehr... Also das erste ist ja nur eine reine übersichtsseite, die einwandfrei ausgeführt wird. Die Änderung ist erst auf der zweiten Seite im "Edit"-Teil...

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy

Sorry hab noch etwas übersehen, was den Fehler wahrscheinlich verusacht hat.

Zeile 66:
[php]
if ($_POST){
//ändern in
if ($_POST != ""){
[/php]

Wenn du auf deiner Hauptseite bist, sollte nun ein Link stehen, wenn du SU 3 bist "Bestand Ändern" jeweils am Ende der Zeile. Dort klicken. Dann gehts in den Edit bereich. Wenn du dort nun änderst und speicherst, wrist du wieder auf die hauptseite geleitet und ein statustext angezeigt. Zeile 70 muss im übrigen auch reaus, hatte ich auch vergessen.

_________________
Das schwarze Schaf der LotGD-Community, 2. platzierter, beim Giga.de Homepage-Award 2007 und 1. platzierter beim German Web Award 2008.
PC-ACTION meint: "Sehr gut" für MotWD!
Mind of the White Dragon => http://my-logd.com/motwd

Hmmmm... Ja, das sieht schonmal viel besser aus, aber wenn ich nun Bestand ändere, wird's komplett geNULLt, das was geändert wird...
Also nirgendwo kommt was, wo man da was ändern könnte, bzw. man wird nicht gefragt, wieviel man haben will...
Vorher hatte ich ja eine Seite, auf der der Bestand angezeigt wurde und dann eine 2. Extra-Seite, wo man das dann hätte ändern können...
Dachte, das könnte so bleiben... Jetzt hab ich eine Seite, die anzeigt, wieviel da ist und wo man alles auf 0 setzen kann...

_________________
10 Gründe, die für Linux sprechen:
Linux ist:
01. stabil
02. schnell
03. sicher
04. flexibel
05. freie Software
06. ressourcenschonend
07. sehr gut dokumentiert
08. beliebt
09. kompatibel zu Windows
10. sexy