Witam

Cały swój projekt zmieniam na biblioteki PDO (dużo roboty z tym mam) i często wyskakują mi błędy. Generalnie z wszystkim sobie radzę, ale nad systemem logowania siedze od godziny i już brakuje mi pomysłów. Skrypt jest od turson (turson.pl).

Rejestracja działą prawidłowo, wszystko zapisuje do bazy itp.

plik config jest require w index.php, tam natomiast include rejestracja i logowanie. (rejestracja działa, jak już pisałem)

[PHP] pobierz, plaintext 
<?PHP
//Formularz rejestracji
function form(){
    echo '
<form method="post" action="">
 <label for="login">Login:</label>
 <input maxlength="32" type="text" name="login" id="login" />
 
 <label for="pass">Hasło:</label>
 <input maxlength="32" type="password" name="password" id="pass" />
 
 <label for="pass_again">Hasło (ponownie):</label>
 <input maxlength="32" type="password" name="password2" id="pass_again" />
 
 <label for="email">Email:</label>
 <input type="text" name="email" maxlength="50" id="email" />
 
 <label for="email_again">Podaj swój Battle.tag: (nie wymagane)</label>
 <input type="text" maxlength="255" name="battle" id="btag" />#<input type="text" maxlength="255" name="tag" id="btag" /><br />
 
 
 <input type="submit" name="wyslano" value="Zarejestruj" />
</form>
 
    ';
}
 
 
if(isset($_POST['wyslano'])){ //Sprawdzamy, czy submit został wciśnięty
    //Usuwamy białe znaki z przesłanych danych
    $login = trim($_POST['login']);
    $password = trim($_POST['password']);
    $password2 = trim($_POST['password2']);
    $email = trim($_POST['email']);
    $battle = trim($_POST['battle']);
    $tag = trim($_POST['tag']);
 
    $errors = NULL; //Tworzymy zmienną przechowująca ewentualne błędy
 
    //Sprawdzamy, czy przesłane dane mają odpowiednią ilość znaków
    if(strlen($login)<3) $errors .= 'Login musi zawierać co najmniej 3 znaki<br>';
    if(strlen($password)<6) $errors .= 'Hasło musi zawierać co najmniej 6 znaków<br>';
    if($password!==$password2) $errors .= 'Hasła nie są takie same<br>';
    if(!preg_match('/\@/', $email) || strlen($email)<5) $errors .= 'Podany adres e-mail jest nieprawidłowy<br>';
 
    //Sprawdzamy czy użytkownik o takim samym loginie już nie istnieje
    $stmt = $baza->prepare("SELECT COUNT(id) FROM users WHERE login=:login");
    $stmt->bindValue(":login", $login, PDO::PARAM_STR);
    $stmt->execute();
    $row = $stmt->fetch();
    if($row[0]>0) $errors .= 'Konto o takim loginie już istnieje<br>';
 
    //Sprawdzamy czy użytkownik o takim samym adresie e-mail już nie istnieje
    $stmt2 = $baza->prepare("SELECT COUNT(id) FROM users WHERE email=:email");
    $stmt2->bindValue(":email", $email, PDO::PARAM_STR);
    $stmt2->execute();
    $row2 = $stmt2->fetch();
    if($row2[0]>0) $errors .= 'Konto o takim adresie e-mail już istnieje<br>';
 
    if(empty($errors)){ //Jeśli nie ma błędów, rejestrujemy użytkownika
        $password = sha1($password); //kodujemy hasło
        $dodaj = $baza->prepare("INSERT INTO users (login, pass, email, date,  battle, tag) VALUES(:login, :password, :email, NOW(),  :battle, :tag)");
        $dodaj->bindValue(":login", $login, PDO::PARAM_STR);
        $dodaj->bindValue(":password", $password, PDO::PARAM_STR);
        $dodaj->bindValue(":email", $email, PDO::PARAM_STR);
        $dodaj->bindValue(":battle", $battle, PDO::PARAM_STR);
        $dodaj->bindValue(":tag", $tag, PDO::PARAM_STR);
        $dodaj->execute();
        echo "Zarejestrowałeś się. Możesz się teraz <a href=\"login.php\">zalogować</a>";
    }
    else{
        echo '<div style="color:red">'.$errors.'</div>';
        form(); //Wyświetlamy formularz
    }
}
else form();
?>
[PHP] pobierz, plaintext 

A o to logowanie:

[PHP] pobierz, plaintext 
<?PHP
//Formularz logowania
function form(){
    echo '
        <form action="" method="post">
            <label for="login">Login</label>
            <input type="text" name="login" required>
            <br>
            <label for="password">Hasło</label>
            <input type="password" name="pass" required>
            <br>
            <input type="submit" name="wyslano" value="Zaloguj się"/>
        </form>
    ';
}
 
if(isset($_POST['wyslano'])){ //Sprawdzamy, czy submit został wciśnięty
    //Usuwamy białe znaki z przesłanych danych
    $login = trim($_POST['login']);
    $pass = trim($_POST['pass']);
 
    //Kodujemy hasło - przy rejestracji również je zakodowaliśmy, wiec przy porównywaniu musi być zakodowane
    $pass = sha1($pass);
 
 
    //Sprawdzamy czy użytkownik o podanych danych istnieje
    $stmt = $baza->prepare("SELECT login, pass FROM users WHERE login=:login AND pass=:pass");
    $stmt->bindValue(":login", $login, PDO::PARAM_STR);
    $stmt->bindValue(":pass", $pass, PDO::PARAM_STR);
    $stmt->execute();
    $row = $stmt->fetch(PDO::FETCH_ASSOC);
    if($stmt->rowCount()!=0){
        echo "Zalogowałeś się!";
        /*
         * Tworzymy sesję dla zalogowanego uzytkownika z:
         * - informacja, ze uzytkownik jest zalogowany
         * - jego id
         */
        session_start();
        $_SESSION['logged'] = true;
        $_SESSION['user_id'] = $row['id'];
 
    }
    else{
        echo '<div style="color:red">Login i/lub hasło są nieprawidłowe</div>';
        form();
    }
}
else form();
?>
[PHP] pobierz, plaintext 

Problem: po wpisaniu login i pass pokazuje Login i/lub hasło są nieprawidłowe.
Tak, tak, ale sytuacja taka, że login i hasło są na 100% prawidłowe. Założyłem już 11 kont i wszędzie ten sam problem. Już nie wiem na co patrzeć...

Struktura bazy danych:

users
id - login - pass - email - date - battle - tag - grupa (interesuje nas przy logowaniu tylko login i pass, prawda?)

http://pl1.php.net/manual/en/pdostatement.rowcount.php


No, i tyle w temacie

Skoro przy logowaniu pobierasz $row, to sprawdzaj czy ten $row jest poprawny a nie ilosc

"najprostsze rozwiązania są najlepsze"

dzięki za rozwiązanie, działa. Nie wiem czemu tego nie zauważyłem.

Jeszcze coś mi nie wychodzi, obojętnie jaki user / pass to i tak loguje. Z tym sprawdzeniem $row coś źle kombinuje.

[PHP] pobierz, plaintext 
    $stmt = $baza->prepare("SELECT login, pass FROM smf_members WHERE login=:login AND pass=:pass");
    $stmt->bindValue(":login", $login, PDO::PARAM_STR);
    $stmt->bindValue(":pass", $pass, PDO::PARAM_STR);
    $stmt->execute();
    $row = $stmt->fetch(PDO::FETCH_ASSOC);
    if($stmt!=0){
        echo "Zalogowałeś się!";
        /*
         * Tworzymy sesję dla zalogowanego uzytkownika z:
         * - informacja, ze uzytkownik jest zalogowany
         * - jego id
         */
        session_start();
        $_SESSION['logged'] = true;
        $_SESSION['user_id'] = $row['id'];
 
    }
    else{
        echo '<div style="color:red">Login i/lub hasło są nieprawidłowe</div>';
        form();
    }
[PHP] pobierz, plaintext 

if($stmt!=0){
a nie
if($stmt->rowCount()!=0){

if($stmt->rowCount()!=0){

cały czas pokazuje nieprawidłowy user / pass, a jest prawidłowe na 100%. Widać mam coś z bazą źle lub rejestracją.

Wyświetl sobie przez echo zmienne $login i $pass, może hasło trzeba zakodować np. sha1 przy logowaniu bo rejestrując kodujesz?

A czego ty robisz if ($stmt!=0){
Przeciez wyraznie ci mowilem ze masz sprawdzac $row.... czy ja po chinsku pisze? $stmt zawsze jest true, nawet jak nie znajdzie rekordu.... to rekord/$row masz sprawdzac

Działą tylko z if($stmt->rowCount()!=0){, warunku na $row coś mi nie wychodziło...

Okazało się, że problemem jest baza danych, pole na hasło miało maksymalnie 32 znaki, podczas kiedy kodowanie wymagało ich więcej.

Można zamknąć


edit: rozwiązane

$nurl->Url('log=profil?id='.$_SESSION['tutaj_problem_mam'].'')
jaki problem?
Jeżeli w logowaniu zapisałeś id usera:
$_SESSION['user_id'] = $row['id'];
to powinieneś wiedzieć co wstawić

nie wiem jak kopiujesz gotowce, że pominąłeś session_start na stronie głównej.


Czy to mysql, czy mysqli czy pdo to zapytania niczym sie nie roznia.