La page landing.php bascule vers la page index.php puisque à la fin code proposé, j'ai ajouté une redirection sans condition vers la page index.php (// si $data['direction'] ne correspond pas aux valeurs testées...)
Il suffit de supprimer cette redirection pour afficher la page.
Tu n'as pas besoin de duppliquer le code pour faire la requête sql, tu peux récupérer les données de l'utilisateur connecté une seule fois.
Pour que les redirections PHP avec la header() fonctionnent correctement et pour éviter l'erreur "headers already sent", il ne faut effectivement aucun affichage ou sortie avant l'utilisation de la fonction header().
La solution recommandée est bien de déplacer tout le traitement php avant l'affichage html.
Egalement, il ne faut pas utiliser echo ou tout autre affichage avant les redirection. Ton message d'erreur pour 'current_password' ou 'success_password' doit donc être affiché dans ton document html.
Evite également de fermer la balise php pour la réouvrir juste après avec une ou plusieurs ligne vide car ceci peut aussi causer l'erreur "headers already sent" et empêcher les redirections php de fonctionner.
<?php
// landing.php
session_start();
require_once 'config.php'; // ajout connexion bdd
// si la session existe pas ou si l'on est pas connecté on redirige
if(!isset($_SESSION['user'])){
header('Location:index.php');
die();
}
if(isset($_GET['err'])){
$err = htmlspecialchars($_GET['err']);
switch($err){
case 'current_password':
$messageAlert = "<div class='alert alert-danger'>Le mot de passe actuel est incorrect</div>";
break;
case 'success_password':
$messageAlert = "<div class='alert alert-success'>Le mot de passe a bien été modifié ! </div>";
break;
}
}
// On récupere les données de l'utilisateur
$req = $bdd->prepare('SELECT * FROM utilisateurs WHERE token = ?');
$req->execute(array($_SESSION['user']));
$data = $req->fetch();
if (!$data) {
// $data vaut false, aucun utilisateur correspondant à token = $_SESSION['user'] dans la table
header('Location: index.php');
die();
}
if ($data['direction'] === 'Direction des Opérations') {
header('Location: direction_des_operations.php');
die();
}
if ($data['direction'] === 'Direction Juridique et Ressources Humaines') {
header('Location: direction_juridique_et_ressources_humaines.php');
die();
}
if ($data['direction'] === 'Direction Technique et Financière') {
header('Location: direction_technique_et_financiere.php');
die();
}
if ($data['direction'] === 'Direction Générale') {
header('Location: direction_generale.php');
die();
}
// si pas de redirection, on affiche le document html :
?>
<!doctype html>
<html lang="fr">
<head>
<title>Espace membre - suivi de l archivage</title>
<!-- Required meta tags -->
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
<!-- Bootstrap CSS -->
<link rel="stylesheet" href="LIEN BOOTSPRAP">
</head>
<body>
<?php if (isset($messageAlert)) echo $messageAlert; ?>
<!-- ... suite du document html ... -->
Comme indiqué dans mon premier message, le contrôle d'accès doit surtout être fait sur les pages cibles (direction_des_operations.php, direction_juridique_et_ressources_humaines.php, etc.).
A ta place, au lieu d'utiliser un input type="submit" pour faire les redirections en fonction de la colonne 'direction' depuis la page landing.php, j'aurai plutôt afficher un lien vers la page concerné en fonction de la colonne 'direction' dans le document html et fait le contrôle d'accès sur les pages cibles.
<?php
// landing.php
session_start();
require_once 'config.php'; // ajout connexion bdd
// si la session existe pas ou si l'on est pas connecté on redirige
if(!isset($_SESSION['user'])){
header('Location:index.php');
die();
}
if(isset($_GET['err'])){
$err = htmlspecialchars($_GET['err']);
switch($err){
case 'current_password':
$messageAlert = "<div class='alert alert-danger'>Le mot de passe actuel est incorrect</div>";
break;
case 'success_password':
$messageAlert = "<div class='alert alert-success'>Le mot de passe a bien été modifié ! </div>";
break;
}
}
// On récupere les données de l'utilisateur
$req = $bdd->prepare('SELECT * FROM utilisateurs WHERE token = ?');
$req->execute(array($_SESSION['user']));
$data = $req->fetch();
if (!$data) {
// $data vaut false, aucun utilisateur correspondant à token = $_SESSION['user'] dans la table
header('Location: index.php');
die();
}
?>
<!doctype html>
<html lang="fr">
<head>
<title>Espace membre - suivi de l archivage</title>
<!-- Required meta tags -->
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
<!-- Bootstrap CSS -->
<link rel="stylesheet" href="LIEN BOOTSPRAP">
</head>
<body>
<?php if (isset($messageAlert)) echo $messageAlert; ?>
<!-- ... suite du document html ... -->
<?php if ($data['direction'] === 'Direction des Opérations') : ?>
<a href="/direction_des_operations.php">Direction des Opérations</a>
<?php endif; ?
<?php if ($data['direction'] === 'Direction Juridique et Ressources Humaines') : ?>
<a href="/direction_juridique_et_ressources_humaines.php">Direction Juridique et Ressources Humaines</a>
<?php endif; ?
</body>
</html>
<?php
// direction_des_operations.php
session_start();
require_once 'config.php'; // ajout connexion bdd
// si la session existe pas ou si l'on est pas connecté on redirige
if(!isset($_SESSION['user'])){
header('Location:index.php');
die();
}
// On récupere les données de l'utilisateur
$req = $bdd->prepare('SELECT * FROM utilisateurs WHERE token = ?');
$req->execute(array($_SESSION['user']));
$data = $req->fetch();
if (!$data) {
// $data vaut false, aucun utilisateur correspondant à token = $_SESSION['user'] dans la table
header('Location: index.php');
die();
}
if ($data['direction'] !== 'Direction des Opérations') {
// l'utilisateur ne fait pas partie du groupe 'Direction des Opérations', accès interdit à cette page
header('Location: index.php');
die();
}
// si l'utilisateur n'est pas redirigé avec la condition juste avant, c'est qu'il fait bien partie du groupe 'Direction des Opérations', on peut donc afficher la page autorisée :
?>
<!doctype html>
<html lang="fr">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
<title>Direction des Opérations</title>
<!-- Bootstrap CSS -->
<link rel="stylesheet" href="LIEN BOOTSPRAP">
</head>
<body>
Page Direction des Opérations...
</body>
</html>
La page landing.php bascule vers la page index.php puisque à la fin code proposé, j'ai ajouté une redirection sans condition vers la page index.php (// si $data['direction'] ne correspond pas aux valeurs testées...)
Il suffit de supprimer cette redirection pour afficher la page.
Tu n'as pas besoin de duppliquer le code pour faire la requête sql, tu peux récupérer les données de l'utilisateur connecté une seule fois.
Pour que les redirections PHP avec la header() fonctionnent correctement et pour éviter l'erreur "headers already sent", il ne faut effectivement aucun affichage ou sortie avant l'utilisation de la fonction header().
La solution recommandée est bien de déplacer tout le traitement php avant l'affichage html.
Egalement, il ne faut pas utiliser echo ou tout autre affichage avant les redirection. Ton message d'erreur pour 'current_password' ou 'success_password' doit donc être affiché dans ton document html.
Evite également de fermer la balise php pour la réouvrir juste après avec une ou plusieurs ligne vide car ceci peut aussi causer l'erreur "headers already sent" et empêcher les redirections php de fonctionner.
[PHP]
<?php
// landing.php
session_start();
require_once 'config.php'; // ajout connexion bdd
// si la session existe pas ou si l'on est pas connecté on redirige
if(!isset($_SESSION['user'])){
header('Location:index.php');
die();
}
if(isset($_GET['err'])){
$err = htmlspecialchars($_GET['err']);
switch($err){
case 'current_password':
$messageAlert = "<div class='alert alert-danger'>Le mot de passe actuel est incorrect</div>";
break;
case 'success_password':
$messageAlert = "<div class='alert alert-success'>Le mot de passe a bien été modifié ! </div>";
break;
}
}
// On récupere les données de l'utilisateur
$req = $bdd->prepare('SELECT * FROM utilisateurs WHERE token = ?');
$req->execute(array($_SESSION['user']));
$data = $req->fetch();
if (!$data) {
// $data vaut false, aucun utilisateur correspondant à token = $_SESSION['user'] dans la table
header('Location: index.php');
die();
}
if ($data['direction'] === 'Direction des Opérations') {
header('Location: direction_des_operations.php');
die();
}
if ($data['direction'] === 'Direction Juridique et Ressources Humaines') {
header('Location: direction_juridique_et_ressources_humaines.php');
die();
}
if ($data['direction'] === 'Direction Technique et Financière') {
header('Location: direction_technique_et_financiere.php');
die();
}
if ($data['direction'] === 'Direction Générale') {
header('Location: direction_generale.php');
die();
}
// si pas de redirection, on affiche le document html :
?>
<!doctype html>
<html lang="fr">
<head>
<title>Espace membre - suivi de l archivage</title>
<!-- Required meta tags -->
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
<!-- Bootstrap CSS -->
<link rel="stylesheet" href="LIEN BOOTSPRAP">
</head>
<body>
<?php if (isset($messageAlert)) echo $messageAlert; ?>
<!-- ... suite du document html ... -->
[/PHP]
Comme indiqué dans mon premier message, le contrôle d'accès doit surtout être fait sur les pages cibles (direction_des_operations.php, direction_juridique_et_ressources_humaines.php, etc.).
A ta place, au lieu d'utiliser un input type="submit" pour faire les redirections en fonction de la colonne 'direction' depuis la page landing.php, j'aurai plutôt afficher un lien vers la page concerné en fonction de la colonne 'direction' dans le document html et fait le contrôle d'accès sur les pages cibles.
[PHP]
<?php
// landing.php
session_start();
require_once 'config.php'; // ajout connexion bdd
// si la session existe pas ou si l'on est pas connecté on redirige
if(!isset($_SESSION['user'])){
header('Location:index.php');
die();
}
if(isset($_GET['err'])){
$err = htmlspecialchars($_GET['err']);
switch($err){
case 'current_password':
$messageAlert = "<div class='alert alert-danger'>Le mot de passe actuel est incorrect</div>";
break;
case 'success_password':
$messageAlert = "<div class='alert alert-success'>Le mot de passe a bien été modifié ! </div>";
break;
}
}
// On récupere les données de l'utilisateur
$req = $bdd->prepare('SELECT * FROM utilisateurs WHERE token = ?');
$req->execute(array($_SESSION['user']));
$data = $req->fetch();
if (!$data) {
// $data vaut false, aucun utilisateur correspondant à token = $_SESSION['user'] dans la table
header('Location: index.php');
die();
}
?>
<!doctype html>
<html lang="fr">
<head>
<title>Espace membre - suivi de l archivage</title>
<!-- Required meta tags -->
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
<!-- Bootstrap CSS -->
<link rel="stylesheet" href="LIEN BOOTSPRAP">
</head>
<body>
<?php if (isset($messageAlert)) echo $messageAlert; ?>
<!-- ... suite du document html ... -->
<?php if ($data['direction'] === 'Direction des Opérations') : ?>
<a href="/direction_des_operations.php">Direction des Opérations</a>
<?php endif; ?
<?php if ($data['direction'] === 'Direction Juridique et Ressources Humaines') : ?>
<a href="/direction_juridique_et_ressources_humaines.php">Direction Juridique et Ressources Humaines</a>
<?php endif; ?
</body>
</html>
[/PHP]
[PHP]
<?php
// direction_des_operations.php
session_start();
require_once 'config.php'; // ajout connexion bdd
// si la session existe pas ou si l'on est pas connecté on redirige
if(!isset($_SESSION['user'])){
header('Location:index.php');
die();
}
// On récupere les données de l'utilisateur
$req = $bdd->prepare('SELECT * FROM utilisateurs WHERE token = ?');
$req->execute(array($_SESSION['user']));
$data = $req->fetch();
if (!$data) {
// $data vaut false, aucun utilisateur correspondant à token = $_SESSION['user'] dans la table
header('Location: index.php');
die();
}
if ($data['direction'] !== 'Direction des Opérations') {
// l'utilisateur ne fait pas partie du groupe 'Direction des Opérations', accès interdit à cette page
header('Location: index.php');
die();
}
// si l'utilisateur n'est pas redirigé avec la condition juste avant, c'est qu'il fait bien partie du groupe 'Direction des Opérations', on peut donc afficher la page autorisée :
?>
<!doctype html>
<html lang="fr">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
<title>Direction des Opérations</title>
<!-- Bootstrap CSS -->
<link rel="stylesheet" href="LIEN BOOTSPRAP">
</head>
<body>
Page Direction des Opérations...
</body>
</html>
[/PHP]
