minor symfony#10676 Removed useless parentheses in Assert annotations (thomasbisignani)

javiereguiluz · javiereguiluz · commit f54067b93028 · 2018-11-16T17:10:19.000+01:00
This PR was merged into the 2.8 branch. Discussion ---------- Removed useless parentheses in Assert annotations To be consistent, this PR removes the useless empty parentheses in the `@Assert` annotations for constraints. Sometimes `@Assert` annotations for constraints without options were used with empty parentheses, sometimes without parentheses. Commits ------- f5980f0 Removed useless parentheses in Assert annotations
diff --git a/best_practices/forms.rst b/best_practices/forms.rst
@@ -169,7 +169,7 @@ blank, add the following in the ``Post`` object::
     class Post
     {
         /**
-         * @Assert\NotBlank()
+         * @Assert\NotBlank
          */
         public $title;
     }
diff --git a/components/validator/resources.rst b/components/validator/resources.rst
@@ -100,7 +100,7 @@ prefixed classes included in doc block comments (``/** ... */``). For example::
     class User
     {
         /**
-        * @Assert\NotBlank()
+        * @Assert\NotBlank
         */
         protected $name;
     }
diff --git a/doctrine/registration_form.rst b/doctrine/registration_form.rst
@@ -63,19 +63,19 @@ With some validation added, your class may look something like this::
 
         /**
          * @ORM\Column(type="string", length=255, unique=true)
-         * @Assert\NotBlank()
-         * @Assert\Email()
+         * @Assert\NotBlank
+         * @Assert\Email
          */
         private $email;
 
         /**
          * @ORM\Column(type="string", length=255, unique=true)
-         * @Assert\NotBlank()
+         * @Assert\NotBlank
          */
         private $username;
 
         /**
-         * @Assert\NotBlank()
+         * @Assert\NotBlank
          * @Assert\Length(max=4096)
          */
         private $plainPassword;
diff --git a/form/embedded.rst b/form/embedded.rst
@@ -25,7 +25,7 @@ of course, by creating the ``Category`` object::
     class Category
     {
         /**
-         * @Assert\NotBlank()
+         * @Assert\NotBlank
          */
         public $name;
     }
@@ -40,7 +40,7 @@ Next, add a new ``category`` property to the ``Task`` class::
 
         /**
          * @Assert\Type(type="AppBundle\Entity\Category")
-         * @Assert\Valid()
+         * @Assert\Valid
          */
         protected $category;
 
diff --git a/forms.rst b/forms.rst
@@ -334,12 +334,12 @@ object.
         class Task
         {
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              */
             public $task;
 
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              * @Assert\Type("\DateTime")
              */
             protected $dueDate;
diff --git a/reference/constraints/Bic.rst b/reference/constraints/Bic.rst
@@ -37,7 +37,7 @@ will contain a Business Identifier Code (BIC).
         class Transaction
         {
             /**
-             * @Assert\Bic()
+             * @Assert\Bic
              */
             protected $businessIdentifierCode;
         }
diff --git a/reference/constraints/Blank.rst b/reference/constraints/Blank.rst
@@ -42,7 +42,7 @@ of an ``Author`` class were blank, you could do the following:
         class Author
         {
             /**
-             * @Assert\Blank()
+             * @Assert\Blank
              */
             protected $firstName;
         }
diff --git a/reference/constraints/Country.rst b/reference/constraints/Country.rst
@@ -29,7 +29,7 @@ Basic Usage
         class User
         {
             /**
-             * @Assert\Country()
+             * @Assert\Country
              */
              protected $country;
         }
diff --git a/reference/constraints/Date.rst b/reference/constraints/Date.rst
@@ -31,7 +31,7 @@ Basic Usage
         class Author
         {
             /**
-             * @Assert\Date()
+             * @Assert\Date
              */
              protected $birthday;
         }
diff --git a/reference/constraints/DateTime.rst b/reference/constraints/DateTime.rst
@@ -31,7 +31,7 @@ Basic Usage
         class Author
         {
             /**
-             * @Assert\DateTime()
+             * @Assert\DateTime
              */
              protected $createdAt;
         }
diff --git a/reference/constraints/IsNull.rst b/reference/constraints/IsNull.rst
@@ -36,7 +36,7 @@ of an ``Author`` class exactly equal to ``null``, you could do the following:
         class Author
         {
             /**
-             * @Assert\IsNull()
+             * @Assert\IsNull
              */
             protected $firstName;
         }
diff --git a/reference/constraints/Language.rst b/reference/constraints/Language.rst
@@ -30,7 +30,7 @@ Basic Usage
         class User
         {
             /**
-             * @Assert\Language()
+             * @Assert\Language
              */
              protected $preferredLanguage;
         }
diff --git a/reference/constraints/Locale.rst b/reference/constraints/Locale.rst
@@ -34,7 +34,7 @@ Basic Usage
         class User
         {
             /**
-             * @Assert\Locale()
+             * @Assert\Locale
              */
              protected $locale;
         }
diff --git a/reference/constraints/NotBlank.rst b/reference/constraints/NotBlank.rst
@@ -40,7 +40,7 @@ class were not blank, you could do the following:
         class Author
         {
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              */
             protected $firstName;
         }
diff --git a/reference/constraints/NotNull.rst b/reference/constraints/NotNull.rst
@@ -34,7 +34,7 @@ class were not strictly equal to ``null``, you would:
         class Author
         {
             /**
-             * @Assert\NotNull()
+             * @Assert\NotNull
              */
             protected $firstName;
         }
diff --git a/reference/constraints/Time.rst b/reference/constraints/Time.rst
@@ -34,7 +34,7 @@ of the day when the event starts:
         class Event
         {
             /**
-             * @Assert\Time()
+             * @Assert\Time
              */
              protected $startsAt;
         }
diff --git a/reference/constraints/UniqueEntity.rst b/reference/constraints/UniqueEntity.rst
@@ -52,7 +52,7 @@ your user table:
              * @var string $email
              *
              * @ORM\Column(name="email", type="string", length=255, unique=true)
-             * @Assert\Email()
+             * @Assert\Email
              */
             protected $email;
 
diff --git a/reference/constraints/Url.rst b/reference/constraints/Url.rst
@@ -32,7 +32,7 @@ Basic Usage
         class Author
         {
             /**
-             * @Assert\Url()
+             * @Assert\Url
              */
              protected $bioUrl;
         }
diff --git a/reference/constraints/Valid.rst b/reference/constraints/Valid.rst
@@ -57,7 +57,7 @@ stores an ``Address`` instance in the ``$address`` property::
         class Address
         {
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              */
             protected $street;
 
diff --git a/validation.rst b/validation.rst
@@ -52,7 +52,7 @@ following:
         class Author
         {
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              */
             public $name;
         }
@@ -536,7 +536,7 @@ class to have at least 3 characters.
         class Author
         {
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              * @Assert\Length(min=3)
              */
             private $firstName;
diff --git a/validation/sequence_provider.rst b/validation/sequence_provider.rst
@@ -178,7 +178,7 @@ entity and a new constraint group called ``Premium``:
         class User
         {
             /**
-             * @Assert\NotBlank()
+             * @Assert\NotBlank
              */
             private $name;
 

Original file line number	Diff line number	Diff line change
@@ -169,7 +169,7 @@ blank, add the following in the ``Post`` object::
`169`	`169`	`class Post`
`170`	`170`	`{`
`171`	`171`	`/**`
`172`		`- * @Assert\NotBlank()`
	`172`	`+ * @Assert\NotBlank`
`173`	`173`	`*/`
`174`	`174`	`public $title;`
`175`	`175`	`}`
Original file line number	Diff line number	Diff line change
@@ -100,7 +100,7 @@ prefixed classes included in doc block comments (``/** ... */``). For example::
`100`	`100`	`class User`
`101`	`101`	`{`
`102`	`102`	`/**`
`103`		`- * @Assert\NotBlank()`
	`103`	`+ * @Assert\NotBlank`
`104`	`104`	`*/`
`105`	`105`	`protected $name;`
`106`	`106`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ will contain a Business Identifier Code (BIC).`
`37`	`37`	`class Transaction`
`38`	`38`	`{`
`39`	`39`	`/**`
`40`		`- * @Assert\Bic()`
	`40`	`+ * @Assert\Bic`
`41`	`41`	`*/`
`42`	`42`	`protected $businessIdentifierCode;`
`43`	`43`	`}`
Original file line number	Diff line number	Diff line change
@@ -42,7 +42,7 @@ of an ``Author`` class were blank, you could do the following:
`42`	`42`	`class Author`
`43`	`43`	`{`
`44`	`44`	`/**`
`45`		`- * @Assert\Blank()`
	`45`	`+ * @Assert\Blank`
`46`	`46`	`*/`
`47`	`47`	`protected $firstName;`
`48`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ Basic Usage`
`29`	`29`	`class User`
`30`	`30`	`{`
`31`	`31`	`/**`
`32`		`- * @Assert\Country()`
	`32`	`+ * @Assert\Country`
`33`	`33`	`*/`
`34`	`34`	`protected $country;`
`35`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ Basic Usage`
`31`	`31`	`class Author`
`32`	`32`	`{`
`33`	`33`	`/**`
`34`		`- * @Assert\Date()`
	`34`	`+ * @Assert\Date`
`35`	`35`	`*/`
`36`	`36`	`protected $birthday;`
`37`	`37`	`}`
Original file line number	Diff line number	Diff line change
@@ -36,7 +36,7 @@ of an ``Author`` class exactly equal to ``null``, you could do the following:
`36`	`36`	`class Author`
`37`	`37`	`{`
`38`	`38`	`/**`
`39`		`- * @Assert\IsNull()`
	`39`	`+ * @Assert\IsNull`
`40`	`40`	`*/`
`41`	`41`	`protected $firstName;`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ Basic Usage`
`30`	`30`	`class User`
`31`	`31`	`{`
`32`	`32`	`/**`
`33`		`- * @Assert\Language()`
	`33`	`+ * @Assert\Language`
`34`	`34`	`*/`
`35`	`35`	`protected $preferredLanguage;`
`36`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ Basic Usage`
`34`	`34`	`class User`
`35`	`35`	`{`
`36`	`36`	`/**`
`37`		`- * @Assert\Locale()`
	`37`	`+ * @Assert\Locale`
`38`	`38`	`*/`
`39`	`39`	`protected $locale;`
`40`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ class were not blank, you could do the following:`
`40`	`40`	`class Author`
`41`	`41`	`{`
`42`	`42`	`/**`
`43`		`- * @Assert\NotBlank()`
	`43`	`+ * @Assert\NotBlank`
`44`	`44`	`*/`
`45`	`45`	`protected $firstName;`
`46`	`46`	`}`