last day (14 days later) » 

10:41 AM
1
A: PHP - MySQL Why is this query never returning false?

SuperDJfunction login($username, $password) { //$user_id = user_id_from_username($username); // remove this line down to the if statement $password = md5($password); $query = mysql_query("SELECT `username`, `password` FROM `users` WHERE `username` = '$username' AND `password` = '$passwor...

 
Thanks, I will try this now
Hmm... Doesn't seem to be working :(
 
@bramhaag Oh wait, I just noticed that you are checking against the $username so I will fix that in the answer.
@bramhaag made another edit
 
It is just doing the same thing :( Cannot tag you for a reason :'(
 
@bramhaag Have tried to put some echos in the code to check where it fails?
@bramhaag and what is the $result in the mysql_result() refurring to?
@bramhaag added the part that gave you an error.
 
Thanks! But for some reason the login part still doesn't work :(
 
10:42 AM
You could also put the following on top of a value that is used almost everywhere. // Force PHP to show errors
error_reporting(E_ALL);
ini_set( 'display_errors','1');
 
Hello sir
 
hello
could you put that code above the functions. I should force php to show you every error
 
Shall I send you the errors if there are some?
 
yes that's fine
 
Notice: A session had already been started - ignoring session_start() in /home/u239117584/public_html/login/connect.php on line 3

Notice: Use of undefined constant login - assumed 'login' in /home/u239117584/public_html/login/processing.php on line 37

Notice: Undefined index: process in /home/u239117584/public_html/login/processing.php on line 51

Notice: Use of undefined constant logout - assumed 'logout' in /home/u239117584/public_html/login/processing.php on line 51
Notice: Use of undefined constant login - assumed 'login' in /home/u239117584/public_html/login/processing.php on line 37 This one is the only one that has something to do with loggin in
 
10:47 AM
hmm notices aren't really important but they could point to a bigger error
do you have to somewhere online?
 
I do
jartexpvp.url.ph/login/index.php This is a old domain, I only tried to make a login system
 
Oke, do you have 1 file that is included in almost every page?
 
I do, it is connect.php
Wait a second! Are you Dutch?
 
ja
 
Hehehe. Dat maakt dit gesprek veel makkelijker :)
 
10:58 AM
als je de session_start() ook alleen op dei pagina hebt en verder nergens dan zou al je al van je eerste notice geen last moeten hebben
 
Maar als ik dan bijv. bij de index.php geen session_start heb, kan ik de errors toch niet ontvangen aangezien ze in een SESSION zitten?
 
wanneer je een bestand include dan gaat alles mee duss ook variabellen etc.
 
Oke, ik gebruik connect.php in 99% van de bestanden, maar niet in de index, dus ik neem aan dat die daar moet blijven staan
 
als je connect.php ook include in de index.php en in connect.php heb je de session_start() staan dan zou dat al iets moeten oplossen
 
Jep, de boventste Notice is weg :)
 
11:02 AM
dan de volgende die met login en logout van processing.php
wanneer word dit bestand aangeroepen?
 
Als je uitlogt via de knop in de navbar.
Dit is de functie waar het om gaat:
if($_GET["process"] === logout) {
if (isset($_SESSION['user_id'])) {
session_destroy();
} else {
echo 'Nope braw';
}


header('Location: index.php');
}
Let maar niet op de echos ;)
 
ik zou daar het volgende van maken:

// Easy to use redirect function
function to($url) {
if(headers_sent()) {
echo ' <script>window.location = "'.$url.'";</script>
<meta http-equiv="refresh" content="0;url='.$url.'">';
} else {
header('Location: '.$url);
exit();
}
}

if(!empty($_GET['proccess'])) {
$process = mysql_real_escape_sting(trim($_GET['proccess']));

if($process === 'logout') {
unset($_SESSION['user_id']);
} else {
echo 'nope braw';
}

to('/');
}
 
Mag ik vragen wat het verschil is qua werking?
 
empty() controleert of de $_GET isset() en of die leeg is
een $_GET geeft altijd een string terug dus vandaar de quotes
 
Maar waarom zou je die to() function maken? Header doet toch in principe het zelfde?
 
11:11 AM
klopt maar bij een header() heb je kans dat je een error krijgt headers already send bla bla bla
op die manier voorkom je dat
tevens zorgt de exit() ervoor dat het script stopt met uitvoeren
 
Dat van exit snap ik ja. Dat van die error van header() wist ik nog niet
 
in de echo staat die if dus om dat op te vangen
eigenlijk zou er om de <meta> tag nog een <noscript> kunnen / moeten
dit word dan alleen uitgevoerd wanneer javascript uitstaat
 
Ohoh... een nieuwe Warning: Unknown: Your script possibly relies on a session side-effect which existed until PHP 4.2.3. Please be advised that the session extension does not consider global variables as a source of data, unless register_globals is enabled. You can disable this functionality and this warning by setting session.bug_compat_42 or session.bug_compat_warn to off, respectively in Unknown on line 0
Ik ga eventjes wat eten. Ik spreek je later nog wel
 
dat komt waarschijnlijk door een session_destroy
 
11:27 AM
Heb jij een idee hoe je de log in sessie kan beeindigen zonder session_destroy ?
 
11:54 AM
unset($_SESSION['user_id']);
Notice: Undefined index: process in /home/u239117584/public_html/login/processing.php on line 51 heeft te maken dat de variabel oid. word aangeroepen voordat het gemaakt is of benoemd
 
Ik gebruik nu unset(), maar ik krijg nogsteeds deze error: Warning: Unknown: Your script possibly relies on a session side-effect which existed until PHP 4.2.3. Please be advised that the session extension does not consider global variables as a source of data, unless register_globals is enabled. You can disable this functionality and this warning by setting session.bug_compat_42 or session.bug_compat_warn to off, respectively in Unknown on line 0
 
Misschien is dit iets wat je bij je hosting kan na vragen
 
Oke. Maar door alle warnings op te lossen werkt het nogsteeds niet...
 
zijn alle notices ook weg?
 
Jep, voor zover ik zie wel ja
 
12:07 PM
heb je die force php error code ook nog aan staan?
 
Nu wel weer
 
probeer het allemaal nog eens
als het niet werkt zou er ergens iets fout moeten gaan
je kan je code ook debuggen door echo's met een duidelijke fout melden neer te zetten
 
Hij heeft nog 1 notice
Notice: Undefined index: proccess in /home/u239117584/public_html/login/processing.php on line 57
 
en dat is?
 
Wacht even, volgensmij heb ik het gefixt
 
12:10 PM
de $_GET[]?
 
Neh, $process = mysql_real_escape_string(trim($_GET['proccess']));
Maar er moest een 'c' weg bij procces
 
werkt het nu wel?
 
Nope
Dat ik ergens wat fout doe snap ik, maar wat... Ik heb werkelijk geen idee
 
word dat processing.php bestand alleen aangeroepen door een get of ook een post of roep je het ook op een andere manier aan?
 
Door post bij de login pagina
De login() function returnd geen false, hierdoor kun je gewoon inloggen omdat er geen errors zijn
 
12:16 PM
Als je voor elk gedeelte wat je aanroep met een post of een get, if($_POST) {} of if($_GET){} omheen zet zou je misschien al iets oplossen
ik krijg wel gebruikersnaam bestaat niet
 
Ik gebruik $_POST en zet de value in een array, zodat ik er makkelijker bij kan
$fields = array(
'username' => $_POST['username'],
'password' => $_POST['password'],
);
 
als je daar if($_POST) {} omheen zet dan word dat alleen aangeroepen wanneer het bestand met een post word aangeroepen
maar welke browser gebruik je?
 
Chrome
Dit staat boven de array
if(isset($_POST['username'], $_POST['password'])) {
 
druk s op f12
 
Dan doe ik mijn WiFi uit ;)
Is f12 gewoon het zelfde als element inspecteren?
 
12:19 PM
je kan beter !empty gebruiken dit controleert namelijk isset en of t leeg is ja of nee
f12 is idd zelfde als element inspecteren
maar als t goed is zie je gelijk dat er een error is
 
Ik stuur de hele lap code wel, want dat staat er ook in
if(isset($_POST['username'], $_POST['password'])) {

$fields = array(
'username' => $_POST['username'],
'password' => $_POST['password'],
);

$login = login($fields['username'], $fields['password']);

foreach($fields as $field => $data) {
if(empty($data)) {
echo 'empty';
$errors[] = 'The ' . $field . ' field is required!';
}
}

if (user_active($fields['username']) != false) {
$errors[] = 'This account isn\'t activated yet!';
}
if (user_exists($fields['username']) === false) {
$errors[] = 'Invalid username!';
 
<script>
$('.dropdown-toggle').dropdown();
</script> moet zijn slideDown of slideUp
 
Das met Bootstrap, dat heeft er in dit geval niks mee te maken aangezien het wel gewerkt heeft
 
<?php
if($_POST) {
$username = $_POST['username'];
$password = $_POST['passowrd'];

if(!empty($username) && !empty($password)) {

$login = login($username, $password);

foreach ($fields as $field => $data) {
if (empty($data)) {
echo 'empty';
$errors[] = 'The ' . $field . ' field is required!';
}
}

if (user_active($username) != false) {
$errors[] = 'This account isn\'t activated yet!';
}

if (user_exists($username) === false) {
$errors[] = 'Invalid username!';
}

// Hier gaat t fout
if ($login === false) {
 
Ik heb inderdaad gemerkt dat het daar fout gaat
 
12:26 PM
werkt het nu wel?
 
Eventjes kijken hoor, want er zitten een paar foutjes in
Nope, nu logt ie helemaal niet meer in
 
zijn er foutmeldingen? ook niet als je connect.php in dit bestand include?
 
Helemaal niks
Alleen als ik connect.php niet include
Dan krijg ik errors dat hij de functies niet kan vinden
 
dan moet je het bestand met de functies ook include
 
Ja, dat is connect.php
Dus, jouw script zorgt dat alle users niet inloggen en die van mij zorgt dat alle users inloggen
:P
 
12:36 PM
vaag
 
Inderdaad, heel vaag
Ik zie dat jij op je site een inlog systeem hebt, mag ik jouw inlog functie misschien weten?
 
haha dat gaat met allemaal classes
 
Pff... Ook erg vervelent...
 
yep
maar ik kan je wel een paar functies geven
<?php
class Session {
public function set($name, $value) {
if(is_array($value)) {
$_SESSION[$name] = $value;
} else {
$_SESSION[$name] = base64_encode($value);
}
}

public function get($name) {
if($this->exists($name)) {
if(is_array($_SESSION[$name])) {
return $_SESSION[$name];
} else {
return base64_decode($_SESSION[$name]);
}
} else {
return false;
}
}

public function exists($name) {
if(!empty($_SESSION[$name])) {
return true;
} else {
return false;
}
}

public function delete($name) {
if($this->exists($name)) {
 
Thanks!
 
12:40 PM
public function isLoggedIn() {
if($this->_session->exists('user') && $this->_db->exists('id', 'users', 'id', $this->_session->get('user'))) {
return true;
} else {
return false;
}
}
 
O gosh... Is dat mysqli ?
 
yep
 
Ja... dat gaat nog wel even duren voordat ik dat snap...
 
<?php
class Database {
public $mysqli;

function __construct() {
$this->mysqli = new mysqli('localhost', '***', '****', '***');
}

public function detail($detail, $table, $column, $value) {
if(is_array($detail)) {
$data = array();

foreach($detail as $key) {
$stmt = $this->mysqli->prepare("SELECT `$key` FROM `$table` WHERE `$column` = ?");
if(is_numeric($value)) {
$stmt->bind_param('i', $value);
} else {
$stmt->bind_param('s', $value);
}
$stmt->execute();
$stmt->bind_result($detail);
$stmt->fetch();
Het meeste is ook zo om te zetten naar mysql
 
HE! Volgensmij begint ie te werken
 
12:43 PM
door wat?
 
In de mysql database de password varchar van 255 naar 32 veranderen...
Ik weet niet of ie al 100% werkt, maar het ziet er wel naar uit
 
dat zou niet moeten uitmaken. helemaal niet omdat je md5 gebruikt is 32 al genoeg dus 255 helemaal
 
Dat dacht ik dus ook, maar ik las net de comments van een video en daaar zeiden ze dat, ik dacht dat het het proberen wel waard was
 
als je goeie en duidelijke tutorials wil moet je hier s kijken youtube.com/user/phpacademy
 
Daar was ik ook aan het kijken :)
PHP Tutorials: Register & Login (Part 5): User Login (Part 3)
Die was ik aan het kijken, van PHPAcedemy natuurlijk
 
12:50 PM
heb je de oop versie of de gewone?
 
De gewone geloof ik
 
daar gebruikt hij dacht ik PDO maar dat doet in feite het zelfde als mysqli
maar wat gaat er nu nog fout?
 
Nee, volgensmij gebruikt hij gewoon MySQL
ATM gaat de user_active funtion een beetje fout
Hij doet niet helemaal wat ik wil
 
o ja die headers already send fout waar ik het eerder overhad
 
Euhm, volgensmij niet. Ik gebruik nu de code van jouw antwoord op de main post
 
12:55 PM
als je bij mijn site gergistreerd zou zijn en zou proberen in te loggen dan krijg je die ook. Dit komt namelijk doordat ik een cookie probeer aan te maken terwijl er al html output is. een cookie gebruikt ook de header() functie
 
Maar ik gebruik geen cookies en ben dat eigenlijk ook niet van plan
 
nee maar daar heb je dezelfde error
en wat gaat er fout in de user_active functie
 
ALLES. Of hij vind alle users '0' (dus dan moeten ze hun account nog activeren) of hij vind alle users '1'
 
en hoe ziet die functie er uit?
 
Op dit moment is alle oude code gecomment
function user_active($username) {
/*$result = mysql_query("SELECT `active` FROM `users` WHERE `username` = '{$username}'");
$return = mysql_result($result);
if ($return > 0 && $return < 3) return true;
else return false;*/

//return (mysql_result(mysql_query("SELECT COUNT(`user_id`) FROM `users` WHERE `username` = '$username' AND `active` > 0 AND `active` < 3"), 0) == 1) ? true : false;

$result = mysql_query("SELECT `active` FROM `users` WHERE `username` = '$username'");
$return = mysql_result($result); // Should be 0 or 1, I don't know this for sure. You need to try it out
Als je van $return dit zou maken (zoals in jouw antwoord)
$return = mysql_result($result,0);
is hij altijd gelijk aan 0
 
1:01 PM
owh oke
 
als je de 0 door een 1 zou vervangen krijg je het zelfde verhaal alleen dan met een 1
 
$result = mysql_query("SELECT `active` FROM `users` WHERE `username` = '$username'");

if($result) {
while($row = mysql_num_rows($result)) {
$active = $row['active'];
}

if($active == 1) {

} else {

}
}
en dat dan?
sorry verkeerde
$result = mysql_query("SELECT `active` FROM `users` WHERE `username` = '$username'");

if($result) {
while($row = mysql_fetch_assoc($result)) {
$active = $row['active'];
}

if($active == 1) {

} else {

}
}
 
Er staat nu '500' niks meer of minder... Dus een Internal Server Error... :(
 
hmm oke
t enige wat ik nog kan bedenken is dat je deze functie:

function detail($detail, $table, $column, $value) {
$stmt = $this->mysqli->prepare("SELECT `$detail` FROM `$table` WHERE `$column` = ?");
$stmt->bind_param('s', $value);
$stmt->execute();
$stmt->bind_result($detail);
$stmt->fetch();

return $detail;
}
omzet naar mysql
daarmee kan je dan heel makkelijk elk detail van een gebruiker ophalen
 
 
1 hour later…
2:33 PM
Naja, t lukt niet... Helaas. Ik geef het op, ik verzin wel even een lulverhaal en dan mankeer ik jouw antwoord als het oplossende antwoord. Echt heel erg bedankt voor je hulp!!
 

last day (14 days later) »