Skip to content

Commit aacdaf3

Browse files
xabbuhxabbuh
committed
use named-arguments to configure validation constraint options
1 parent 171af72 commit aacdaf3

24 files changed

+130
-130
lines changed

‎components/console/helpers/questionhelper.rst

+4-4
Original file line numberDiff line numberDiff line change
@@ -480,10 +480,10 @@ invalid answer and will only be able to proceed if their input is valid.
480480
use Symfony\Component\Validator\Validation;
481481

482482
$question = new Question('Please enter the name of the bundle', 'AcmeDemoBundle');
483-
$validation = Validation::createCallable(new Regex([
484-
'pattern' => '/^[a-zA-Z]+Bundle$/',
485-
'message' => 'The name of the bundle should be suffixed with \'Bundle\'',
486-
]));
483+
$validation = Validation::createCallable(new Regex(
484+
pattern: '/^[a-zA-Z]+Bundle$/',
485+
message: 'The name of the bundle should be suffixed with \'Bundle\'',
486+
));
487487
$question->setValidator($validation);
488488

489489
Validating a Hidden Response

‎components/options_resolver.rst

+1-1
Original file line numberDiff line numberDiff line change
@@ -394,7 +394,7 @@ returns ``true`` for acceptable values and ``false`` for invalid values::
394394

395395
// ...
396396
$resolver->setAllowedValues('transport', Validation::createIsValidCallable(
397-
new Length(['min' => 10 ])
397+
new Length(min: 10)
398398
));
399399

400400
In sub-classes, you can use :method:`Symfony\\Component\\OptionsResolver\\OptionsResolver::addAllowedValues`

‎components/validator.rst

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ characters long::
3636

3737
$validator = Validation::createValidator();
3838
$violations = $validator->validate('Bernhard', [
39-
new Length(['min' => 10]),
39+
new Length(min: 10),
4040
new NotBlank(),
4141
]);
4242

‎components/validator/metadata.rst

+4-4
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ the ``Author`` class has at least 3 characters::
2424
$metadata->addPropertyConstraint('firstName', new Assert\NotBlank());
2525
$metadata->addPropertyConstraint(
2626
'firstName',
27-
new Assert\Length(["min" => 3])
27+
new Assert\Length(min: 3)
2828
);
2929
}
3030
}
@@ -55,9 +55,9 @@ Then, add the Validator component configuration to the class::
5555
{
5656
public static function loadValidatorMetadata(ClassMetadata $metadata): void
5757
{
58-
$metadata->addGetterConstraint('passwordSafe', new Assert\IsTrue([
59-
'message' => 'The password cannot match your first name',
60-
]));
58+
$metadata->addGetterConstraint('passwordSafe', new Assert\IsTrue(
59+
message: 'The password cannot match your first name',
60+
));
6161
}
6262
}
6363

‎components/validator/resources.rst

+4-4
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,10 @@ In this example, the validation metadata is retrieved executing the
4242
public static function loadValidatorMetadata(ClassMetadata $metadata): void
4343
{
4444
$metadata->addPropertyConstraint('name', new Assert\NotBlank());
45-
$metadata->addPropertyConstraint('name', new Assert\Length([
46-
'min' => 5,
47-
'max' => 20,
48-
]));
45+
$metadata->addPropertyConstraint('name', new Assert\Length(
46+
min: 5,
47+
max: 20,
48+
));
4949
}
5050
}
5151

‎controller/upload_file.rst

+5-5
Original file line numberDiff line numberDiff line change
@@ -75,14 +75,14 @@ so Symfony doesn't try to get/set its value from the related entity::
7575
// unmapped fields can't define their validation using attributes
7676
// in the associated entity, so you can use the PHP constraint classes
7777
'constraints' => [
78-
new File([
79-
'maxSize' => '1024k',
80-
'mimeTypes' => [
78+
new File(
79+
maxSize: '1024k',
80+
mimeTypes: [
8181
'application/pdf',
8282
'application/x-pdf',
8383
],
84-
'mimeTypesMessage' => 'Please upload a valid PDF document',
85-
])
84+
mimeTypesMessage: 'Please upload a valid PDF document',
85+
)
8686
],
8787
])
8888
// ...

‎form/without_class.rst

+4-4
Original file line numberDiff line numberDiff line change
@@ -96,12 +96,12 @@ but here's a short example::
9696
{
9797
$builder
9898
->add('firstName', TextType::class, [
99-
'constraints' => new Length(['min' => 3]),
99+
'constraints' => new Length(min: 3),
100100
])
101101
->add('lastName', TextType::class, [
102102
'constraints' => [
103103
new NotBlank(),
104-
new Length(['min' => 3]),
104+
new Length(min: 3),
105105
],
106106
])
107107
;
@@ -153,10 +153,10 @@ This can be done by setting the ``constraints`` option in the
153153
$resolver->setDefaults([
154154
'data_class' => null,
155155
'constraints' => new Collection([
156-
'firstName' => new Length(['min' => 3]),
156+
'firstName' => new Length(min: 3),
157157
'lastName' => [
158158
new NotBlank(),
159-
new Length(['min' => 3]),
159+
new Length(min: 3),
160160
],
161161
]),
162162
]);

‎reference/constraints/All.rst

+4-4
Original file line numberDiff line numberDiff line change
@@ -79,12 +79,12 @@ entry in that array:
7979
{
8080
public static function loadValidatorMetadata(ClassMetadata $metadata): void
8181
{
82-
$metadata->addPropertyConstraint('favoriteColors', new Assert\All([
83-
'constraints' => [
82+
$metadata->addPropertyConstraint('favoriteColors', new Assert\All(
83+
constraints: [
8484
new Assert\NotBlank(),
85-
new Assert\Length(['min' => 5]),
85+
new Assert\Length(min: 5),
8686
],
87-
]));
87+
));
8888
}
8989
}
9090

‎reference/constraints/AtLeastOneOf.rst

+12-12
Original file line numberDiff line numberDiff line change
@@ -115,23 +115,23 @@ The following constraints ensure that:
115115
{
116116
public static function loadValidatorMetadata(ClassMetadata $metadata): void
117117
{
118-
$metadata->addPropertyConstraint('password', new Assert\AtLeastOneOf([
119-
'constraints' => [
120-
new Assert\Regex(['pattern' => '/#/']),
121-
new Assert\Length(['min' => 10]),
118+
$metadata->addPropertyConstraint('password', new Assert\AtLeastOneOf(
119+
constraints: [
120+
new Assert\Regex(pattern: '/#/'),
121+
new Assert\Length(min: 10),
122122
],
123-
]));
123+
));
124124
125-
$metadata->addPropertyConstraint('grades', new Assert\AtLeastOneOf([
126-
'constraints' => [
127-
new Assert\Count(['min' => 3]),
128-
new Assert\All([
129-
'constraints' => [
125+
$metadata->addPropertyConstraint('grades', new Assert\AtLeastOneOf(
126+
constraints: [
127+
new Assert\Count(min: 3),
128+
new Assert\All(
129+
constraints: [
130130
new Assert\GreaterThanOrEqual(5),
131131
],
132-
]),
132+
),
133133
],
134-
]));
134+
));
135135
}
136136
}
137137

‎reference/constraints/CardScheme.rst

+4-4
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,12 @@ on an object that will contain a credit card number.
7777
{
7878
public static function loadValidatorMetadata(ClassMetadata $metadata): void
7979
{
80-
$metadata->addPropertyConstraint('cardNumber', new Assert\CardScheme([
81-
'schemes' => [
80+
$metadata->addPropertyConstraint('cardNumber', new Assert\CardScheme(
81+
schemes: [
8282
Assert\CardScheme::VISA,
8383
],
84-
'message' => 'Your credit card number is invalid.',
85-
]));
84+
message: 'Your credit card number is invalid.',
85+
));
8686
}
8787
}
8888

‎reference/constraints/Choice.rst

+10-10
Original file line numberDiff line numberDiff line change
@@ -100,10 +100,10 @@ If your valid choice list is simple, you can pass them in directly via the
100100
new Assert\Choice(['New York', 'Berlin', 'Tokyo'])
101101
);
102102
103-
$metadata->addPropertyConstraint('genre', new Assert\Choice([
104-
'choices' => ['fiction', 'non-fiction'],
105-
'message' => 'Choose a valid genre.',
106-
]));
103+
$metadata->addPropertyConstraint('genre', new Assert\Choice(
104+
choices: ['fiction', 'non-fiction'],
105+
message: 'Choose a valid genre.',
106+
));
107107
}
108108
}
109109
@@ -182,9 +182,9 @@ constraint.
182182
183183
public static function loadValidatorMetadata(ClassMetadata $metadata): void
184184
{
185-
$metadata->addPropertyConstraint('genre', new Assert\Choice([
186-
'callback' => 'getGenres',
187-
]));
185+
$metadata->addPropertyConstraint('genre', new Assert\Choice(
186+
callback: 'getGenres',
187+
));
188188
}
189189
}
190190
@@ -250,9 +250,9 @@ you can pass the class name and the method as an array.
250250
251251
public static function loadValidatorMetadata(ClassMetadata $metadata): void
252252
{
253-
$metadata->addPropertyConstraint('genre', new Assert\Choice([
254-
'callback' => [Genre::class, 'getGenres'],
255-
]));
253+
$metadata->addPropertyConstraint('genre', new Assert\Choice(
254+
callback: [Genre::class, 'getGenres'],
255+
));
256256
}
257257
}
258258

‎reference/constraints/Collection.rst

+22-22
Original file line numberDiff line numberDiff line change
@@ -139,8 +139,8 @@ following:
139139
140140
public static function loadValidatorMetadata(ClassMetadata $metadata): void
141141
{
142-
$metadata->addPropertyConstraint('profileData', new Assert\Collection([
143-
'fields' => [
142+
$metadata->addPropertyConstraint('profileData', new Assert\Collection(
143+
fields: [
144144
'personal_email' => new Assert\Email(),
145145
'short_bio' => [
146146
new Assert\NotBlank(),
@@ -150,8 +150,8 @@ following:
150150
]),
151151
],
152152
],
153-
'allowMissingFields' => true,
154-
]));
153+
allowMissingFields: true,
154+
));
155155
}
156156
}
157157
@@ -267,15 +267,15 @@ you can do the following:
267267
268268
public static function loadValidatorMetadata(ClassMetadata $metadata): void
269269
{
270-
$metadata->addPropertyConstraint('profileData', new Assert\Collection([
271-
'fields' => [
270+
$metadata->addPropertyConstraint('profileData', new Assert\Collection(
271+
fields: [
272272
'personal_email' => new Assert\Required([
273273
new Assert\NotBlank(),
274274
new Assert\Email(),
275275
]),
276276
'alternate_email' => new Assert\Optional(new Assert\Email()),
277277
],
278-
]));
278+
));
279279
}
280280
}
281281
@@ -291,28 +291,28 @@ groups. Take the following example::
291291

292292
use Symfony\Component\Validator\Constraints as Assert;
293293

294-
$constraint = new Assert\Collection([
295-
'fields' => [
294+
$constraint = new Assert\Collection(
295+
fields: [
296296
'name' => new Assert\NotBlank(['groups' => 'basic']),
297297
'email' => new Assert\NotBlank(['groups' => 'contact']),
298298
],
299-
]);
299+
);
300300

301301
This will result in the following configuration::
302302

303-
$constraint = new Assert\Collection([
304-
'fields' => [
305-
'name' => new Assert\Required([
306-
'constraints' => new Assert\NotBlank(['groups' => 'basic']),
307-
'groups' => ['basic', 'strict'],
308-
]),
309-
'email' => new Assert\Required([
310-
"constraints" => new Assert\NotBlank(['groups' => 'contact']),
311-
'groups' => ['basic', 'strict'],
312-
]),
303+
$constraint = new Assert\Collection(
304+
fields: [
305+
'name' => new Assert\Required(
306+
constraints: new Assert\NotBlank(groups: ['basic']),
307+
groups: ['basic', 'strict'],
308+
),
309+
'email' => new Assert\Required(
310+
constraints: new Assert\NotBlank(groups: ['contact']),
311+
groups: ['basic', 'strict'],
312+
),
313313
],
314-
'groups' => ['basic', 'strict'],
315-
]);
314+
groups: ['basic', 'strict'],
315+
);
316316

317317
The default ``allowMissingFields`` option requires the fields in all groups.
318318
So when validating in ``contact`` group, ``$name`` can be empty but the key is

‎reference/constraints/Compound.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,9 @@ you can create your own named set or requirements to be reused consistently ever
3535
return [
3636
new Assert\NotBlank(),
3737
new Assert\Type('string'),
38-
new Assert\Length(['min' => 12]),
38+
new Assert\Length(min: 12),
3939
new Assert\NotCompromisedPassword(),
40-
new Assert\PasswordStrength(['minScore' => 4]),
40+
new Assert\PasswordStrength(minScore: 4),
4141
];
4242
}
4343
}

‎reference/constraints/Count.rst

+6-6
Original file line numberDiff line numberDiff line change
@@ -82,12 +82,12 @@ you might add the following:
8282
8383
public static function loadValidatorMetadata(ClassMetadata $metadata): void
8484
{
85-
$metadata->addPropertyConstraint('emails', new Assert\Count([
86-
'min' => 1,
87-
'max' => 5,
88-
'minMessage' => 'You must specify at least one email',
89-
'maxMessage' => 'You cannot specify more than {{ limit }} emails',
90-
]));
85+
$metadata->addPropertyConstraint('emails', new Assert\Count(
86+
min: 1,
87+
max: 5,
88+
minMessage: 'You must specify at least one email',
89+
maxMessage: 'You cannot specify more than {{ limit }} emails',
90+
));
9191
}
9292
}
9393

‎reference/constraints/CssColor.rst

+9-9
Original file line numberDiff line numberDiff line change
@@ -110,15 +110,15 @@ the named CSS colors:
110110
{
111111
$metadata->addPropertyConstraint('defaultColor', new Assert\CssColor());
112112
113-
$metadata->addPropertyConstraint('accentColor', new Assert\CssColor([
114-
'formats' => Assert\CssColor::HEX_LONG,
115-
'message' => 'The accent color must be a 6-character hexadecimal color.',
116-
]));
117-
118-
$metadata->addPropertyConstraint('currentColor', new Assert\CssColor([
119-
'formats' => [Assert\CssColor::BASIC_NAMED_COLORS, Assert\CssColor::EXTENDED_NAMED_COLORS],
120-
'message' => 'The color "{{ value }}" is not a valid CSS color name.',
121-
]));
113+
$metadata->addPropertyConstraint('accentColor', new Assert\CssColor(
114+
formats: Assert\CssColor::HEX_LONG,
115+
message: 'The accent color must be a 6-character hexadecimal color.',
116+
));
117+
118+
$metadata->addPropertyConstraint('currentColor', new Assert\CssColor(
119+
formats: [Assert\CssColor::BASIC_NAMED_COLORS, Assert\CssColor::EXTENDED_NAMED_COLORS],
120+
message: 'The color "{{ value }}" is not a valid CSS color name.',
121+
));
122122
}
123123
}
124124

‎reference/constraints/DivisibleBy.rst

+3-3
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,9 @@ The following constraints ensure that:
9292
{
9393
$metadata->addPropertyConstraint('weight', new Assert\DivisibleBy(0.25));
9494
95-
$metadata->addPropertyConstraint('quantity', new Assert\DivisibleBy([
96-
'value' => 5,
97-
]));
95+
$metadata->addPropertyConstraint('quantity', new Assert\DivisibleBy(
96+
value: 5,
97+
));
9898
}
9999
}
100100

‎reference/constraints/Email.rst

+3-3
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,9 @@ Basic Usage
7070
7171
public static function loadValidatorMetadata(ClassMetadata $metadata): void
7272
{
73-
$metadata->addPropertyConstraint('email', new Assert\Email([
74-
'message' => 'The email "{{ value }}" is not a valid email.',
75-
]));
73+
$metadata->addPropertyConstraint('email', new Assert\Email(
74+
message: 'The email "{{ value }}" is not a valid email.',
75+
));
7676
}
7777
}
7878

0 commit comments

Comments
 (0)