Конечно, это хороший подход?Возможно, вам лучше написать метод для обновления или создания $ deplacement и передать 2 аргумента: переменную $ data из формы и user_id, например:
if ($form->isSubmitted() && $form->isValid()) {
$em = $this->getDoctrine()->getManager();
$userId = 2;
$formData->userId = $userId;
$deplacement->update($formData);
$em->persist($deplacement);
$em->flush($deplacement);
return $this->redirectToRoute('deplacement_show', array('id' => $deplacement->getId()));
}
Также я думаю, что вам не нужнослушатель событий.Почему ты это используешь?