[Validator] Added abstract method Constraint::targets() to define whether constraints can be put onto properties, classes or both

Author: Bernhard Schussek

src/Symfony/Component/Validator/Constraint.php

@@ -28,8 +28,24 @@
  */
 abstract class Constraint
 {
+    /**
+     * The name of the group given to all constraints with no explicit group
+     * @var string
+     */
     const DEFAULT_GROUP = 'Default';
 
+    /**
+     * Marks a constraint that can be put onto classes
+     * @var string
+     */
+    const CLASS_CONSTRAINT = 'class';
+
+    /**
+     * Marks a constraint that can be put onto properties
+     * @var string
+     */
+    const PROPERTY_CONSTRAINT = 'property';
+
     /**
      * @var array
      */
@@ -173,4 +189,15 @@ public function validatedBy()
     {
         return get_class($this) . 'Validator';
     }
+
+    /**
+     * Returns whether the constraint can be put onto classes, properties or
+     * both
+     *
+     * This method should return one or more of the constants
+     * Constraint::CLASS_CONSTRAINT and Constraint::PROPERTY_CONSTRAINT.
+     *
+     * @return string|array  One or more constant values
+     */
+    abstract public function targets();
 }

src/Symfony/Component/Validator/Constraints/All.php

@@ -24,4 +24,12 @@ public function requiredOptions()
     {
         return array('constraints');
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/AssertFalse.php

@@ -14,4 +14,12 @@
 class AssertFalse extends \Symfony\Component\Validator\Constraint
 {
     public $message = 'This value should be false';
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/AssertTrue.php

@@ -14,4 +14,12 @@
 class AssertTrue extends \Symfony\Component\Validator\Constraint
 {
     public $message = 'This value should be true';
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/AssertType.php

@@ -31,4 +31,12 @@ public function requiredOptions()
     {
         return array('type');
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/Blank.php

@@ -14,4 +14,12 @@
 class Blank extends \Symfony\Component\Validator\Constraint
 {
     public $message = 'This value should be blank';
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/Choice.php

@@ -29,4 +29,12 @@ public function defaultOption()
     {
         return 'choices';
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/Collection.php

@@ -23,4 +23,12 @@ public function requiredOptions()
     {
         return array('fields');
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/Country.php

@@ -14,4 +14,12 @@
 class Country extends \Symfony\Component\Validator\Constraint
 {
     public $message = 'This value is not a valid country';
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }

src/Symfony/Component/Validator/Constraints/Date.php

@@ -14,4 +14,12 @@
 class Date extends \Symfony\Component\Validator\Constraint
 {
     public $message = 'This value is not a valid date';
+
+    /**
+     * {@inheritDoc}
+     */
+    public function targets()
+    {
+        return self::PROPERTY_CONSTRAINT;
+    }
 }