Почему не работает ajax?

Question

[hypero]

Всем привет.
Почему не работает ajax сохранение в бд?

<?php
require_once('head-start.php');
require_once('connect.php'); ?>

<div class="page">
    <div class="container">
        <h1>Изменяем запись</h1>
        <?php $id = $_GET['order_id'];
        $sql = mysqli_query($connect, "SELECT * FROM orders WHERE order_id = '$id'");
        $row = mysqli_fetch_assoc($sql);
        if(isset($_POST['change'])) {
            $order_name = $_POST['order_name'];
            $order_description = $_POST['order_desc'];
            $order_price = $_POST['order_price'];
            $user_id = $_POST['user_id'];
            $order_date = $_POST['order_date'];
            $change_sql = mysqli_query($connect, "UPDATE orders SET order_name='$order_name', order_description='$order_description', order_price='$order_price', user_id='$user_id', order_date='$order_date' WHERE order_id='$id'");
        }
        if($change_sql) {
            echo '1';
        } else {
            echo mysqli_error($connect);
        }
        ?>

        <form action="edit.php?order_id=<?php echo $id; ?>" method="post">
            <input type="text" value="<?php echo $row['order_name']; ?>" name="order_name" placeholder="Заголовок">
            <textarea name="order_desc" id="" cols="30" rows="10"><?php echo $row['order_description']; ?></textarea>
            <input type="text" value="<?php echo $row['order_price']; ?>" name="order_price" placeholder="Цена">
            <input type="text" value="<?php echo $row['user_id']; ?>" name="user_id" placeholder="Исполнитель">
            <input type="text" value="<?php echo $row['order_date']; ?>" name="order_date" placeholder="Дата">
            <input type="submit" name="change" value="Отправить">
        </form>
    </div>
</div>
<?php require_once('head-end.php'); ?>

$(document).ready(function(){
    $("form").submit(function(e){
        e.preventDefault();
        let order_name = $('input[name="order_name"]').val();
        let order_desc = $('textarea[name="order_desc"]').val();
        let order_price = $('input[name="order_price"]').val();
        let user_id = $('input[name="user_id"]').val();
        let order_date = $('input[name="order_date"]').val();
        $.ajax({
            type: "POST",
            url: "edit.php",
            dataType: "html",
            data: {
                order_name: order_name,
                order_description: order_desc,
                order_price: order_price,
                user_id: user_id,
                order_date: order_date
            },
            success: function(data) {
                alert(data);
            }
        });
    });
});

Comments

[alexalexes]

Пробивайте ошибки в запросах при помощи mysqli_error()
Но с легкого прищура на такое написание запроса сразу можно сказать, что будет ругаться на попытку сравнить order_id как число и строку '$id' (кавычки там стоят осознано, а не просто что, так принято подставлять параметры).
"SELECT * FROM orders WHERE order_id = '$id'"
PS: При подготовке запросов с параметрами, не применяйте конкатенацию строк. Для этого есть функции prepare и bind (а вообще, это называется подготавливаемые запросы). Это позволит исключить использование sql-инъекций на вашем коде.

[hypero]

alexalexes, без ajax всё норм, с ajax не добавляет

[alexalexes]

F12 в браузере, вкладка "Сеть", после отправки ajax, если посмотреть вкладку Ответ на запросе к edit.php что там выводится. Какой код статуса ответа?

[hypero]

alexalexes, 200

[scottparker]

hypero, где в данных отправляемых через ajax поле change?

[hypero]

scottparker, он нужен? там через событие submit

[alexalexes]

Во-первых:
У вас один из параметров передается get-ом:
$id = $_GET['order_id'];
Ajax передает все через метод POST, в url никак не фигурирует order_id как GET параметр, так что он будет неопределен, и запрос "SELECT * FROM orders WHERE order_id = '$id'" ничего не вернет. Запрос "UPDATE orders SET order_name='$order_name', order_description='$order_description', order_price='$order_price', user_id='$user_id', order_date='$order_date' WHERE order_id='$id'" выдаст ошибку.
Во-вторых:

url: "edit.php?order_id=" + какой-то_id, // а тут дописать get-параметр
            dataType: "html",
            data: {
                order_name: order_name,
                order_description: order_desc,
                order_price: order_price,
                user_id: user_id,
                order_date: order_date,
                change: 1 // нужно что-нибудь присвоить ченджу, чтобы сработал  if(isset($_POST['change']))
            },

[scottparker]

hypero, и что что через сабмит? проверка на наличие этого параметра в пост есть?

[hypero]

alexalexes, url: "edit.php?order_id=" + какой-то_id как я тут выведу id? я же не могу в js вписать php

[alexalexes]

Возьмите url, который указан в атрибуте action формы, возможно, там будет order_id. Но изначально, order_id должен быть в адресе страницы, на которой отрывается эта форма.

let action = $('form').attr('action');
        $.ajax({
            type: "POST",
            url: action,

Answer 1

[lUser L]

Потому что у формы экшон -
action="edit.php?order_id=<?php echo $id; ?>"

А у аякса
url: "edit.php",

spoiler

Вообще всем советую и себе и тебе, вообще всем - дебажить код. Научитесь дебажить код. Выдели для себя 1 день и посмотри, почитай статьи как правило дебажить.

Тут в принципе дело решается одним var_dump-ом в edit.php

Решение сам до думаешь ?

Upd

ну тогда очевидно нужно передать гет параметр, только не нужно пихать в джс пхп. Я пишу с телефона, но попробую: в форме добавь инпут с типом hidden

<input type="hidden" class="order-id" data-order-id="<?php echo $id; ?>">

Далее в аяксе:

$("form").submit(function(e){
        e.preventDefault();

 var id = $('.order-id').data('order-id');
        let order_name = $('input[name="order_name"]').val();
        let order_desc = $('textarea[name="order_desc"]').val();
        let order_price = $('input[name="order_price"]').val();
        let user_id = $('input[name="user_id"]').val();
        let order_date = $('input[name="order_date"]').val();
        $.ajax({
            type: "POST",
            url: `edit.php?order_id=${id}`,
            dataType: "html",
            data: {
                order_name: order_name,
                order_description: order_desc,
                order_price: order_price,
                user_id: user_id,
                order_date: order_date,
                change: true
            },
            success: function(data) {
                alert(data);
            }
        });
    });
});

Comments

[hypero]

Гет параметры надо прописать в ajax?

[lUser L]

hypero, откуда мне знать, я же не видел edit.php) но судя по коду -да

[hypero]

Эмиль 🔥, edit.php сверху.