Problème lors d'un preRemove

Version imprimable

Bonjour,

J'essaie de faire un remove dans un EventListenner et j'ai toujours la même erreur'
Il n'y a pas de relation entre les deux tables...je sais pas propre mais c'est comme ça.

En gros une table couleur, et une table couleurToProduit. Du coup, lorsque je supprime une couleur, j'aimerais supprimer les associations dans couleurToProduit.

Mon listenner :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 <?php namespace Cuisine\AppBundle\Entity\Listener; use Doctrine\ORM\Event\LifecycleEventArgs; use Cuisine\AppBundle\Entity\Couleur; use Cuisine\AppBundle\Entity\CouleursToProduit; class CouleurListener { public function preRemove(LifecycleEventArgs $event) { $entity = $event->getEntity(); $em = $event->getEntityManager()->getRepository('CuisineAppBundle:CouleursToProduit'); $results = $em->findByCouleurId($entity->getId()); foreach ($results as $result) { if ($result instanceOf CouleursToProduit) { $em->remove($result); $em->flush(); } } } }
L'erreur obtenue:
Code:

1 2 Undefined method 'remove'. The method name must start with either findBy or findOneBy!
Je me casse la tête dessus depuis 3 jours. Si quelqu'un a une idée, une solution je suis preneur :)

Merci d'avance

18/07/2016, 18h02
MehrezLabidi

le

Code:

$results = $em->findByCouleurId($entity->getId());

est erroné bien sur .
des exemples ici
si ton

Code:

findByCouleurId

est defini ailleurs ,montre nous comment tu fais

Pensez à valoriser les réponses pertinantes, cliquez sur le bouton vert +1:plusser: pour indiquer votre accord avec la solution proposée.
Pensez à utiliser la balise [code] pour afficher du code, elle est cachée sous le bouton [#] dans l'éditeur.
Une discussion est terminée ? Alors le bouton :resolu: est votre ami !

Il est défini dans un fichier :

Code:

1
2
 
CouleursToProduitRepository.php

le code est le suivant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
<?php
 
namespace Cuisine\AppBundle\Entity;
 
use Doctrine\ORM\EntityRepository;
use Cuisine\AppBundle\Entity\CouleursToProduit;
 
 
class CouleursToProduitRepository extends EntityRepository
{
 
	public function findByProduitId($produit_id){
 
		$query = $this->_em->createQuery(
		    'SELECT c
		    FROM CuisineAppBundle:Couleur c
		    WHERE c.produit_id = :produit_id'
		)->setParameter(":produit_id", $produit_id);
 
		$products = $query->getResult();
		return $products;
	}
 
	public function findByCouleurId($couleur_id){
 
		$query = $this->_em->createQuery(
		    'SELECT cp
		    FROM CuisineAppBundle:CouleursToProduit cp
		    WHERE cp.couleur_id = :couleur_id'
		)->setParameter(":couleur_id", $couleur_id);
 
		$products = $query->getResult();
		return $products;
	}
 
}

Je viens d'essayer en changeant par :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
<?php
namespace Cuisine\AppBundle\Entity\Listener;
 
use Doctrine\ORM\Event\LifecycleEventArgs;
use Cuisine\AppBundle\Entity\Couleur;
use Cuisine\AppBundle\Entity\CouleursToProduit;
 
class CouleurListener
{
 
    public function preRemove(LifecycleEventArgs $event)
    {
 
    	$entity = $event->getEntity();
 
    	$em = $event->getEntityManager()->getRepository('CuisineAppBundle:CouleursToProduit');
 
  		$results = $em->findBy(array("couleurId" => $entity->getId()));
 
  		foreach ($results as $result) {
 
  			$em->remove($result->getId());
  			$em->flush();
  		}
    }
}

et toujours le même problème

18/07/2016, 23h01
MehrezLabidi

un petit changement peu résoudre le problème.

Code:

$em = $event->getEntityManager()->getRepository('CuisineAppBundle:CouleursToProduit');

devient

Code:

$em = $args->getEntityManager();

Pensez à valoriser les réponses pertinantes, cliquez sur le bouton vert +1 :plusser: pour indiquer votre accord avec la solution proposée.
Pensez à utiliser la balise [code] pour afficher du code, elle est cachée sous le bouton [#] dans l'éditeur.
Une discussion est terminée ? Alors le bouton :resolu: est votre ami !
18/07/2016, 23h18
anleclerc

En faisant comme cela j'ai une erreur 500 tout de suite après, avec le findBy, comme si il ne trouvait plus la table CouleursToProduit
19/07/2016, 00h00
MehrezLabidi

Oui c'est sur.

Code:

$em = $args->getEntityManager();

et

Code:

$results = $em->getRepository('CuisineAppBundle:CouleursToProduit')->findBy(array("couleurId" => $entity->getId()));

$em sert seulement à gerer (remove,persist,detach..) une instance alors que $em->getRepository te permet de se connecter (findBy,findAll ..) à la table entiere.

Pensez à valoriser les réponses pertinantes, cliquez sur le bouton vert +1 :plusser: pour indiquer votre accord avec la solution proposée.
Pensez à utiliser la balise [code] pour afficher du code, elle est cachée sous le bouton [#] dans l'éditeur.
Une discussion est terminée ? Alors le bouton :resolu: est votre ami !
19/07/2016, 15h32
anleclerc

Merci pour ces précisions. Je débute sur Doctrine.

Je n'ai plus l'erreur, je n'ai plus aucune erreur d'ailleurs. Par contre il ne remove pas. Si je mets un die dans la boucle avant le remove, il passe dedans, si je le mets après le remove, il ne passe jamais dans la boucle, le code doit sortir de la boucle au premier remove mais aucune erreur...

Je continue de regarder :)
19/07/2016, 15h38
MehrezLabidi

$em->remove s'applique à un objet ,alors c'était quoi le

Code:

$result->getId()

!!
pour des raisons de performances, on mets pas flush dans une boucle ,mais juste dehors car $em contient des modifications pour divers objets et attaquer la base plusieurs fois au meme temps . ça tue :)

Non non mais j'ai changé le code, il ressemble à ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public function preRemove(LifecycleEventArgs $event)
    {
 
    	$entity = $event->getEntity();
 
    	$em = $event->getEntityManager();
    	$results = $em->getRepository('CuisineAppBundle:CouleursToProduit')->findBy(array("couleurId" => $entity->getId()));
 
  		foreach ($results as $result) {
    		$em->remove($result);
  		}
  		$em->flush();
 
  		die('stop');
    }

19/07/2016, 16h02
MehrezLabidi

Je crois pas que $results contient un array d'objet, je sens qu'il est vide .
Juste avant la boucle ,si ca retourne un array vide c'est fini.

Code:

var_dump( $results);exit;

Il contient ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
array (size=3)
  0 => 
    object(Cuisine\AppBundle\Entity\CouleursToProduit)[96]
      private 'id' => int 14
      private 'couleurId' => int 7
      private 'produitId' => int 7
  1 => 
    object(Cuisine\AppBundle\Entity\CouleursToProduit)[95]
      private 'id' => int 50
      private 'couleurId' => int 7
      private 'produitId' => int 8
  2 => 
    object(Cuisine\AppBundle\Entity\CouleursToProduit)[100]
      private 'id' => int 425
      private 'couleurId' => int 7
      private 'produitId' => int 29

En fait si je fait un var_dump du result au premier passage dans la boucle il me sort bien le premier object result, mais après le remove, plus rien

C'est bon ça fonctionne, j'ai rajouté une vérification de l'instance pour être sur que cela ne passe pas n'importe quand :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
<?php
namespace Cuisine\AppBundle\Entity\Listener;
 
use Doctrine\ORM\Event\LifecycleEventArgs;
use Cuisine\AppBundle\Entity\Couleur;
use Cuisine\AppBundle\Entity\CouleursToProduit;
 
class CouleurListener
{
 
    public function preRemove(LifecycleEventArgs $event)
    {
 
    	$entity = $event->getEntity();
 
    	if ($entity instanceof Couleur) {
 
	    	$em = $event->getEntityManager();
	    	$results = $em->getRepository('CuisineAppBundle:CouleursToProduit')->findBy(array("couleurId" => $entity->getId()));
 
	  		foreach ($results as $result) {
				$em->remove($result);
	  		}
 
	    }
    }
 
}