在非对象上调用成员函数 removeElement()

Call to a member function removeElement() on a non-object

我正在创建一个表单,以便能够向一组链接到特定课程的学生发送电子邮件。默认情况下,给定课程的所有学生都必须 selected,但消息的发件人必须能够删除 select 学生以将他们排除在接收消息之外。发给全群没问题。从 select.

中删除学生时会出现问题

我将 Sonata Admin 的 sonata_type_model 与自定义查询结合使用。在生成的表单上,如果我不更改 select 选项并提交表单,一切正常。当我从列表中删除项目时,提交表单后出现错误:

错误:在 /xxx/xxx/xxx/vendor/sonata-project/doctrine-orm-admin-bundle/Model/ModelManager.php 行 607

中调用非对象的成员函数 removeElement()

经过两天的寻找答案,希望这里有人可以帮助我朝着正确的方向前进。这是我使用的一些代码:

管理员:

$em = $this->modelManager->getEntityManager('Stnu\EduBundle\Entity\DealItem');
    $query = $em->createQueryBuilder('d')
            ->select('d')
            ->from('StnuEduBundle:DealItem', 'd')
            ->innerJoin('d.deal', 'de')
            ->where('d.course = :course')
            ->andWhere('de.status = :status')
            ->setParameter('course',$course)
            ->setParameter('status','order');

    $defaults = $query->getQuery()->getResult();


    $formMapper
            ->with('Certificaten verzenden cursus \''. $title .'\'', array('description' => 'Begeleidende tekst e-mail'))
                ->add('dealItems', 'sonata_type_model', array(
                    'required' => true,
                    'expanded' => false,
                    'btn_add' => false,
                    'multiple' => true,
                    'label' => 'Verzenden aan',
                    'query' => $query,
                    'property' => 'deal.user',
                    'data' => $defaults,
                    'validation_groups' => false
                ))
                ->add('subject', 'text', array('required' => true, 'label' => 'Onderwerp', 'data' => $subject))
                ->add('body', 'textarea', array('label' => 'Bericht', 'required' => false, 'data' => $body, 'attr' => array('class' => 'tinymce', 'data-theme' => 'fullpage', 'style' => 'height: 350px')));

控制器:

/**
 * Create action
 *
 * @return Response
 *
 * @throws AccessDeniedException If access is not granted
 */
public function createAction()
{

    // the key used to lookup the template
    $templateKey = 'edit';

    if (false === $this->admin->isGranted('CREATE')) {
        throw new AccessDeniedException();
    }

    $object = $this->admin->getNewInstance();

    $this->admin->setSubject($object);


    /** @var $form \Symfony\Component\Form\Form */
    $form = $this->admin->getForm();
    $form->setData($object);


    if ($this->getRestMethod()== 'POST') {

        $object->setDealItems($object->getDealItems());

        $form->submit($this->get('request'));

错误在此时出现。

实体:

<?php

namespace Stnu\EduBundle\Entity;

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

/**
 * DocsEmail
 * 
 * @ORM\Entity
 */
class CertificateEmail {

    /**
     * @var integer
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;


    /**
     * @ORM\ManyToMany(targetEntity="DealItem")
     * @ORM\JoinTable(name="certificateemails_dealitems",
     *      joinColumns={@ORM\JoinColumn(name="certificateEmail_id", referencedColumnName="id")},
     *      inverseJoinColumns={@ORM\JoinColumn(name="dealItem_id", referencedColumnName="id")}
     *      )
     */
    private $dealItems;

    private $subject;

    private $body;

    private $extraEmailTo;

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

    /**
     * Add dealItem
     *
     */
    public function addDealItem(\Stnu\EduBundle\Entity\DealItem $dealItem) {

        $this->dealItems->add($dealItem);
        //$this->dealItems[] = $dealItem;
        return $this;
    }

    /**
     * Remove dealItem
     */
    public function removeDealItem(\Stnu\EduBundle\Entity\DealItem $dealItem) {

        foreach ($this->dealItems as $item) {
            if ($dealItem === $item) {
                // manager of Stnu\EduBundle\Entity\DealItem
                $entityManager->remove($dealItem);
            }
        }

    }

    /**
     * Get dealItems
     *
     * @return \Doctrine\Common\Collections\Collection 
     */
    public function getDealItems() {

        return $this->dealItems;
    }

    public function setDealItems($dealItems) {
        $this->dealItems = new ArrayCollection();

        if (count($dealItems) > 0) {
            foreach ($dealItems as $dealItem) {
                $this->addDealItem($dealItem);
            }
        }

        return $this;
    }


    /**
     * Set subject
     *
     * @param string $subject
     */
    public function setSubject($subject) {
        $this->subject = $subject;

        return $this;
    }

    /**
     * Get subject
     *
     * @return string 
     */
    public function getSubject() {
        return $this->subject;
    }

    /**
     * Set body
     *
     * @param string $body
     */
    public function setBody($body) {
        $this->body = $body;

        return $this;
    }

    /**
     * Get body
     *
     * @return string 
     */
    public function getBody() {
        return $this->body;
    }

    /**
     * Set extraEmailTo
     *
     * @param string $extraEmailTo
     */
    public function setExtraEmailTo($extraEmailTo) {
        $this->extraEmailTo = $extraEmailTo;

        return $this;
    }

    /**
     * Get extraEmailTo
     *
     * @return string 
     */
    public function getExtraEmailTo() {
        return $this->extraEmailTo;
    }

}

希望有人能帮帮我!

删除操作前请检查您的 collection。

ldd($object->getDealItems()); // or var_dump();die();

我认为当调用删除操作时,您的 属性 dealItems 是空的。

我相信这离您的问题的答案越来越近了。问题是,你完全问错了问题。重要的东西在下面,在 "EDIT - Custom Multiple Select Field".

你的removeDealItem方法全错了。试试这个:

    public function removeDealItem(\Stnu\EduBundle\Entity\DealItem $dealItem)
    {
        $this->dealItems->removeElement( $dealItem );

        return $this;
    }

您没有 $entityManager 可以在这里拜访……您也不需要。 Doctrine 将检查您要删除的实体是否存在,如果存在则将其删除。您不需要遍历集合中的现有元素,当然也不需要在数据库级别执行任何操作。

    public function addDealItem(\Stnu\EduBundle\Entity\DealItem $dealItem)
    {
        // Getting fancy - check if the item exists before adding it
        if( !$this->dealItems->contains($dealItem) )
        {
            $this->dealItems->add($dealItem);
        }
        return $this;
    }

添加一个项目也很容易...我们甚至可以花点心思,使用 Doctrine ArrayCollection::contains() 方法在添加元素之前检查它是否存在。您的 addDealItem() 方法没有任何问题 - 我只是想向您展示 contains() 作为让 ArrayCollection class 为您完成工作的更直观的说明。

编辑 - 自定义多个 Select 字段

好的 - 在阅读了您关于不一定需要保留数据的评论后,我想我会提供这个简化示例来说明如何创建自定义多 select 框。请理解这是一个 'bare basics' 示例 - 但它应该引导您朝着正确的方向前进。显然,在不知道您的 DealItem 实体的结构的情况下,我只是猜测您需要访问的特定字段才能获取发送电子邮件所需的数据。

所以 - 在你的控制器中 - 首先我们得到数据:

$query = $em->createQueryBuilder('d')
            ->select('d')
            ->from('StnuEduBundle:DealItem', 'd')
            ->innerJoin('d.deal', 'de')
            ->where('d.course = :course')
            ->andWhere('de.status = :status')
            ->setParameter('course',$course)
            ->setParameter('status','order');

$defaults = $query->getQuery()->getResult();

$choices = array();

foreach( $defaults as $dealItem )
{
    $choices[ $dealItem->getEmailAddress() ] = $dealItem->getStudentName();
}

现在我们需要一个对象来接收数据。我从您的评论中收集到的是您不想保留数据,并且您只为您的 CertificateEmail 对象创建了一个实体,以便您可以构建一个表单。馊主意。 您不需要实体 - 所以一开始就不要构建实体。为了证明这一点,我将使用 stdClass 对象来实现:

$certificateEmail = new \stdClass();

$certificateEmail->dealItems = array();
$certificateEmail->subject   = '';
$certificateEmail->body      = '';

然后我们构建表单:

$form = $this->createFormBuilder( $certificateEmail )
             ->add( 'dealItems', 'choice', array(
                         'choices'  => $choices,
                         'multiple' => true,
                         'required' => true,
                         'label'    => 'Verzenden aan' ) )
             ->add( 'subject', 'text', array( 'required' => true, 'label' => 'Onderwerp' ) )
             ->add( 'body', 'textarea', array( 'required' => false, 'label' => 'Bericht' ) )
             ->getForm();

最后,把它扔到模板上:

return $this->render( 'template.html.twig', array( 'form' => $form->createView() ) );

而且,希望你可以从那里开始:)