PHP/MYSQL headers allready send

Pascalus

Hi,

//Edit

PHP

<?
 mysql_connect('#','#','#');
 mysql_select_db("#");






mysql_query("UPDATE benutzerdaten SET Vorname='{$_REQUEST['vorname']}' AND Nachname='{$_REQUEST['nachname']}' AND  where Id='{$_REQUEST['id']}'");


header ("Location: intern.php?ok=1");


?>

Alles anzeigen

//Edit

mfg
Pascalus

//Edit: Hab den dummen Fehler selbst gefunden^^
Habe jetzt aber ein anderes Problem. Die Daten, die geändert werden sollen, bleiben in der Db wie sie waren
Bitte um Hilfe

Tobse

Sicher werden sie das nicht weil

PHP

AND where

auch nicht korrekt ist.
mach nur

PHP

where id='bla'

und in zukunft benutzt du bitte dinge wie

PHP

error_reporting(E_ALL);

und

PHP

echo mysql_error();

driver

versuch mal des

PHP

$result = mysql_query("UPDATE benutzerdaten SET (vorname, nachname,id) values ('".{$_REQUEST['vorname']}."','".{$_REQUEST['nachname']}."','".{$_REQUEST['id']}."')");

Pascalus

Hi,

danke für die Hilfe, aber hab beim klick auf den erstellen Button den Fehler gefunden :-D.

Mein neues Problem ist nun aber, dass die Daten, die Upgedated (Ah, es lebe das Denglisch :D) in der DB nicht verändert werden, obwohl ich sicher bin, dass die Felder existieren und dass es kein Rechtschreibfehler (o.ä.) bei den Feldname gibt.

mfg

//Edit: @Vorposter: Estmal Danke für die Hilfe, aber jetzt wird doch auch die ID geupdated. Es sollen aber nur die Vor/-Nachname Felder geupdatet werden, wo die ID die des User ist...

PS: Man, ich bin heute nur am Post editieren^^

Grevas

Schau dir die Version von driver an. Wenn du schon Arrays drin hast, solltest du sie nicht wie normale Variablen innerhalb eines Strings schreiben. Ansonsten Poste doch mal die aktuelle Version, wenn du schon was veränderst.

driver

noch ne kleine nachfrage - sicher dass du die id ändern willst und kannst (autowert) ?

Pascalus

Neee^^

Also das ist ein Teil meiner benutzerdaten Tabelle:

Id | Vorname | Nachname

Wenn nun jemand seine Daten in der intern.php ändert, erscheint sein Vorname und Nachname in 2 Textboxen. Wenn der User jetzt auf Speichern klickt, wird die Intern2.php aufgerufen.

Wenn dieser User nun die ID 1 hat, sollen nur Vor/-Nachname von diesem User(ID1) geändert werden.

Hoffe, ich konnte es verständlich rüber bringen

mfg

driver

dann stimmt die anweisung nicht.... die müsste so sein

PHP

$result = mysql_query("UPDATE benutzerdaten SET (vorname, nachname) values ('".{$_REQUEST['vorname']}."','".{$_REQUEST['nachname']}."') where id=".{$_REQUEST['id']});

</span></span>

Pascalus

Hi,

danke nochmal für die Hilfe.

Aber irgenwie steh ich gerade völlig aufm Schlauch^^

Bei

PHP

$result = mysql_query("UPDATE benutzerdaten SET (vorname, nachname) values ('".{$_REQUEST['vorname']}."','".{$_REQUEST['nachname']}."') where id=".{$_REQUEST['id']});

kommt folgende Fehlermedlung

PHP

Parse error:  syntax error, unexpected '{' in /home/htdocs/pascalus/html/dson/Socom3/de/plattform/intern2.php on line 20

Dann hab ich mich nochmal selbst rangetraut:

PHP

$id = $_POST['id'];
$vorname = $_POST['vorname'];
$nachname = $_POST['nachname'];






mysql_query("UPDATE benutzerdaten SET Vorname='$vorname', Nachname='$nachname' WHERE Id='$id'");

Aber da passiert garnix (-> keine Fehlermeldung und kein Update in der DB)

Hilfe!!!

mfg

Grevas

1. schreib hinter der mysql_query noch or die(mysql_error())
also

PHP

mysql_query("UPDATE benutzerdaten 
SET Vorname='$vorname', Nachname='$nachname' 
WHERE Id='$id'") or die(mysql_error());

2. debug halt mal, lass dir die $id ausgeben und schau selbst nach ob diese ID in der DB ist (und zwar in der Tabelle benutzerdaten)

Pascalus

Hi,
also ich bin jetzt etwas baff, denn wenn ich mir die id (oder auch alle anderen Variablen) ausgeben lasse, bleibt die Seite weiß O.O.

Find ich seltsam.
Hier ist mal ein Ausschnett aus intern.php

PHP

<?
$id = $_SESSION['user_id'];
$nickname = $_SESSION['user_nickname'];
$email = $_SESSION['user_email'];
$vorname = $_SESSION['user_vorname'];
$nachname = $_SESSION['user_nachname'];
$ort = $_SESSION['user_ort'];
$fse = $_SESSION['user_fse'];
$handle = $_SESSION['user_handle'];
$bemerkung = $_SESSION['user_bemerkung'];
?>




<form action="intern2.php" method="post">




<table>


<tr><td>BenutzerId:                  </td><td>  <? echo "<input type='text' value='". $id ."' name='id' disabled>";  ?>                </td></tr>
<tr><td>Nickname:                    </td><td>    <? echo "<input type='text' value='". $nickname ."' name='nickname' disabled>";  ?>  </td></tr>
<tr><td>Email:                       </td><td>    <? echo "<input type='text' value='". $email ."' name='email' disabled>";  ?>        </td></tr>
<tr><td>Vorname:                     </td><td>    <? echo "<input type='text' value='". $vorname ."' name='vorname'>";  ?>             </td></tr>
<tr><td>Nachname:                    </td><td>    <? echo "<input type='text' value='". $nachname ."' name='nachname'>";  ?>           </td></tr>
<tr><td>Ort:                         </td><td>    <? echo "<input type='text' value='". $ort ."' name='ort'>";  ?>                     </td></tr>
<tr><td>Lieblings Socom Edition:     </td><td>    <? echo "<input type='text' value='". $fse ."' name='fse'>";  ?>                     </td></tr>
<tr><td>Online Handle:               </td><td>    <? echo "<input type='text' value='". $handle ."' name='handle'>";  ?>               </td></tr>
<tr><td>Bemerkung:                   </td><td>    <? echo "<input type='text' value='". $bemerkung ."' name='bemerkung'>";  ?>         </td></tr>
<tr><td>                             </td><td>    <input type="submit" value="Speichern">        </td></tr>


</table>
</form>


  <hr>
  <a href="logout.php">Ausloggen</a>
</body>
</html>

Alles anzeigen

Die Variablen oben stammen aus der login.php:

PHP

<?php
// Session starten
session_start ();


// Datenbankverbindung aufbauen
$connectionid = mysql_connect ("l#", "#", "#");
if (!mysql_select_db ("#", $connectionid))
{
  die ("Keine Verbindung zur Datenbank");
}


$sql = "SELECT ".
    "Id, Nickname, Nachname, Vorname, email, ort, fse, handle, bemerkung ".
  "FROM ".
    "benutzerdaten ".
  "WHERE ".
    "(Nickname like '".$_REQUEST['name']."') AND ".
    "(code like 'ja') AND ".
    "(Kennwort = '".md5 ($_REQUEST['pwd'])."')";
$result = mysql_query ($sql);


if (mysql_num_rows ($result) > 0)
{
  // Benutzerdaten in ein Array auslesen.
  $data = mysql_fetch_array ($result);


  // Sessionvariablen erstellen und registrieren
  $_SESSION['user_id'] = $data['Id'];
  $_SESSION['user_nickname'] = $data['Nickname'];
  $_SESSION['user_nachname'] = $data['Nachname'];
  $_SESSION['user_vorname'] = $data['Vorname'];
  $_SESSION['user_email'] = $data['email'];


  $_SESSION['user_alter'] = $data['alter'];
  $_SESSION['user_ort'] = $data['ort'];
  $_SESSION['user_fse'] = $data['fse'];
  $_SESSION['user_handle'] = $data['handle'];
  $_SESSION['user_bemerkung'] = $data['bemerkung'];


  header ("Location: intern.php");
}
else
{
  header ("Location: formular.php?fehler=1");
}
?>

Alles anzeigen

Kann es vllt sein,dass ich die Variable zu oft übergeben habe?

Weiß gerade echt nicht weiter...

mfg

Pascalus

Grevas

Könntest du das ganze ein wenig genauer beschreiben? Bleibt die intern.php weiß? Wenn ja:
gibt es vor der Tabelle noch irgendwelche Bedingungen? Sonst kann ich mir das nämlich nicht erklären, keine Werte - ok, aber komplett weiß wäre seehr merkwürdig.

Wenn nur die Werte fehlen; sicher, dass du auf der aufgerufenen Seite auch die Session wieder startest? Hast ja geschrieben, dass es nur ein ausschnitt ist...

Pascalus

Hi,

ich hab den Fehler gefuden!!

PHP

<tr><td>Nickname:                    </td><td>    <? echo "<input type='text' value='". $nickname ."' name='nickname' disabled>";  ?>  </td></tr>

Es lag an dem disabled! Wenn ich das disabled entferne, klappt alles, wie es soll!

Vielen Dank an alle die mir versucht haben zu helfen

Aber eine Frage habe ich noch:
Wenn ich kein disabled einfügen kann, wie kann ich sonst noch verhindern,dass der User in diesem Feld was ändert? Geht das i-wie mit readonly oder so?

thx+mfg
Pascalus

Grevas

Ja das macht man mit readonly...

http://de.selfhtml.org/html/formulare/eingabe.htm#nurlesen

aber auch wenn du felder readonly einstellst, niemals den wert daraus weiterverwenden "nur" weil der user es nicht editieren kann... Sollte man genauso behandeln wie jedes inputfeld

Bandit

Und nicht vergessen: http://de.php.net/manual/de/func…cape-string.php

Pascalus

OK. Vielen Dank für die zahlreiche Hilfe!

PHP/MYSQL headers allready send

Tags

Benutzer online in diesem Thema