0

Хочу сделать возможным редактирование конкретной статьи, а не всех имеющихся, не могу построить правильный запрос:

В этом случае возникает ошибка:

<meta charset="utf-8"> 
<link rel="stylesheet" href="css/style.css">

<?php   

    include 'bd.php';

    $zagolovok = $_POST['zagolovok'];
    $smallopisanie = $_POST['smallopisanie'];
    $opisanie = $_POST['opisanie']; 

    $editid  = $_GET[edit];

    $result = mysql_query("UPDATE news WHERE id = '$editid' SET zagolovok = replace('$zagolovok','zagolovok','$zagolovok')");
    if (!$result) {
        die('Неверный запрос: ' . mysql_error());
    }

    if ($result) {
        echo "<center><p style='color: #404040; font-size: 24px; text-decoration: none; font-family: 'Gerbera''>Статья отредактирована</p><br><a style='color: #1d97e1; font-size: 24px; text-decoration: none; font-family: 'Gerbera'' href='adminpanel'>Назад в админ-панель</div></a></center>";
    }

?>

В этом случае редактируются все статьи:

<?php   

    include 'bd.php';

    $zagolovok = $_POST['zagolovok'];
    $smallopisanie = $_POST['smallopisanie'];
    $opisanie = $_POST['opisanie']; 

    $result = mysql_query("UPDATE news SET zagolovok = replace('$zagolovok','zagolovok','$zagolovok' WHERE id = '$editid')");
    if (!$result) {
        die('Неверный запрос: ' . mysql_error());
    }

    if ($result) {
        echo "<center><p style='color: #404040; font-size: 24px; text-decoration: none; font-family: 'Gerbera''>Статья отредактирована</p><br><a style='color: #1d97e1; font-size: 24px; text-decoration: none; font-family: 'Gerbera'' href='adminpanel'>Назад в админ-панель</div></a></center>";
    }

?>
6

1 ответ 1

0

исправьте последовательность

$result = mysql_query("UPDATE `news`  SET `zagolovok` = REPLACE(`zagolovok`,'zagolovok','$zagolovok') WHERE `id` = '$editid' ");

но я бы порекомендовал немного другой код по проще

$result = mysql_query('
  UPDATE `news`
  SET `zagolovok` = \''.$zagolovok.'\' 
  WHERE `id` = \''.$editid.'\' 
  ');

В вашем коде я не увидел правильно ли берется переменная $editid Также хотел обратить ваше внимание на неправильное применение синтаксиса функции REPLACE

" $editid пуст" - пока он пуст ни один запрос у вас не получится.

у вас в коде должно быть что то вроде

$editid = $_POST['editid '];

Рекомендую прислушаться к пользователю @Mike - и отказаться от использования mysql_query в пользу php PDO

3
  • В таком случае он вообще не хочет заменять содержимое
    – user273992
    Commented 13 нояб. 2017 в 6:54
  • @user273992 Начните со строки $editid = $_GET[edit]; она вообще то должна как минимум предупреждения давать, если конечно у вас нет константы с именем edit. А потом срочно переходите на PDO и не подставляйте значения переменных в текст запроса никогда, иначе вас взломают через неделю после того как этот код заработает в интернете
    – Mike
    Commented 13 нояб. 2017 в 7:15
  • @Mike, $editid пуст, он передается на страницу редактирования информации, а вот сохранения уже нет. imgur.com/a/IfxCf
    – user273992
    Commented 13 нояб. 2017 в 7:18

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.