Future validation of password

[caouecs]

laravel/framework#36960

with new tags, but not defined for the moment

[andrey-helldar]

I now have one less package to support (andrey-helldar/strong-password) 😅

[caouecs]

the pull request was merger but without translations...

[luisprmat]

@caouecs, No, laravel/framework#36960 was merged on April 23 and it works perfectly with translations. i tested it with Spanish Language.

it happens that this time the messages are wrapped by the translator through the fail method of the Illuminate\Validation\Rules\Password class.

    $this->fail('The :attribute must contain at least one uppercase and one lowercase letter.');
...
    protected function fail($messages)
    {
        $messages = collect(Arr::wrap($messages))->map(function ($message) {
            return __($message);  // Call translator
        })->all();

        $this->messages = array_merge($this->messages, $messages);

        return false;
    }

I tried this code in the register form provided by Laravel Breeze, modifying app/Http/Controllers/Auth/RegisteredUserController.php in the store method.

use Illuminate\Validation\Rules\Password;

...

$request->validate([
    'name' => 'required|string|max:255',
    'email' => 'required|string|email|max:255|unique:users',
    'password' => [
        'required',
        'string',
        'confirmed',
        Password::min(2)
            ->symbols()
            ->numbers()
            ->mixedCase()
            ->letters()
            ->uncompromised()
    ],
]);

and i got

This feature yet doesn't available in Laravel 8.38.0 but i tested in dev mode (branch 8.x-dev). I guess it will be available in the next release (8.39.0 ?).
For this reason I added two PR ( en, es) since the tests do not pass after modifying the file source/en.json and it did not allow me to make a single pull request from my fork as I always do.

Hopefully @andrey-helldar can fix this soon.

[andrey-helldar]

I remembered why the test was giving an error.

You added new keys to the source/en.json file, but did not add them to all languages.

In this case, we just accept the PR without paying attention to phpunit.

When the PR is accepted, an automatic script is launched that will put down all the necessary keys (php app/keys.php)

[luisprmat]

laravel/framework#36960

with new tags, but not defined for the moment

Feature supported since version 8.39.0 which was released today April 27