Уведомление пользователя по email. Нету ли тут косяка?

Question

[dev400]

Сюда приходят данные из ajax

<?php
$model=new models_List;
$id=$_POST['id'];
$user_id = $_POST['user_id'];
$email = Models_UserData::getUserEmail($user_id);

unset($_POST['url']);
unset($_POST['id']);
unset($_POST['user_id']);



if($model->manageOrder($_POST, $id, $user_id)) {
    $response = array("msg" => "Заказ отредактирован, клиент уведомлен по Email", "status" => true);
    $model->orderNotice($email);
}

else
    $response = array("msg" => "Не удалось отредактировать заказ", "status" => false);
    echo json_encode($response);

?>

тут идет отправка уведомления пользователю

if($model->manageOrder($_POST, $id, $user_id)) {
    $response = array("msg" => "Заказ отредактирован, клиент уведомлен по Email", "status" => true);
    $model->orderNotice($email);
}

если метод
$model->manageOrder
вернул true, то вот так
$model->orderNotice($email);
мы уведомляем пользователя по email.

Нормально ли именно так (в этой части кода) уведомлять пользователя по email?

Answer 1

[Archakov Dennis]

Как-бы нормально. Но лучше фильтровать данные POST от (XSS,SQL) инъекций.