Skip to content

Commit

Permalink
extract action
Browse files Browse the repository at this point in the history
  • Loading branch information
taylorotwell committed Sep 11, 2020
1 parent 9215e54 commit a9e68f2
Show file tree
Hide file tree
Showing 2 changed files with 47 additions and 22 deletions.
43 changes: 43 additions & 0 deletions src/Actions/ConfirmPassword.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
<?php

namespace Laravel\Fortify\Actions;

use Illuminate\Contracts\Auth\StatefulGuard;
use Laravel\Fortify\Fortify;

class ConfirmPassword
{
/**
* Confirm that the given password is valid for the given user.
*
* @param \Illuminate\Contracts\Auth\StatefulGuard $guard
* @param mixed $user
* @param string $password
* @return bool
*/
public function __invoke(StatefulGuard $guard, $user, string $password)
{
$username = config('fortify.username');

return is_null(Fortify::$confirmPasswordsUsingCallback) ? $guard->validate([
$username => $user->{$username},
'password' => $password,
]) : $this->confirmPasswordUsingCustomCallback($user, $password);
}

/**
* Confirm the user's password using a custom callback.
*
* @param mixed $user
* @param string $password
* @return bool
*/
protected function confirmPasswordUsingCustomCallback($user, string $password)
{
return call_user_func(
Fortify::$confirmPasswordsUsingCallback,
$user,
$password
);
}
}
26 changes: 4 additions & 22 deletions src/Http/Controllers/ConfirmablePasswordController.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@
use Illuminate\Contracts\Auth\StatefulGuard;
use Illuminate\Http\Request;
use Illuminate\Routing\Controller;
use Laravel\Fortify\Actions\ConfirmPassword;
use Laravel\Fortify\Contracts\ConfirmPasswordViewResponse;
use Laravel\Fortify\Contracts\FailedPasswordConfirmationResponse;
use Laravel\Fortify\Contracts\PasswordConfirmedResponse;
use Laravel\Fortify\Fortify;

class ConfirmablePasswordController extends Controller
{
Expand Down Expand Up @@ -49,12 +49,9 @@ public function show(Request $request)
*/
public function store(Request $request)
{
$username = config('fortify.username');

$confirmed = is_null(Fortify::$confirmPasswordsUsingCallback) ? $this->guard->validate([
$username => $request->user()->{$username},
'password' => $request->input('password')
]) : $this->confirmPasswordUsingCustomCallback($request);
$confirmed = app(ConfirmPassword::class)(
$this->guard, $request->user(), $request->input('password')
);

if ($confirmed) {
$request->session()->put('auth.password_confirmed_at', time());
Expand All @@ -64,19 +61,4 @@ public function store(Request $request)
? app(PasswordConfirmedResponse::class)
: app(FailedPasswordConfirmationResponse::class);
}

/**
* Confirm the user's password using a custom callback.
*
* @param \Illuminate\Http\Request $request
* @return bool
*/
protected function confirmPasswordUsingCustomCallback(Request $request)
{
return call_user_func(
Fortify::$confirmPasswordsUsingCallback,
$request->user(),
$request->input('password')
);
}
}

0 comments on commit a9e68f2

Please sign in to comment.