Update gebruiker

Door Enrico van der List op 23-09-2015 12:13

2.237 views

Wat is er verkeerd aan want op een of andere manier kan ik mijn gegevens niet updaten wanneer ik ze aanpas. Ik heb deze code letterlijk overgenomen van een tutorial. Maar bij hem werkt het wel en bij mij niet.. Misschien zien jullie iets over het hoofd wat ik niet heb gezien...


<?php require "php_conectie/Connectie2.php";?>
<?php
error_reporting(E_ALL);
ini_set('display_errors',true);

session_start();
	if(isset($_SESSION["UserID"])){
	}else{
		header('Location: Login.php');
	}
?>
<?php
	$User = $_SESSION["UserID"];
	$result = $con->query("select * from user where UserID='$User'");
	$row = $result->fetch_array(MYSQLI_BOTH);	
	$_SESSION["FirstName"] = $row['Fname'];
	$_SESSION["LastName"] = $row['Lname'];
	$_SESSION["Email"] = $row['Email'];
	$_SESSION["PW"] = $row['Password'];
?>
<?php
	if(isset($_POST['Update'])){
		
		$UpdateFName = $_POST['FirstName'];
		$UpdateLName = $_POST['LastName'];
		$UpdateEmail = $_POST['Email'];
		$UpdateFPassword = $_POST['Password'];
		
		$sql = $con->query("UPDATE user SET Fname = '{$UpdateFName}', Lname = '{$UpdateLName}', Email='{$UpdateEmail}', Password='{$UpdateFPassword}' where UserID= $User");
		
		header ('Location: Account.php');
		
	}
?>
<!doctype html>
<html>
<head>
<meta charset="UTF-8">
<title>Register</title>

<link href="css/Master.css" rel="stylesheet" type="text/css">
<link href="css/Menu.css" rel="stylesheet" type="text/css">

</head>

<body>
	<div class="Container">
    	<div class="Header"></div>
        <div class="Menu">
        	<div id="Menu">
            	<nav>
                	<ul class="cssmenu">
                    	<li><a href="Account.php">Account</a></li>
                        <li><a href="LogOut.php">Uitloggen</a></li>
                    </ul>
                </nav>
           </div>
        </div>
        <div class="LeftBody"></div>
        <div class="RightBody">
        <form action="" method="post" name="form1">
            	<div class="FormElement">
                	<input name="FirstName" type="text" required="required" class="TField" id="FirstName" placeholder="Voornaam" value="<?php echo $_SESSION["FirstName"];?>">
                </div>
                <div class="FormElement">
                	<input name="LastName" type="text" required="required" class="TField" id="LastName" placeholder="Achternaam" value="<?php echo $_SESSION["LastName"];?>">
                </div>
                <div class="FormElement">
                	<input name="Email" type="email" required="required" class="TField" id="Email" placeholder="E-mail" value="<?php echo $_SESSION["Email"];?>">
                </div>
                <div class="FormElement">
                	<input name="Password" type="password" required="required" class="TField" id="Password" placeholder="Wachtwoord" value="<?php echo $_SESSION["PW"];?>">
                </div>
                <div class="FormElement">
                	<input name="Update" type="submit" class="button" id="Update" value="Update info" >
                </div>
            </form>
        </div>
        <div class="Footer"></div>
     </div>
</body>
</html>

Ivo P

23-09-2015 13:52

zet op regel 30 de query eens op het scherm (echo)
en check ook even of de query gelukt is.

je redirect nu botweg naar een pagina, maar je weet daardoor niet of alles wel lukte en waarom niet.

---
trouwens wat zonde om op regel 13 ev. de user op te halen, als je eventueel nog een update gaat uitvoeren en daarna redirect en dus de data opgehaald hebt voor niets.

---
regel 6 zou een foutmelding moeten geven omdat je op regel 1 ¿> gebruikt en op regel 2, na een ENTER, weer <¿php hebt staan.
Daar zou een Headers-Already-Sent error uit moeten komen.

Thomas van den Heuvel

23-09-2015 14:17 gewijzigd op 23-09-2015 14:21

Ivo P op 23/09/2015 13:52:44
regel 6 zou een foutmelding moeten geven omdat je op regel 1 ¿> gebruikt en op regel 2, na een ENTER, weer <¿php hebt staan.
Daar zou een Headers-Already-Sent error uit moeten komen.

:)

Volgens mij was dat een dingetje van PHP, deze stript een newline nadat je een php-blok afsluit met ?>. Dus je krijgt geen headers sent error denk ik.

Zet na een header('Location: ...') wel altijd een exit;

Daarnaast ben ik die tutorial ook aan het volgend voor de gein, volgens mij klopt er geen biet van de character encoderingen.

Wat je nog aan verbeteringen zou kunnen aanbrengen is oa controleren of je queryresultaten hebt voordat je ze ophaalt, mogelijk lost dat iets op.

Enrico van der List

23-09-2015 14:36

Thomas van den Heuvel op 23/09/2015 14:17:46

[quote="Ivo P op 23/09/2015 13:52:44"]regel 6 zou een foutmelding moeten geven omdat je op regel 1 ¿> gebruikt en op regel 2, na een ENTER, weer <¿php hebt staan.
Daar zou een Headers-Already-Sent error uit moeten komen.

Johan K

23-09-2015 23:27 gewijzigd op 23-09-2015 23:30

Ik zit op de 15 minuten en vind het verschrikkelijk worden:

1. Regel 3 start de sessie regel 17 herhaald regel 3 op niets af.
2. Hij werkt met "capitalizedLetters" in classes en id's en ook in php array's $SESSION[], doe dit nooit.
3. De headers worden al verzonden doordat er op regel 1 een linebreak staat, en als een nieuwe gebruiker langs komt word er een nieuwe sessie gemaakt waarvan UserId niet bestaat, zondoende doet 'header("location: LogIn.php")' helemaal niets en loopt het script gewoon verder en word er een query uitgevoerd die niet bestendig is tegen mysql injecties.

Op schaal van 0 tot 10 op veiligheid is dit toch wel een 2je aangezien jouw browser ook een redirect (header("Location: ...")) gewoon kan negeren via een plugin.

Daarbij kan regel 4->7 (of bij jouw 7 t/m 10)gewoon:
<?php
if( !isset($_SESSION["UserID"])){
header('Location: Login.php');
die();
}
?>

Kortom, leer niet van deze gast want hij bakt er niets van.
Wat er fout gaat in dit script is doordat jouw form niet gelinked staat met PHP. Specefiek:

<form action="" method="post" name="form1"> -> 
<form action="updateAccount.php" method="post">

Enrico van der List

16-10-2015 13:20 gewijzigd op 16-10-2015 13:35

Hij werkt wel weer bijna alleen krijg ik nu errors terwijl ik wel kan uploaden met dezelfde code...

Notice: Undefined variable: POST in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 28
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 34
Warning: file_get_contents(): Filename cannot be empty in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 34
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 35
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 36 Warning: getimagesize(): Filename cannot be empty in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 36
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 39



<?php
  if(isset($_POST['Update'])){
	
	//De image uit het form wordt opgeslagen in een variabele.
	$profileImage = $POST["myimg"];																		//<------------- error
	
	//Upload van afbeelding naar server en database.
	$resultaat = mysqli_query($con, "SELECT MAX(UserID) FROM users");//Kijkt of database image bestaat.
	$rij = mysqli_fetch_array($resultaat);
	$rij = $rij['MAX(UserID)'] + 1;//Verandert de naam in een nummer van de foto aan de hand van de database + 1.
	$image = addslashes(file_get_contents($_FILES['myimg']['tmp_name']));											//<------------- error
	$image_name = addslashes($_FILES['myimg']['name']);//Naam bestand.											//<------------- error
	$imageSize = getimagesize($_FILES['myimg']['tmp_name']);//Grootte bestand.									//<------------- error
	$filePath = "images/profilephoto/".$image_name;//Locatie en naam van het bestand dat is geupload.

	move_uploaded_file($_FILES['myimg']['tmp_name'], "images/profilephoto/".$image_name);//Plaatst de foto op de server	//<------------- error
	
        
        $UpdateFName = $_POST['FirstName'];
        $UpdateLName = $_POST['LastName'];
        $UpdateEmail = $_POST['Email'];
        $UpdateFPassword = $_POST['Password'];
        
		$StorePassword = password_hash($UpdateFPassword, PASSWORD_BCRYPT, array('cost' => 10));
		
        $sql = $con->query("UPDATE users SET Fname = '{$UpdateFName}', Lname = '{$UpdateLName}', Email='{$UpdateEmail}', Password='{$StorePassword}' , ProfileImage='{$image_name}' , ProfilePath='{$filePath}',  WHERE UserID= $ID");
    }
?>

[size=xsmall]Toevoeging op 16/10/2015 13:20:58:[/size]

Enrico van der List op 16/10/2015 13:20:20

Hij werkt wel weer bijna alleen krijg ik nu errors terwijl ik wel kan uploaden met dezelfde code op een andere pagina...

Notice: Undefined variable: POST in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 28
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 34
Warning: file_get_contents(): Filename cannot be empty in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 34
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 35
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 36 Warning: getimagesize(): Filename cannot be empty in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 36
Notice: Undefined index: myimg in /home/i296429/domains/i296429.iris.fhict.nl/public_html/page/WP31/Week1/UpdateAccount.php on line 39
  if(isset($_POST['Update'])){
	
	//De image uit het form wordt opgeslagen in een variabele.
	$profileImage = $POST["myimg"];																		//<------------- error
	
	//Upload van afbeelding naar server en database.
	$resultaat = mysqli_query($con, "SELECT MAX(UserID) FROM users");//Kijkt of database image bestaat.
	$rij = mysqli_fetch_array($resultaat);
	$rij = $rij['MAX(UserID)'] + 1;//Verandert de naam in een nummer van de foto aan de hand van de database + 1.
	$image = addslashes(file_get_contents($_FILES['myimg']['tmp_name']));											//<------------- error
	$image_name = addslashes($_FILES['myimg']['name']);//Naam bestand.											//<------------- error
	$imageSize = getimagesize($_FILES['myimg']['tmp_name']);//Grootte bestand.									//<------------- error
	$filePath = "images/profilephoto/".$image_name;//Locatie en naam van het bestand dat is geupload.

	move_uploaded_file($_FILES['myimg']['tmp_name'], "images/profilephoto/".$image_name);//Plaatst de foto op de server	//<------------- error
	
        
        $UpdateFName = $_POST['FirstName'];
        $UpdateLName = $_POST['LastName'];
        $UpdateEmail = $_POST['Email'];
        $UpdateFPassword = $_POST['Password'];
        
		$StorePassword = password_hash($UpdateFPassword, PASSWORD_BCRYPT, array('cost' => 10));
		
        $sql = $con->query("UPDATE users SET Fname = '{$UpdateFName}', Lname = '{$UpdateLName}', Email='{$UpdateEmail}', Password='{$StorePassword}' , ProfileImage='{$image_name}' , ProfilePath='{$filePath}',  WHERE UserID= $ID");
    }
?>

Onbekende gebruiker

16-10-2015 13:29

Enrico, kleine tip.
Stop met het gebruiken van hoofdletters in je bestandnamen. Dit gaat problemen opleveren later als het op een Linux distrubutie staat.

Gebruik gewoon lowercase en streepjes. Linux-word-daar-blijer-van, en wij ook.

Enrico van der List

16-10-2015 13:36

Rickert Bombaklats op 16/10/2015 13:29:06

Enrico, kleine tip.
Stop met het gebruiken van hoofdletters in je bestandnamen. Dit gaat problemen opleveren later als het op een Linux distrubutie staat.

Gebruik gewoon lowercase en streepjes. Linux-word-daar-blijer-van, en wij ook.

maar waarom doet de code het wel op een andere pagina en andere projecten en deze weer niet...

Thomas van den Heuvel

16-10-2015 13:38 gewijzigd op 16-10-2015 13:39

Enrico, als $_FILES niet wordt herkend is de kans groot dat je de toevoeging:

enctype="multipart/form-data"

bent vergeten in je <form> tag.

(EDIT: dit zorgt ervoor dat je files kunt uploaden, anders worden deze simpelweg niet meegestuurd als je je formulier post)

Daarnaast trekt die tutorial reeks op stront. Ik heb de maker al verzocht deze reeks offline te halen. Voor zowel de variant met als zonder password-hashing heb ik bewezen en aangetoond dat deze beide vatbaar zijn voor SQL-injectie. Los van de enorme security-gaten mist de applicatie die in die serie wordt gebouwd ook een heleboel logica.

Reageren

Inloggen om te reageren