problema al validar un formulario embebido

[daymerrf]

Hola comunidad hace algunos días tengo un problema con symfony 2.3, tengo el siguiente escenario: Una entidad Usuario y una Teléfono, con una relación de Uno a Mucho, es decir un usuario puede tener muchos teléfonos. Muestro un formulario en donde el usuario puede ir dinámicamente insertando hasta 3 teléfonos. Y en caso de editar el usuario dicha información puede modificar sus teléfonos o eliminar alguno o insertarlo en caso de ser posible. Cuando un usuario se va a crear nuevo todo funciona perfectamente, el gran problema esta cuando un usuario desea editar sus teléfonos, el problema consiste en que el usuario al modificar un teléfono e insertar uno numero no valido, que no cumple con una expresión regular que defino, el formulario cuando lo envió me da error y me muestra perfectamente el mensaje, pero cuando reviso en la base de datos, el numero se modifico o se inserto aunque este mal escrito. He estado buscando y todo parece estar en cuando ejecuto el

Código:

$form->bind($request),

al parecer al ejecutar esta línea ya se insertan los teléfonos en la base de datos y luego cuando ejecuto el

Código:

$form->isValid()

, me da los errores, pero ya se ha insertado los campos de teléfonos en, la base de datos.
Espero que me puedan ayudar con esto, si necesitan mas información para encontrar la solución yo se las puedo subir.
Gracias a todos los que me puedan ayudar de antemano.

[hhs]

De entrada el uso de bind es obsoleto, debes de utiliza handleRequest($request), por otro lado ese método no realiza el inser en la base de datos. Me temo que el isValid() no esta usando las reglas que puso para validar el telefono. Puedes publicar tu entidad y tu form para ver que estas haciendo ?

[daymerrf]

@hhs: aqui te dejo las entitades y los formularios que estoy utilizando. Te los muestro solo con los atributos necesarios.
Entidades

Código PHP:

<?php
 
namespace AppBundle\Entity;
 
use Doctrine\ORM\Mapping as ORM;
use Symfony\Bridge\Doctrine\Validator\Constraints as DoctrineAssert;
use Symfony\Component\Validator\Constraints as Assert;
 
/**
 * User
 *
 * @ORM\Table(name="user")
 * @ORM\Entity(repositoryClass="AppBundle\Entity\UserRepository")
 */
class User
{
    /**
     * @var integer
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;
    
    /**
     * @var Phones
     *
     * @ORM\OneToMany(targetEntity="Phone", mappedBy="user", cascade={"persist" , "remove"})    
     */
    private $phones;
    
    /**
     * Get id
     *
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }
    
    /**
     * Constructor
     */
    public function __construct()
    {       
        $this->phones = new \Doctrine\Common\Collections\ArrayCollection();
    }
 
    /**
     * Add phones
     *
     * @param \AppBundle\Entity\Phone $phones
     * @return User
     */
    public function addPhone(\AppBundle\Entity\Phone $phone)
    {        
        $phone->setUser($this);
        $this->phones->add($phone);    
    }
 
    /**
     * Remove phones
     *
     * @param \AppBundle\Entity\Phone $phones
     */
    public function removePhone(\AppBundle\Entity\Phone $phone)
    {
        $this->phones->removeElement($phone);
    }
 
    /**
     * Get phones
     *
     * @return \Doctrine\Common\Collections\Collection 
     */
    public function getPhones()
    {
        return $this->phones;
    }     
    
    public function setPhones(\Doctrine\Common\Collections\Collection $phones)
    {
        die('setPhones');
        $this->phones = $phones;
        foreach ($phones as $phone) {
            $phone->setUser($this);
        }
    }
        
}
 

Código PHP:

<?php
 
namespace AppBundle\Entity;
 
use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Validator\Constraints as Assert;
use Symfony\Bridge\Doctrine\Validator\Constraints as DoctrineAssert;
 
/**
 * Phone
 *
 * @ORM\Table()
 * @ORM\Entity(repositoryClass="AppBundle\Entity\PhoneRepository")
 */
class Phone
{
    /**
     * @var integer
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;
 
    /**
     * @var string
     *
     * @ORM\Column(name="number", type="string", length=20)
     * @Assert\Regex(pattern="/^\+53\ ?[0-9]{8}$/", message = "field.phone")
     */
    private $number;
 
    /**
     * @var \User
     *
     * @ORM\ManyToOne(targetEntity="User", inversedBy="phones", cascade={"persist"})
     * @ORM\JoinColumns({
     *   @ORM\JoinColumn(name="user", referencedColumnName="id", onDelete="Cascade", nullable=false)
     * })
     */
    private $user;
 
    /**
     * Get id
     *
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }
 
    /**
     * Set number
     *
     * @param string $number
     * @return Phone
     */
    public function setNumber($number)
    {
        $this->number = $number;
        return $this;
    }
 
    /**
     * Get number
     *
     * @return string 
     */
    public function getNumber()
    {
        return $this->number;
    }
    
    /**
     * Set user
     *
     * @param \AppBundle\Entity\User $user
     * @return Phone
     */
    public function setUser(\AppBundle\Entity\User $user)
    {
        $this->user = $user;
 
        return $this;
    }
 
    /**
     * Get user
     *
     * @return \AppBundle\Entity\User 
     */
    public function getUser()
    {
        return $this->user;
    }
    
    public function __toString() {
        return $this->number;
    }
}
 

Formularios

Código PHP:

<?php
 
namespace AppBundle\Form;
 
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
use Symfony\Component\Validator\Constraints\Regex;
use Symfony\Component\Validator\Constraints\NotBlank;
 
class PhoneType extends AbstractType {
 
    /**
     * @param FormBuilderInterface $builder
     * @param array $options
     */
    public function buildForm(FormBuilderInterface $builder, array $options) {
        $builder
                ->add('number', 'text', array(
                    'max_length' => 15,
                    'required' => false,
                    'label' => null,
                    'constraints' => array(
                        new Regex(array(
                            'pattern' => "/^\+53\ ?[0-9]{8}$/",
                            'message' => 'field.phone'
                                )),
                        new NotBlank()
                    )
                ))
        ;
    }
 
    /**
     * @param OptionsResolverInterface $resolver
     */
    public function setDefaultOptions(OptionsResolverInterface $resolver) {
        $resolver->setDefaults(array(
            'data_class' => 'AppBundle\Entity\Phone'
        ));
    }
 
    /**
     * @return string
     */
    public function getName() {
        return 'appbundle_phone';
    }
 
}
 

Código PHP:

<?php
 
namespace AppBundle\Form;
 
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
use Symfony\Component\Validator\Constraints\NotBlank;
use Symfony\Component\Validator\Constraints\Email;
use Symfony\Component\Validator\Constraints\Length;
use AppBundle\Form\PhoneType;
 
class UserType extends AbstractType {
 
    /**
     * @param FormBuilderInterface $builder
     * @param array $options
     */
    public function buildForm(FormBuilderInterface $builder, array $options) {
        $builder                
                ->add('phones', 'collection', array(
                    'label' => false,
                    'type' => new PhoneType(),
                    'allow_add' => true,
                    'allow_delete' => true,
                    'by_reference' => false,
                    'cascade_validation' => true,
                ))
        ;
    }
 
    /**
     * @param OptionsResolverInterface $resolver
     */
    public function setDefaultOptions(OptionsResolverInterface $resolver) {
        $resolver->setDefaults(array(
            'data_class' => 'AppBundle\Entity\User'
        ));
    }
 
    /**
     * @return string
     */
    public function getName() {
        return 'appbundle_user';
    }
 
}
 

[hhs]

Primer paso, el constraints se utiiza cuando tu formulario no tiene una entidad asociada, asi que quita eso del formulario PhoneType y cambia el "cascade_validation" a false en UserType.
Para validar una entidad que tiene relaciones le tienes que decir que delegue esa responsabilidad a la entidad asociada a la relación en el Assert. Eso se logra usando el Asset Valid: http://symfony.com/doc/current/refer...nts/Valid.html

[daymerrf]

Ya hice todo lo que me dijiste, y leyendo en la url que me dejaste el atributo phones en la entidad User le modifique las anotaciones de la siguiente manera:

Código PHP:

/**
     * @var Phones
     *
     * @ORM\OneToMany(targetEntity="Phone", mappedBy="hostal", cascade={"persist" , "remove"})  
     * @Assert\Valid
     */
    protected $phones;
 

pero lamentablemente me sigue dando el mismo error, me valida, pero cuando reviso la base de datos el teléfono ha sido modificado, voy a ponerte también a continuación el código del controlador para que también lo veas

Código PHP:

<?php
 
namespace AppBundle\Controller\Admin;
 
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;
 
class UserController extends Controller {
 
 public function editAction(Request $request, $id) {
        $translator = $this->get('translator');
        $em = $this->getDoctrine()->getManager();
        $user = $em->getRepository('AppBundle:User')->findOneById($id);
 
        if (!$user)
            return Helpers::error404($this, $translator->trans('user.not.found', array(), 'admin'), $translator->trans('back.users', array(), 'admin'), 'fa fa-bed fa-fw', $this->generateUrl('admin_users'));
   
        $originalPhones = array();
        // Create an array of the current phones objects in the database
        foreach ($user->getPhones() as $phone) {
            $originalPhones[] = $phone;
        }
 
       
        $form = $this->createForm(new UserType(), $user);
 
        if ($request->getMethod() == 'POST') {
            
            $form->handleRequest($request);
 
            if ($form->isValid()) {
 
                //process to update the phones
                // filter $originalPhones to contain tags no longer present
                foreach ($user->getPhones() as $phone) {
                    foreach ($originalPhones as $key => $toDel) {
                        if ($toDel->getId() === $phone->getId()) {
                            unset($originalPhones[$key]);
                        }
                    }
                }
 
                // remove the relationship between the phone and the User
                foreach ($originalPhones as $phone) {
                    $user->getPhones()->removeElement($phone);
                    $em->remove($phone);
                }
 
                $em->persist($user);
                $em->flush();
                             
                return $this->redirect($this->generateUrl('admin_users'));
            }
        }
        
        return $this->render('AppBundle:Admin/user:form.html.twig', array(
                    'form' => $form->createView(),                   
                    'action' => 'Edit',`                    
        ));
    }
}
 

[hhs]

Si agregaste el assert en la entidad Phone ?