Как изменить строку из БД?

Question

[Lpfo]

<form class="msg-wall" method="post" action="/vendor/wall.php">
            <p>
                Message Wall:
            </p>
            <input type="text" name="wall" placeholder="What are you thinking about?">
            <button type="submit">Publish</button>

            <?php foreach($check_message as $row) { ?>
                <div class="content">
                    <p class="msg"><?= $row['message'] ?></p>
                    <input type="text" name="edit">
                    <a href="?ed=<?= $row['id'] ?>">edit</a>
                </div>
            <?php } ?>

</form>

Логика вроде простая. В параграфе какая-то строка из БД, я хочу ее просто изменить на текст, который введу в input. После нажатия на ссылку с текстом edit изменения должны отобразиться.

Вот так я пытался реализовать:

if (isset($_GET['ed'])) {
    $id = $_GET['ed'];
    $value = $_POST['edit'];
    mysqli_query($connect, "UPDATE message SET message = '$value' WHERE id = $id");
    header('Location: ../profile.php');
}

Это работает, если вместо $value я передаю любой конкретно заданный текст. А здесь просто пустая строка получается, видимо в $value ничего не лежит почему-то...
Как можно исправить мой код, чтобы он работал?

Answer 1

[AUser0]

Поскольку у вас в определении формы в action не вписан никакой id (например action="/vendor/wall.php?id=111"), то и переменная $_GET['id'] во время исполнения PHP-скрипта не определена, не существует, не назначена, поэтому и код внутри if (isset($_GET['id'])) не должен отрабатываться.

Но лучше не вписывать id внутрь action, а добавить в форму скрытую переменную <input type=hidden name=id value='{$row['id']}'>, и поменять в скрипте на $_POST['id'].

Comments

[Lpfo]

Как-то так надо сделать?

<form class="msg-wall" method="post" action="/vendor/wall.php">
            <p>
                Message Wall:
            </p>
            <input type="text" name="wall" placeholder="What are you thinking about?">
            <button type="submit">Publish</button>

            <?php foreach($check_message as $row) { ?>
                <div class="content">
                    <p class="msg"><?= $row['message'] ?></p>
                    <input type="text" name="edit">
                    <input type="hidden" name="id" value="<?= $row['id'] ?>">
                    <a href="?ed=<?= $row['id'] ?>">edit</a>
                </div>
            <?php } ?>

        </form>

if (isset($_GET['ed'])) {
    $id = $_POST['id'];
    $value = $_POST['edit'];
    mysqli_query($connect, "UPDATE message SET message = '$value' WHERE id = $id");
    header('Location: ../profile.php');
}

[AUser0]

Если нужна проверка через переменную - тогда и её добавляйте в форму <input type="hidden" name="ed" value="edit">, а в условии не забудьте поменять $_GET на $_POST.

Хотя логичнее поменять условие на: if (!empty($_POST['id'])) {.
Ещё один момент, ваш код совершенно не защищён от вредоносных SQL-иньекций.