Hallo allemaal,

Ik heb een probleempje met het updaten.
De bedoeling is dat de gebruiker als zijn gegevens kan veranderen eigenlijk werkt het hele formulier perfect,
behalve dat er dus geen veranderingen plaatsvinden,
ik krijg verder geen errors of niks.

nu ga ik even een paar keer enter drukken zodat de code niet in de google adds komt.....







de vraag is dus eigenlijk gewoon waarom update hij niet?

Code:

mysql_query("UPDATE members SET firstname = " . mysql_real_escape_string($_POST['firstname']) . ", lastname = " . mysql_real_escape_string($_POST['lastname']) . ", companyname = " . mysql_real_escape_string($_POST['companyname']) . ", kvk = " . mysql_real_escape_string($_POST['kvk']) . ", street = " . mysql_real_escape_string($_POST['street']) . ", housenumber = " . mysql_real_escape_string($_POST['housenumber']) . ", zipcode = " . mysql_real_escape_string($_POST['zipcode']) . ", city = " . mysql_real_escape_string($_POST['city']) . ", country = " . mysql_real_escape_string($_POST['country']) . ", phonenumber = " . mysql_real_escape_string($_POST['phonenumber']) . ", emailadres = " . mysql_real_escape_string($_POST['emailadres']) . " WHERE id = '" . $_SESSION['data']['id'] . "' AND password = '" . $_SESSION['data']['password'] . "'");

PHP Code:

mysql_query("
UPDATE `members` SET 
`firstname` = '". mysql_real_escape_string($_POST['firstname']) ."', 
`lastname` = '". mysql_real_escape_string($_POST['lastname']) ."', 
`companyname` = '". mysql_real_escape_string($_POST['companyname']) ."', 
`kvk` = '". mysql_real_escape_string($_POST['kvk']) ."', 
`street` = '". mysql_real_escape_string($_POST['street']) ."', 
`housenumber` = '". mysql_real_escape_string($_POST['housenumber']) ."', 
`zipcode` = '". mysql_real_escape_string($_POST['zipcode']) ."', 
`city` = '". mysql_real_escape_string($_POST['city']) ."', 
`country` = '". mysql_real_escape_string($_POST['country']) ."', 
`phonenumber` = '". mysql_real_escape_string($_POST['phonenumber']) ."', 
`emailadres` = '". mysql_real_escape_string($_POST['emailadres']) ."' 
WHERE `id` = '". $_SESSION['data']['id'] . "' AND `password` = '". $_SESSION['data']['password'] . "'
"); 


Wat staat er in $_SESSION['data'] ?

PHP Code:

print_r($_SESSION['data']); 


de id en de password

Array ( [id] => 1 [password] => 9cdfb439c7876e703e307864c9167a15 )

wat dus maar 1 row kan opleveren

de update werkt nu het lag waarschijnlijk aan die ``

deze heb ik niet eerder hoeven te gebruiken, bedankt!

Gooi eens "or die mysql_error()" er achter ?

Nee, het lag eraan dat je geen quotes om je string variabelen had staan :)

Heeft die al verandert neem ik aan.. ?

Edit:

Oh, hehe het is laat. Ik ga naar bed..

Array ( [id] => 1 [password] => 9cdfb439c7876e703e307864c9167a15 )

Misschien een handige tip, zet niet zomaar wachtwoorden online.
Als je deze md5 encrypt kom je erachter.

uberhaupt: als je een id hebt om een row op te zoeken dan is het volkomen nutteloos om ook aan de hand van het password de row te zoeken. Dat maakt de query echt niet sneller.

Dus haal het gehele password stuk er af.

ik ga er hierbij natuurlijk wel vanuit dat id uniek is.

Origineel gepost door Rein S

uberhaupt: als je een id hebt om een row op te zoeken dan is het volkomen nutteloos om ook aan de hand van het password de row te zoeken. Dat maakt de query echt niet sneller.

Dus haal het gehele password stuk er af.

ik ga er hierbij natuurlijk wel vanuit dat id uniek is.

En dan edit je 1 keer de sessie $_SESSION['data']['id'] en je kan zo even alle gegevens van een andere gebruker wijzigen.. Maak dan gebruik van salts..

Origineel gepost door Robin L

En dan edit je 1 keer de sessie $_SESSION['data']['id'] en je kan zo even alle gegevens van een andere gebruker wijzigen.. Maak dan gebruik van salts..

Zover ik weet verander je die niet zo snel.?
En anders geef je iedere gebruiker gewoon een code van 16 tekens en zet je die in je session ipv het id

Origineel gepost door RubenP

Zover ik weet verander je die niet zo snel.?
En anders geef je iedere gebruiker gewoon een code van 16 tekens en zet je die in je session ipv het id

Sessie's zijn zeker wel te bewerken, en overigens 'NEVER TRUST USER INPUT'.