Discussion :

[dubitoph]

Bonjour,

Dans une boucle, je cherche à concaténer des collections de cette manière :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$communes = $obj->getCommunes();
 
$localites = "";
 
foreach ($communes->toArray() as $com)
   {
      $localites->AddItem($com->getLocalites());        
   }

Cependant, j'obtiens l'erreur suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Fatal error: Call to a member function AddItem() on a non-object in...

Quelqu'un aurait une idée sur la manière de procéder pour pouvoir merger ces différents arrayCollection?

Merci d'avance pour votre aide.

[stealth35]

pas besoin de faire un toArray

[dubitoph]

Bonjour et merci pour ton intervention. Du fait que $obj->getCommunes() retourne une collection, je devais employer le toArray().

Je viens de trouver la solution que voici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
    if($typeElemInit == 'commune')
     {
      $localites = $obj->getLocalites();      
     }
    else
     {      
      $communes = $obj->getCommunes();
 
      $localites = new ArrayCollection();
 
      foreach ($communes->toArray() as $com)
       {
        foreach ($com->getLocalites()->toArray() as $locByCom)
         {
          $localites->add($locByCom);
         }        
       }
     }
 
    $retour = "";
 
    foreach ($localites->toArray() as $loc)
     {
      $retour .= $loc->getId() . '_' . $loc->getNom() . '+';
     }
 
    //Supprimer le dernier '+'
    if($retour != "")
      {
       $retour = substr($retour, 0, strlen($retour) - 1);
      }         
 
    //return new \Symfony\Component\HttpFoundation\Response($retour);
    return $retour;

.

[stealth35]

Collection c'est un ArrayAccess, donc direct foreach dessus ...
tu te compliques et tu bouffes de le mémoire pour rien la

[dubitoph]

OK et merci pour tes bons conseils. Voici donc ma méthode ré-écrite dans le repository :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
38
39
40
41
42
43
44
45
46
47
48
 public function getLocalitesByElem($idElemInit, $typeElemInit)
	{	
		$em = $this->getEntityManager();
 
    switch($typeElemInit)
     {
      case 'commune':
        $obj = $em->getRepository('HotelsGestionAnnoncesBundle:Commune')->find($request->get('idElemInit'));
      break;
 
      case 'province':
        $obj = $em->getRepository('HotelsGestionAnnoncesBundle:Province')->find($request->get('idElemInit'));
      break;
 
      case 'departement':
        $obj = $em->getRepository('HotelsGestionAnnoncesBundle:Departement')->find($request->get('idElemInit'));
      break;
 
      case 'region':
       $obj = $em->getRepository('HotelsGestionAnnoncesBundle:Region')->find($idElemInit);
      break;
 
      default:
        $obj = $em->getRepository('HotelsGestionAnnoncesBundle:Pays')->find($request->get('idElemInit'));
      break;
     }
 
    if($typeElemInit == 'commune')
     {
      $localites = $obj->getLocalites();      
     }
    else
     {      
      $communes = $obj->getCommunes();
 
      $localites = array();
 
      foreach ($communes as $com)
       {
        foreach ($com->getLocalites() as $locByCom)
         {
          $localites[] = $locByCom;
         }        
       }
     }
 
    return $localites;
	}

J'y fais appel de cette manière dans un formulaire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
            ->add('localite','entity', array('class'=>'Hotels\GestionAnnoncesBundle\Entity\Localite',
                                             'query_builder' => function(LocaliteRepository $er) use ($region)
                                                                  {
                                                                   return $er->getLocalitesByElem($region->getId(), 'region');
                                                                  }, 
                                             'required' => false))
        ;

Cependant, j'obtiens l'erreur :

Expected argument of type "Doctrine\ORM\QueryBuilder", "array" given

[rafleboss]

tu retourne un array alors qu'il attend un queryBuilder...

[rafleboss]

au lieu de retourner un tableau car si j'ai bien compris ton code tu veux retourner qu'une colonne de ta table. Utilise les requetes doctrine! utilise partial si tu veux retourner qu'une seul colonne

[dubitoph]

Merci beaucoup de l'intérêt que tu portes à mon problème.

La méthode getLocalitesByElem() me retourne un tableau d'objets car elle peut-être appelée dans différentes situations (pas uniquement depuis un formulaire).

Y a-t-il donc une possibbilité d'affecter un tableau d'objets à un champ de type 'entity' (en ne passant donc pas par array( 'query_builder' => ...)?

[rafleboss]

au lieu de mettre query_builder tu peux rajouter ce champs
'choices'=>ton_tableau_d'objets !

[dubitoph]

Merci beaucoup pour ton aide. Je venais juste de tenter cette solution :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
            ->add('localite','entity', array('class'=>'Hotels\GestionAnnoncesBundle\Entity\Localite',
                                             'choices' => function(LocaliteRepository $er) use ($region)
                                                                  {
                                                                   return $er->getLocalitesByElem($region->getId(), 'region');
                                                                  }, 
                                             'required' => false))

Cependant, j'obtiens l'erreur suivante :

Catchable Fatal Error: Argument 1 passed to Hotels\GestionAnnoncesBundle\Form\{closure}() must be an instance of Hotels\GestionAnnoncesBundle\Entity\LocaliteRepository, none given in C:\wamp\www\Symfony\src\Hotels\GestionAnnoncesBundle\Form\AdresseType.php line 23

Or, j'ai bien ceci au début de mon formulaire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

use Hotels\GestionAnnoncesBundle\Entity\LocaliteRepository;

[rafleboss]

franchement je n'ai jamais fait sa! donc je peux pas t'aider sur cette erreur par contre localite est dans ton Form AdresseType c'est bien?? Moi je te conseil d'appeler la fonction dans ton contrôleur et de passer le tableau en paramètre afin d'éviter d'appeler ton fonction N fois la génération d'adresse. De plus pour ton fichier LocaliteRepository je n'utiliserai pas le foreach alors que tu peux faire autrement !! tu prends des ressources pour rien... enfin j'espère que sa t'aidera!! Mais essaie de passer le tableau en paramètre !