Как заменить фотографию сущности?

Question

[Иван Антонов]

Когда я добавляю компанию, мне нужно загрузить её логотип:
Загружаю я так:
AddCompanyType:

$builder->add('image', new FileType(), array('label' => 'Логотип'));

FileType:

$builder->add('file', 'file', array('label' => false));

Загрузка происходит нормально. Но я не могу сообразить как мне отредактировать картинку, когда я редактирую компанию.
Нужна такая же форма, с чистым полем загрузки логотипа. При сохранении - если поле логотипа пустое, тогда логотип не обновлять; если новый логотип задан, тогда обновить его.

Answer 1

[Алексей Скобкин]

Ну вы же в контроллере можете описать логику обновления сущности, а не только автоматически заполнить её новыми данными из формы.

Comments

[Иван Антонов]

Я так и хотел. Проблема в том что я не могу в EditCompanyType изменить поле в FileType, я хотел дописать в опции
'data_class' => null

[Алексей Скобкин]

Иван Антонов: Это не обязательно делать. Вы можете взять полученный файл из нового объекта, который получен из формы и ещё не перзистится в БД (пока явно не вызовете $em->persist($object)), посмотреть, есть ли там фотография и, если есть - обновить в том объекте, который перзистится.

[Иван Антонов]

а я просто добавил в FileType 'data_class' => null. и все работает и при добавлении и при изменении. :)

[Алексей Скобкин]

Иван Антонов: Ну, поздравляю, вы сделали свою логику обновления сущности :)

[Иван Антонов]

Алексей Скобкин: но из-за того что это поле обязательное, при попытке сохранить без фотки он мне кидает исключение:

Catchable Fatal Error: Argument 1 passed to Proxies\__CG__\AppBundle\Entity\File::setFile() must be an instance of Symfony\Component\HttpFoundation\File\UploadedFile, null given, called in

[Иван Антонов]

Алексей Скобкин: дальше $form->handleRequest($request); не идет

[Алексей Скобкин]

Иван Антонов:

но из-за того что это поле обязательное, при попытке сохранить без фотки он мне кидает исключение

Ну так:
1. Если вы не всегда загружаете фотки - сделайте столбец nullable в маппинге.
2. Не надо перзистить ту сущность, которую получили из формы. Просто получите из неё фотографию и обновите ту сущность, которую получили из БД.

[Иван Антонов]

Алексей Скобкин: а как nullable у связывающего поля поставить?

/**
     * @var \AppBundle\Entity\File
     *
     * @ORM\ManyToOne(targetEntity="AppBundle\Entity\File", cascade={"persist"})
     * @ORM\JoinColumns({
     *   @ORM\JoinColumn(name="image", referencedColumnName="id")
     * })
     */
    private $image;

[Иван Антонов]

Алексей Скобкин: нашел. Только вот FK не дает обновить БД, если только удалять все и по новой создавать БД.

@ORM\JoinColumn(name="image", referencedColumnName="id", nullable=false)

[Алексей Скобкин]

Иван Антонов:

/**
     * @var \AppBundle\Entity\File
     *
     * @ORM\ManyToOne(targetEntity="AppBundle\Entity\File", cascade={"persist"})
     * @ORM\JoinColumns({
     *   @ORM\JoinColumn(name="image", referencedColumnName="id")
     * })
     */
    private $image;

А зачем у вас тут лишний код? Можно просто сделать так:

/**
     * @var \AppBundle\Entity\File
     *
     * @ORM\ManyToOne(targetEntity="AppBundle\Entity\File", cascade={"persist"})
     * @ORM\JoinColumn(name="image", nullable=true, options={"default" = null})
     */
    private $image;

Только вот FK не дает обновить БД

Ошибку в студию.

[Иван Антонов]

Алексей Скобкин: лишний код сгенерирован автоматически, я уж не стал каждую сущность обходить и переписывать.
делаю команду
php app\console doctrine:schema:update --force
Ошибка

[Doctrine\DBAL\Exception\DriverException]
  An exception occurred while executing 'ALTER TABLE companies CHANGE image i
  mage BIGINT NOT NULL':
  SQLSTATE[HY000]: General error: 1832 Cannot change column 'image': used in
  a foreign key constraint 'FK_8244AA3AC53D045F'
  [Doctrine\DBAL\Driver\PDOException]
  SQLSTATE[HY000]: General error: 1832 Cannot change column 'image': used in
  a foreign key constraint 'FK_8244AA3AC53D045F'
  [PDOException]
  SQLSTATE[HY000]: General error: 1832 Cannot change column 'image': used in
  a foreign key constraint 'FK_8244AA3AC53D045F'

[Иван Антонов]

Алексей Скобкин:

2. Не надо перзистить ту сущность, которую получили из формы. Просто получите из неё фотографию и обновите ту сущность, которую получили из БД.

не понял.

[Алексей Скобкин]

Иван Антонов: Два момента:
1. Почему вы пытаетесь сделать поле image BIGINT?
2. NOT NULL здесь как бы символизирует, что вы сделали поле не nullable. Я вам говорил сделать наоборот. Вы и выше мне маппинг неправильный привели, но я думал, что это старый код был.

[Иван Антонов]

Алексей Скобкин: а вот, сработало. я наверно ошибся в первый раз.

[Алексей Скобкин]

Иван Антонов: Наверное, ошиблись. Вам было нужно сделать nullable=true, а вы откуда-то nullable=false скопипастили.

[Иван Антонов]

Алексей Скобкин: Вот только обязательствами заполнения логотипа связывает не форма компании, а подформа картинки :(

$builder->add('file', 'file', array('label' => false));

В контексте картинки я не могу это поле сделать необязательным. так что нужно другое решение.
По-моему должно быть типовое решение. Вы делали подобное, можете показать как реализовывали?

[Алексей Скобкин]

Иван Антонов: Я думал что вам картинка не всегда нужна в сущности. Зачем же вы тогда делали поле nullable...
По поводу обработки наличия фотографии в форме - ещё раз повторяю - нужно описать логику обновления данных сущности из формы. Вы можете показать код обработки формы и сказать, что вам возвращает $form->getData() (естественно, только в случае, если $form->isValid())?

[Иван Антонов]

Алексей Скобкин:

Зачем же вы тогда делали поле nullable

для теста

нужно описать логику обновления данных сущности из формы

как же я опишу, если ошибка возникает на этапе handleRequest?

[Алексей Скобкин]

Иван Антонов:

для теста

Какой смысл тестировать то, что вам не нужно? nullable поле подразумевает, что у вас могут существовать записи без фотографий.

как же я опишу, если ошибка возникает на этапе handleRequest?

Вы уже начинайте прикладывать к вопросам ошибки, если хотите, чтобы вам помогали. Телепаты - в отпуске. Если же вы про

Catchable Fatal Error: Argument 1 passed to Proxies\__CG__\AppBundle\Entity\File::setFile() must be an instance of Symfony\Component\HttpFoundation\File\UploadedFile, null given, called in

то объявите поле формы как необязательное с помощью опции required.

[Алексей Скобкин]

Алексей Скобкин: И код уже приведите, в конце-то концов. Если его много - можно ссылкой на gist или любой другой pastebin.

[Иван Антонов]

Алексей Скобкин: https://gist.github.com/antonowano/314a2998e30183609a67

[Иван Антонов]

Алексей Скобкин:

Какой смысл тестировать то, что вам не нужно?

Я проверял будет ли ошибка если это поле сделать необязательным.

объявите поле формы как необязательное с помощью опции required.

не помогло.

[Алексей Скобкин]

Иван Антонов:

https://gist.github.com/antonowano/314a2998e30183609a67

Прошу прощения, но сначала комментарии к тому, что в глаза бросается.

$form = $this->createForm(new AddCompanyType($em));
// ...
$builder->get('user')->addModelTransformer(new EntityToNumberTransformer($this->manager, 'AppBundle:Users'));

Что вы такое странное делаете? Зачем вам менеджер сущностей вообще в формы пробрасывать?
class Companies
Не в тему, конечно, но почему у вас класс Companies называется, если он представляет одну компанию? Аналогично с Files и т.п.

/**
     * @ORM\Column(name="address", type="json_array", length=65535, nullable=true)
     */
    private $address;
// ...
    /**
     * @var string
     *
     * @ORM\Column(name="functions", type="json_array", length=65535, nullable=true)
     */
    private $functions;

Вы уверены, что это хорошее решение? Зачем вам там массив? Если у вас будут разные атомарные данные храниться, может стоит сделать хранение в отдельной таблице и проставить связь один-ко-многим?
class EditCompanyType extends AddCompanyType
Зачем вам вообще две формы. Могли бы и одной справиться. Но это не суть.

Теперь по делу.

2. Не надо перзистить ту сущность, которую получили из формы. Просто получите из неё фотографию и обновите ту сущность, которую получили из БД.

не понял.

public function editCompanyAction($id, Request $request)
    {
        $em = $this->getDoctrine()->getManager();
        $entity = $em->getRepository('AppBundle:Companies')->find($id);

Я имел в виду, что здесь вы получаете сущность из БД и она перзистится (находится под управлением EntityManager). Вы её, кстати, даже не проверяете - может вам null вернулся.

if ($form->isSubmitted() && $form->isValid()) {
            $entity = $form->getData();

А вот здесь вы почему-то берёте и перезаписываете ранее вытащенную из БД сущность тем, что пришло из формы. Зачем?
Если получите в отдельную переменную - можно будет по ней проверять, что у вас там в file и, если таки этот файл есть - привязывать его к первой $entity.

А теперь напомните, что именно у вас не работает в данной версии кода. Какая ошибка, какая строка.

[Алексей Скобкин]

Денис: Кстати, про странные описания связи я тоже спрашивал, да.

[Иван Антонов]

Простите что опущу большинство наводящих вопросов, для меня сейчас это не так важно.
Денис лишнее в контроллере убрал и прикрепил сущность Files
https://gist.github.com/antonowano/a0889f6bab787bf2e2ac

[Иван Антонов]

Алексей Скобкин чтобы вас распутать, сейчас я расскажу все по порядку.
Столкнулся с проблемой, когда для редактирования компании решил использовать форму добавления компании.
Открывая на сайте форму редактирования компании, выпадала ошибка:

The form's view data is expected to be an instance of class Symfony\Component\HttpFoundation\File\File, but is a(n) string. You can avoid this error by setting the "data_class" option to null or by adding a view transformer that transforms a(n) string to an instance of Symfony\Component\HttpFoundation\File\File

После прочтения ошибки я в форму FileType добавил data_class, так:

$builder->add('file', 'file', array(
    'label' => false,
    'data_class' => null     // <-- here
));

Форма у меня теперь открывается. Загрузка фотографии работает.
НО, когда я сохраняю форму не добавляя в поле фотографию, мне возвращается ошибка:

Catchable Fatal Error: Argument 1 passed to Proxies\__CG__\AppBundle\Entity\Files::setFile() must be an instance of Symfony\Component\HttpFoundation\File\UploadedFile, null given, called in D:\xampp\htdocs\52realty.local\vendor\symfony\symfony\src\Symfony\Component\PropertyAccess\PropertyAccessor.php on line 410 and defined

Сначала поле было обязательным, поэтому я подумал что дело в этом, но попробовав изменить nullable у сущности и required у формы я понял что дело не в этом, потому что ошибка осталась.

[Иван Антонов]

Денис:
if (null !== $this->file) вы имели ввиду это: if (null !== $file) ?

[Иван Антонов]

Денис: Ура. Теперь работает, спасибо. Даже 'data_class' у поля файла убрал и все равно работает.

[Алексей Скобкин]

Иван Антонов: Рад, что вы разобрались. Успехов!

[Алексей Скобкин]

Денис: Спасибо, что закончили помогать автору, а то я отвалился от обсуждения :)