Почему mysqli_fetch_array() не получает параметры?

Question

[Stoble]

<?php 
  
    if(isset($_SESSION['cart'])){ 
          
        $sql="SELECT * FROM products WHERE id IN (";
        foreach($_SESSION['cart'] as $id => $value) { 
            $sql.=$id.","; 
        } 
        $sql=substr($sql, 0, -1).") ORDER BY name ASC"; 

        echo $sql;
        $query=mysqli_query($connect, $sql); 
        while($row=mysqli_fetch_array($query)){ 
              
        ?> 
            <p><?php echo $row['name'] ?> x <?php echo $_SESSION['cart'][$row['id_product']]['quantity'] ?></p> 
        <?php 
              
        } 
    ?> 
        <hr /> 
        <a href="index.php?page=cart">В корзину</a> 
    <?php 
          
    }else{ 
          
        echo "<p>Корзина пуста.</p>"; 
          
    } 
  
?>

Comments

[Stoble]

Вот какую ошибку выдает

[FanatPHP]

про SQL инъекции когда-нибудь слышал?

Answer 1

[Антон]

session_start() не забыли?
А в целом, что в вашем понимании означает "не получает параметры" ? Как они вообще в $_SESSION попадают?

Comments

[Stoble]

Попадают из другого кода, session_start() не забыл,если я правильно понял ,то ошибка в этой части кода

$sql="SELECT * FROM products WHERE id IN (";
        foreach($_SESSION['cart'] as $id => $value) { 
            $sql.=$id.","; 
        } 
        $sql=substr($sql, 0, -1).") ORDER BY name ASC";

[Антон]

Stoble, ну вот у вас там выводится $sql - сюда его напишите, что у вас там получается.
И, суда по комменту, у вас там ошибка в синтаксисе запроса. Её можно вывести с помощию mysqli_error()

[Stoble]

Вроде бы исправил, теперь другой вопрос: Как обрезать первую запятую в скобках

[Антон]

Stoble, если отвечать именно на вопрос - как обрезать первую запятую - то собрать айдишники в отдельную строку и также через функцию substr(). Другой вариант - совместить функции array_keys() - чтобы получить массив ключей из $_SESSION['cart'] и сделать join(",",$ids_array) - т.е. сразу готовую строку с айдишниками через запятую. Ещё проверьте что у вас в $_SESSION['cart'], потому что, судя по коду - у вас там пустой айдишник.
Ну и на закуску почитать про плейсхолдеры.

[Stoble]

Спасибо за ответ