Problem z sesją w PHP - Nie mogę się odnieść do id użytkownika Opcje

[Baku12345]

Witam
Działanie mojego skryptu ma być proste, ma polegać na tym, że po zalogowaniu się danego użytkownika wyświetli mu się lista kursów ze statusami do których ma dostęp a do których nie.

Mam w bazie między innymi takie tabele jak:
- users (id, name, login, password) - tabela przechowuje dane użytkownika
- courses (id, lp, name, url, category_id) - tabela przechowuje kursy, linki do nich i id kategorii do jakiej należy dany kurs
- access_courses (id, id_user, id_course, status) - tabela przechowuje informacje który użytkownik ma dostęp do którego kursu i wyświetla odpowiedni status

To mi działa w połączeniu z kodem, który sobie napisałem, ale problem jest w linii 75. Dokładnie w zapytaniu

[SQL] pobierz, plaintext 
SELECT * FROM courses c, access_courses ac WHERE ac.id_user = $_SESSION['id'] && ac.id_course = c.id ORDER BY lp ASC
[SQL] pobierz, plaintext 

Jeżeli to wstawię to jest problem, natomiast jeżeli wstawię na sztywno id użytkownika np. tak

[SQL] pobierz, plaintext 
SELECT * FROM courses c, access_courses ac WHERE ac.id_user = 1 && ac.id_course = c.id ORDER BY lp ASC
[SQL] pobierz, plaintext 

to wszystko jest idealnie. Jednak tak jak wspomniałem będzie więcej użytkowników i id chcę pobierać z sesji.

Poniżej umieszczam swój fragment kodu. Proszę o przeanalizowanie go i podpowiedź jak to naprawić.

[PHP] pobierz, plaintext 
<?php
    .
    . // tu nieistotny fragment kodu
    .
    if (isset($_POST['login']) && isset($_POST['password']))
        {
            if (!empty($_POST['login']) && !empty($_POST['password']))
                {
                    $login = addslashes(strip_tags($_POST['login']));
                    $password = addslashes(strip_tags(md5($_POST['password'])));
 
                    $stmt = $pdo->prepare("SELECT * FROM users where login = '".$login."' && password = '".$password."';");
                    $stmt->execute();
 
                    if ($row = $stmt->fetchAll(PDO::FETCH_ASSOC))
                        {
                            $_SESSION['id'] = $row['id'];
                            $_SESSION['online'] = 1;
                            $_SESSION['time'] = time();
                            $_SESSION['user'] = $login;
                            $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
                        }
                    else
                        {
                            $logon = new cTemplate;
 
                            $logon->assign['msg'] = "podałeś błędny login lub hasło!<br>spróbuj ponownie";
                            $page->assign['container'] = $logon->parse("templates/logon.html");
                            $page->assign['title'] = "Logowanie";
                            $main->assign['page'] = $page->parse("templates/page.html");
                            echo $main->parse("templates/index.html");
                            exit;
                        }
                }
            else
                {
                    $logon = new cTemplate;
 
                    $logon->assign['msg'] = "nie wprowadziłeś wszystkich danych<br>spróbuj ponownie";
                    $page->assign['container'] = $logon->parse("templates/logon.html");
                    $page->assign['title'] = "Logowanie";
                    $main->assign['page'] = $page->parse("templates/page.html");
                    echo $main->parse("templates/index.html");
                    exit;
                }
        }
 
    if (isset($_SESSION['online']) && isset($_SESSION['time']) && $_SESSION['user'] && ($_SESSION['online']===1))
        {
            if (time()-$_SESSION['time'] < 900)
                {
                    $_SESSION['time'] = time();
 
                    if (!isset($_GET['action']))
                        {
                            $_GET['action'] = "";
                        }
 
                    .
                    . // tu nieistotny fragment kodu
                    .
 
                    switch ($_GET['action'])
                        {
                            default:
                                if (isset($_GET['cat_id']))
                                    {
                                        $cat_id = $_GET['cat_id'];
 
                                        $stmt = $pdo->prepare("SELECT * FROM courses c, access_courses ac WHERE ac.id_user = 1 && ac.id_course = c.id && c.category_id = $cat_id ORDER BY lp ASC");
                                        $stmt->execute();
                                    }
                                else
                                    {
                                        $stmt = $pdo->prepare("SELECT * FROM courses c, access_courses ac WHERE ac.id_user = $_SESSION['id'] && ac.id_course = c.id ORDER BY lp ASC");
                                        $stmt->execute();
                                    }
 
                                .
                                . // tu fragment wyświetlający listę kursów dostępnych dla danego użytkownika
                                .
                        }
 
                    $page->assign['container'] = $panel->parse("templates/panel.html");
                }
            else
                {
                    $logon = new cTemplate;
 
                    $logon->assign['msg'] = "przekroczono czas bezczynności<br>zaloguj się ponownie";
                    $page->assign['container'] = $logon->parse("templates/logon.html");
                    $page->assign['title'] = "Logowanie";
                }
        }
    else
        {
            $logon = new cTemplate;
 
            $logon->assign['msg'] = "<br>podaj login i hasło<br>";
            $page->assign['container'] = $logon->parse("templates/logon.html");
            $page->assign['title'] = "Witaj na portalu";
        }
 
    $main->assign['page'] = $page->parse("templates/page.html");
 
    echo $main->parse("templates/index.html");
?>
[PHP] pobierz, plaintext 

Wydaje mi się że problem może być z tym że sesja nie jest globalna, ale może się mylę. Proszę o pomoc.
Z góry bardzo dziękuję.

[Baku12345]

A no faktycznie głupi błąd, powinienem był pobrać tylko jeden wiersz spełniający warunek czyli wstawić

[PHP] pobierz, plaintext 
$_SESSION['id'] = $row[0]['id'];
$_SESSION['user'] = $row[0]['name'];
[PHP] pobierz, plaintext 

zamiast

[PHP] pobierz, plaintext 
$_SESSION['id'] = $row['id'];
$_SESSION['user'] = $row['name'];
[PHP] pobierz, plaintext 

Poza tym błąd miałem w warunku pobierałem

[PHP] pobierz, plaintext 
if (isset($_SESSION['id']) && isset($_SESSION['time']) && isset($_SESSION['name']))
[PHP] pobierz, plaintext 

zamiast

[PHP] pobierz, plaintext 
if (isset($_SESSION['id']) && isset($_SESSION['time']) && isset($_SESSION['user']))
[PHP] pobierz, plaintext 

Dzięki wszystkim za podpowiedzi, teraz działają mi już obie wersje skryptu (IMG:style_emoticons/default/smile.gif)

Mam jeszcze tylko jedno ostatnie pytanko odnośnie
Usuwanie znaków z hasła też specjalnie mądre nie jest.

Teraz mój kod w części odpowiadającej za logowanie wygląda tak

[PHP] pobierz, plaintext 
if (isset($_POST['login']) && isset($_POST['password']))
        {
            if (!empty($_POST['login']) && !empty($_POST['password']))
                {
                    $login = addslashes(strip_tags($_POST['login']));
                    $password = hash_hmac('sha256', $_POST['password'], '2NYbH5qS8J');
 
                    $stmt = $pdo->prepare("SELECT * FROM users WHERE login = :login && password = :password");
                    $stmt -> execute(array(':login' => $login, ':password' => $password));
 
                    if ($row = $stmt->fetch(PDO::FETCH_ASSOC))
                        {
                            $_SESSION[$session_data] = array('id' => $row['id'], 'time' => $now, 'user' => $row['name']);
                        }
                    else
                        {
                            echo "Podałeś błędny login lub hasło";
                            exit;
                        }
                }
            else
                {
                    echo "Nie wprowadzłeś wszystkich danych";
                    exit;
                }
        }
[PHP] pobierz, plaintext 

Czy teraz jest już ok, w sensie bezpiecznie? Czy addslashes i strip_tags przy loginie jest uzasadnione czy może pominąć? I podobne pytanie przy haśle czy dodać addslashes? A jeśli tak to w takiej formie

[PHP] pobierz, plaintext 
addslashes(hash_hmac('sha256', $_POST['password'], '2NYbH5qS8J'));
[PHP] pobierz, plaintext 

czy takiej

[PHP] pobierz, plaintext 
hash_hmac('sha256', addslashes($_POST['password']), '2NYbH5qS8J');
[PHP] pobierz, plaintext 

Jeszcze raz dzięki za odpowiedzi, to już ostatnie moje pytanie w tym temacie (IMG:style_emoticons/default/smile.gif)

Ten post edytował Baku12345 23.09.2016, 05:55:39