0

Как сделать запрос на вставку NULL? Точнее как сделать это через переменную $date_contract_finish

if ($_GET['date_contract_finish'] == ''){
  $date_contract_finish = NULL;
}else{
  $date_contract_finish = $_GET['date_contract_finish'];
}
$sql = "UPDATE project_registry SET 
        date_contract_finish = '$date_contract_finish' 
         WHERE id = '$id'"

В итоге вставляется пустая строка, а необходимо вставить null.

5
  • а null это не пустая строка ?
    – madfan41k
    7 июн 2018 в 9:23
  • ага, понял вопрос
    – madfan41k
    7 июн 2018 в 9:26
  • я бы на вашем месте озадачился валидацией данных по крайней мере, а то передача значения 1970-01-01' # вам обновит всю таблицу целиком.
    – teran
    7 июн 2018 в 9:35
  • это как если бы я делал обычный запрос в субд update project_registry set date_contract_finish = NULL where id = 32; 7 июн 2018 в 9:36
  • честно говоря я и пытаюсь победить проблему, когда при пустой строке в input у меня в поле дата уходит 1900-01-01 вместо NULL 7 июн 2018 в 9:38

2 ответа 2

1
if ($_GET['date_contract_finish'] == ''){
  $date_contract_finish = "NULL";
}else{
  $date_contract_finish = "'" . $_GET['date_contract_finish'] ."'";
}

$sql = "UPDATE project_registry SET 
        date_contract_finish = $date_contract_finish
         WHERE id = '$id'"
6
  • я так пробовал, и ".$date_contract_finish." тоже пробовал, не выходит 7 июн 2018 в 9:35
  • @ИгорьТюлькин кавычка там лишняя в запросе а так вариант рабочий, только не менее дырявый чем ваш код в вопросе
    – teran
    7 июн 2018 в 9:37
  • так скажите как лучше) 7 июн 2018 в 9:38
  • да, кавычку не заметил, код пправил
    – madfan41k
    7 июн 2018 в 9:41
  • ну а что бы он не был дырявым, нужно использовать хотя бы PDO
    – madfan41k
    7 июн 2018 в 9:41
1

Используйте механизм prepared statements. Что PDO что mysqli корректно передают NULL через prepared statements. И, что существенно важнее, закроете открытую дыру для sql-инъекции.

$stmt = $pdo->prepare("UPDATE project_registry SET 
    date_contract_finish = :date_contract_finish
     WHERE id = :id");
$stmt->execute([
    'date_contract_finish' => empty($_GET['date_contract_finish']) ? null : $_GET['date_contract_finish'],
    'id' => $id,
]);

Если вам действительно не нужно валидировать формат. Хотя скорее всего вам сперва нужно проверить, что вам передали что-то похожее на дату.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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