Как можно скрыть раздел меню для неавторизованного пользователя?

Question

[Andre548]

Привет, ребят подскажите как можно скрыть раздел. Без цикла можно было бы использовать if else, а вот с циклом у меня прям проблема.
вот код

<ul class="list-inside bullet-list-item flex flex-wrap justify-between -mx-5 -my-2">
    <?php foreach ($menu as $value) { ?>
        <li class="px-5 py-2">
            <a class="<?= selectedMenu($value['path']) ? 'text-orange cursor-default' : 'text-gray-600 hover:text-orange' ?>"
               href="<?= $value['path'] ?>"><?= cutString($value['title']) ?></a></li>
    <?php } ?>
</ul>

Answer 1

[Сергей delphinpro]

Не здесь нужно скрывать.
В том месте где у вас формируется массив ссылок проверяйте уровень доступа и не добавляйте ссылки, если доступа нет.

Comments

[Andre548]

Была мысль такая, но не понимаю как потом в цикле данный момент реализовать

[Сергей delphinpro]

Andre548, как вы определяете уровень доступа (авторизован/неавторизован)? И как вы формируете массив $menu?

[Andre548]

Массив

<?php

function getMenu()
{
    $mainMenu = [
        [
            'title' => 'Главная',
            'path' => '/',
            'sort' => 1,
        ],
        [
            'title' => 'Каталог',
            'path' => '/templates/catalog/',
            'sort' => 2,
        ],
        [
            'title' => 'Сидки',
            'path' => '/templates/discounts/',
            'sort' => 3,
        ],
        [
            'title' => 'О нас',
            'path' => '/templates/about/',
            'sort' => 4,
        ],
    ];

    return $mainMenu;
}

и функция для сверки авторизован или нет

<?php

function isAuth() {
    $sessionName = $_SESSION['user_email'];
    if ($sessionName) {

        return true;
    }
    return false;
}

[Dark_Dante]

Andre548, ну все же просто

function getMenu(bool $isAuthorized = false)
{

    return array_merge([
            [
                'title' => 'Главная',
                'path' => '/',
                'sort' => 1,
            ],
            [
                'title' => 'Каталог',
                'path' => '/templates/catalog/',
                'sort' => 2,
            ],
            [
                'title' => 'Сидки',
                'path' => '/templates/discounts/',
                'sort' => 3,
            ],
        ],
        $isAuthorized ? [ //массив с пунктами меню, доступные только авторизованным
            [ 
                'title' => 'О нас',
                'path' => '/templates/about/',
                'sort' => 4,
            ],
        ] : [],
    );
}

[Andre548]

Dark_Dante, Я всего пару месяцев изучаю php, для меня пока не все так просто. Спасибо)

[Сергей delphinpro]

Ну не знаю. например так:

function getMenu($access = 'guest')
{
    $mainMenu = [
        [
            'title' => 'Главная',
            'path' => '/',
            'sort' => 1,
            'access' => ['user','guest'],
        ],
        [
            'title' => 'Каталог',
            'path' => '/templates/catalog/',
            'sort' => 2,
            'access' => ['user','guest'],
        ],
        [
            'title' => 'Сидки',
            'path' => '/templates/discounts/',
            'sort' => 3,
            'access' => ['user'], // только для зарегистрированных
        ],
        [
            'title' => 'О нас',
            'path' => '/templates/about/',
            'sort' => 4,
            'access' => ['user','guest'],
        ],
    ];

    return array_filter($mainMenu, function($item) use ($access) {
      return in_array($access, $item['access']);
    });
}

$menu = getMenu(isAuth() ? 'user' : 'guest');

[Dark_Dante]

Сергей delphinpro, Это ты ему сложно написал. Не догадается вынести роли в константы, будет сравнивать значения, где нибудь опечатается в текстовых значениях и результат окажется неожиданным

[Сергей delphinpro]

Dark_Dante, все мы ошибаемся =) на ошибках и учимся. Зато это гибкий подход. На случай увеличения количества ролей.

[Andre548]

Сергей delphinpro, Спасибо

Answer 2

[Dark_Dante]

<ul class="list-inside bullet-list-item flex flex-wrap justify-between -mx-5 -my-2">
    <?php 
 if ($isAuthorized) { //тут условие проверки авторизации
        foreach ($menu as $value) {
 ?>
        <li class="px-5 py-2">
            <a class="<?= selectedMenu($value['path']) ? 'text-orange cursor-default' : 'text-gray-600 hover:text-orange' ?>"
               href="<?= $value['path'] ?>"><?= cutString($value['title']) ?></a></li>
    <?php }
}
 ?>
</ul>

Comments

[Andre548]

При таком условии, он все меню скроет, если не авторизован, а нужно 1 пункт только. Была мысль в массиве с меню добавить к пункту значение 'только для авторизированных' но как потом в цикле это проверить не пойму

[Dark_Dante]

Andre548, ну так перенеси условие где формируется меню

<ul class="list-inside bullet-list-item flex flex-wrap justify-between -mx-5 -my-2">
    <?php foreach ($menu as $value) {
        if (/*тут условие выводить или не выводит пункт меню*/) {
         ?>
        <li class="px-5 py-2">
            <a class="<?= selectedMenu($value['path']) ? 'text-orange cursor-default' : 'text-gray-600 hover:text-orange' ?>"
               href="<?= $value['path'] ?>"><?= cutString($value['title']) ?></a></li>
    <?php 
        } 
    }
    ?>
</ul>

В чем проблемы то? А вообще человек ниже тебе верно сказал, это надо делать не тут, а логике формирования данных для ссылок в меню

[Andre548]

Dark_Dante, Спасибо, буду пробовать