Symfony, Как корректно обновить сущность, имеющую поле типа «Файл»?

Question

[Артемио Вегас]

Всех приветствую! Изучаю Symfony
Проблема такая, имеется сущность Task, в которой есть свойства

<?php

namespace AppBundle\Entity;

use Doctrine\ORM\Mapping as ORM;

/**
 * Task
 *
 * @ORM\Table(name="task")
 * @ORM\Entity(repositoryClass="AppBundle\Repository\TaskRepository")
 */
class Task
{
     /**
     * @ORM\Column(type="string", nullable=true)
    */    
    private $originalFileName;
    /**
     * @ORM\Column(type="string", nullable=true)
    */    
    private $fileName;

В которых при сохранении пишется оригинальное имя файла, и хешированное имя файла на сервере.

Есть форма

<?php
namespace AppBundle\Form;

use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\OptionsResolver\OptionsResolver;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\Form\Extension\Core\Type\FileType;

class TaskType extends AbstractType
{
    /**
     * {@inheritdoc}
     */
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
                ->add('taskName',TextType::class)
                ->add('fileName', FileType::class, array('label' => 'Документ'))                
                ;
    }/**
     * {@inheritdoc}
     */
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults(array(
            'data_class' => 'AppBundle\Entity\Task'
        ));
    }

Есть экшн редактирование сущности.
Если при редактировании сущности у которой уже есть загруженный файл соответственно поля originalFileName и fileName заполнены, если файл инпут пустой, то старое значение originalFileName в БД становится null
Как сделать так, чтобы если значение из формы файл инпут пустое, то старое не перезатиралось?
Загрузку файла реализовывал c помощью
документации

Comments

[bears]

Покажите код загрузки, подозреваю, что там Вы сами затираете originalFileName

Answer 1

[voronkovich]

В представленном коде должно затираться свойство fileName, т.к. оно у вас указано в форме TaskType. Вообще говоря, в него должен сохраняться экземпляр UploadedFile (или null если поле не было заполнено при отправке формы), а у вас в аннотации указано, что это строковое поле.

Вам не нужно делать "мэппинг" поля с файлом в сущность (укажите 'mapped' => false):

public function buildForm(FormBuilderInterface $builder, array $options)
{
    $builder
        ->add('taskName',TextType::class)
        ->add('file', FileType::class, [ 'mapped' => false, 'label' => 'Документ'])                
    ;
}

Экземпляр UploadedFile можно получить вручную:

$uploadedFile = $form['file']->getData();

$fileName = 'myfile.txt';

$uploadedFile->move('public/uploads', $fileName);

$task->setFileName('public/uploads/'.$fileName);

Comments

[Артемио Вегас]

Спасибо,Олег, если не делаем мэппинг и проверяем в контроллере значение из $form['file']->getData() то всё ок работает.
А если допустим использовать листенер Доктрины как описано в документации https://symfony.com/doc/2.8/controller/upload_file... в листенер как то можно заинжектить объект сабмиченной формы, чтобы можно было проверить значение файл_инпута там?

[voronkovich]

Артемио Вегас, Да, но придется подписаться на два события:

• FormEvents::POST_SET_DATA - для сохранения ссылки на экземпляр UploadedFile
  
• prePersist - для сохранения файла на диск и обновления свойста fileName.
  

См. события доктрины https://www.doctrine-project.org/projects/doctrine...
И события форм: https://symfony.com/doc/current/form/events.html

[Артемио Вегас]

voronkovich, верно ли я понимаю, это нужно будет в конструктор доктрин листенера типа такого:

<?php
// src/AppBundle/EventListener/BrochureUploadListener.php
namespace AppBundle\EventListener;

use Symfony\Component\HttpFoundation\File\UploadedFile;
use Doctrine\ORM\Event\LifecycleEventArgs;
use Doctrine\ORM\Event\PreUpdateEventArgs;
use AppBundle\Entity\Product;
use AppBundle\FileUploader;

class BrochureUploadListener
{
    private $uploader;

    public function __construct(FileUploader $uploader)
    {
        $this->uploader = $uploader;
    }

    public function prePersist(LifecycleEventArgs $args)
    {
        $entity = $args->getEntity();

        $this->uploadFile($entity);
    }

    public function preUpdate(PreUpdateEventArgs $args)
    {
        $entity = $args->getEntity();

        $this->uploadFile($entity);
    }

    private function uploadFile($entity)
    {
        // upload only works for Product entities
        if (!$entity instanceof Product) {
            return;
        }

        $file = $entity->getBrochure();

        // only upload new files
        if ($file instanceof UploadedFile) {
            $fileName = $this->uploader->upload($file);
            $entity->setBrochure($fileName);
        }
    }
}

передать объект формы ?
Гуглю, что то не нахожу примеры когда в доктрин листенер, инжектят объект формы (=

[voronkovich]

Артемио Вегас, Лучше используйте подписчики (subscribers). Начиная с SF4 им отдается предпочтение.

Вам не нужно внедрять форму. Вам нужно подписаться на одно из ее событий и она сама упадет к вам в руки. Вот пример: https://symfony.com/doc/current/form/events.html#e...

В обработчике вы сможете сохранить экземпляр файла до момента сохранения сущности:
$this->uploadedFile = $form['file']->getData(); // Как-то так

С другой стороны, может вам лучше вернуться к вашему первоначальному варианту. Просто добавить в сущность свойство uploadedFile (с сеттером и геттером) и снять 'mapped' => "false"