je bug sur une condition multiple

bonsoir
déjà on ne fais pas de comparaison a une fonction ton empty.

on va simplifier la chose.
on vérifie dans un 1er temps que ma session auth ne soit pas vide. je suppose qu'il aura une autorisation à 0 ou 1 par défaut.

il faut pour modifier un post que l'utilisateur est une autorisation superieur ou égale à 2 OU qu'il soit l'auteur du post

Hello :)

Alors pour les conditions... Disons que c'est entièrement à refaire :p

Bon, le mieux serait de prendre une logique de php orienté objet mais en procédural:

<?php

// En POO ce serait un modèle, j'ai reproduis un tableau, et je fais retourner false si il n'y a pas de session auth
function current_user($reps) {
    if (!isset($_SESSION['auth'])) return false;
    $auth = $_SESSION['auth'];
    $role = $auth?->authorisation;
    $id = $auth?->id;
    $user_id = $reps->userid;

    return [
        $auth,
        $role,
        $id,
        $user_id,
    ];
}

// On vérifie que le user est l'auteur
function is_owner($user) {
    return $user['id'] == $user['user_id'];
}

// On vérifie si le user est un admin ou un modo
function is_elevated($user) {
    return in_array($user['role'], [2, 3]);
}

// On vérifie si la personne peut éditer, la fontion utilise les deux fonctions précédentes
function can_edit($user) {
    return is_owner($user) || is_elevated($user);
}

// On enregistre le current user dans une variable, plus pratique
$user = current_user();

// Et on attaque la condition
if ($user && can_edit($user)) {

?>

<a class="p-2 bd-highlight" href="<?= $router->generate('editerep', ['id' => $reps->topicsrep]) ?>">
    <i class="far fa-edit"></i> Editer
</a>

<?php } ?>

Voila, il y a beaucoup plus de code mais au moins c'est lisible, du coup plus simple à débugger en cas de problème :p

D'aileurs regardes des cours PHP sur le MVC, ça va pas mal te faciliter la gestion des users, et même la POO (class, interfaces, traits, contracts)

Bonjour,

Cette condition semble être correcte ?

en tout cas elle fontionne, je suis bête de vérifiez 3 et 4 (sauf si je peut le faire avec un array) alors qu'il suffit de faire supérieur ou égal et je vérifie la session avant, vue comme ça je peut mieux comprendre la version POO car j'ai 2 version de la même app une procédural et une poo (qui est en pause a cause de mon router j'ai pas compris comme l'utiliser dans mes class et en dehors).

juste pour savoir, ça || est egal a OR et && et egale a AND ?

          <?php 
            if (isset($_SESSION['auth'])){
              if($_SESSION['auth']->id == $topic->usersid or $_SESSION["auth"]->authorization >= 3) { 
          ?>
            <a class="p-2 bd-highlight" href="<?= $router->generate('editetopic', ['id' => $topic->topicsid]) ?>"><i class="far fa-edit"></i> Editer</a> 
            <?php 
              }
            } 
          ?>

Alors ça parrait bien, si tu utilises PHP8 tu peux encore plus simplifier ta condition en virant le isset:

<?php 
    if($_SESSION['auth']?->id == $topic->usersid or $_SESSION["auth"]?->authorization >= 3) { 
?>

<a class="p-2 bd-highlight" href="<?= $router->generate('editetopic', ['id' => $topic->topicsid]) ?>"><i class="far fa-edit"></i> Editer</a> 

<?php 
} 
?>

Le ? il sert de isset, si ça n'existe pas ça renverra null, et dans un if le null est traité comme un false (enfin sauf si tu fais une comparaison strict avec ===)

Ensuite dans ta condition faudrait pas mettre >= 2 au lieux de 3 pour autoriser les modos?

Et oui, || c'est OR et && c'est AND :p

Alors je suis en php 7 car html purifier 4.15 ne fonctionne pas en php 8 et donc le ? ne fonctionne pas non plus.

Par contre on est d'accord pour dire que si je post un topic ou édite un topic et une réponse c'est pareil je dois vérifié si on a une session et si l'utilisateur est un membre un modo ou admin

Pour ce faire j'ai fais cette vérif car ça sert a rien d'afficher un lien ou non si le script n'est pas sur.

if (isset($_SESSION['auth'])){

     $firstTopic = $db->prepare("SELECT id, f_topic_content, f_user_id FROM f_topics WHERE id = ? ");

    $firstTopic->execute([intval($match['params']['id'])]);

    $topic = $firstTopic->fetch();

    if($_SESSION['auth']->id == $topic->f_user_id or $_SESSION["auth"]->authorization >= 2){

        /**
        * La sauvegarde
        **/
        if(isset($_POST['topics'])){
            checkCsrf();//on verifie les faille csrf
            $content = trim($_POST['f_topic_content']);
            $error = '';
            if(grapheme_strlen($content) < 100){

                $error .= errors(['content ok']);

            }
            if(empty($error)){

                $u = [$content, intval($params['id'])];

                $db->prepare("UPDATE f_topics SET f_topic_content = ? WHERE id = ?")->execute($u);

                    setFlash('Votre message a bien étais modifier');

                    redirect($router->generate('viewtopic', ['id' => $params['id']]));
            }

        }

    }else{

        setFlash("Vous n'avez pas le bon rang pour editer ce topic ou ce topic n'est pas le votre",'orange');
        redirect('forum');
    }
}else{

    setFlash('Vous devez être connecter pour acceder a cette page','orange');
    redirect('forum');
}

Dernière chose, peut-on faire une vérification comme ceci ? if($_SESSION["auth"]->authorization == in_array($_SESSION["auth"]->authorization, [2,3])) ça parait plus facile a maintenir et au moins ça vérifie seulement si 2 et 3 exist

C'est vrai que toutes mes imbrication c'est pas fameux !

je remet tout mon code au propre donc c'est l'occation de mieux faire, je met des redirection pour évité de retrouver les token en get, a la base c'est ce que je faisait mais j'utilise AltoRouter maintenant ce qui change pas mal de chose par exemple j'ai plus besoin de vérifié si j'ai un id en get il le gère tout seul, j'ai juste fais une condition avec les fonctions

is_array($match) is_callable($match['target']) call_user_func_array($match['target'], $match['params'])

si la route ne match pas je redirige tout vers la page d'erreur, ça gère même GET|POST

$router->map('GET'     , '/forum-[*:slug]-[i:id]', 'viewforums','forum-tags');
$match = $router->match();

et pour cette condition que tu souligne

if(grapheme_strlen($content) < 100){
    // Euh la ça veut dire que même si le message est ok tu auras un champ dans $error, du coup la condition suivante elle va etre à false vu que c'est pas empty nan?
    $error .= errors(['content ok']);
}

dans un premier j'ai mis grapheme parce que 1 smiley ça fait plusieurs octets d'où l'utilisation de utf8_mb4 et donc ça peut valoir plus qu'un caractères le simple strlen faisait pas la différence, et ça fonctionne comme c'est censé fonctionner si quelqu'un post du vide ou moins de 100 caractères ça me fait l'erreur, voilà pourquoi je vérifie pas empty ça me parait pas pertinant vu que la fonction vérifie le nombre de caractères je ne vérifie que dans le value du formulaire, mais est-ce que les espaces sont des caractères ??

J'ai plus qu'a géré les redirections avec les maxpage et j'aurais fini il me manque juste la page qui va se créer quand le max perpage atteind sa limite l'ors de la création d'une réponse, ensuite je passe a poo a 100%.