Discussion :

[pc.bertineau]

Bonjour,

Sur un champ text non "required", sans contrainte NotBlank ou autre, la valeur correspondante de mon entité n'est pas écrasée par une chaine vide.

Doctrine tente de faire un setMonChamp(null) qui n'écrase pas l'ancienne valeur... Si je fais un setMonChamp('') à la main ça marche bien...

Avez-vous rencontré ce problème ?

[winzou]

Salut,

C'est etrange oui, mais pourquoi un setTonChamp(null) n'ecrase pas l'ancienne valeur ? A partir du moment ou Doctrine appelle bien cette methode de l'objet, c'est a toi de gerer le cas dans cette methode.

[pc.bertineau]

Je n'en sais rien, "mon entité est brut de génération".

J'ai remarque que le problème se pose pour une entité qui est "proxyfiée" par Doctrine2, pour les autres je peux setter une valeur vide sans problème...
edit: FAUX, j'en ai qui marche, même proxyfiée...

[pc.bertineau]

Voici le code côté Controller :

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
   public function editProfilAction()
   {
      $user = $this->get('security.context')->getToken()->getUser();
      $profile = $user->getCustomerAccount();
 
      $form = $this->get('form.factory')
         ->createBuilder('form', $profile)
         ->add('companyName', 'text', array ('label' => "youraccount.companyName", 'required' => false))
         ->add('contactFullname', 'text', array ('label' => "youraccount.contactName", 'required' => true))
         ->add('phoneNumber', 'text', array ('label' => "youraccount.phoneNumber", 'required' => true))
         ->add('address1', 'text', array ('label' => "youraccount.address1", 'required' => false))
         ->add('address2', 'text', array ('label' => "youraccount.address2", 'required' => false))
         ->add('zip', 'text', array ('label' => "youraccount.zipcode", 'required' => false))
         ->add('city', 'text', array ('label' => "youraccount.city", 'required' => false))
         ->add('state', 'choice', array ('label' => "shipcountry", 'choices' => array ('FR' => 'FR', 'DE' => 'DE', 'UK' => 'UK'), 'required' => true))
         ->getForm();
 
      $request = $this->get('request');
      if ($request->getMethod() == 'POST')
      {
         $form->bindRequest($request);
 
         if ($form->get('contactFullname')->isValid() && $form->get('phoneNumber')->isValid() && $form->get('state')->isValid())
         {
            $em = $this->get('doctrine')->getEntityManager();
            $em->persist($profile);
            $em->flush();
            return $this->redirect($this->generateUrl('_account'));
         }
      }
 
      return $this->render('CompanyAccountBundle:EditAccount:editProfil.html.twig'));

[pc.bertineau]

Je n'y comprends rien, j'ai écrit un test qui ressort une entité de ma base, setCompanyName(null) puis persist et flush... et là ça marche bien...

[pc.bertineau]

J'explique le code de mon contrôlleur :

Je récupère le customerAccount lié à ma table user (via la colonne user.customer_account_id). Doctrine m'a mappé une association ManyToOne et donc je récupère le customerAccount en question directement depuis le User connecté :

Code :

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

1
2
$user = $this->get('security.context')->getToken()->getUser();
$customerAccount = $user->getCustomerAccount();

Seulement je ne récupère pas une entité réelle mais une ProxyClass auto-générée par Doctrine2.

Je pense que certaines classes sont automatiquement "proxyfiée" quand elles sont chargées... mais franchement je ne saurai dire sur quelq critères

Et donc quand je souhaite sauvegarder les données de mon form, un champ textuel initialement non-vide que je souhaiterais vider n'est pas sauvegardé...

Mais là où ça devient intéressant, c'est que si j'essaie de modifier un customerAccount autre que celui de mon user courant, les données sont bien mises à jour, même avec une chaîne vide !

Code :

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

1
2
 
      $customerAccount = $em->getRepository('Company\SellerBundle\Entity\CustomerAccount')->findOneBy(array ('customerAccountId' => 8));

Il semble donc qu'il y ait un rapport avec les proxy...

[winzou]

Uhm interessant. J'avais deja lu un truc sur les proxy Doctrine qui expliquait bien mais pas moyen de remettre la main dessus...

Tu t'en sers souvent de ton CustomerAccount ? Une alternative (que je fais perso) est de personnaliser le UserProvider pour recuperer non seulement le User, mais aussi le CustomerAccount via une jointure. Et ce a chaque page. Ainsi, quand tu feras $user->getCustomerAccount(), ca ne va pas trigger une requete. Et tu auras une vraie entite, pas un proxy fantome a la Doctrine :p

[pc.bertineau]

Ahh! Je vais tester ça, ça m'a l'air intéressant

[pc.bertineau]

Merci Winzou, ça marche du tonnerre !!

Néanmoins j'ai déclaré le bug sur Symfony car ne pas pouvoir faire un setXxxx(null) sur un proxy est bien embêtant...
https://github.com/symfony/symfony/issues/1417