多对多关系设置器

Question

我的实体之间存在多对多关系：采购订单和供应商。当我想在我的Symfony项目中将供应商添加到订单时，我总是收到以下错误消息：

属性“suppliers”在类“Acme\AppBundle\Entity\PurchaseOrder”中不是公共的。也许您应该创建方法“setSuppliers（）”？

当我自己在PurchaseSOEntity中创建setSubplers（）函数时：

public function setSuppliers(\Acme\AppBundle\Entity\Supplier $suppliers )
{
    $this->suppliers = $suppliers;

    return $this;
}

我收到以下错误消息:

可捕获的致命错误：传递给Doctrine\通用\集合\Array集合::__construct（）的参数1必须是数组类型、给定对象、在第519行的 /var/www/symfony/vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php中调用并在第47行的 /var/www/symfony/vendor/doctrine/collections/lib/Doctrine/Common/Collections/ArrayCollection.php中定义

有什么想法吗？

/**
 * @Route("order/{id}/supplieradd", name="order_supplieradd")
 * @Secure(roles="ROLE_ADMIN")
 */
public function newSupplierAction(Request $request, $id)
{
    $purchaseOrder = $this->getDoctrine()
    ->getRepository('AcmeAppBundle:PurchaseOrder')
    ->find($id);

    if (!$purchaseOrder) {
        throw $this->createNotFoundException(
                'No order found for id '.$id
        );
    }

    $form = $this->createForm(new AddSupplierType(), $purchaseOrder);

    // process the form on POST
    if ($request->isMethod('POST')) {
        $form->bind($request);
        if ($form->isValid()) {             

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

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

            return new Response('Added Supplier to Order with ID '.$articleOrder->getId());
        }
    }

    return $this->render('AcmeAppBundle:BasicData:newSupplier.html.twig', array(
            'form' => $form->createView(),
            'id' => $id,
    ));
}

和我的添加供应商类型.php

public function buildForm(FormBuilderInterface $builder, array $options)
{
    $builder->add('suppliers', 'entity', array(
         'class' => 'AcmeAppBundle:Supplier',
         'property' => 'name',
    ));
}

PurchaseSO和供应商实体的某些部分：

 class PurchaseOrder{
  ...
         /**
         * @ORM\ManyToMany(targetEntity="Supplier", mappedBy="purchaseOrders")     
         */
    private $suppliers;

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

/**
 * Add suppliers
 *
 * @param \Acme\AppBundle\Entity\Supplier $suppliers
 * @return PurchaseOrder
 */
public function addSupplier(\Acme\AppBundle\Entity\Supplier $suppliers)
{
    $this->suppliers[] = $suppliers;

    return $this;
}

/**
 * Remove suppliers
 *
 * @param \Acme\AppBundle\Entity\Supplier $suppliers
 */
public function removeSupplier(\Acme\AppBundle\Entity\Supplier $suppliers)
{
    $this->suppliers->removeElement($suppliers);
}

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

class Supplier{
    ...

    /**
     * @ORM\ManyToMany(targetEntity="PurchaseOrder", inversedBy="suppliers")
     * @ORM\JoinTable(name="suppliers_purchaseOrders")
     */
    private $purchaseOrders;
}

新的添加删除集方法：

 /**
     * Add supplier
     *
     * @param \Acme\AppBundle\Entity\Supplier $supplier
     * @return PurchaseOrder
     */
public function addSupplier(\Acme\AppBundle\Entity\Supplier $supplier)
{
    $this->suppliers->add($supplier);

    return $this;
}

/**
 * Remove supplier
 *
 * @param \Acme\AppBundle\Entity\Supplier $supplier
 */
public function removeSupplier(\Acme\AppBundle\Entity\Supplier $supplier)
{
    $this->suppliers->removeElement($supplier);
}

public function setSuppliers($supplier)
{
     if ( is_array($supplier) ) {
        $this->suppliers = $supplier ;
    } else {
        $this->suppliers->clear() ;
        $this->suppliers->add($supplier) ;
    }
}

Answer 1

您的setSuplers方法不应该将Array集合作为第一个参数吗？

public function setSuppliers(ArrayCollection $suppliers )
{
    $this->suppliers = $suppliers;

    return $this;
}

当你在构造函数中将其初始化为 ArrayCollection 时。使用 setSuppliers 方法，您可以将 ArrayCollection 转换为单个 Supplier 对象。

您的表单将返回一个选定的 Suplier 对象。如果这是你想要的，你为什么不使用OnyToMany实现呢？否则，您可以将多个选项添加到表单域中。然后可以将多个 Suplier 对象作为数组返回。

Answer 2

问题：

>

语法不正确的方法名称及其参数：

public function addSupplier(\Acme\AppBundle\Entity\Supplier $suppliers)

方法说addSupplier(单数)但你接受供应商(复数)

您需要对此方法进行重构：

public function addSupplier(Supplier $supplier)
{
    $this->suppliers->add($supplier) ;
}

也：

public function removeSupplier(Supplier $supplier)
{
    $this->suppliers->removeElement($supplier) ;
}

如果您像我的答案那样做，则 getter 和 setter 方法将起作用：在多对多关系的形式中设置 multiple='false' symfony2

Symfony会找到add...()并移除...()方法本身。因此，如果关系是“供应商”，它会找到addSupplier。或者如果relation是“categories”，它会找到addCategory()和removeCategory()。