How to transfer one object with many related objects

Question

I have entities Basketand BasketItem:

/**
 * Acme\BasketBundle\Entity\Basket
 *
 * @ORM\Entity(repositoryClass="Acme\BasketBundle\Repository\BasketRepository")
 * @ORM\Table(name="orders")
 * @ORM\HasLifecycleCallbacks()
 */
class Basket
{
  /**
   * @var integer $id
   *
   * @ORM\Column(name="id", type="integer")
   * @ORM\Id
   * @ORM\GeneratedValue(strategy="AUTO")
   */
  private $id;

  // ...

  /**
    * @ORM\OneToMany(targetEntity="BasketItem", mappedBy="order_id",cascade={"all"})
    */
  protected $items;

  // ...

  public function __construct() {
    $this->items = new ArrayCollection();
  }

  /**
   * Add item
   *
   * @param BasketItem $item
   */
  public function addItem(BasketItem $item)
  {
    $key = $this->find($item->getProduct()->getId());

    if ($key === false) {
        $this->items->add($item);
    } else {
        $this->items->get($key)->raiseQuantity($item->getQuantity());
    }
  }

  /**
   * Find an item (if present)
   *
   * @param integer $id
   * @return integer
   */
  public function find($id)
  {
    foreach ($this->items as $key => $item) {
        if ($item->getProduct()->getId() == $id)
            return $key;
    }
    return false;
  }
}

/**
 * Acme\BasketBundle\Entity\BasketItem
 *
 * @ORM\Entity
 * @ORM\Table(name="order_items")
 */
class BasketItem
{
  /**
   * @var integer $id
   *
   * @ORM\Column(name="id", type="integer")
   * @ORM\Id
   * @ORM\GeneratedValue(strategy="AUTO")
   */
  private $id;

  // ...

  /**
   * @ORM\ManyToOne(targetEntity="Basket", inversedBy="items")
   * @ORM\JoinColumn(name="order_id", referencedColumnName="id")
   */
  private $basket;

  // ...
}

---

Now, when I create a basket and fill it with items, I have problems with saving it in db.

The following does not work properly.

$basket = new Basket();
$basket->addItem(new BasketItem($product1, 1));
$basket->addItem(new BasketItem($product2, 2));

$em->persist($basket);
$em->flush();

Then I consulted the manual more carefully :

$basket = new Basket();
$basket->addItem(new BasketItem($product1, 1));
$basket->addItem(new BasketItem($product2, 2));

$em->persist($basket);
foreach ($basket->getItems() as $item) {
    $em->persist($item);
}
$em->flush();

This did not work as expected.

In both cases, all data is stored in the database , but the basket items are not associated with the basket, i.e. order_idobject BasketItem NULL.

Can someone explain to me what I'm doing wrong, please? Please note that I am new to the Doctrine. Thank!

---

---

---

EDIT

I'm really confused. Here is my simplified test:

<?php

namespace Amsel\BasketBundle\Entity;

use Doctrine\ORM\Mapping as ORM;
use Doctrine\Common\Collections\ArrayCollection;

/**
 * Amsel\BasketBundle\Entity\Basket
 *
 * @ORM\Entity(repositoryClass="Amsel\BasketBundle\Repository\BasketRepository")
 * @ORM\Table(name="orders")
 */
class Basket
{
    /**
     * @var integer $id
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;

    /**
     * @var ArrayCollection $items
     *
     * @ORM\OneToMany(targetEntity="BasketItem", mappedBy="basket", cascade={"all"})
     */
    protected $items;

    public function __construct() {
        $this->items = new ArrayCollection();
    }

    /**
     * Add item
     *
     * @param BasketItem $item
     */
    public function addItem(BasketItem $item)
    {
        $this->items->add($item);
    }

    /**
     * Get id
     *
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * Get items
     *
     * @return Doctrine\Common\Collections\Collection 
     */
    public function getItems()
    {
        return $this->items;
    }
}

<?php

namespace Amsel\BasketBundle\Entity;

use Doctrine\ORM\Mapping as ORM;


/**
 * Amsel\BasketBundle\Entity\BasketItem
 *
 * @ORM\Entity(repositoryClass="Amsel\BasketBundle\Repository\BasketItemRepository")
 * @ORM\Table(name="order_items")
 */
class BasketItem
{
    /**
     * @var integer $id
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;

    /**
     * @var Basket $basket
     *
     * @ORM\ManyToOne(targetEntity="Basket", inversedBy="items")
     * @ORM\JoinColumn(name="order_id", referencedColumnName="id")
     */
    protected $basket;


    /**
     * Get id
     *
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * Set basket
     *
     * @param Amsel\BasketBundle\Entity\Basket $basket
     */
    public function setBasket(\Amsel\BasketBundle\Entity\Basket $basket)
    {
        $this->basket = $basket;
    }

    /**
     * Get basket
     *
     * @return Amsel\BasketBundle\Entity\Basket 
     */
    public function getBasket()
    {
        return $this->basket;
    }
}

public function testAction(Request $request) {

    $em = $this->getDoctrine()->getEntityManager();

    $basket = new Basket();
    $basket->addItem(new BasketItem());
    $basket->addItem(new BasketItem());

    $em->persist($basket);

    try {
        $em->flush();
    } catch(Exception $e) {
        die('ERROR: '.$e->getMessage());
    }        
    die ('end');
}

- (BasketItem) , (Basket).

Answer 1

, - - Basket BasketItem.

public function testAction(Request $request) {

    $em = $this->getDoctrine()->getEntityManager();

    $basket = new Basket();

    $bi1 = new BasketItem();
    $bi1->setBasket($basket);

    $bi2 = new BasketItem();
    $bi2->setBasket($basket);

    $basket->addItem($bi1);
    $basket->addItem($bi2);

    $em->persist($basket);

    try {
        $em->flush();
    } catch(Exception $e) {
        die('ERROR: '.$e->getMessage());
    }        
    die ('end');
}

, , , .

, , !

Answer 2

Basket: mappedBy , .

/**  
 * @ORM\OneToMany(targetEntity="BasketItem", mappedBy="basket",cascade={"all"})  
 */  
 protected $items;

Answer 3

Symfony Doctrine, . , , , , Basket BasketItem

$bi->setBasket($basket);

u , :

$basket->addItem($bi);

u need to change the addItem method and removeItem method

/**
 * Add item
 *
 * @param BasketItem $item
 */
public function addItem(BasketItem $item)
{
    $item->setBasket($this);
    $this->items->add($item);
}

/**
 * Remove item
 *
 * @param BasketItem $item
 */
public function removeItem(BasketItem $item)
{
    $item->setBasket(null);
    $this->items->removeElement($item);
}

[EDIT] see this: http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/association-mapping.html#owning-and-inverse-side-on-a-manytomany- association