Skip to content

Commit f4f5eb1

Browse files
authored
feature #1464 [make:registration-form] use validateEmailConfirmationFromRequest() when verifying emails
* [make:registration-form] use the new validateEmailConfirmationRequest method * use constructor property promotion
1 parent 33cb766 commit f4f5eb1

File tree

3 files changed

+8
-11
lines changed

3 files changed

+8
-11
lines changed

src/Maker/MakeRegistrationForm.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@
5656
use Symfony\Component\Validator\Validation;
5757
use Symfony\Contracts\Translation\TranslatorInterface;
5858
use SymfonyCasts\Bundle\VerifyEmail\Exception\VerifyEmailExceptionInterface;
59-
use SymfonyCasts\Bundle\VerifyEmail\Model\VerifyEmailSignatureComponents;
6059
use SymfonyCasts\Bundle\VerifyEmail\SymfonyCastsVerifyEmailBundle;
60+
use SymfonyCasts\Bundle\VerifyEmail\VerifyEmailHelper;
6161
use SymfonyCasts\Bundle\VerifyEmail\VerifyEmailHelperInterface;
6262

6363
/**
@@ -444,13 +444,13 @@ private function getMissingComponentsComposerMessage(): ?string
444444
$missing = false;
445445
$composerMessage = 'composer require';
446446

447-
// verify-email-bundle 1.1.1 includes support for translations and a fix for the bad expiration time bug.
448-
// we need to check that if the bundle is installed, it is version 1.1.1 or greater
447+
// verify-email-bundle 1.17.0 includes the new validateEmailConfirmationFromRequest method.
448+
// we need to check that if the bundle is installed, it is version 1.17.0 or greater
449449
if (class_exists(SymfonyCastsVerifyEmailBundle::class)) {
450-
$reflectedComponents = new \ReflectionClass(VerifyEmailSignatureComponents::class);
450+
$reflectedComponents = new \ReflectionClass(VerifyEmailHelper::class);
451451

452-
if (!$reflectedComponents->hasMethod('getExpirationMessageKey')) {
453-
throw new RuntimeCommandException('Please upgrade symfonycasts/verify-email-bundle to version 1.1.1 or greater.');
452+
if (!$reflectedComponents->hasMethod('validateEmailConfirmationFromRequest')) {
453+
throw new RuntimeCommandException('Please upgrade symfonycasts/verify-email-bundle to version 1.17.0 or greater.');
454454
}
455455
} else {
456456
$missing = true;

src/Resources/skeleton/registration/RegistrationController.tpl.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,8 @@
77
class <?= $class_name; ?> extends AbstractController
88
{
99
<?php if ($will_verify_email): ?>
10-
private <?= $generator->getPropertyType($email_verifier_class_details) ?>$emailVerifier;
11-
12-
public function __construct(<?= $email_verifier_class_details->getShortName() ?> $emailVerifier)
10+
public function __construct(private <?= $generator->getPropertyType($email_verifier_class_details) ?>$emailVerifier)
1311
{
14-
$this->emailVerifier = $emailVerifier;
1512
}
1613

1714
<?php endif; ?>

src/Resources/skeleton/verifyEmail/EmailVerifier.tpl.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public function sendEmailConfirmation(string $verifyEmailRouteName, UserInterfac
4141
*/
4242
public function handleEmailConfirmation(Request $request, UserInterface $user): void
4343
{
44-
$this->verifyEmailHelper->validateEmailConfirmation($request->getUri(), $user-><?= $id_getter ?>(), $user-><?= $email_getter?>());
44+
$this->verifyEmailHelper->validateEmailConfirmationFromRequest($request, $user-><?= $id_getter ?>(), $user-><?= $email_getter?>());
4545

4646
$user->setIsVerified(true);
4747

0 commit comments

Comments
 (0)