Discussion :

[andaman]

Bonjour,

Je ne comprends pas mon erreur suivante. Il me dit que cela présente une boucle de redirection. Je n'ai jamais eu cela.

J'ai bien session_start en haut de page. Et quand je fais un print_r je n'ai rien en session.

la fonction isLogged :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
public function isLogged(){
        return(isset($_SESSION['bo'])) ? true : false;
 }

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
if($auth->isLogged() == true){
    echo 'true';
}else{
     echo 'false';
     header("Location: index.php?section=login");
}

Merci d'avance

[sabotage]

Déjà tu ne peux pas faire echo + header.

Ensuite pour pouvoir te répondre il manque un peu des explications : ou sont positionnés les bouts de code que tu nous montres et dans quel cas (true / false) du if tombe tu ?

[andaman]

Merci sabotage

je n'avais posté que le code qui me semblait nécéssaire
Donc voici comme demandé. Mais le problème reste le même...

la page index.php

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
session_start();
ob_start();
 
require 'tools/config.php';
require 'tools/server.php';
require 'classes/database.class.php';
 
spl_autoload_register(function($class){
    include 'classes/'.$class.'.class.php';
});
 
$html = new PageHtml($name_website, $address_back);
$auth = new Auth();
 
if($auth->isLogged() == true){
    include 'tools/menu.php';
}else{
    header("Location: index.php?section=login");
}
$html->footerAdmin($name_website, $address_back);
ob_end_flush();
echo '<pre>';
print_r($_SESSION);
echo '<pre>';

et la classe (elle n'est pas finie) qui va avec:

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
class Auth{
    public $login;
    public $pswd;
    public $email;
 
    public function __construct() {
        echo'coucou la classe<br>';
    }
    public function setLogin($login){
        $this->login = $login;
    }
 
    /**
     * Permet de voir si l'utilisateur est en session
     * @return BOLEEN
     */
    public function isLogged(){
        return(isset($_SESSION['bo']) && !empty($_SESSION['bo'])) ? true : false;
    }
}

Merci d'avance

[sabotage]

Tu fais une redirection de index.php vers index.php ca fait donc une boucle.

Au passage, isset() est déjà inclus dans !empty()

[andaman]

Merci sabotage

Donc je peux faire simplement ceci?

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
return(isset($_SESSION['bo'])) ? true : false;

Je viens de modifier mon code en faisant ceci et ce la fonctionne. est ce la bonne manière?

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
if($auth->isLogged() == true){
    include 'tools/menu.php';
}else{
    if(empty($_GET['section'])){
        header("Location: index.php?lang=fr&section=login");
    }
}

Merci d'avance

[sabotage]

C'est l'inverse : c'est isset() qui est inclus dans !empty()

Pour ta condition, c'est difficile de te répondre sans connaitre toute ta logique de sécurité. Prends juste bien en compte ce qui se passe dans les cas qui ne sont pas dans la condition :
Par exemple si on n'est pas loggué mais qu'on fourni une section.