Запись не обновляется при успешной вставке

Question

Запись не обновляется при успешной вставке

0

У меня есть строка, как показано ниже.

Я извлекаю id и имя из db, а другие опции выбора должны обновляться в определенной строке.

Мой идентификатор и имя, как показано ниже

$sql = "SELECT id, name,day,week,month,pickup,drop_location FROM crm LIMIT 10";
        $resultset = mysqli_query($conn, $sql) or die("database error:". mysqli_error($conn));
        while( $emp = mysqli_fetch_assoc($resultset) ) {        
        ?>  
            <tr style="text-align: center;">  
                <td><?php echo $emp["id"]; ?></td>

Я обновляю строку, но она не работает.

form.php

<form action="drop.php" name = "myForm" method="post">
                <td id="s1">    
                    <select style="width: 150px;" class="blue" id="pick" data-style="btn-info" name="pickup">
                        <option value="ward" id="ward1">Floor 1</option>
                        <option value="ward1" id="ward2">Floor 2</option>
                        <option value="ward2" id="ward3">Floor 3</option>
                    </select>

                </td>
                <td id="s2">
                    <select style="width: 150px;" class="blue" id="drop" data-style="btn-success" name="drop_location">
                        <option value="ward"  id="ward1">Floor 1</option>
                        <option value="ward1" id="ward2">Floor 2</option>
                        <option value="ward2" id="ward3">Floor 3</option>
                    </select>

                </td>
                <td id="s3">
                    <button class="btn btn-primary" type="submit" name="submit">submit</button>
                    <select style="width: 150px;" class="blue" id="stat" data-style="btn-warning" name="stat">

                        <option value="allocate">Allocated</option>
                        <option value="cancel">Cancelled</option>   
                    </select>

                </td>  

                <td id="s4">
                    <select style="width: 150px;" class="blue" id="status" data-style="btn-primary" name="status">
                        <option value="pending">pending</option>
                        <option value="Allocated">Allocated</option>
                        <option value="Engaged">Engaged</option>
                    </select>

                </td>  

            </form>
            </tr>

drop.php

 <?php
    /* Attempt MySQL server connection. Assuming you are running MySQL
    server with default setting (user 'root' with no password) */
    $mysqli = new mysqli("localhost", "root", "", "redeem");

    // Check connection
    if($mysqli === false){
        die("ERROR: Could not connect. " . $mysqli->connect_error);
    }


    // Escape user inputs for security
    $pickup = $mysqli->real_escape_string($_POST['pickup']);
    $drop_location = $mysqli->real_escape_string($_POST['drop_location']);
    $stat = $mysqli->real_escape_string($_POST['stat']);
    $status = $mysqli->real_escape_string($_POST['status']);

    // attempt insert query execution
    $sql = "UPDATE crm SET pickup='$pickup',drop_location='$drop_location',stat='$stat', status='$status' WHERE id = 'id' ";

    if($mysqli->query($sql) === true){
        echo "Records inserted successfully.";
    } else{
        echo "ERROR: Could not able to execute $sql. " . $mysqli->error;
    }

    // Close connection
    $mysqli->close();
    ?>

Я создал идентификатор для выбора, подобного этому

 $(document).ready(function() {
$("#pick, #drop, #stat, #status").on('change', function () {
    alert($(this).val());
});

Я не могу понять, где ошибка. Пожалуйста, любезно помогите мне.

Заранее спасибо!!!

Gowtham 21 фев. 2018, в 10:26

Источник

1

WHERE id = 'id' вероятно, неправильный идентификатор для использования.
Nigel Ren 21 фев. 2018, в 09:21
0

Какую ошибку вы получаете прямо сейчас?
Amit Merchant 21 фев. 2018, в 09:23
0

это потому, что вы не указали ID для обновления
Masivuye Cokile 21 фев. 2018, в 09:23
0

как я могу обновить идентификатор
Gowtham 21 фев. 2018, в 09:25
0

Вы можете добавить в форму скрытое поле с идентификатором, а затем выбрать его в значениях $_POST .
Nigel Ren 21 фев. 2018, в 09:28
0

@NigelRen подскажите, пожалуйста, как это сделать.
Gowtham 21 фев. 2018, в 09:38

Показать ещё 4 комментария

Теги:

php

mysql

mysqli

2 ответа

0

Вы не принимаете, какой идентификатор необходимо обновить. И в запросе вы используете id = 'id' чтобы заменить id переменной $ id, чтобы обновить значение.

// Escape user inputs for security
    $pickup = $mysqli->real_escape_string($_POST['pickup']);
    $drop_location = $mysqli->real_escape_string($_POST['drop_location']);
    $stat = $mysqli->real_escape_string($_POST['stat']);
    $status = $mysqli->real_escape_string($_POST['status']);
    $id= $mysqli->real_escape_string($_POST['id']);   // Get id from post need to be updated

    // attempt insert query execution
    $sql = "UPDATE crm SET 
            pickup='$pickup',drop_location='$drop_location',stat='$stat', status='$status' 
            WHERE id = $id ";   // use $id variable

    if($mysqli->query($sql) === true){
        echo "Records inserted successfully.";
    } else{
        echo "ERROR: Could not able to execute $sql. " . $mysqli->error;
    }

Naincy 21 фев. 2018, в 07:02

0

как я могу это принять
Gowtham 21 фев. 2018, в 09:28
0

когда вы создаете вышеуказанную форму, вам нужно поместить первичный ключ этой записи в скрытое поле, и во время публикации вы можете получить ее и обновить запись.
Naincy 21 фев. 2018, в 09:29
0

Ваш код уязвим для инъекций SQL
Your Common Sense 21 фев. 2018, в 09:32
0

@Naincy, не могли бы вы помочь мне с этим запросом.
Gowtham 21 фев. 2018, в 09:32
0

// Escape user inputs for security real_escape_string() // Escape user inputs for security Как real_escape_string() укрепляет безопасность? @Naincy
Masivuye Cokile 21 фев. 2018, в 09:34

Показать ещё 3 комментария

Ещё вопросы

WHERE id = 'id' вероятно, неправильный идентификатор для использования.
Какую ошибку вы получаете прямо сейчас?
это потому, что вы не указали ID для обновления
как я могу обновить идентификатор
Вы можете добавить в форму скрытое поле с идентификатором, а затем выбрать его в значениях $_POST .
@NigelRen подскажите, пожалуйста, как это сделать.
когда вы создаете вышеуказанную форму, вам нужно поместить первичный ключ этой записи в скрытое поле, и во время публикации вы можете получить ее и обновить запись.
@Naincy, не могли бы вы помочь мне с этим запросом.
// Escape user inputs for security real_escape_string() // Escape user inputs for security Как real_escape_string() укрепляет безопасность? @Naincy

Masivuye Cokile · Accepted Answer · 2018-02-21T06-52-00.000Z

Вы уже извлекаете записи из базы данных, так что вам нужно, чтобы внутри вашей формы было скрытое поле, которое будет иметь значение каждого идентификатора записи.

добавьте это в вашу форму:

<input type="hidden" name="id" value="<?=$emp['id']?>">

Затем вы сможете обновить запись с помощью этого идентификатора, я также предлагаю вам использовать подготовленный оператор для предотвращения внедрения sql.

<?php
/* Attempt MySQL server connection. Assuming you are running MySQL
server with default setting (user 'root' with no password) */
$mysqli = new mysqli("localhost", "root", "", "redeem");

// Check connection
if ($mysqli === false) {
    die("ERROR: Could not connect. " . $mysqli->connect_error);
}



$pickup        = isset($_POST['pickup']) ? $_POST['pickup'] : null;
$drop_location = isset($_POST['drop_location']) ? $_POST['drop_location'] : null;
$stat          = isset($_POST['stat']) ? $_POST['stat'] : null;
$status        = isset($_POST['status']) ? $_POST['status'] : null;
$id            = isset($_POST['id']) ? intval($_POST['id']) : null;


// attempt insert query execution
$sql = "UPDATE crm SET pickup = ? ,drop_location= ? ,stat= ? , status= ?  WHERE id = ? ";

$stmt = $mysqli->prepare($sql); //prepare statement
$stmt->bind_param("ssssi", $pickup, $drop_location, $stat, $status, $id);


if ($mysqli->execute()) { //execute the query
    echo "Records inserted successfully.";
} else {
    echo "ERROR: Could not able to execute $sql. " . $mysqli->error;
}

// Close connection
$mysqli->close();
?>

Спасибо, братан, который мне очень помог. Спасибо за вашу помощь.