Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP & SQL

E-Mailadres weergeven (Opgelost)

Waflix - 03/08/2011 17:11

Lid

Jawel hoor, ik heb weer een vraag.

Deze keer maak ik gebruik van een inlogscript, geschreven door Dennisvb.
Ik heb het wat uitgebreid door ook de mogelijkheid toe te voegen je e-mailadres in te voeren. Dat heb ik gedaan door een kolom toe te voegen genaamd 'mail'.
Helaas lukt het mij maar niet om dit e-mailadres weer te geven op mijn site, en ik heb weten te concluderen dat dit komt door een fout in het bestand 'session.php'.
Originele code session.php

php code - Bekijk de code zonder highlighting - Klap code in

<?php
 session_start(); // start een sessie of zet een sessie voort

 // als de gebruiker is ingelogd
 if(isset($_SESSION['suser'])) {
   // het volgende timeout deel is optioneel - dit mag worden weggelaten
   // * timeout gedeelte *
   $now = time();
   // als er meer tijd is verstreken dan smaxidle
   // sinds het aanmaken van de sessie
   if($now - $_SESSION['stime'] > $_SESSION['smaxidle']) {
     // breek de sessie af, de gebruiker dient opnieuw in te loggen
     $_SESSION = array();
     session_destroy();
   } else {
     // ververs anders de sessietijd. Dit zorgt er voor
     // dat de gebruiker ingelogd blijft zolang deze actief is.
     $_SESSION['stime'] = $now;
   }
   // * einde timeout gedeelte *
 // v1.2 extra functionaliteit, onthouden login
 } elseif(isset($_COOKIE['login_cookie'])) {
   // bekijk de waarden van de cookie en als deze kloppen met de database - start alsnog een sessie
   // aanname - er is een verbinding met de database
   list($id, $wacht) = split(";", $_COOKIE['login_cookie']);
   $res = mysql_query("SELECT id, pass, level, last_ip FROM users WHERE id='".$id."'") or die(mysql_error());
   if(mysql_num_rows($res) > 0) {
     $row = mysql_fetch_assoc($res);
     if(!strcmp($wacht, $row['pass']) && $_SERVER['REMOTE_ADDR'] == $row['last_ip']) {
       // init session
       $_SESSION['suser']    = $naam;
       $_SESSION['slevel']   = $row['level'];
       $_SESSION['stime']    = time();
       $_SESSION['smaxidle'] = 60 * 60;

       // update cookie
       // gebruik hierbij wederom het id en het versleutelde wachtwoord
       setcookie("login_cookie", $id.";".$wacht, time()+3600*24*31*2, "/");
     } else {
       // password of ip komt niet overeen - unset het cookie en beeindig de sessie
       setcookie("login_cookie", "", time(), "/");
       $_SESSION = array();
       session_destroy();
     }
     // geef resultaten vrij
     unset($row);
     mysql_free_result($res);
   } else {
     // gebruiker onbekend, cookie vervalst ?
     $_SESSION = array();
     session_destroy();
   }
   // ververs de pagina
   header("Location: ".$_SERVER['REQUEST_URI']);
 }
 ?>

<?php
 session_start(); // start een sessie of zet een sessie voort
 
 // als de gebruiker is ingelogd
 if(isset($_SESSION['suser'])) {
   // het volgende timeout deel is optioneel - dit mag worden weggelaten
   // * timeout gedeelte *
   $now = time();
   // als er meer tijd is verstreken dan smaxidle
   // sinds het aanmaken van de sessie
   if($now - $_SESSION['stime'] > $_SESSION['smaxidle']) {
     // breek de sessie af, de gebruiker dient opnieuw in te loggen
     $_SESSION = array();
     session_destroy();
   } else {
     // ververs anders de sessietijd. Dit zorgt er voor
     // dat de gebruiker ingelogd blijft zolang deze actief is.
     $_SESSION['stime'] = $now;
   }
   // * einde timeout gedeelte *
 // v1.2 extra functionaliteit, onthouden login
 } elseif(isset($_COOKIE['login_cookie'])) {
   // bekijk de waarden van de cookie en als deze kloppen met de database - start alsnog een sessie
   // aanname - er is een verbinding met de database
   list($id, $wacht) = split(";", $_COOKIE['login_cookie']);
   $res = mysql_query("SELECT id, pass, level, last_ip FROM users WHERE id='".$id."'") or die(mysql_error());
   if(mysql_num_rows($res) > 0) {
     $row = mysql_fetch_assoc($res);
     if(!strcmp($wacht, $row['pass']) && $_SERVER['REMOTE_ADDR'] == $row['last_ip']) {
       // init session
       $_SESSION['suser']    = $naam;
       $_SESSION['slevel']   = $row['level'];
       $_SESSION['stime']    = time();
       $_SESSION['smaxidle'] = 60 * 60;
 
       // update cookie
       // gebruik hierbij wederom het id en het versleutelde wachtwoord
       setcookie("login_cookie", $id.";".$wacht, time()+3600*24*31*2, "/");
     } else {
       // password of ip komt niet overeen - unset het cookie en beeindig de sessie
       setcookie("login_cookie", "", time(), "/");
       $_SESSION = array();
       session_destroy();
     }
     // geef resultaten vrij
     unset($row);
     mysql_free_result($res);
   } else {
     // gebruiker onbekend, cookie vervalst ?
     $_SESSION = array();
     session_destroy();
   }
   // ververs de pagina
   header("Location: ".$_SERVER['REQUEST_URI']);
 }
 ?>

Mijn aangepaste (vooral ook mee gerommelde) code

php code - Bekijk de code zonder highlighting - Klap code in

<?php
	session_start();

	if(isset($_SESSION['suser'])) {
		$now = time();
		if($now - $_SESSION['stime'] > $_SESSION['smaxidle']) {
			$_SESSION = array();
			session_destroy();
		} else {
			$_SESSION['stime'] = $now;
		}
	} elseif(isset($_COOKIE['login_cookie'])) {
		list($id, $wacht) = split(";", $_COOKIE['login_cookie']);
		if( isset($_SESSION['mail'])) {
			$res = mysql_query("SELECT id, pass, mail, level, last_ip FROM users WHERE id='".$id."'") or die(mysql_error());
		} else {
			$res = mysql_query("SELECT id, pass, level, last_ip FROM users WHERE id='".$id."'") or die(mysql_error());
		}
		if(mysql_num_rows($res) > 0) {
			$row = mysql_fetch_assoc($res);
			if(!strcmp($wacht, $row['pass']) && $_SERVER['REMOTE_ADDR'] == $row['last_ip']) {
				if( isset($_SESSION['mail'])) {
					$_SESSION['suser']    = $naam;
					$_SESSION['mail']     = $mail;
					$_SESSION['slevel']   = $row['level'];
					$_SESSION['stime']    = time();
					$_SESSION['smaxidle'] = 60 * 60;
				} else {
					$_SESSION['suser']    = $naam;
					$_SESSION['slevel']   = $row['level'];
					$_SESSION['stime']    = time();
					$_SESSION['smaxidle'] = 60 * 60;
				}

			setcookie("login_cookie", $id.";".$wacht, time()+3600*24*31*2, "/");
			} else {
				setcookie("login_cookie", "", time(), "/");
				$_SESSION = array();
				session_destroy();
			}
			unset($row);
			mysql_free_result($res);
		} else {
			$_SESSION = array();
			session_destroy();
		}
		header("Location: ".$_SERVER['REQUEST_URI']);
	}
?>

<?php
	session_start();
 
	if(isset($_SESSION['suser'])) {
		$now = time();
		if($now - $_SESSION['stime'] > $_SESSION['smaxidle']) {
			$_SESSION = array();
			session_destroy();
		} else {
			$_SESSION['stime'] = $now;
		}
	} elseif(isset($_COOKIE['login_cookie'])) {
		list($id, $wacht) = split(";", $_COOKIE['login_cookie']);
		if( isset($_SESSION['mail'])) {
			$res = mysql_query("SELECT id, pass, mail, level, last_ip FROM users WHERE id='".$id."'") or die(mysql_error());
		} else {
			$res = mysql_query("SELECT id, pass, level, last_ip FROM users WHERE id='".$id."'") or die(mysql_error());
		}
		if(mysql_num_rows($res) > 0) {
			$row = mysql_fetch_assoc($res);
			if(!strcmp($wacht, $row['pass']) && $_SERVER['REMOTE_ADDR'] == $row['last_ip']) {
				if( isset($_SESSION['mail'])) {
					$_SESSION['suser']    = $naam;
					$_SESSION['mail']     = $mail;
					$_SESSION['slevel']   = $row['level'];
					$_SESSION['stime']    = time();
					$_SESSION['smaxidle'] = 60 * 60;
				} else {
					$_SESSION['suser']    = $naam;
					$_SESSION['slevel']   = $row['level'];
					$_SESSION['stime']    = time();
					$_SESSION['smaxidle'] = 60 * 60;
				}
 
			setcookie("login_cookie", $id.";".$wacht, time()+3600*24*31*2, "/");
			} else {
				setcookie("login_cookie", "", time(), "/");
				$_SESSION = array();
				session_destroy();
			}
			unset($row);
			mysql_free_result($res);
		} else {
			$_SESSION = array();
			session_destroy();
		}
		header("Location: ".$_SERVER['REQUEST_URI']);
	}
?>

5 antwoorden

Gesponsorde links

lemoinet - 03/08/2011 17:35
PHP gevorderde		en als je php code - Bekijk de code zonder highlighting - Klap code in $_SESSION['mail'] = $mail; $_SESSION['mail'] = $mail; verandert in php code - Bekijk de code zonder highlighting - Klap code in $_SESSION['mail'] = $row['mail']; $_SESSION['mail'] = $row['mail'];

Martijn2008 - 03/08/2011 17:36
PHP beginner		Ik denk dat je de code als volgt moet aanpassen: Regel 14: php code - Bekijk de code zonder highlighting - Klap code in if(!isset($_SESSION['mail'])) { if(!isset($_SESSION['mail'])) { Regel 24: php code - Bekijk de code zonder highlighting - Klap code in $_SESSION['mail'] = $row['mail']; $_SESSION['mail'] = $row['mail']; Bij regel 23 zet ik m'n vraagtekens, omdat je variabele $naam nooit eerder in het script hebt gedefinieerd. Overigens wil ik je erop wijzen dat je de comments beter in de code kunt laten staan. Als je later wijzigingen wilt maken kun je sneller begrijpen hoe en waarom je bepaalde aannames/besluiten hebt genomen. Waarom wil je een e-mailadres eigenlijk in een sessie-object stoppen, dat kun je toch gewoon aan de hand van het gebuikers_ID uit de database lezen?

Waflix - 03/08/2011 17:49
Lid		lemoinet schreef: en als je [..code..] verandert in [..code..] Helaas, maar dit werkt niet. ---- Martijn2008 schreef: Ik denk dat je de code als volgt moet aanpassen: Regel 14: [..code..] Regel 24: [..code..] Bij regel 23 zet ik m'n vraagtekens, omdat je variabele $naam nooit eerder in het script hebt gedefinieerd. Overigens wil ik je erop wijzen dat je de comments beter in de code kunt laten staan. Als je later wijzigingen wilt maken kun je sneller begrijpen hoe en waarom je bepaalde aannames/besluiten hebt genomen. Waarom wil je een e-mailadres eigenlijk in een sessie-object stoppen, dat kun je toch gewoon aan de hand van het gebuikers_ID uit de database lezen? Die regel 23 zit ook in het originele script, en zonder al mijn aanpassingen werkt dit gewoon. In mijn aangepaste bestanden (zoals deze) zitten geen comments, maar in de back-up bestanden (de originele bestanden) staan wel de comments erbij. Dus dat wordt geen probleem. Doch bedankt voor de tip. En hoe zou ik dan het e-mailadres aand de hand van het ID kunnen lezen, volgens jou? (Ik zeg niet dat het onmogelijk is, het is gewoon een vraag)

Martijn2008 - 03/08/2011 17:53 (laatste wijziging 03/08/2011 17:56)
PHP beginner		Is je probleem nu verholpen? Om gegevens uit je database op te halen aan de hand van de primary key(id) willen de andere Sitemasters en ik je graag helpen. Denk dan dat je dit topic het best dan kunt sluiten en een nieuwe kunt aanmaken.

Waflix - 03/08/2011 18:04
Lid		^ Helaas heeft het bewerken van beide regels niets opgeleverd. ==!NIEUW TOPIC!==

Gesponsorde links

Je moet ingelogd zijn om een reactie te kunnen posten.

Actiefste leden van de maand

Actieve forumberichten