Discussion :

[yamatoshi]

Bonjour, j'ai suivi ce tuto https://gist.github.com/danvbe/4476697 pour pouvoir gérer à partir de FOSUserBundle et HWI la connexion automatique via facebook. J'ai réussi et cela marche très bien. Par contre lors de l'enregistrement d'un nouvel utilisateur via facebook, j'ai voulu modifier la fonction loadUserByOAuthUserResponse du fichier UserProvider.php pour créer en même temps une nouvelle ligne dans une table que j'ai appelé membre. En faisant des recherches sur les erreurs que j'obtiens j'ai bien compris que qu'on ne pouvait pas faire appel à Doctrine dans un service en faisant un simple via $em = $this->container->get('doctrine')->getEntityManager();

Aussi, j'ai fais plusieurs recherches et tenté plusieurs solutions dont celle-ci donnée sur ce forum mais j'obtients toujours cette erreur de Call to e member Function...

Code :

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

Error: Call to a member function getEntityManager() on a non-object in D:\www\myapp\src\MyAPp\UserBundle\OAuth\UserProvider.php line 114

Service.yml

Code :

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

1
2
3
4
5
services:
    my_user_provider:
        class: "%my_user_provider.class%"
        arguments: [@fos_user.user_manager,{facebook: facebook_id, google: google_id} ]
        arguments: [@doctrine.orm.entity_manager]

UserProvider.php

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
use Symfony\Bundle\DoctrineBundle\Registry;
use Doctrine\ORM\EntityRepository;

class UserProvider extends FOSUBUserProvider
{
        protected $em;
        public function _construct(EntityManager $em)
        {      
             $em = $this->em;
        }

        /**
        * {@inheritdoc}
        */
        public function loadUserByOAuthUserResponse(UserResponseInterface $response){
           
            [...]
            $user_Id = $user->getId();
            $membre = $em->getEntityManager()->getRepository('MyAppMembreBundle:Membre')->findOneBy(array('id_user' => $user_Id));
            [...]
        }
}

Merci pour vos explications qui me permettrons de comprendre où ça bloque.

[Nico_F]

Haha, c'est mignon, tu as fait la bonne démarche, mais tu n'as pas compris ce que tu as copié-collé (ou alors c'est une simple erreur d'inattention).

Code :

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

$membre = $em->getEntityManager()->getRepository('MyAppMembreBundle:Membre')->findOneBy(array('id_user' => $user_Id));

Cette ligne : $em est déjà censé être ton entityManager, plus besoin d'appeler la méthode qui va te le récupérer puisque c'est déjà ça. Et vu que tu l'as injecté et qu'il est en attribut de ta classe c'est $this->em.

Code :

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

$membre = $this->em->getRepository('MyAppMembreBundle:Membre')->findOneBy(array('id_user' => $user_Id));

Ca devrait aller mieux

[Nico_F]

Tu peux aussi faire encore plus propre en déclarant ton repository en tant que service et en n'injectant que le repository et non l'entityManager tout entier.

À moins que tu en aies besoin pour de la persistance plus tard ... mais je ne sais pas ce que tu as après.

++

[yamatoshi]

Cette ligne : $em est déjà censé être ton entityManager, plus besoin d'appeler la méthode qui va te le récupérer puisque c'est déjà ça. Et vu que tu l'as injecté et qu'il est en attribut de ta classe c'est $this->em.

Code :

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

$membre = $this->em->getRepository('MyAppMembreBundle:Membre')->findOneBy(array('id_user' => $user_Id));

Ca devrait aller mieux

Bonjour Nico F. J'ai bien compris ta remarque. Mais lorsque je fais comme tu as dit, j'ai toujours le message d'erreur

Code :

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

Error: Call to a member function getRepository() on a non-object in D:\www\myapp\src\MyApp\UserBundle\OAuth\UserProvider.php line 116

Et ma ligne 116

Code :

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

$membre = $this->em->getRepository('MyAppMembreBundle:Membre')->findOneBy(array('id_user' => $user_Id));

Je ne comprends plus rien.

[Nico_F]

Ton problème vient de l'injection de dépendance.
Le code que je t'ai donné est bon.

Si tu continues à avoir le problème c'est que ta variable $em ne contient pas un entityManager.
Si ta variable ne contient pas un entityManager, c'est parce que l'injection de dépendance n'a pas fonctionné.
Si l'injection de dépendance n'a pas fonctionné c'est parce que tu l'as mal configuré

Code :

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

1
2
3
4
5
services:
    my_user_provider:
        class: "%my_user_provider.class%"
        arguments: [@fos_user.user_manager,{facebook: facebook_id, google: google_id} ]
        arguments: [@doctrine.orm.entity_manager]

Donne nous le contenu complet de ce fichier : c'est là que ça bloque.
Compte tenu de la définition de ton construit les paramètres "arguments" ne sont pas bon : il y en a deux dans l'injection, un seul dans le construct : ça ne va pas.
Si ton UserProvider étend de FOSUBUserProvider, il est possible que l'injection de services doive être hérité également.

Avec le fichier de service complet ce sera plus facile de t'aider.

[goabonga]

Slt,
_construct n'est pas __construct !

Ensuite si tu ovverides le constructeur de ton FOSUBUserProvider il faudra lui repasser les arguments.
puis $this->em = $em;

Code :

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

1
2
3
4
5
6
7
8
9
 
class UserProvider extends FOSUBUserProvider
{
        protected $em;
        public function __construct($fosParams,$fbParms, EntityManager $em)
        {      
             $this->em = $em;
             parent::__construct($fosParams,$fbParms);
        }

Dans ta config :

Code :

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

1
2
 
        arguments: [@fos_user.user_manager,{facebook: facebook_id, google: google_id}, @doctrine.orm.entity_manager]

Un conseil il faut apprendre le php avant de s'attaquer a un tel outil !