Doctrine 2 - ORM

EntityManager::remove($entity), EntityManager::flush() does not remove the entity from DB

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Critical Critical
  • Resolution: Invalid
  • Affects Version/s: None
  • Fix Version/s: 2.0-RC1
  • Component/s: ORM
  • Security Level: All
  • Labels:
    None
  • Environment:
    PgSQL 8.4, current Doctrine 2 from Git

Description

Test.php
/* 
 * @Entity
 * @Table(indexes={@Index(name="name", columns={"name"})})
 */
class ProductAttribute
{

    /**
     * ID
     * 
     * @Id @Column(type="integer") @GeneratedValue
     *
     * @var integer
     */
    protected $id;
    /**
     * Name
     *
     * @Column
     *
     * @var string
     */
    protected $name;
    /**
     * Values
     *
     * @OneToMany(targetEntity="ProductAttributeValue", mappedBy="attribute")
     *
     * @var array
     */
    protected $values;

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

    /**
     * Sets ID
     *
     * @param integer $id ID
     *
     * @return void
     */
    public function setId($id = null)
    {
        $this->id = $id;
    }

    /**
     * Gets name
     *
     * @return string Name
     */
    public function getName()
    {
        return $this->name;
    }

    /**
     * Sets name
     *
     * @param string $name Name
     *
     * @return void
     */
    public function setName($name = null)
    {
        $this->name = $name;
    }

    /**
     * Gets product attribute values
     *
     * @return array Product attribute values
     */
    public function getValues()
    {
        return $this->values;
    }

    /**
     * Adds product attribute value
     *
     * @param ProductAttributeValue $value Product attribute value
     * 
     * @return void
     */
    public function addValue(ProductAttributeValue $value)
    {
        $value->setAttribute($this);
        $this->values[] = $value;
    }

}

$attribute = $em->find('ProductAttribute', 1);
$em->remove($attribute);
$em->flush();

The upper code won't remove the entity from the database.... Corresponding DQL (DELETE ProductAttribute a WHERE a.id = 1) will remove it from the database...

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Petr Motejlek added a comment -

I just found out, that when I have an entity containing a *ToMany column with cascade=

{"remove"}

set, all the entities are removed, except for the one...

Show
Petr Motejlek added a comment - I just found out, that when I have an entity containing a *ToMany column with cascade= {"remove"} set, all the entities are removed, except for the one...
Hide
Permalink
Benjamin Eberlei added a comment -

I cannot reproduce this issue. Can you try to build a reproducible test-case? Have a look at tests/Doctrine/Tests/ORM/Functional/Ticket/* and build a test-case like this one here:

<?php

namespace Doctrine\Tests\ORM\Functional\Ticket;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Tests\Models\CMS\CmsUser;
use Doctrine\Tests\Models\CMS\CmsGroup;

require_once __DIR__ . '/../../../TestInit.php';

class DDC824Test extends \Doctrine\Tests\OrmFunctionalTestCase
{
    public function setUp()
    {
        $this->useModelSet('cms');
        parent::setUp();
    }

    public function testRemoveWithCascade()
    {
        $user = new CmsUser();
        $user->username = "beberlei";
        $user->name = "Benjamin";
        $user->status = "active";

        $phone1 = new \Doctrine\Tests\Models\CMS\CmsPhonenumber();
        $phone1->phonenumber = "1234";
        $phone2 = new \Doctrine\Tests\Models\CMS\CmsPhonenumber();
        $phone2->phonenumber = "1235";

        $user->addPhonenumber($phone1);
        $user->addPhonenumber($phone2);

        $this->_em->persist($user);
        $this->_em->flush();
        $this->_em->clear();

        $user = $this->_em->find(get_class($user), $user->id);
        $phone1 = $user->phonenumbers[0];
        $phone2 = $user->phonenumbers[1];

        $this->assertTrue($this->_em->contains($user));
        $this->assertTrue($this->_em->contains($phone1));
        $this->assertTrue($this->_em->contains($phone2));

        $this->_em->remove($user);

        $this->assertFalse($this->_em->contains($user));
        $this->assertFalse($this->_em->contains($phone1));
        $this->assertFalse($this->_em->contains($phone2));
    }
}
Show
Benjamin Eberlei added a comment - I cannot reproduce this issue. Can you try to build a reproducible test-case? Have a look at tests/Doctrine/Tests/ORM/Functional/Ticket/* and build a test-case like this one here: <?php namespace Doctrine\Tests\ORM\Functional\Ticket; use Doctrine\Common\Collections\ArrayCollection; use Doctrine\Tests\Models\CMS\CmsUser; use Doctrine\Tests\Models\CMS\CmsGroup; require_once __DIR__ . '/../../../TestInit.php'; class DDC824Test extends \Doctrine\Tests\OrmFunctionalTestCase { public function setUp() { $ this ->useModelSet('cms'); parent::setUp(); } public function testRemoveWithCascade() { $user = new CmsUser(); $user->username = "beberlei" ; $user->name = "Benjamin" ; $user->status = "active" ; $phone1 = new \Doctrine\Tests\Models\CMS\CmsPhonenumber(); $phone1->phonenumber = "1234" ; $phone2 = new \Doctrine\Tests\Models\CMS\CmsPhonenumber(); $phone2->phonenumber = "1235" ; $user->addPhonenumber($phone1); $user->addPhonenumber($phone2); $ this ->_em->persist($user); $ this ->_em->flush(); $ this ->_em->clear(); $user = $ this ->_em->find(get_class($user), $user->id); $phone1 = $user->phonenumbers[0]; $phone2 = $user->phonenumbers[1]; $ this ->assertTrue($ this ->_em->contains($user)); $ this ->assertTrue($ this ->_em->contains($phone1)); $ this ->assertTrue($ this ->_em->contains($phone2)); $ this ->_em->remove($user); $ this ->assertFalse($ this ->_em->contains($user)); $ this ->assertFalse($ this ->_em->contains($phone1)); $ this ->assertFalse($ this ->_em->contains($phone2)); } }
Hide
Permalink
Petr Motejlek added a comment -

I found out that I cannot reproduce the same behaviour with different project of mine either. It seems there's simply something rotten inside the one. I'll try investigating it a bit farther, but I think you can close this issue for now. Thanks for your time .

Show
Petr Motejlek added a comment - I found out that I cannot reproduce the same behaviour with different project of mine either. It seems there's simply something rotten inside the one. I'll try investigating it a bit farther, but I think you can close this issue for now. Thanks for your time .

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Petr Motejlek
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: