Флажки и действия PHP

Question

У меня есть следующий код, показанный ниже, чтобы позволить администратору добавлять определенного пользователя в несколько ролей, выбирая из доступных ролей (флажки).Код прекрасно работает для достижения этой цели, но он также должен удалить пользователя, для которого когда-либо роль (флажок) не была выбрана.Флажки позволяют администратору как добавлять, так и удалять пользователя из выбранных ролей.Это та часть, которую я не могу понять.

Если бы кто-нибудь мог провести меня через это, это было бы действительно здорово.Спасибо!

<?php
// declare variables
$msg = '';

// ------------------------------------------------------------------
// UPDATE USER INFO
// ------------------------------------------------------------------
if(isset($_POST['UpdateUser']))
{
    // get user id from query string sent
    $sent_id = mysqli_real_escape_string($conn, $_GET['uid']);
    if(isset($sent_id) && !empty($sent_id) && is_numeric($sent_id) && $sent_id > 0)
    {
        $user_id = $sent_id;

        // ------------------------------------------------------------------
        // ADD CURRENT USER TO SELECTED ROLES
        // ------------------------------------------------------------------
        if(isset($_POST['checked']))
        {
            // get selected role names
            $checked = $_POST['checked'];

            $i1=0;
            $i2=0;
            foreach($checked as $role_name)
            {
                // get role id
                $get_role_id = mysqli_query($conn, "SELECT RoleId FROM roles WHERE RoleName = '$role_name'")
                or die($dataaccess_error.mysqli_error($conn));

                $row = mysqli_fetch_array($get_role_id);
                $role_id = $row['RoleId'];

                // check if user already exist in role
                $check_if_exist = mysqli_query($conn, "SELECT UserId, RoleId, RoleName FROM users_in_roles WHERE UserId = $user_id AND RoleId = $role_id AND RoleName = '$role_name' LIMIT 1")
                or die($dataaccess_error);

                if(mysqli_num_rows($check_if_exist) == 0)
                {
                    // add user to roles
                    $add_user_to_roles = mysqli_query($conn, "INSERT INTO users_in_roles(UserId, RoleId, RoleName) VALUES($user_id, $role_id, '$role_name')")
                    or die($dataaccess_error.mysqli_error($conn));

                    $count1 = $i1++ + 1;
                    $count2 = $i2;
                    $msg = "<div class='msgBox3'>SUCCESS: USER have been ADDED to ($count1) ROLES - AND ALREADY EXISTS in ($count2).</div>";
                }
                elseif(mysqli_num_rows($check_if_exist) == 1)
                {
                    $count1 = $i1;
                    $count2 = $i2++ +1;
                    $msg = "<div class='msgBox4'>NOTE: USER have been ADDED to ($count1) ROLES - AND ALREADY EXISTS in ($count2).</div>";
                }
            }
        }
        else
        {
            $msg = $msg_error2;
        }
    }
}
?>

Answer 1

Удалите все существующие отношения пользователя / роли, прежде чем создавать новые из представленных флажков. Вы также можете вставить все роли в один оператор MySQL, если вы установите значения флажка для role_id вместо role_name. Кроме того, у таблицы users_in_roles нет причин дублировать столбец role_name. Вот что я бы использовал (упрощенные псевдо-утверждения, конечно):

// New data structure
mysqli_query("DROP TABLE users_in_roles");
mysqli_query("CREATE TABLE users_in_roles ( user_id ..., role_id ....)");

....

// Build SQL fragments for multi-row INSERT.
$a = array();
foreach($checked as $role_id) {
 $a[]= sprintf('(%d, %d)', $user_id, $role_id);
}
// Delete existing user/role relationships for the user.
mysqli_query("DELETE FROM users_in_roles WHERE user_id = $user_id");
// Insert multiple rows in a single statement.
mysqli_query(
 sprintf('INSERT INTO users_in_roles (user_id, role_id) VALUES %s', 
   implode(', ', $a)));

Этот метод выполняет то же самое - используя только два SQL-оператора. Один недостаток: в маловероятном случае ошибки до / во время оператора INSERT роли не назначаются. Вы можете выполнить DELETE и INSERT в транзакции.

Answer 2

Простое решение состоит в том, чтобы использовать что-то кроме флажков, так как они не отправляют значения, когда не проверено .

В качестве альтернативы, вы можете оставить флажки и создать массив PHP возможных значений ролей. В вашем текущем цикле, вы можете удалить эти значения при проверке. После цикла в массиве останутся только непроверенные роли, и вы сможете удалить их соответствующим образом.

$roles = array('key' => 'Nice Name', 'admin' => 'Administrator');

Это можно использовать не только для вывода флажков, но и для управления ролями. В идеале это должно быть в таблице базы данных.