Skip to content

Commit 1cd514e

Browse files
committed
Rename validate method to checkValidations for clarity in DTO validation.
1 parent dfea275 commit 1cd514e

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

src/base/dto/ValidatableDtoTrait.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ trait ValidatableDtoTrait
77
private static ?\WeakMap $__validationInputMap = null;
88
private ?ValidationResult $__validationResult = null;
99

10-
public function validate(?ValidationContext $ctx = null): ValidationResult
10+
public function checkValidations(?ValidationContext $ctx = null): ValidationResult
1111
{
1212
$context = $ctx ?? new ValidationContext($this->getValidationInputData());
1313
$result = DtoValidationEngine::validate(
@@ -21,7 +21,7 @@ public function validate(?ValidationContext $ctx = null): ValidationResult
2121

2222
public function isValid(?ValidationContext $ctx = null): bool
2323
{
24-
return $this->validate($ctx)->isValid();
24+
return $this->checkValidations($ctx)->isValid();
2525
}
2626

2727
/**

src/controller/UserController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ public function deleteUsers()
250250
}
251251
$requestDto = new DeleteUserRequestDto(false);
252252
$requestDto->fromArray($payload);
253-
$validation = $requestDto->validate(new ValidationContext(
253+
$validation = $requestDto->checkValidations(new ValidationContext(
254254
$payload,
255255
[],
256256
true,

tests/base/dto/DtoValidationEngineTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public function testValidateRejectsUnknownFieldsAndRuleViolations(): void
5151
'unknown' => 'boom',
5252
]);
5353

54-
$result = $dto->validate();
54+
$result = $dto->checkValidations();
5555
$this->assertFalse($result->isValid());
5656
$codes = array_column($result->getErrors(), 'code');
5757
$this->assertContains('unknown_field', $codes);
@@ -69,7 +69,7 @@ public function testValidateAppliesDefaultsAndAcceptsValidPayload(): void
6969
'status' => 'active',
7070
]);
7171

72-
$result = $dto->validate();
72+
$result = $dto->checkValidations();
7373
$this->assertTrue($result->isValid());
7474
$this->assertSame('guest', $dto->role);
7575
$this->assertNull($dto->comment);
@@ -85,7 +85,7 @@ public function testCsrfProtectedDtoValidatesTokenAndRejectsReplay(): void
8585
'_csrf' => $tokens['token'],
8686
'_csrf_key' => $tokens['key'],
8787
]);
88-
$first = $dto->validate();
88+
$first = $dto->checkValidations();
8989
$this->assertTrue($first->isValid());
9090

9191
$dtoReplay = new DemoCsrfDto(false);
@@ -94,7 +94,7 @@ public function testCsrfProtectedDtoValidatesTokenAndRejectsReplay(): void
9494
'_csrf' => $tokens['token'],
9595
'_csrf_key' => $tokens['key'],
9696
]);
97-
$replay = $dtoReplay->validate();
97+
$replay = $dtoReplay->checkValidations();
9898
$this->assertFalse($replay->isValid());
9999
$this->assertContains('invalid_csrf', array_column($replay->getErrors(), 'code'));
100100
}

0 commit comments

Comments
 (0)