Symfony4 Doctrine Получаю ошибку при сохранении в базу, как можно исправить (association field)?

Question

[symnoob]

доброе время суток,
Получаю ошибку при сохранении в базу, как можно исправить?

Expected value of type "MyBundle\HotelBundle\Model\Pic" for association field "MyBundle\HotelBundle\Model\Hotel#$pics", got "NULL" instead.

Answer 1

[Decadal]

вам нужно сделать тип для поля pics nullable=true и убрать строгий тайпхинт если он есть

Comments

[symnoob]

а какие недостатки могут возникнуть если убрать строгий тайпхинт? В документациях всегда они присутствуют.
Да и Доктрина их сома всегда генерирует
и как можно быстро получить в контроллере верный тип?

[Decadal]

symnoob, недостатки в том что вы можете допустить ошибку и словить её не так быстро как хотелось бы.

В вашем случае если вы допускаете что может быть pics nullable, следует ставить такую конструкцию:

public function setPic( ? Pic $pic) {
   $this->pics = $pic; 
}

знак вопроса в тайпхинте означает, что вы можете передать null, а не только экземпляр класса Pic.

Обратите внимание что nullable hint (знак вопроса перед типом ) доступен только с php 7.1

[profesor08]

А может лучше сначала ициализировать Pic, даже если из бд пришел null, только правильно это обработать? А то как-то неправильно получается, в setPic ты ожидаешь Pic, а вместо него прилетает null, и ты потом этим null инициализируешь, а потом тебе еще каждый раз проверять на null. Как-то это все не очень выглядит.

Разве что тебя это не должно заботить что там дальше будет с объектом. Но тогда можно просто написать public function setPic($pic), а в доках к функции указать тип для значения, чтоб IDE все понимала, и не парить себе мозги.

[Decadal]

profesor08

А может лучше сначала ициализировать Pic,

У вас есть информация о том что автору вопроса обязательно нужен Pic?

как-то неправильно получается, в setPic ты ожидаешь Pic, а вместо него прилетает null,

откуда у вас такие сведения, что в setPic он ожидает Pic? Если он хочет удалить Pic для Hotel, пусть делает setPic(null);
Null это корректное значение для отсутствия записей. Или отели не бывают без картинок?

[symnoob]

теперь вот такая ошибка, что можно сделать?

Return value of MyBundle\HotelBundle\Model\Hotel::getPics() must be an instance of MyBundle\HotelBundle\Model\Pic or null, instance of Doctrine\ORM\PersistentCollection returned

[Decadal]

symnoob, для начала вы должны определиться:
Pics это много картинок?
или Pics это одна картинка?
Если одна, то вам не нужно было брать коллекцию для этого поля. (у вас видимо указана связь многие к одному)
Коллекция - это когда много. И строгого тайпхинта на Pics у коллекции быть не должно

Для коллекций нужен несколько другой набор методов:

addPic(Pic $pic) //добавить одну картинку
removePic(Pic $pic) //удалить одну (конкретную) картинку

setPics(Collection $pics);

getPics() : ? Collection;

Я вам смогу помочь если вы приведете полный код своих сущностей - отеля и картинки. Чтобы я увидел связи между ними.

[symnoob]

у меня один отель много картинок, тобишь one-to-many

А что писать в конструкторе Модели/Entity?
пытаюсь прописать:

public function __construct(string $id)
    {
        $this->id = $id;
        $this->pics = new Collection();
    }

IDE Ругается, заставляет изменить на

<code>    
public function __construct(string $id)
    {
        $this->id = $id;
        $this->pics = new ArrayCollection();
    }
</code>

<?xml version="1.0" encoding="utf-8"?>
<doctrine-mapping xmlns="http://doctrine-project.org/schemas/orm/doctrine-mapping"
                  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                  xsi:schemaLocation="http://doctrine-project.org/schemas/orm/doctrine-mapping http://doctrine-project.org/schemas/orm/doctrine-mapping.xsd">
    <entity name="MyBundle\HotelBundle\Model\Hotel" table="ex_hotels" repository-class="MyBundle\HotelBundle\Repository\HotelRepository">
        <id name="id" type="guid">
            <generator strategy="NONE"/>
        </id>

        <field name="title" type="string"/>
        <field name="description" type="text"/>
        <field name="startDate" type="datetime" nullable="true"/>
        <field name="endDate" type="datetime" nullable="true"/>

        <one-to-many field="pics" target-entity="MyBundle\HotelBundle\Model\Pic"
                     mapped-by="hotel">
            <cascade>
                <cascade-all/>
            </cascade>
        </one-to-many>

    </entity>
</doctrine-mapping>

<?xml version="1.0" encoding="utf-8"?>
<doctrine-mapping xmlns="http://doctrine-project.org/schemas/orm/doctrine-mapping"
                  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                  xsi:schemaLocation="http://doctrine-project.org/schemas/orm/doctrine-mapping http://doctrine-project.org/schemas/orm/doctrine-mapping.xsd">
    <entity name="MyBundle\HotelBundle\Model\Pic" table="ex_pics" repository-class="MyBundle\HotelBundle\Repository\PicRepository">

        <id name="id" type="integer" column="id">
            <generator strategy="AUTO"/>
        </id>

        <field name="mediaFile" type="integer"/>

        <many-to-one field="hotel" target-entity="MyBundle\HotelBundle\Model\Hotel" inversed-by="pics">
            <cascade>
                <cascade-all/>
            </cascade>
            <join-columns>
                <join-column name="idHotels" on-delete="CASCADE" referenced-column-name="id" nullable="false"/>
            </join-columns>
        </many-to-one>

    </entity>
</doctrine-mapping>

Hotel Entitty:

protected $pics;

    public function __construct(string $id)
    {
        $this->id = $id;
        $this->pics = new ArrayCollection();

    public function addPic(? Collection $pics)
    {
        $this->pics[] = $pics;

        return $this;
    }

    public function removePic($pics)
    {
        $this->pics->removeElement($pics);
    }

    public function getPics(): ? Collection
    {
        return $this->pics;
    }

Pic Entitty:

protected $hotel;


    public function setHotel($hotel)
    {
        $this->hotel = $hotel;

        return $this;
    }

    public function getHotel()
    {
        return $this->hotel;
    }

[Decadal]

symnoob,

IDE Ругается, заставляет изменить на

IDE правильно подсказывает, я вам дал неполную информацию

Hotel:

protected $pics;

// вы уверены что тип у айди - стринг? и что его надо передавать в конструктор?
    public function __construct(string $id)
    {
        $this->id = $id;
        $this->pics = new ArrayCollection();
   }


// здесь элемент должен быть строго экземпляром класса Pic, потому что вы знаете какой элемент хотите добавить и вы не хотите добавлять "пустые" картинки
        /**
     * Add pic
     *
     * @param Pic $pic
     */
    public function addPic(Pic $pic)
    {
        if (! $this->pics->contains($pic)) {
            $this->pics[] = $pic;
        }
    }

// здесь элемент должен быть строго экземпляром класса Pic, потому что вы знаете какой элемент хотите удалить
    public function removePic(Pic $pic)
    {
        $this->pics->removeElement($pics);
    }

// можете получить null, если не проинициализируете поле по умолчанию в конструкторе. Но обычно вы будете получать этим методом список картинок которые есть у отеля
    public function getPics(): ? ArrayCollection
    {
        return $this->pics;
    }

// можете передать null, либо, если угодно, пустую коллекцию - чтобы удалить сразу все картинки. 
// в иных случаях вы знаете что должны передать всю коллекцию картинок за раз.
    public function setPics(? ArrayCollection $pics)
    {
        $this->pics = $pics;
    }

[symnoob]

Decadal,
теперь вот такая ошибка:

Return value of MyBundle\\HotelBundle\\Model\\Hotel::getPics() must be an instance of Doctrine\\Common\\Collections\\ArrayCollection or null, instance of Doctrine\\ORM\\PersistentCollection returned

Может это как то связано с сериалайзером?

[Decadal]

symnoob, укажите Doctrine\Common\Collections\Collection - интерфейс для коллекций
вместо явного ArrayCollection