Any solution for this bug ? I've got same problem…

me too as @ehibes

i have the same problem? does anyone know some fixture or workaround for this issue ??

I'm having the same problem using multiple table inheritance

Was this ever addressed or a work around found?

I'm having the same problem

Is this issue ever going to be fixed? It's unbelievable that it's been open for over a year already!

@gentisaliu did you attempt at fixing it? this is FOSS

I'm trying to right now.

It seems this is rather a Doctrine issue as @datiecher rightfully points out. I don't have the time to go deep in the Doctrine code right now.

As a shortcut I would suggest adding an option in UniqueEntity declaring the entity whose repository is to be used or implement a repository method and set the 'repositoryMethod' in UniqueEntity accordingly.

I have the same problem. Has anyone found a solution ?

@Razmo There is no solution for this yet, it appears. I'd suggest adding a custom repository to your entity and implementing the findBy"fieldname" method yourself in there.

I dug into code and found that inside UniqueEntityValidator::validate, $className is populated with the child FQCN, not the parent which have the constraint. If I change $className to the right FQCN, everything works fine.

I have a piece of code that solve the problem, but is very far from a good code to commit inside source. Someone can give me some insights?

        $className = $this->context->getClassName();
        $class = $em->getClassMetadata($className);

        if($class->inheritanceType == \Doctrine\ORM\Mapping\ClassMetadataInfo::INHERITANCE_TYPE_JOINED) {
            $className = $class->rootEntityName;
            $class = $em->getClassMetadata($className);
        }

I'm not sure if this is related to Doctrine, since the $className is wrongly populated, but I could not track yet from where it is being populated. My guess is that $className should be populated with the FQCN of the Entity where @UniqueEntity is declared.

+1

Adding a boolean (inheritedFields) option to the constraint can maybe solve this problem :

UniqueEntity.php

class UniqueEntity extends Constraint
{
    public $message = 'This value is already used.';
    public $service = 'doctrine.orm.validator.unique';
    public $em = null;
    public $repositoryMethod = 'findBy';
    public $fields = array();
    public $errorPath = null;
    public $ignoreNull = true;
    public $inheritedFields = false;

UniqueEntityValidator.php

if($constraint->inheritedFields) {
    $repository = $em->getRepository($class->rootEntityName);
}
else {
    $repository = $em->getRepository(get_class($entity));
}
$result = $repository->{$constraint->repositoryMethod}($criteria);

User.php

Class User
{
    /**
     * @UniqueEntity("email", inheritedFields=true)
     */
    private $email;
}

@kassner

In Symfony\Component\Validator\Mapping\ClassMetadataFactory the parent class constraints are merged to the child constraints.

public function getMetadataFor($value)
{
    // ...

    // Include constraints from the parent class
    if ($parent = $metadata->getReflectionClass()->getParentClass()) {
        $metadata->mergeConstraints($this->getMetadataFor($parent->name));
    }

Maybe check in UniqueEntityValidator::validate if the fields are only present in the parent class and, in that case, switch to the parent repository.

We have to consider multi level of inheritance too.

This quick fix works good for me but is a BC break and support only one level of inheritance.

namespace Symfony\Bridge\Doctrine\Validator\Constraints;

class UniqueEntityValidator extends ConstraintValidator
{
    // ...

    public function validate($entity, Constraint $constraint)
    {
        // ...

        $involvedClasses = [];
        foreach ($fields as $fieldName) {
            if (!$class->hasField($fieldName) && !$class->hasAssociation($fieldName)) {
                throw new ConstraintDefinitionException(sprintf("The field '%s' is not mapped by Doctrine, so it cannot be validated for uniqueness.", $fieldName));
            }

            $property = $class->reflFields[$fieldName];
            $involvedClasses[$property->getDeclaringClass()->getName()] = true;
            $criteria[$fieldName] = $property->getValue($entity);

            // ...
        }

        /*
         * We have to switch to parent repository if fields are only present in parent.
         * Does not support multi level inheritance.
         * @see https://github.com/symfony/symfony/issues/4087
         */
        if (count($involvedClasses) == 1) {
            $repository = $em->getRepository(key($involvedClasses));
        } else {
            $repository = $em->getRepository(get_class($entity));
        }

        $result = $repository->{$constraint->repositoryMethod}($criteria);

        // ...
}

There is a workaround with repositoryMethod
See http://www.kassner.com.br/en/2013/11/27/symfony-2-inheritance-and-uniqueentity-workaround/

For everyone else affected by this: Another workaround is documented here:
http://jayroman.com/blog/symfony2-quirks-with-doctrine-inheritance-and-unique-constraints

+1

The solution presented here worked for me:

http://jayroman.com/blog/symfony2-quirks-with-doctrine-inheritance-and-unique-constraints

I then ran into another issue after this which was caused by needing to validate at both levels which resulted in 2 validation errors instead of the anticipated 1 but that can be resolved via validation groups so that's not a big deal and certainly better than exceptions.