Что не так с SQL запросом?

Question

[Samuello]

Всем привет, подскажите пожалуйста, что не так с SQL запросом
Голова не пашит уже

if (isset($_POST['date_btn']))
{
	$surname       = $_POST['surname'];
	$name          = $_POST['name'];
	$patronymic    = $_POST['patronymic'];
	$birth_date    = $_POST['birth_date'];
	$e_mail        = $_POST['e-mail'];
	$phone_number  = $_POST['phone_number'];
	$region        = $_POST['region'];
	$city          = $_POST['city'];
	$school        = $_POST['school'];
	$work_position = $_POST['work_position'];

	mysqli_query($connect, "UPDATE users SET surname = '$surname', name = '$name', patronymic = '$patronymic', birth_date = '$birth_date', e_mail = '$e_mail' phone_number = '$phone_number' region = '$region' city = '$city' school = '$school' work_position = '$work_position' WHERE login = '$login';");

	header('Location: https://********.**/?page=date');
}

Comments

[JokerS_Smile]

По моему запятых не хватает в запросе

[Samuello]

JokerS_Smile, Вы правы, спасибо

[Samuello]

tommy-vercetti, Точно, спасибо

[serginhold]

tommy-vercetti, в mysqli тоже есть prepared statements :)

Answer 1

[Vitaliy Orlov]

запятых не хватает

e_mail = '$e_mail' phone_number = '$phone_number' region

должно быть

e_mail = '$e_mail', phone_number = '$phone_number', region

и советую почитать про sql инъекции и mysqli prepare

Comments

[Vitsliputsli]

Samuello, обязательно прочитайте, пришедшие извне данные обязательно нужно подготавливать (в принципе, там все достаточно просто).