CODEWITHSUNDEEP
symfonydoctrine
isundil
isundil

Reputation: 1111

Doctrine - A new entity was found through the relationship

since 2 weeks, we are having this problem while trying to flush new elements:

CRITICAL: Doctrine\ORM\ORMInvalidArgumentException:

A new entity was found through the relationship 'Comment#capture' that was not configured to cascade persist operations for entity

But the capture is already in the database, and we are getting it by a findOneBy, so if we cascade persist it, or persist it, we get a

Table constraint violation: duplicate entry.

The comments are created in a loop with differents captures, with a new, and all required field are set.

With all of the entities persisted and / or got by a findOne (and all valid), the flush still fails.

I'm on this issue since a while, so please help me

Upvotes: 101

Views: 128083

Answers (10)

LupusMichaelis
LupusMichaelis

Reputation: 101

The issue is the identity of the entity's object that is colliding with an other entity's object. I guess you were cloning the capture object.

In my case, I was cloning a related entity. As the f prop of E was a new object, but kept the same value for the primary key, Doctrine didn't know what do to do of it.

class E
{
  /** @ORM relationship spec
    */
  private $f;

  public function __construct(F $f)
  {
    $this->f = $f;
  }

  public function __clone()
  {
    $this->f = clone $f;
  }
}

Here solutions are:

  • not to clone the related entity
  • overload F::__clone to ensure the primary key's cleansed
  • copying the object (with the default reference behaviour)

(I'm aware the topic is old, but I wish I had a clearer answer, even if the answers here helped me a lot)

Upvotes: 0

Manuel
Manuel

Reputation: 127

I've seen this problem as well when you have Entities with a OneToMany <-> ManyToOne relationship.

If at the database level you have included an "on delete cascade" (when you delete the "One" side), and you don't include the concomitant cascade={"remove"} as in:

'... OneToMany(targetEntity="WhateverEntity", mappedBy="aProperty", cascade={"remove"})

Then this problem might arise.

Upvotes: 1

Sergei Krivosheenko
Sergei Krivosheenko

Reputation: 1366

I want to tell about my case as that might be helpful to somebody.

Given two entities: AdSet and AdSetPlacemnt. AdSet has the following property:

/**
 * @ORM\OneToOne(targetEntity="AdSetPlacement", mappedBy="adSet", cascade={"persist"})
 *
 * @JMS\Expose
 */
protected $placement;

Then error appears when I try to delete some AdSet objects in a cycle after 1st iteration

foreach($adSetIds as $adSetId) {
    /** @var AdSet $adSet */
    $adSet = $this->adSetRepository->findOneBy(["id" => $adSetId]);

    $this->em->remove($adSet);
    $this->em->flush();
}

Error

A new entity was found through the relationship 'AppBundle\Entity\AdSetPlacement#adSet' that was not configured to cascade persist operations for entity: AppBundle\Entity\AdSet@00000000117d7c930000000054c81ae1. To solve this issue: Either explicitly call EntityManager#persist() on this unknown entity or configure cascade persist this association in the mapping for example @ManyToOne(..,cascade={"persist"}). If you cannot find out which entity causes the problem implement 'AppBundle\Entity\AdSet#__toString()' to get a clue.

Solution

The solution was to add "remove" to $placement cascade options to be: cascade={"persist","remove"}. This guarantees that Placement also becomes detached. Entity manager will "forget" about Placement object thinking of it as "removed" once AdSet is removed.

Bad alternative

When trying to figure out what's going on I've seen a couple answers or recommendations to simply use entity manager's clear method to completely clear persistence context.

foreach($adSetIds as $adSetId) {
    /** @var AdSet $adSet */
    $adSet = $this->adSetRepository->findOneBy(["id" => $adSetId]);

    $this->em->remove($adSet);
    $this->em->flush();

    $this->em->clear();
}

So that code also works, the issue gets solved but it's not always what you really wanna do. Indeed it's happens quite rarely that you actually need to clear entity manager.

Upvotes: 0

Dennis
Dennis

Reputation: 8101

Refreshing the entity in question helped my case.

/* $item->getProduct() is already set */

/* Add these 3 lines anyway */
$id = $item->getProduct()->getId();            
$reference = $this->getDoctrine()->getReference(Product::class, $id);
$item->setProduct($reference);

/* Original code as follows */
$quote->getItems()->add($item);
$this->getDoctrine()->persist($quote);
$this->getDoctrine()->flush();

Despite my $item already having a Product set elsewhere, I was still getting the error.

Turns out it was set via a different instance of EntityManager.

So this is a hack of sorts, by retrieving id of the existing product, and then retrieving a reference of it, and using setProduct to "refresh" the whatever connection. I later fixed it by ensuring I have and use only a single instance of EntityManager in my codebase.

Upvotes: 10

drakonli
drakonli

Reputation: 2424

My answer is relevant for topic, but not very relevant for your particular case, so for those googling I post this, as the answers above did not help me.

In my case, I had the same error with batch-processing entities that had a relation and that relation was set to the very same entity.

WHAT I DID WRONG:

When I did $this->entityManager->clear(); while processing batch of entities I would get this error, because next batch of entities would point to the detached related entity.

WHAT WENT WRONG:

  1. I did not know that $this->entityManager->clear(); works the same as $this->entityManager->detach($entity); only detaches ALL of the repositorie`s entities.

  2. I thought that $this->entityManager->clear(); also detaches related entities.

WHAT I SHOULD HAVE DONE:

I should have iterated over entities and detach them one by one - that would not detach the related entity that the future entities pointed to.

I hope this helps someone.

Upvotes: 57

shukshin.ivan
shukshin.ivan

Reputation: 11340

I got this error too when tried to add new entity. 

A new entity was found through the relationship 'Application\Entity\User#chats' 
that was not configured to cascade persist operations for entity: ###. 
To solve this issue: Either explicitly call EntityManager#persist() on this unknown entity or
configure cascade persist  this association in the mapping for example @ManyToOne(..,cascade={"persist"}).

My case was that I tried to save entity, that shouldn't be saved. Entity relations was filled and tried to be saved (User has Chat in Many2Many, but Chat was a temporary entity), but there were some collisions.

So If I use cascade={"persist"} I get unwanted behaviour - trash entity is saved. My solution was to remove non-saving entity out of any saving entities:

// User entity code
public function removeFromChats(Chat $c = null){
    if ($c and $this->chats->contains($c)) {
        $this->chats->removeElement($c);
    }
}

Saving code

/* some code witch $chat entity */
$chat->addUser($user);

// saving
$user->removeFromChats($chat);
$this->getEntityManager()->persist($user);
$this->getEntityManager()->flush();

Upvotes: 0

Mirza Selimovic
Mirza Selimovic

Reputation: 1739

I had the same problem and it was the same EntityManager. I wanted to insert an object related ManyToOne. And I don't want a cascade persist.

Example : 

$category = $em->find("Category", 10);

$product = new Product();
$product->setCategory($category)

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

This throws the same exception for me.

So the solution is :

$category = $em->find("Category", 10);

$product = new Product();
$product->setCategory($category)

$em->merge($product);
$em->flush();

Upvotes: 67

Roberto
Roberto

Reputation: 562

The error: 'Comment#capture' that was not configured to cascade persist operations for entity

The problem:

/**
 * @ORM\ManyToOne(targetEntity="Capture", inversedBy="comments")
 * @ORM\JoinColumn(name="capture_id", referencedColumnName="id",nullable=true)
 */
 protected $capture;

dont configured the cascade persist

try with this:

/**
 * @ORM\ManyToOne(targetEntity="Capture", inversedBy="comments", cascade={"persist", "remove" })
 * @ORM\JoinColumn(name="capture_id", referencedColumnName="id",nullable=true)
 */
 protected $capture;

Upvotes: 13

ownking
ownking

Reputation: 2006

In my case a too early call of 

$this->entityManager->clear();

caused the problem. It also disappeared by only doing a clear on the recent object, like

$this->entityManager->clear($capture);

Upvotes: 53

Damien
Damien

Reputation: 5882

First of all, you should take better care of your code, I see like 3 differents indentations in your entity and controller - this is hard to read, and do not fit the Symfony2 coding standards.

The code you show for your controller is not complete, we have no idea from where $this->activeCapture is coming. Inside you have a $people['capture'] which contains a Capture object I presume. This is very important.

If the Capture in $people['capture'] is persisted / fetched from another EntityManager than $this->entityManager (which, again, we do not know from where it come), Doctrine2 have no idea that the object is already persisted.

You should make sure to use the same instance of the Doctrine Entity Manager for all those operations (use spl_object_hash on the EM object to make sure they are the same instance).

You can also tell the EntityManager what to do with the Capture object.

// Refreshes the persistent state of an entity from the database
$this->entityManager->refresh($captureEntity);

// Or
// Merges the state of a detached entity into the 
// persistence context of this EntityManager and returns the managed copy of the entity.
$captureEntity = $this->entityManager->merge($captureEntity);

If this does not help, you should provide more code.

Upvotes: 11

Related Questions