Doctrine 2 - ORM

cascade remove revert order

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Invalid
  • Affects Version/s: Git Master
  • Fix Version/s: None
  • Component/s: ORM
  • Labels:
    None

Description

cascadeRemove incorrect work

UnitOfWork

in commit() (336)

// Entity deletions come last and need to be in reverse commit order
if ($this->entityDeletions) {
for ($count = count($commitOrder), $i = $count - 1; $i >= 0; --$i)

{ $this->executeDeletions($commitOrder[$i]); }

}

----------------------------------------------------------------
in doRemove (1630)

// Cascade first, because scheduleForDelete() removes the entity from the identity map, which
// can cause problems when a lazy proxy has to be initialized for the cascade operation.
$this->cascadeRemove($entity, $visited);

......

$this->scheduleForDelete($entity);

in array we get

child
child
parent

revert in commit - first remove parent and get foregein key error

if move cascadeRemove after switch all woork correct

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ivan Borzenkov added a comment -

patch

Show
Ivan Borzenkov added a comment - patch
Hide
Permalink
Ivan Borzenkov added a comment - - edited 

class Domain
{
...
    /**
     * @ORM\OneToMany(targetEntity="Query", mappedBy="domain", cascade={"all"})
     */
    protected $q;
...
}

class Query
{
...
    /**
     * @ORM\ManyToOne(targetEntity="Domain", inversedBy="q")
     */
    protected $domain;
...
}

if remove Domain - get mysql error foreign key

Show
Ivan Borzenkov added a comment - - edited class Domain { ... /** * @ORM\OneToMany(targetEntity= "Query" , mappedBy= "domain" , cascade={ "all" }) */ protected $q; ... } class Query { ... /** * @ORM\ManyToOne(targetEntity= "Domain" , inversedBy= "q" ) */ protected $domain; ... } if remove Domain - get mysql error foreign key

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Ivan Borzenkov
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: